Re: [U-Boot] Regressions in MTD / SPI FLASH

2019-09-25 Thread Vignesh Raghavendra
18 Written: OK > OK > AXS# sf read 0x8100 0x18 0x10 && echo OK > device 0 offset 0x180000, size 0x10 > 0b 00 18 00 00 | [16B in] 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 > [ret 0] > SF: 16 bytes @ 0x18 Read: OK > OK > AXS# md.b 0x8100 0

Re: [U-Boot] Regressions in MTD / SPI FLASH

2019-09-24 Thread Eugeniy Paltsev
_ From: Vignesh Raghavendra Sent: Monday, September 23, 2019 08:31 To: Eugeniy Paltsev Cc: uboot-snps-...@synopsys.com; Alexey Brodkin Subject: Re: [U-Boot] Regressions in MTD / SPI FLASH Hi Eugeniy On 12/09/19 5:59 PM, Eugeniy Paltsev wrote: > Hi Vignesh, > > I

Re: [U-Boot] Regressions in MTD / SPI FLASH

2019-09-12 Thread Eugeniy Paltsev
-boot@lists.denx.de; Alexey Brodkin; tr...@konsulko.com; uboot-snps-...@synopsys.com Subject: Re: [U-Boot] Regressions in MTD / SPI FLASH Hi Eugeniy, One more request: I am trying to find a better way to identify parts that don't support 4byte addressing. Could you enable CONFIG_SPI_FLASH_SFDP_S

Re: [U-Boot] Regressions in MTD / SPI FLASH

2019-09-10 Thread Vignesh Raghavendra
Hi Eugeniy, One more request: I am trying to find a better way to identify parts that don't support 4byte addressing. Could you enable CONFIG_SPI_FLASH_SFDP_SUPPORT and also enable debug prints in spi_mem_exec_op() (in drivers/spi/spi-mem.c like before) and provide logs? Just logs of "sf probe"

Re: [U-Boot] Regressions in MTD / SPI FLASH

2019-09-10 Thread Vignesh Raghavendra
On 10/09/19 5:11 PM, Eugeniy Paltsev wrote: > Hi Vignesh, > > that patch helps - both erase and write works fine. > Thanks for testing! I will cleanup the patches and send formal patches to the list with your tested by. Regards Vignesh > For n25q512ax3: > Tested-by: "Eugeniy Paltsev " > >

Re: [U-Boot] Regressions in MTD / SPI FLASH

2019-09-10 Thread Eugeniy Paltsev
Hi Vignesh, that patch helps - both erase and write works fine. For n25q512ax3: Tested-by: "Eugeniy Paltsev " --- Eugeniy Paltsev From: Vignesh Raghavendra Sent: Tuesday, September 10, 2019 08:07 To: Eugeniy Paltsev; Jagan Teki Cc: u-boot@lists.denx.

Re: [U-Boot] Regressions in MTD / SPI FLASH

2019-09-09 Thread Vignesh Raghavendra
Hi, On 10/09/19 12:18 AM, Eugeniy Paltsev wrote: > Hi! > Comments are inlined: > >> On 04/09/19 11:37 PM, Eugeniy Paltsev wrote: >>> We faced with regressions caused by >>> commit c4e8862308d4 (mtd: spi: Switch to new SPI NOR framework) >>> This switch was performed by removing entire u-boot spi-

Re: [U-Boot] Regressions in MTD / SPI FLASH

2019-09-09 Thread Eugeniy Paltsev
Hi! Comments are inlined: >On 04/09/19 11:37 PM, Eugeniy Paltsev wrote: >> We faced with regressions caused by >> commit c4e8862308d4 (mtd: spi: Switch to new SPI NOR framework) >> This switch was performed by removing entire u-boot spi-flash >> core implementation and copying it from another proj

Re: [U-Boot] Regressions in MTD / SPI FLASH

2019-09-09 Thread Schrempf Frieder
Hi Eugeniy, On 04.09.19 20:07, Eugeniy Paltsev wrote: > We faced with regressions caused by > commit c4e8862308d4 (mtd: spi: Switch to new SPI NOR framework) > This switch was performed by removing entire u-boot spi-flash > core implementation and copying it from another project. > However the swi

Re: [U-Boot] Regressions in MTD / SPI FLASH

2019-09-06 Thread Vignesh Raghavendra
Hi, On 04/09/19 11:37 PM, Eugeniy Paltsev wrote: > We faced with regressions caused by > commit c4e8862308d4 (mtd: spi: Switch to new SPI NOR framework) > This switch was performed by removing entire u-boot spi-flash > core implementation and copying it from another project. > However the switch i

[U-Boot] Regressions in MTD / SPI FLASH

2019-09-04 Thread Eugeniy Paltsev
We faced with regressions caused by commit c4e8862308d4 (mtd: spi: Switch to new SPI NOR framework) This switch was performed by removing entire u-boot spi-flash core implementation and copying it from another project. However the switch is performed without proper testing and investigations about