[U-Boot] [PATCH] ARM: dts: am437x-idk: Fix QSPI compatible string

Jagan Teki jagan at openedev.com
Thu Oct 13 15:29:52 CEST 2016


On Thu, Oct 13, 2016 at 6:24 PM, Tom Rini <trini at konsulko.com> wrote:
> On Thu, Oct 13, 2016 at 05:45:52PM +0530, Jagan Teki wrote:
>> On Thu, Oct 13, 2016 at 3:53 PM, Vignesh R <vigneshr at ti.com> wrote:
>> > Unlike Linux kernel, U-Boot depends on "spi-flash" compatible to probe
>> > m25p80 spi-nor devices. Hence, add "spi-flash" compatible string to
>> > m25p80 node. Without this patch, flash device DT data is not parsed and
>> > QSPI operates in unsupported mode leading to data corruption.
>> >
>> > Signed-off-by: Vignesh R <vigneshr at ti.com>
>>
>> Applied to u-boot-spi/master
>
> ... I don't like that we need a non u-boot prefixed string here for the
> binding to work as that will lead to harder re-syncs later on the dt
> files.  Why aren't we matching on the existing part?  Thanks!

Most of these compatibles from Linux, and we're unable to support as
of now. and moment we moved to spi-nor the we will maintain the
compatible of all these like Linux.

Too speed-up, can you please look at the last approach was submitted on [1]

[1] [PATCH RFC v8 00/16] SPI-NOR/MTD addition

thanks!
-- 
Jagan Teki
Free Software Engineer | www.openedev.com
U-Boot, Linux | Upstream Maintainer
Hyderabad, India.


More information about the U-Boot mailing list