[U-Boot] [PATCH] mtd: sf_probe: add jedec,spi-nor compatible

Simon Goldschmidt simon.k.r.goldschmidt at gmail.com
Wed Nov 14 09:08:55 UTC 2018


On Wed, Nov 14, 2018 at 9:59 AM Jagan Teki <jagan at amarulasolutions.com> wrote:
>
> On Sat, Nov 10, 2018 at 1:14 AM Simon Goldschmidt
> <simon.k.r.goldschmidt at gmail.com> wrote:
> >
> > On 06.11.2018 10:24, Neil Armstrong wrote:
> > > There is no reason not to use the Linux "jedec,spi-nor" binding in u-boot.
> > > This patch adds this compatible into sf_probe and will avoid changing the
> > > device tree files to add a u-boot specific compatible to probe a spi flash.
> > >
> > > This change is needed to enable SPI NOR Flash support using the non-modified
> > > device tree file for Amlogic Meson GX SoC based boards.
> > >
> > > Signed-off-by: Neil Armstrong <narmstrong at baylibre.com>
> >
> > This is definitively a move in the right direction! I was only days away
> > from sending this myself ;-)
> >
> > Reviewed-by: Simon Goldschmidt <simon.k.r.goldschmidt at gmail.com>
> >
> > I would still need more changes to U-Boot to boot the Linux socfpga
> > devicetrees without patches (they have compatible = "n25q256a" etc.),
> > but then again, maybe it's easier to change the Linux ones to include
> > "jedec,spi-nor" than trying the effort of changing U-Boot here...
>
> It's better to revert/remove "spi-flash" from Linux sync dts(s) files
> along with this compatible change, u-boot specific stuff like sandbox
> still use the legacy.

Is this a reply to me to change the scofpga dts files or to Neil to
remove "spi-flash" from U-Boot with his patch?

Because, sure, once U-Boot supports "jedec-spi-nor", I'll make sure
the Linux devicetrees get that, too (in addition to the "n25q..."
compatibles) so that they just work in U-Boot and can be synced again.

Simon


More information about the U-Boot mailing list