Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1339094

Re: [PATCH v3 2/3] ARM: dts: Change iio_hwmon nodes to use hypen in node names

From Rob Herring <robh@kernel.org>
Newsgroups linux.kernel
Subject Re: [PATCH v3 2/3] ARM: dts: Change iio_hwmon nodes to use hypen in node names
Date 2016-02-22 04:00 +0100
Message-ID <r4OT8-7hh-29@gated-at.bofh.it> (permalink)
References <r2G3D-399-3@gated-at.bofh.it> <r2G3E-399-9@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


On Tue, Feb 16, 2016 at 10:30:54AM +0530, Sanchayan Maity wrote:
> Change iio_hwmon nodes to use hypen in node names instead of
> underscore.
> 
> Signed-off-by: Sanchayan Maity <maitysanchayan@gmail.com>
> ---
>  Documentation/devicetree/bindings/iio/iio-bindings.txt | 2 +-
>  arch/arm/boot/dts/imx23.dtsi                           | 2 +-
>  arch/arm/boot/dts/imx28.dtsi                           | 2 +-
>  3 files changed, 3 insertions(+), 3 deletions(-)

No real objection here though I don't think this is really necessary. 
While we prefer '-' to '_' it's not really strong enough to change. It 
bothers me more that the node name leaks to userspace.

Rob

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH v3 0/3] Allow iio_hwmon to accept hypen in node names Sanchayan Maity <maitysanchayan@gmail.com> - 2016-02-16 06:10 +0100
  [PATCH v3 2/3] ARM: dts: Change iio_hwmon nodes to use hypen in node names Sanchayan Maity <maitysanchayan@gmail.com> - 2016-02-16 06:10 +0100
    Re: [PATCH v3 2/3] ARM: dts: Change iio_hwmon nodes to use hypen in  node names Rob Herring <robh@kernel.org> - 2016-02-22 04:00 +0100
  Re: [PATCH v3 0/3] Allow iio_hwmon to accept hypen in node names Guenter Roeck <linux@roeck-us.net> - 2016-02-17 04:40 +0100

csiph-web