Re: [PATCH 2/2] dt-bindings: power: sbs-battery: re-document "ti,bq20z75"
Hi, On Tue, Jun 12, 2018 at 01:20:42PM -0700, Brian Norris wrote: > This compatible property was documented before the driver was renamed to > "SBS" (see commit e57f1b68c406 ("devicetree-bindings: Propagate > bq20z75->sbs rename to dt bindings")). The driver has continued to > support this property as an alternative to "sbs,sbs-battery", and > because we've noticed there are some lingering TI specifics (in the > manufacturer-specific portion of the SBS spec), we'd like to start using > this property again to differentiate. > > In typical DT fashion, the , specifics should be > used ahead of the generic "sbs,sbs-battery" string, so we can handle > vendor specifics -- so document this. Language borrowed mostly from > Documentation/devicetree/bindings/power/supply/sbs_sbs-charger.txt > > Also fixup the example to use this property (it's already implying that > it's "bq20z75@b"); fixup the node name to be generic ("battery", not > ""); and fixup some whitespace. > > Signed-off-by: Brian Norris > Acked-by: Rhyland Klein > Reviewed-by: Rob Herring > --- Thanks, queued to power-supply-next. -- Sebastian signature.asc Description: PGP signature
Re: [PATCH 2/2] dt-bindings: power: sbs-battery: re-document "ti,bq20z75"
Hi, On Tue, Jun 12, 2018 at 01:20:42PM -0700, Brian Norris wrote: > This compatible property was documented before the driver was renamed to > "SBS" (see commit e57f1b68c406 ("devicetree-bindings: Propagate > bq20z75->sbs rename to dt bindings")). The driver has continued to > support this property as an alternative to "sbs,sbs-battery", and > because we've noticed there are some lingering TI specifics (in the > manufacturer-specific portion of the SBS spec), we'd like to start using > this property again to differentiate. > > In typical DT fashion, the , specifics should be > used ahead of the generic "sbs,sbs-battery" string, so we can handle > vendor specifics -- so document this. Language borrowed mostly from > Documentation/devicetree/bindings/power/supply/sbs_sbs-charger.txt > > Also fixup the example to use this property (it's already implying that > it's "bq20z75@b"); fixup the node name to be generic ("battery", not > ""); and fixup some whitespace. > > Signed-off-by: Brian Norris > Acked-by: Rhyland Klein > Reviewed-by: Rob Herring > --- Thanks, queued to power-supply-next. -- Sebastian signature.asc Description: PGP signature
[PATCH 2/2] dt-bindings: power: sbs-battery: re-document "ti,bq20z75"
This compatible property was documented before the driver was renamed to "SBS" (see commit e57f1b68c406 ("devicetree-bindings: Propagate bq20z75->sbs rename to dt bindings")). The driver has continued to support this property as an alternative to "sbs,sbs-battery", and because we've noticed there are some lingering TI specifics (in the manufacturer-specific portion of the SBS spec), we'd like to start using this property again to differentiate. In typical DT fashion, the , specifics should be used ahead of the generic "sbs,sbs-battery" string, so we can handle vendor specifics -- so document this. Language borrowed mostly from Documentation/devicetree/bindings/power/supply/sbs_sbs-charger.txt Also fixup the example to use this property (it's already implying that it's "bq20z75@b"); fixup the node name to be generic ("battery", not ""); and fixup some whitespace. Signed-off-by: Brian Norris Acked-by: Rhyland Klein Reviewed-by: Rob Herring --- v2: add Rhyland's Acked-by v3: no change v4: add Rob's Reviewed-by v5: * note "sbs,sbs-battery" as a fallback for vendor,part-number * improve the Example * I still kept the {Acked,Reviewed}-by, since the substance of the change is essentially the same. Apologies if that is not proper. --- .../bindings/power/supply/sbs_sbs-battery.txt| 12 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/Documentation/devicetree/bindings/power/supply/sbs_sbs-battery.txt b/Documentation/devicetree/bindings/power/supply/sbs_sbs-battery.txt index c40e8926facf..4e78e51018eb 100644 --- a/Documentation/devicetree/bindings/power/supply/sbs_sbs-battery.txt +++ b/Documentation/devicetree/bindings/power/supply/sbs_sbs-battery.txt @@ -2,7 +2,11 @@ SBS sbs-battery ~~ Required properties : - - compatible : "sbs,sbs-battery" + - compatible: ",", "sbs,sbs-battery" as fallback. The + part number compatible string might be used in order to take care of + vendor specific registers. + Known ,: + ti,bq20z75 Optional properties : - sbs,i2c-retry-count : The number of times to retry i2c transactions on i2c @@ -14,9 +18,9 @@ Optional properties : Example: - bq20z75@b { - compatible = "sbs,sbs-battery"; - reg = < 0xb >; + battery@b { + compatible = "ti,bq20z75", "sbs,sbs-battery"; + reg = <0xb>; sbs,i2c-retry-count = <2>; sbs,poll-retry-count = <10>; sbs,battery-detect-gpios = < 122 1>; -- 2.18.0.rc1.242.g61856ae69a-goog
[PATCH 2/2] dt-bindings: power: sbs-battery: re-document "ti,bq20z75"
This compatible property was documented before the driver was renamed to "SBS" (see commit e57f1b68c406 ("devicetree-bindings: Propagate bq20z75->sbs rename to dt bindings")). The driver has continued to support this property as an alternative to "sbs,sbs-battery", and because we've noticed there are some lingering TI specifics (in the manufacturer-specific portion of the SBS spec), we'd like to start using this property again to differentiate. In typical DT fashion, the , specifics should be used ahead of the generic "sbs,sbs-battery" string, so we can handle vendor specifics -- so document this. Language borrowed mostly from Documentation/devicetree/bindings/power/supply/sbs_sbs-charger.txt Also fixup the example to use this property (it's already implying that it's "bq20z75@b"); fixup the node name to be generic ("battery", not ""); and fixup some whitespace. Signed-off-by: Brian Norris Acked-by: Rhyland Klein Reviewed-by: Rob Herring --- v2: add Rhyland's Acked-by v3: no change v4: add Rob's Reviewed-by v5: * note "sbs,sbs-battery" as a fallback for vendor,part-number * improve the Example * I still kept the {Acked,Reviewed}-by, since the substance of the change is essentially the same. Apologies if that is not proper. --- .../bindings/power/supply/sbs_sbs-battery.txt| 12 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/Documentation/devicetree/bindings/power/supply/sbs_sbs-battery.txt b/Documentation/devicetree/bindings/power/supply/sbs_sbs-battery.txt index c40e8926facf..4e78e51018eb 100644 --- a/Documentation/devicetree/bindings/power/supply/sbs_sbs-battery.txt +++ b/Documentation/devicetree/bindings/power/supply/sbs_sbs-battery.txt @@ -2,7 +2,11 @@ SBS sbs-battery ~~ Required properties : - - compatible : "sbs,sbs-battery" + - compatible: ",", "sbs,sbs-battery" as fallback. The + part number compatible string might be used in order to take care of + vendor specific registers. + Known ,: + ti,bq20z75 Optional properties : - sbs,i2c-retry-count : The number of times to retry i2c transactions on i2c @@ -14,9 +18,9 @@ Optional properties : Example: - bq20z75@b { - compatible = "sbs,sbs-battery"; - reg = < 0xb >; + battery@b { + compatible = "ti,bq20z75", "sbs,sbs-battery"; + reg = <0xb>; sbs,i2c-retry-count = <2>; sbs,poll-retry-count = <10>; sbs,battery-detect-gpios = < 122 1>; -- 2.18.0.rc1.242.g61856ae69a-goog
Re: [PATCH 2/2] dt-bindings: power: sbs-battery: re-document "ti,bq20z75"
On 5/31/2018 8:32 PM, Brian Norris wrote: > This compatible property was documented before the driver was renamed to > "SBS" (see commit e57f1b68c406 ("devicetree-bindings: Propagate > bq20z75->sbs rename to dt bindings")). The driver has continued to > support this property as an alternative to "sbs,sbs-battery", and > because we've noticed there are some lingering TI specifics (in the > manufacturer-specific portion of the SBS spec), we'd like to start using > this property again to differentiate. > > Cc: Rhyland Klein > Signed-off-by: Brian Norris > --- > .../devicetree/bindings/power/supply/sbs_sbs-battery.txt| 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git > a/Documentation/devicetree/bindings/power/supply/sbs_sbs-battery.txt > b/Documentation/devicetree/bindings/power/supply/sbs_sbs-battery.txt > index c40e8926facf..a7a9c3366f82 100644 > --- a/Documentation/devicetree/bindings/power/supply/sbs_sbs-battery.txt > +++ b/Documentation/devicetree/bindings/power/supply/sbs_sbs-battery.txt > @@ -2,7 +2,7 @@ SBS sbs-battery > ~~ > > Required properties : > - - compatible : "sbs,sbs-battery" > + - compatible : "sbs,sbs-battery" or "ti,bq20z75" > > Optional properties : > - sbs,i2c-retry-count : The number of times to retry i2c transactions on i2c > Acked-by: Rhyland Klein -- nvpublic
Re: [PATCH 2/2] dt-bindings: power: sbs-battery: re-document "ti,bq20z75"
On 5/31/2018 8:32 PM, Brian Norris wrote: > This compatible property was documented before the driver was renamed to > "SBS" (see commit e57f1b68c406 ("devicetree-bindings: Propagate > bq20z75->sbs rename to dt bindings")). The driver has continued to > support this property as an alternative to "sbs,sbs-battery", and > because we've noticed there are some lingering TI specifics (in the > manufacturer-specific portion of the SBS spec), we'd like to start using > this property again to differentiate. > > Cc: Rhyland Klein > Signed-off-by: Brian Norris > --- > .../devicetree/bindings/power/supply/sbs_sbs-battery.txt| 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git > a/Documentation/devicetree/bindings/power/supply/sbs_sbs-battery.txt > b/Documentation/devicetree/bindings/power/supply/sbs_sbs-battery.txt > index c40e8926facf..a7a9c3366f82 100644 > --- a/Documentation/devicetree/bindings/power/supply/sbs_sbs-battery.txt > +++ b/Documentation/devicetree/bindings/power/supply/sbs_sbs-battery.txt > @@ -2,7 +2,7 @@ SBS sbs-battery > ~~ > > Required properties : > - - compatible : "sbs,sbs-battery" > + - compatible : "sbs,sbs-battery" or "ti,bq20z75" > > Optional properties : > - sbs,i2c-retry-count : The number of times to retry i2c transactions on i2c > Acked-by: Rhyland Klein -- nvpublic
[PATCH 2/2] dt-bindings: power: sbs-battery: re-document "ti,bq20z75"
This compatible property was documented before the driver was renamed to "SBS" (see commit e57f1b68c406 ("devicetree-bindings: Propagate bq20z75->sbs rename to dt bindings")). The driver has continued to support this property as an alternative to "sbs,sbs-battery", and because we've noticed there are some lingering TI specifics (in the manufacturer-specific portion of the SBS spec), we'd like to start using this property again to differentiate. Cc: Rhyland Klein Signed-off-by: Brian Norris --- .../devicetree/bindings/power/supply/sbs_sbs-battery.txt| 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/devicetree/bindings/power/supply/sbs_sbs-battery.txt b/Documentation/devicetree/bindings/power/supply/sbs_sbs-battery.txt index c40e8926facf..a7a9c3366f82 100644 --- a/Documentation/devicetree/bindings/power/supply/sbs_sbs-battery.txt +++ b/Documentation/devicetree/bindings/power/supply/sbs_sbs-battery.txt @@ -2,7 +2,7 @@ SBS sbs-battery ~~ Required properties : - - compatible : "sbs,sbs-battery" + - compatible : "sbs,sbs-battery" or "ti,bq20z75" Optional properties : - sbs,i2c-retry-count : The number of times to retry i2c transactions on i2c -- 2.17.0.921.gf22659ad46-goog
[PATCH 2/2] dt-bindings: power: sbs-battery: re-document "ti,bq20z75"
This compatible property was documented before the driver was renamed to "SBS" (see commit e57f1b68c406 ("devicetree-bindings: Propagate bq20z75->sbs rename to dt bindings")). The driver has continued to support this property as an alternative to "sbs,sbs-battery", and because we've noticed there are some lingering TI specifics (in the manufacturer-specific portion of the SBS spec), we'd like to start using this property again to differentiate. Cc: Rhyland Klein Signed-off-by: Brian Norris --- .../devicetree/bindings/power/supply/sbs_sbs-battery.txt| 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/devicetree/bindings/power/supply/sbs_sbs-battery.txt b/Documentation/devicetree/bindings/power/supply/sbs_sbs-battery.txt index c40e8926facf..a7a9c3366f82 100644 --- a/Documentation/devicetree/bindings/power/supply/sbs_sbs-battery.txt +++ b/Documentation/devicetree/bindings/power/supply/sbs_sbs-battery.txt @@ -2,7 +2,7 @@ SBS sbs-battery ~~ Required properties : - - compatible : "sbs,sbs-battery" + - compatible : "sbs,sbs-battery" or "ti,bq20z75" Optional properties : - sbs,i2c-retry-count : The number of times to retry i2c transactions on i2c -- 2.17.0.921.gf22659ad46-goog