The thread is so old that the mailing list has changed. Mea culpa. Here is a copy. Linus ---------- Forwarded message ---------- From: Linus Walleij <linus.walleij@xxxxxxxxxx> Date: Mon, Sep 8, 2014 at 1:31 PM Subject: Re: [PATCH v2] ARM: dt: fix up PL011 device tree bindings To: "devicetree-discuss@xxxxxxxxxxxxxxxx" <devicetree-discuss@xxxxxxxxxxxxxxxx>, Grant Likely <grant.likely@xxxxxxxxxx>, Rob Herring <robh+dt@xxxxxxxxxx> Cc: "linux-arm-kernel@xxxxxxxxxxxxxxxxxxx" <linux-arm-kernel@xxxxxxxxxxxxxxxxxxx>, Linus Walleij <linus.walleij@xxxxxxxxxx> On Mon, May 12, 2014 at 11:37 AM, Linus Walleij <linus.walleij@xxxxxxxxxx> wrote: > Make the map match the reality, the current binding text is > nonsense: > > - The clock required for the clocking of the serial port > must come first and is not optional (as the driver will > otherwise proceed to grab and use the apb_pclk as uartclk), > and the apb_pclk that clocks the logic must come second > as the code will retrieve the first clock by index, > whereas the PrimeCell but will explicitly look for > "apb_pclk" so this can be specified later, as it is > looked up by name. > > - The pin control state "default" is the only mandated > state, the sleep state is entirely optional. > > We also add an example to avoid further confusion. > > Reported-by: Arnd Bergmann <arnd@xxxxxxxx> > Signed-off-by: Linus Walleij <linus.walleij@xxxxxxxxxx> > --- > ChangeLog v1->v2: > - Rename amba_pclk to apb_pclk > - Split clock/clocks properties and indicate that listin one > clock only is deprecated I don't see either this or Mark's related patch being applied and the DT binding for PL011 left in a sad state. Can we please apply this? Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html