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


Groups > linux.kernel > #1256895 > unrolled thread

[PATCH v3 3/4] mfd: arizona: Remove bindings covered in new subsystem specific docs

Started byCharles Keepax <ckeepax@opensource.wolfsonmicro.com>
First post2015-10-27 17:20 +0100
Last post2015-10-28 10:20 +0100
Articles 4 — 3 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  [PATCH v3 3/4] mfd: arizona: Remove bindings covered in new subsystem specific docs Charles Keepax <ckeepax@opensource.wolfsonmicro.com> - 2015-10-27 17:20 +0100
    Re: [PATCH v3 3/4] mfd: arizona: Remove bindings covered in new  subsystem specific docs Lee Jones <lee.jones@linaro.org> - 2015-10-28 10:10 +0100
      Re: [PATCH v3 3/4] mfd: arizona: Remove bindings covered in new  subsystem specific docs Lee Jones <lee.jones@linaro.org> - 2015-10-28 10:20 +0100
      Re: [PATCH v3 3/4] mfd: arizona: Remove bindings covered in new  subsystem specific docs Chanwoo Choi <cw00.choi@samsung.com> - 2015-10-28 10:20 +0100

#1256895 — [PATCH v3 3/4] mfd: arizona: Remove bindings covered in new subsystem specific docs

FromCharles Keepax <ckeepax@opensource.wolfsonmicro.com>
Date2015-10-27 17:20 +0100
Subject[PATCH v3 3/4] mfd: arizona: Remove bindings covered in new subsystem specific docs
Message-ID<qof8B-7mI-7@gated-at.bofh.it>
Binding documentation has been added in new subsystem specific binding
documents. This patch removes the now duplicate documentation from the
MFD documentation.

Signed-off-by: Charles Keepax <ckeepax@opensource.wolfsonmicro.com>
---

Changes since v2:
 - No longer moving the core regulators into the regulator binding document,
   as they are properties of the device as a whole not just the regulators
 - Moved the maintainer updates into a seperate patch

Lee, I have removed your ack because the patch has changed a fair amount.
Apologies if this would not have been your preference.

Thanks,
Charles

 Documentation/devicetree/bindings/mfd/arizona.txt |   15 +++------------
 1 files changed, 3 insertions(+), 12 deletions(-)

diff --git a/Documentation/devicetree/bindings/mfd/arizona.txt b/Documentation/devicetree/bindings/mfd/arizona.txt
index a8fee60..18be0cb 100644
--- a/Documentation/devicetree/bindings/mfd/arizona.txt
+++ b/Documentation/devicetree/bindings/mfd/arizona.txt
@@ -44,7 +44,6 @@ Required properties:
 Optional properties:
 
   - wlf,reset : GPIO specifier for the GPIO controlling /RESET
-  - wlf,ldoena : GPIO specifier for the GPIO controlling LDOENA
 
   - wlf,gpio-defaults : A list of GPIO configuration register values. Defines
     for the appropriate values can found in <dt-bindings/mfd/arizona.txt>. If
@@ -67,21 +66,13 @@ Optional properties:
     present, the number of values should be less than or equal to the
     number of inputs, unspecified inputs will use the chip default.
 
-  - wlf,hpdet-channel : Headphone detection channel.
-    ARIZONA_ACCDET_MODE_HPL or 1 - Headphone detect mode is set to HPDETL
-    ARIZONA_ACCDET_MODE_HPR or 2 - Headphone detect mode is set to HPDETR
-    If this node is not mentioned or if the value is unknown, then
-    headphone detection mode is set to HPDETL.
-
   - DCVDD-supply, MICVDD-supply : Power supplies, only need to be specified if
     they are being externally supplied. As covered in
     Documentation/devicetree/bindings/regulator/regulator.txt
 
-Optional subnodes:
-  - ldo1 : Initial data for the LDO1 regulator, as covered in
-    Documentation/devicetree/bindings/regulator/regulator.txt
-  - micvdd : Initial data for the MICVDD regulator, as covered in
-    Documentation/devicetree/bindings/regulator/regulator.txt
+Also see child specific device properties:
+  Regulator - ../regulator/arizona-regulator.txt
+  Extcon    - ../extcon/extcon-arizona.txt
 
 Example:
 
-- 
1.7.2.5

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

[toc] | [next] | [standalone]


#1257818 — Re: [PATCH v3 3/4] mfd: arizona: Remove bindings covered in new subsystem specific docs

FromLee Jones <lee.jones@linaro.org>
Date2015-10-28 10:10 +0100
SubjectRe: [PATCH v3 3/4] mfd: arizona: Remove bindings covered in new subsystem specific docs
Message-ID<qouU1-EQ-1@gated-at.bofh.it>
In reply to#1256895
On Tue, 27 Oct 2015, Charles Keepax wrote:

> Binding documentation has been added in new subsystem specific binding
> documents. This patch removes the now duplicate documentation from the
> MFD documentation.
> 
> Signed-off-by: Charles Keepax <ckeepax@opensource.wolfsonmicro.com>
> ---
> 
> Changes since v2:
>  - No longer moving the core regulators into the regulator binding document,
>    as they are properties of the device as a whole not just the regulators
>  - Moved the maintainer updates into a seperate patch
> 
> Lee, I have removed your ack because the patch has changed a fair amount.
> Apologies if this would not have been your preference.

That was the right thing to do.

> Thanks,
> Charles
> 
>  Documentation/devicetree/bindings/mfd/arizona.txt |   15 +++------------
>  1 files changed, 3 insertions(+), 12 deletions(-)

Applied, thanks.

> diff --git a/Documentation/devicetree/bindings/mfd/arizona.txt b/Documentation/devicetree/bindings/mfd/arizona.txt
> index a8fee60..18be0cb 100644
> --- a/Documentation/devicetree/bindings/mfd/arizona.txt
> +++ b/Documentation/devicetree/bindings/mfd/arizona.txt
> @@ -44,7 +44,6 @@ Required properties:
>  Optional properties:
>  
>    - wlf,reset : GPIO specifier for the GPIO controlling /RESET
> -  - wlf,ldoena : GPIO specifier for the GPIO controlling LDOENA
>  
>    - wlf,gpio-defaults : A list of GPIO configuration register values. Defines
>      for the appropriate values can found in <dt-bindings/mfd/arizona.txt>. If
> @@ -67,21 +66,13 @@ Optional properties:
>      present, the number of values should be less than or equal to the
>      number of inputs, unspecified inputs will use the chip default.
>  
> -  - wlf,hpdet-channel : Headphone detection channel.
> -    ARIZONA_ACCDET_MODE_HPL or 1 - Headphone detect mode is set to HPDETL
> -    ARIZONA_ACCDET_MODE_HPR or 2 - Headphone detect mode is set to HPDETR
> -    If this node is not mentioned or if the value is unknown, then
> -    headphone detection mode is set to HPDETL.
> -
>    - DCVDD-supply, MICVDD-supply : Power supplies, only need to be specified if
>      they are being externally supplied. As covered in
>      Documentation/devicetree/bindings/regulator/regulator.txt
>  
> -Optional subnodes:
> -  - ldo1 : Initial data for the LDO1 regulator, as covered in
> -    Documentation/devicetree/bindings/regulator/regulator.txt
> -  - micvdd : Initial data for the MICVDD regulator, as covered in
> -    Documentation/devicetree/bindings/regulator/regulator.txt
> +Also see child specific device properties:
> +  Regulator - ../regulator/arizona-regulator.txt
> +  Extcon    - ../extcon/extcon-arizona.txt
>  
>  Example:
>  

-- 
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]


#1257834 — Re: [PATCH v3 3/4] mfd: arizona: Remove bindings covered in new subsystem specific docs

FromLee Jones <lee.jones@linaro.org>
Date2015-10-28 10:20 +0100
SubjectRe: [PATCH v3 3/4] mfd: arizona: Remove bindings covered in new subsystem specific docs
Message-ID<qov3I-If-29@gated-at.bofh.it>
In reply to#1257818
On Wed, 28 Oct 2015, Chanwoo Choi wrote:
> On 2015년 10월 28일 18:03, Lee Jones wrote:
> > On Tue, 27 Oct 2015, Charles Keepax wrote:
> > 
> >> Binding documentation has been added in new subsystem specific binding
> >> documents. This patch removes the now duplicate documentation from the
> >> MFD documentation.
> >>
> >> Signed-off-by: Charles Keepax <ckeepax@opensource.wolfsonmicro.com>
> >> ---
> >>
> >> Changes since v2:
> >>  - No longer moving the core regulators into the regulator binding document,
> >>    as they are properties of the device as a whole not just the regulators
> >>  - Moved the maintainer updates into a seperate patch
> >>
> >> Lee, I have removed your ack because the patch has changed a fair amount.
> >> Apologies if this would not have been your preference.
> > 
> > That was the right thing to do.
> > 
> >> Thanks,
> >> Charles
> >>
> >>  Documentation/devicetree/bindings/mfd/arizona.txt |   15 +++------------
> >>  1 files changed, 3 insertions(+), 12 deletions(-)
> > 
> > Applied, thanks.
> 
> If you possible, could you pick patch1[1] which has already the acked tag of me?
> [1] [PATCH v3 1/4] extcon: arizona: Add extcon specific device tree binding document

Yes, no problem.

-- 
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]


#1257835 — Re: [PATCH v3 3/4] mfd: arizona: Remove bindings covered in new subsystem specific docs

FromChanwoo Choi <cw00.choi@samsung.com>
Date2015-10-28 10:20 +0100
SubjectRe: [PATCH v3 3/4] mfd: arizona: Remove bindings covered in new subsystem specific docs
Message-ID<qov3I-If-31@gated-at.bofh.it>
In reply to#1257818
Hi Lee,

On 2015년 10월 28일 18:03, Lee Jones wrote:
> On Tue, 27 Oct 2015, Charles Keepax wrote:
> 
>> Binding documentation has been added in new subsystem specific binding
>> documents. This patch removes the now duplicate documentation from the
>> MFD documentation.
>>
>> Signed-off-by: Charles Keepax <ckeepax@opensource.wolfsonmicro.com>
>> ---
>>
>> Changes since v2:
>>  - No longer moving the core regulators into the regulator binding document,
>>    as they are properties of the device as a whole not just the regulators
>>  - Moved the maintainer updates into a seperate patch
>>
>> Lee, I have removed your ack because the patch has changed a fair amount.
>> Apologies if this would not have been your preference.
> 
> That was the right thing to do.
> 
>> Thanks,
>> Charles
>>
>>  Documentation/devicetree/bindings/mfd/arizona.txt |   15 +++------------
>>  1 files changed, 3 insertions(+), 12 deletions(-)
> 
> Applied, thanks.

If you possible, could you pick patch1[1] which has already the acked tag of me?
[1] [PATCH v3 1/4] extcon: arizona: Add extcon specific device tree binding document

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

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web