Hi Chris, On Thu, Jan 12, 2017 at 7:11 PM, Chris Brandt <chris.brandt@xxxxxxxxxxx> wrote: > Now that all the clocks in the boot loader are disabled before booting > the kernel, and the mstp driver has been fixed for RZ/A1, here is a typo > that was missed during original testing. > > Fixes: 7c8522b7047c ("ARM: dts: r7s72100: add sdhi clock to device tree") > Signed-off-by: Chris Brandt <chris.brandt@xxxxxxxxxxx> > --- > include/dt-bindings/clock/r7s72100-clock.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/include/dt-bindings/clock/r7s72100-clock.h b/include/dt-bindings/clock/r7s72100-clock.h > index 29e01ed..318ab14 100644 > --- a/include/dt-bindings/clock/r7s72100-clock.h > +++ b/include/dt-bindings/clock/r7s72100-clock.h > @@ -46,6 +46,6 @@ > > /* MSTP12 */ > #define R7S72100_CLK_SDHI0 3 > -#define R7S72100_CLK_SDHI1 2 > +#define R7S72100_CLK_SDHI1 1 This is strange. There are two SDHI channels, but the STBCR12 documentation (all versions up to rev. 3.00) says the register has MSTP bits for four SD host interfaces? Can you please enlighten me? Thanks! Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds