lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite for Android: free password hash cracker in your pocket
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <54F37BA5.2080605@cogentembedded.com>
Date:	Sun, 01 Mar 2015 23:50:45 +0300
From:	Sergei Shtylyov <sergei.shtylyov@...entembedded.com>
To:	Roman Volkov <rvolkov@...os.org>
CC:	Rob Herring <robh+dt@...nel.org>, Pawel Moll <pawel.moll@....com>,
	Mark Rutland <mark.rutland@....com>,
	Ian Campbell <ijc+devicetree@...lion.org.uk>,
	Kumar Gala <galak@...eaurora.org>,
	Russell King <linux@....linux.org.uk>,
	devicetree@...r.kernel.org, Alexey Charkov <alchark@...il.com>,
	linux-kernel@...r.kernel.org, linux-arm-kernel@...ts.infradead.org
Subject: Re: [PATCH v3 1/2] dts: vt8500: Add SDHC node to DTS file for WM8650

Hello.

On 03/01/2015 10:49 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@...os.org>
>>> Reviewed-by: Alexey Charkov <alchark@...il.com>
>>> ---
>>>    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@...0a000 {

>>      According to the ePAPR standard, the device nodes should have
>> generic names according to their functions, so this node should be
>> named just "sd@...0a000".

>> WBR, Sergei

> Sergei,

> Why not "sdmmc" or "sdhci" or "sdio" then? It would be great to name

    SDHCI is the SD host controller standard, while the names should describe 
"the function of the device and not its precise programming model". SDIO isn't 
the physical interface name, the physical interface underneath is still SD. 
"mmcsd" or "mmc-sd" would be appropriate, however just "mmc" is typically used.

> nodes in same way, but there is no exact requirement in ePAPR v1.1
> regarding SD/MMC controllers.

    Yes, MMC/SD node names are not yet standardized.

> Also, in Linux I don't see that someone
> strictly follows this requirement.

    Because people for the most part are not aware of it, despite 
http://www.devicetree.org/Device_Tree_Usage points to ePAPR and the section 
2.4 of this page also tells to use generic names.

> This renaming will affect driver,
> since .compatible strings likely also need changing.

    Why on earth they'll need change?!

> Roman

WBR, Sergei

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ