Re: [PATCH v2 0/1] Add Kindle generation 6/7 support

[Alexander Kurz <akurz@blala.de>]

On Mon, 17 Nov 2025, Sascha Hauer wrote:

> On Mon, Nov 17, 2025 at 06:46:00AM +0000, Alexander Kurz wrote:
> > Changes in v2:
> > - Link to v1: https://lore.barebox.org/barebox/aRMOO3jH4a16GBI6@pengutronix.de/
> > - Remove explicit RAM size settings from lowlevel.c and dts since this can be
> >   autodetected. As consequence, both DP75SDI variants (Paperwhite 2 and 3) now
> >   share the same dts.
> > - Rebase to current
> > - Attempts to use barebox,deep-probe did unfortunately not work yet.
> >   Brief description of the problem:
> >   Device specific data is stored in /dev/mmc1.boot0.userdata, this data
> >   is read by barebox and passed as ATAGs to the vendor kernel, without this
> >   data, the device won't work properly.
> >   An attempt has been made to use barebox,deep-probe (see patch below), however,
> >   mmc1 will not be available later, resulting e.g. in an error
> >   > Could not read board info from /dev/mmc1.boot0.userdata
> >   during startup.
> >   Example patch:
> > # diff --git a/arch/arm/boards/kindle-mx6sl/board.c b/arch/arm/boards/kindle-mx6sl/board.c
> > index ff8083a756..5c18e074c9 100644
> > --- a/arch/arm/boards/kindle-mx6sl/board.c
> > +++ b/arch/arm/boards/kindle-mx6sl/board.c
> > @@ -194,15 +194,10 @@ late_initcall(kindle_mx6sl_late_init);
> >  
> >  static int kindle6_device_init(void)
> >  {
> > -       struct device *dev;
> > -
> >         if (!is_mx6sl_kindle())
> >                 return 0;
> >  
> > -       /* Probe the eMMC to allow reading the board serial and revision */
> > -       dev = get_device_by_name("mmc1");
> > -       if (dev)
> > -               dev_set_param(dev, "probe", "1");
> > +       of_device_ensure_probed_by_alias("mmc1");
> 
> On return of this function the MMC host is probed, but the card is not
> yet detected. You can do that by adding a:
> 
> 	device_detect_by_name("mmc1");
> 
> Does it work with this line added?
Thanks, with this change, the this works now with barebox,deep-probe.

> 
> Sascha

Thanks, Alexander
> 
> >  
> >         defaultenv_append_directory(defaultenv_kindle_mx6sl);
> >  
> > # diff --git a/arch/arm/dts/imx6sl-kindle-dp75sdi.dts b/arch/arm/dts/imx6sl-kindle-dp75sdi.dts
> > index 9dc4a0c867..05fdd77a47 100644
> > --- a/arch/arm/dts/imx6sl-kindle-dp75sdi.dts
> > +++ b/arch/arm/dts/imx6sl-kindle-dp75sdi.dts
> > @@ -12,7 +12,7 @@
> >  / {
> >         model = "kindle-dp75sdi";
> >         compatible = "amazon,imx6sl-kindle-dp75sdi",  "fsl,imx6sl";
> > -       barebox,disable-deep-probe;
> > +       barebox,deep-probe;
> >  };
> >  
> >  &iomuxc {
> > 
> >