Re: [PATCH v3 1/2] dts: vt8500: Add SDHC node to DTS file for WM8650
From: Roman Volkov
Date: Sun Mar 01 2015 - 14:51:41 EST
Ð Sun, 01 Mar 2015 20:59:13 +0300
Sergei Shtylyov <sergei.shtylyov@xxxxxxxxxxxxxxxxxx> ÐÐÑÐÑ:
> On 3/1/2015 7:06 PM, Roman Volkov wrote:
>
> > Since WM8650 has the same 'WMT' SDHC controller as WM8505, and the
> > driver is already in the kernel, this node enables the controller
> > support for WM8650
>
> > Signed-off-by: Roman Volkov <rvolkov@xxxxxxxxx>
> > Reviewed-by: Alexey Charkov <alchark@xxxxxxxxx>
> > ---
> > arch/arm/boot/dts/wm8650.dtsi | 9 +++++++++
> > 1 file changed, 9 insertions(+)
>
> > diff --git a/arch/arm/boot/dts/wm8650.dtsi
> > b/arch/arm/boot/dts/wm8650.dtsi index b1c59a7..e12213d 100644
> > --- a/arch/arm/boot/dts/wm8650.dtsi
> > +++ b/arch/arm/boot/dts/wm8650.dtsi
> > @@ -187,6 +187,15 @@
> > interrupts = <43>;
> > };
> >
> > + sdhc@d800a000 {
>
> According to the ePAPR standard, the device nodes should have
> generic names according to their functions, so this node should be
> named just "sd@d800a000".
>
> WBR, Sergei
>
Sergei,
Why not "sdmmc" or "sdhci" or "sdio" then? It would be great to name
nodes in same way, but there is no exact requirement in ePAPR v1.1
regarding SD/MMC controllers. Also, in Linux I don't see that someone
strictly follows this requirement. This renaming will affect driver,
since .compatible strings likely also need changing.
Roman
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/