public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Jagan Teki <jteki@openedev.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [RESEND 1/2] stm32: stm32f4: move flash driver to mtd driver location
Date: Fri, 11 Mar 2016 00:08:20 +0530	[thread overview]
Message-ID: <56E1BF1C.1090002@openedev.com> (raw)
In-Reply-To: <1E25CFEBFF5B6D40BBC7682080D80524380A276399@SAFEX1MAIL5.st.com>

On Thursday 10 March 2016 11:47 PM, Vikas MANOCHA wrote:
> Hi Jagan,
>
>> -----Original Message-----
>> From: Jagan Teki [mailto:jteki at openedev.com]
>> Sent: Thursday, March 10, 2016 9:58 AM
>> To: Vikas MANOCHA
>> Cc: u-boot at lists.denx.de; Albert Aribaud; Antonio Borneo; Heiko Schocher;
>> Kamil Lulko; Matt Porter; rev13 at wp.pl; Scott Wood; Simon Glass; Stefan
>> Roese; Thomas Chou; Vadzim Dambrouski
>> Subject: Re: [RESEND 1/2] stm32: stm32f4: move flash driver to mtd driver
>> location
>>
>> On Thursday 10 March 2016 11:22 PM, Vikas MANOCHA wrote:
>>> Hi,
>>>
>>>> -----Original Message-----
>>>> From: Jagan Teki [mailto:jteki at openedev.com]
>>>> Sent: Thursday, March 10, 2016 9:46 AM
>>>> To: Vikas MANOCHA
>>>> Cc: u-boot at lists.denx.de; Albert Aribaud; Antonio Borneo; Heiko
>>>> Schocher; Kamil Lulko; Matt Porter; rev13 at wp.pl; Scott Wood; Simon
>>>> Glass; Stefan Roese; Thomas Chou; Vadzim Dambrouski
>>>> Subject: Re: [RESEND 1/2] stm32: stm32f4: move flash driver to mtd
>>>> driver location
>>>>
>>>> On Thursday 10 March 2016 11:11 PM, Vikas MANOCHA wrote:
>>>>> Hi Jagan,
>>>>>
>>>>>> -----Original Message-----
>>>>>> From: jteki at openedev.com [mailto:jteki at openedev.com]
>>>>>> Sent: Wednesday, March 09, 2016 9:02 PM
>>>>>> To: Vikas MANOCHA
>>>>>> Cc: u-boot at lists.denx.de; Albert Aribaud; Antonio Borneo; Heiko
>>>>>> Schocher; Kamil Lulko; Matt Porter; rev13 at wp.pl; Scott Wood; Simon
>>>>>> Glass; Stefan Roese; Thomas Chou; Vadzim Dambrouski
>>>>>> Subject: Re: [RESEND 1/2] stm32: stm32f4: move flash driver to mtd
>>>>>> driver location
>>>>>>
>>>>>> On 2016-03-09 23:18, Vikas MANOCHA wrote:
>>>>>>> Hi Jagan ,
>>>>>>>
>>>>>>>> On Mar 9, 2016, at 6:53 PM, Jagan Teki <jteki@openedev.com>
>> wrote:
>>>>>>>>
>>>>>>>>> On Thursday 10 March 2016 04:48 AM, Vikas Manocha wrote:
>>>>>>>>> Same flash driver can be used by other stm32 families like stm32f7.
>>>>>>>>> Better place for this driver would be mtd driver location.
>>>>>>>>
>>>>>>>> What kind of flash is it, parallel or serial NOR?
>>>>>>>
>>>>>>> It is embedded 256 bit NOR flash.
>>>>>>
>>>>>> OK, I thought it's similar to SST25L.
>>>>>>
>>>>>> Can't we detect this through cfi_flash framework?
>>>>>
>>>>> No, it is not possible.
>>>>
>>>> Sorry, please add some details why it couldn't?
>>>
>>> The embedded flash in stm32 is not cfi flash. Cfi tables to find the flash
>> characteristics are not implemented.
>>
>> OK, all your saying that this not quite exactly a cfi, different kind wrt sst32.
>> true?
>
> Yes, flash is embedded in the stm32 micro & details like data width etc are not relevant for the user.
> Sst32  ? , sorry but I am not aware of it and sst32L.
>
>> then please convert this into MTD driver model format.
>>
>> If needed take an example of existing MTD uclass driver.
>
> Can you please point me to it ?

The board your using with this driver has fdt and dm support?

If so, it is quite straight forward, take an example of this 
drivers/mtd/altera_qspi.c

- register with MTD_UCLASS
- add your erase/write and read ops through mtd_ops

  reply	other threads:[~2016-03-10 18:38 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-03-09 23:18 [U-Boot] [RESEND 1/2] stm32: stm32f4: move flash driver to mtd driver location Vikas Manocha
2016-03-09 23:18 ` [U-Boot] [RESEND 2/2] stm32f746-disco: enable flash support Vikas Manocha
2016-03-27 22:23   ` [U-Boot] [U-Boot, RESEND, " Tom Rini
2016-03-10  2:52 ` [U-Boot] [RESEND 1/2] stm32: stm32f4: move flash driver to mtd driver location Jagan Teki
2016-03-10  4:18   ` Vikas MANOCHA
2016-03-10  5:01     ` jteki at openedev.com
2016-03-10 17:41       ` Vikas MANOCHA
2016-03-10 17:46         ` Jagan Teki
2016-03-10 17:52           ` Vikas MANOCHA
2016-03-10 17:58             ` Jagan Teki
2016-03-10 18:17               ` Vikas MANOCHA
2016-03-10 18:38                 ` Jagan Teki [this message]
2016-03-21 21:53                   ` Vikas MANOCHA
2016-03-27 22:23 ` [U-Boot] [U-Boot, RESEND, " Tom Rini

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=56E1BF1C.1090002@openedev.com \
    --to=jteki@openedev.com \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox