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

Groups > linux.kernel > #1336045

Re: [PATCH v3 0/3] Allow iio_hwmon to accept hypen in node names

Show all headers | View raw

On 02/15/2016 09:00 PM, Sanchayan Maity wrote:
> Hello,
>
> Initial versions of the patchset intended to introduce the use of
> iio_hwmon for Vybrid SoC's. Currently the iio_hwmon driver has two
> users and as per the binding documentation, the node names use the
> underscore. Use of hypen in device tree node names is acceptable,
> however currently the hwmon core code rejects it due to perhaps user
> space not liking the hypen and getting confused.
>
> As per Guenter Roeck's suggestion, the first patch looks for a hypen
> and replaces it with a underscore in the probe call of iio_hwmon driver
> before calling hwmon_device_register_with_groups for registration. Any
> users of iio_hwmon driver can now use hypen in the node name.
>
> As per Shawn's suggestion, the existing users have been changed to use
> the hypen instead of the underscore and the iio-bindings document
> updated. Third patch introduces the usage for Vybrid itself.
>
> Patchset is based on top of shawn's for-next branch.
>
> Feedbacks and comments most welcome. Thank you for feedbacks.
>
> @Guenter
> I hope I understood and implemented correctly what you had in mind.
>
Yes.

Series applied to hwmon-next (if the dts changes also end up in an arm branch,
no harm done).

Thanks,
Guenter

Back to linux.kernel | Previous | Next — Previous 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