On Sun, Jun 10, 2012 at 8:11 PM, Rob Herring <robherring2@xxxxxxxxx> wrote: > On 06/10/2012 12:24 PM, Fabio Estevam wrote: >> From: Fabio Estevam <fabio.estevam@xxxxxxxxxxxxx> >> >> Fix gpio number assignment, so that it can match the number stated in the comment. >> >> Cc: >> Signed-off-by: Fabio Estevam <fabio.estevam@xxxxxxxxxxxxx> >> --- >> .../devicetree/bindings/mmc/fsl-imx-esdhc.txt | 4 ++-- >> 1 files changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/Documentation/devicetree/bindings/mmc/fsl-imx-esdhc.txt b/Documentation/devicetree/bindings/mmc/fsl-imx-esdhc.txt >> index c7e404b..fea541e 100644 >> --- a/Documentation/devicetree/bindings/mmc/fsl-imx-esdhc.txt >> +++ b/Documentation/devicetree/bindings/mmc/fsl-imx-esdhc.txt >> @@ -29,6 +29,6 @@ esdhc@70008000 { >> compatible = "fsl,imx51-esdhc"; >> reg = <0x70008000 0x4000>; >> interrupts = <2>; >> - cd-gpios = <&gpio0 6 0>; /* GPIO1_6 */ >> - wp-gpios = <&gpio0 5 0>; /* GPIO1_5 */ >> + cd-gpios = <&gpio1 6 0>; /* GPIO1_6 */ >> + wp-gpios = <&gpio1 5 0>; /* GPIO1_5 */ > > Are you sure all these aren't just DT is 0 based and the chip is 1 based > numbering? We start counting gpio number from 0 in i.mx, so the DT number and the GPIO number should match. -- To unsubscribe from this list: send the line "unsubscribe linux-doc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html