Am Sonntag, den 01.03.2015, 08:23 +0100 schrieb Sascha Hauer:
> On Fri, Feb 27, 2015 at 08:22:17PM +0100, Lucas Stach wrote:
> > Signed-off-by: Lucas Stach <dev@xxxxxxxxxx>
> > ---
> > arch/arm/boards/nvidia-beaver/board.c | 4 ++++
> > 1 file changed, 4 insertions(+)
> >
> > diff --git a/arch/arm/boards/nvidia-beaver/board.c b/arch/arm/boards/nvidia-beaver/board.c
> > index d270301..b4d856e 100644
> > --- a/arch/arm/boards/nvidia-beaver/board.c
> > +++ b/arch/arm/boards/nvidia-beaver/board.c
> > @@ -19,6 +19,7 @@
> > #include <gpio.h>
> > #include <i2c/i2c.h>
> > #include <init.h>
> > +#include <mach/tegra-bbu.h>
> >
> > static int nvidia_beaver_fs_init(void)
> > {
> > @@ -53,6 +54,9 @@ static int nvidia_beaver_device_init(void)
> >
> > barebox_set_hostname("beaver");
> >
> > + tegra_bbu_register_emmc_handler("eMMC", "/dev/disk0.boot0",
> > + BBU_HANDLER_FLAG_DEFAULT);
>
> 'disk0' is not a persistent name and will change when your SD slots are
> probed in a different order. You should add aliases to the device nodes
> and use of_alias_get() to obtain them, see for example imx-esdhc.c
>
It's not a problem for these patches, as both boards have their
environment on disk0.boot1, so the eMMC will always be the first SDMMC
device to be detected and consequently always get number 0 assigned.
But in general I agree with you and will add the alias handling in a
follow on series.
Regards,
Lucas
_______________________________________________
barebox mailing list
barebox@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/barebox
