From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bin Meng Date: Wed, 17 Dec 2014 16:02:07 +0800 Subject: [U-Boot] [PATCH v3 0/9] sf: Update flash params for supported read commands and sector size In-Reply-To: References: <1418215892-17617-1-git-send-email-bmeng.cn@gmail.com> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Jagan, On Wed, Dec 17, 2014 at 3:59 PM, Jagan Teki wrote: > On 15 December 2014 at 19:21, Bin Meng wrote: >> Hi Jagan, >> >> On Thu, Dec 11, 2014 at 3:40 PM, Bin Meng wrote: >>> Hi Jagan, >>> >>> On Thu, Dec 11, 2014 at 3:26 PM, Jagan Teki wrote: >>>> Hi Bin, >>>> >>>> On 11 December 2014 at 08:34, Bin Meng wrote: >>>>> Hi Jagan, >>>>> >>>>> On Thu, Dec 11, 2014 at 2:41 AM, Jagan Teki wrote: >>>>>> Hi Bin, >>>>>> >>>>>> On 10 December 2014 at 18:21, Bin Meng wrote: >>>>>>> This series update SPI flash supported read commands per datasheet >>>>>>> in the flash params table, and change flash sector size to 4KiB as >>>>>>> long as flash supports sector erase (20h) command, to ensure >>>>>>> 'sf erase offset +len' work on 4KiB boundary instead of 64KiB when >>>>>>> given SECT_4K. >>>>>>> >>>>>>> Changes in v3: >>>>>>> - Rebase with Jagan's patch series @ http://patchwork.ozlabs.org/patch/419154/ >>>>>>> >>>>>>> >>>>>>> Bin Meng (9): >>>>>>> sf: Update SST flash params >>>>>>> sf: Update Atmel flash params >>>>>>> sf: Update EON flash params >>>>>>> sf: Update GigaDevice flash params >>>>>>> sf: Update Macronix flash params >>>>>>> sf: Update Spansion flash params >>>>>>> sf: Update Micron flash params >>>>>>> sf: Update Winbond flash params >>>>>>> sf: Give proper spacing between flash table params >>>>>> >>>>>> Thanks for the updates - have you verified these changes? >>>>> >>>>> I verified some, but not all of these flash parts. The update is based >>>>> on flash datasheet, so if something is broken, eg before this series >>>>> the flash advertises only READ_NORM and after my series it is changed >>>>> to READ_FULL, and let's say QUAD_IO_FAST is not working, it is very >>>>> likely that the SPI controller driver has some bugs when supporting >>>>> QUAD_IO_FAST. >>>> >>>> Since these updates were tested before, I will skip these for this PR. >>>> Will test all the rest (except these) and send the PR soon. >>>> >>>> Let me know your inputs? >>> >>> I am fine, as long as this PR will not contains other commits which >>> modify the same sf_params.c to introduce more flash support. We can >>> test these updates and if everything looks fine, apply these first and >>> ask other commits to rebase on this series to introduce more flash >>> support. >>> >> >> Do you have any additional comments about this patch series besides >> the S25FL128S_64K and S25FL256S_64K sector size? If not, I can send >> the v4. > > I'm thinking about the other flashes too, since these params were taken from > previous working and Linux mtd. > You mean 'thinking about' or 'testing'? I should say previously they might not be 100% working as per datasheet some flash params currently are apparently wrong. Regards, Bin