On Mon, Aug 2, 2010 at 9:11 PM, Roy Zang <tie-fei.zang@xxxxxxxxxxxxx> wrote: > Signed-off-by: Roy Zang <tie-fei.zang@xxxxxxxxxxxxx> > --- > Documentation/powerpc/dts-bindings/fsl/esdhc.txt | 2 ++ > arch/powerpc/boot/dts/p4080ds.dts | 1 + > 2 files changed, 3 insertions(+), 0 deletions(-) > > diff --git a/Documentation/powerpc/dts-bindings/fsl/esdhc.txt b/Documentation/powerpc/dts-bindings/fsl/esdhc.txt > index 8a00407..64bcb8b 100644 > --- a/Documentation/powerpc/dts-bindings/fsl/esdhc.txt > +++ b/Documentation/powerpc/dts-bindings/fsl/esdhc.txt > @@ -14,6 +14,8 @@ Required properties: > reports inverted write-protect state; > - sdhci,1-bit-only : (optional) specifies that a controller can > only handle 1-bit data transfers. > + - sdhci,auto-cmd12: (optional) specifies that a controller can > + only handle auto CMD12. I read this, but I still don't understand what it means. What does "auto CMD12" mean? Are there other kinds of CMD12? Part of this might be my ignorance about how eSDHC works, but it could be clearer. Also, you can feel free to merge this patch with patch 1 in your series. I makes sense to update the documentation and the driver at the same time. g. -- Grant Likely, B.Sc., P.Eng. Secret Lab Technologies Ltd. -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html