Re: [PATCH 05/22] ARM: dtsi: axp209: add AXP209 ADC subnode

From	Chen-Yu Tsai <wens@csie.org>
Newsgroups	linux.kernel
Subject	Re: [PATCH 05/22] ARM: dtsi: axp209: add AXP209 ADC subnode
Date	2017-01-05 10:10 +0100
Message-ID	<sWcdz-Is-1@gated-at.bofh.it> (permalink)
References	<sVdY5-1dY-17@gated-at.bofh.it> <sViuJ-4xe-1@gated-at.bofh.it> <sWals-7XR-1@gated-at.bofh.it> <sWbrb-bR-5@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On Thu, Jan 5, 2017 at 4:08 PM, Quentin Schulz
<quentin.schulz@free-electrons.com> wrote:
> On 05/01/2017 06:51, Chen-Yu Tsai wrote:
>> On Tue, Jan 3, 2017 at 12:37 AM, Quentin Schulz
>> <quentin.schulz@free-electrons.com> wrote:
>>> X-Powers AXP209 PMIC has multiple ADCs, each one exposing data from the
>>> different power supplies connected to the PMIC.
>>>
>>> This adds the ADC subnode for AXP20X PMIC.
>>>
>>> Signed-off-by: Quentin Schulz <quentin.schulz@free-electrons.com>
>>> ---
>>>  arch/arm/boot/dts/axp209.dtsi | 5 +++++
>>>  1 file changed, 5 insertions(+)
>>>
>>> diff --git a/arch/arm/boot/dts/axp209.dtsi b/arch/arm/boot/dts/axp209.dtsi
>>> index 675bb0f..2a4e8ee 100644
>>> --- a/arch/arm/boot/dts/axp209.dtsi
>>> +++ b/arch/arm/boot/dts/axp209.dtsi
>>> @@ -53,6 +53,11 @@
>>>         interrupt-controller;
>>>         #interrupt-cells = <1>;
>>>
>>> +       axp209_adc: axp209_adc {
>>
>> Node name should be generic. Please change it to "adc".
>>
>
> OK, do I keep the label as is?
>
> axp209_adc: adc {

Yup. The label is for dereferencing and stuff, and exists in a global scope.
You wouldn't want 2 label names clashing.

ChenYu

>
> Thanks,
> Quentin
>
> --
> Quentin Schulz, Free Electrons
> Embedded Linux and Kernel engineering
> http://free-electrons.com

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

Thread

Re: [PATCH 05/22] ARM: dtsi: axp209: add AXP209 ADC subnode Chen-Yu Tsai <wens@csie.org> - 2017-01-05 08:10 +0100
  Re: [PATCH 05/22] ARM: dtsi: axp209: add AXP209 ADC subnode Quentin Schulz <quentin.schulz@free-electrons.com> - 2017-01-05 09:20 +0100
    Re: [PATCH 05/22] ARM: dtsi: axp209: add AXP209 ADC subnode Chen-Yu Tsai <wens@csie.org> - 2017-01-05 10:10 +0100

csiph-web