Subject: [PATCH 7/7] Documentation: Update binding for Nomadik and DBx5x based platforms

From: [email-masked] (Lee Jones)
Date: Wed, 29 May 2013 20:01:39 +0100

On Wed, 29 May 2013, Linus Walleij wrote:

> On Tue, May 28, 2013 at 11:38 AM, Lee Jones <[email-masked]> wrote:
> >> >  Required properties:
> >> > -- compatible: "stericsson,nmk-pinctrl", "stericsson,nmk-pinctrl-db8540",
> >> > -              "stericsson,nmk-pinctrl-stn8815"
> >> > +- compatible: "stericsson,db8500-pinctrl", "stericsson,db8540-pinctrl",
> >> > +              "stericsson,stn8815-pinctrl"
> >>
> >> OK...
> >>
> >> >         pinctrl at 80157000 {
> >> > -               compatible = "stericsson,nmk-pinctrl";
> >> > +               compatible = "stericsson,u8500-pinctrl";
> >>
> >> This does not look like a supported compatible string?
> >
> > Do you still think this, or did you change you mind?
> 
> Que? It is not among the listed compatible strings?
> 
> > If so, what makes you think it? How does it differ from:
> >
> > compatible = "arm,cortex-a9-twd-timer";
> > compatible = "arm,rtc-pl031";
> > compatible = "stericsson,db8500-gpio",
> > compatible = "stericsson,db8500-musb",
> > compatible = "stericsson,db8500-dma40",
> > compatible = "stericsson,db8500-prcmu";
> > compatible = "stericsson,db8500-prcmu-timer-4";
> > compatible = "stericsson,db8500-thermal";
> > compatible = "stericsson,db8500-prcmu-regulator";
> 
> It says u8500-pinctrl, not db8500-pinctrl.
> 
> Could it be that you're not actually seeing your own typo?

Yes, that's exactly what happened.

I thought you were mentioning the way it was laid out.

I'll fixup.

-- 
Lee Jones
Linaro ST-Ericsson Landing Team Lead
Linaro.org ? Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog