* [PATCH 1/1] of/documentation: Update s5m8767-regulator bindings document
@ 2013-06-24 9:36 Sachin Kamat
2013-06-24 15:25 ` Mark Brown
2013-07-03 9:55 ` Mark Brown
0 siblings, 2 replies; 6+ messages in thread
From: Sachin Kamat @ 2013-06-24 9:36 UTC (permalink / raw)
To: devicetree-discuss
Cc: linux-doc, linux-kernel, grant.likely, rob, broonie, sachin.kamat,
patches
s5m8767 regulator is used on Exynos platforms which use pin controller
to configure GPIOs. Update the example accordingly.
Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
---
.../bindings/regulator/s5m8767-regulator.txt | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/Documentation/devicetree/bindings/regulator/s5m8767-regulator.txt b/Documentation/devicetree/bindings/regulator/s5m8767-regulator.txt
index a35ff99..7364f71 100644
--- a/Documentation/devicetree/bindings/regulator/s5m8767-regulator.txt
+++ b/Documentation/devicetree/bindings/regulator/s5m8767-regulator.txt
@@ -103,13 +103,13 @@ Example:
s5m8767,pmic-buck-default-dvs-idx = <0>;
- s5m8767,pmic-buck-dvs-gpios = <&gpx0 0 1 0 0>, /* DVS1 */
- <&gpx0 1 1 0 0>, /* DVS2 */
- <&gpx0 2 1 0 0>; /* DVS3 */
+ s5m8767,pmic-buck-dvs-gpios = <&gpx0 0 0>, /* DVS1 */
+ <&gpx0 1 0>, /* DVS2 */
+ <&gpx0 2 0>; /* DVS3 */
- s5m8767,pmic-buck-ds-gpios = <&gpx2 3 1 0 0>, /* SET1 */
- <&gpx2 4 1 0 0>, /* SET2 */
- <&gpx2 5 1 0 0>; /* SET3 */
+ s5m8767,pmic-buck-ds-gpios = <&gpx2 3 0>, /* SET1 */
+ <&gpx2 4 0>, /* SET2 */
+ <&gpx2 5 0>; /* SET3 */
s5m8767,pmic-buck2-dvs-voltage = <1350000>, <1300000>,
<1250000>, <1200000>,
--
1.7.9.5
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH 1/1] of/documentation: Update s5m8767-regulator bindings document
2013-06-24 9:36 [PATCH 1/1] of/documentation: Update s5m8767-regulator bindings document Sachin Kamat
@ 2013-06-24 15:25 ` Mark Brown
2013-06-25 6:26 ` Sachin Kamat
2013-07-03 9:55 ` Mark Brown
1 sibling, 1 reply; 6+ messages in thread
From: Mark Brown @ 2013-06-24 15:25 UTC (permalink / raw)
To: Sachin Kamat
Cc: devicetree-discuss, linux-doc, linux-kernel, grant.likely, rob,
patches
[-- Attachment #1: Type: text/plain, Size: 287 bytes --]
On Mon, Jun 24, 2013 at 03:06:57PM +0530, Sachin Kamat wrote:
> s5m8767 regulator is used on Exynos platforms which use pin controller
> to configure GPIOs. Update the example accordingly.
This smells bad, why does a driver using GPIOs through the GPIO API see
a change in the binding?
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 1/1] of/documentation: Update s5m8767-regulator bindings document
2013-06-24 15:25 ` Mark Brown
@ 2013-06-25 6:26 ` Sachin Kamat
2013-06-28 11:35 ` Mark Brown
0 siblings, 1 reply; 6+ messages in thread
From: Sachin Kamat @ 2013-06-25 6:26 UTC (permalink / raw)
To: Mark Brown
Cc: devicetree-discuss, linux-doc, linux-kernel, grant.likely, rob,
patches
On 24 June 2013 20:55, Mark Brown <broonie@kernel.org> wrote:
> On Mon, Jun 24, 2013 at 03:06:57PM +0530, Sachin Kamat wrote:
>> s5m8767 regulator is used on Exynos platforms which use pin controller
>> to configure GPIOs. Update the example accordingly.
>
> This smells bad, why does a driver using GPIOs through the GPIO API see
> a change in the binding?
There is no change in the bindings, but just a correction in the
documentation to reflect the
implementation. Earlier when Samsung platforms did not have pinctrl
driver, legacy GPIO driver
was used which took those 5 parameters. Now since we are using
pinctrl, we need only 3 parameters.
The document was somehow not updated to reflect this change.
--
With warm regards,
Sachin
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 1/1] of/documentation: Update s5m8767-regulator bindings document
2013-06-25 6:26 ` Sachin Kamat
@ 2013-06-28 11:35 ` Mark Brown
2013-06-30 5:01 ` Sachin Kamat
0 siblings, 1 reply; 6+ messages in thread
From: Mark Brown @ 2013-06-28 11:35 UTC (permalink / raw)
To: Sachin Kamat
Cc: devicetree-discuss, linux-doc, linux-kernel, grant.likely, rob,
patches
[-- Attachment #1: Type: text/plain, Size: 522 bytes --]
On Tue, Jun 25, 2013 at 11:56:12AM +0530, Sachin Kamat wrote:
> There is no change in the bindings, but just a correction in the
> documentation to reflect the
> implementation. Earlier when Samsung platforms did not have pinctrl
> driver, legacy GPIO driver
> was used which took those 5 parameters. Now since we are using
> pinctrl, we need only 3 parameters.
> The document was somehow not updated to reflect this change.
So there was a previous change to the code that mistakenly didn't update
the binding document?
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 1/1] of/documentation: Update s5m8767-regulator bindings document
2013-06-28 11:35 ` Mark Brown
@ 2013-06-30 5:01 ` Sachin Kamat
0 siblings, 0 replies; 6+ messages in thread
From: Sachin Kamat @ 2013-06-30 5:01 UTC (permalink / raw)
To: Mark Brown
Cc: devicetree-discuss, linux-doc, linux-kernel, grant.likely, rob,
patches
On 28 June 2013 17:05, Mark Brown <broonie@kernel.org> wrote:
> On Tue, Jun 25, 2013 at 11:56:12AM +0530, Sachin Kamat wrote:
>
>> There is no change in the bindings, but just a correction in the
>> documentation to reflect the
>> implementation. Earlier when Samsung platforms did not have pinctrl
>> driver, legacy GPIO driver
>> was used which took those 5 parameters. Now since we are using
>> pinctrl, we need only 3 parameters.
>> The document was somehow not updated to reflect this change.
>
> So there was a previous change to the code that mistakenly didn't update
> the binding document?
Yes, that is correct. In fact the changes happened during the review
process itself. Hence there is no commit history of this change.
However the binding document was not updated during the review phase
and the first version itself got committed as it is.
--
With warm regards,
Sachin
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 1/1] of/documentation: Update s5m8767-regulator bindings document
2013-06-24 9:36 [PATCH 1/1] of/documentation: Update s5m8767-regulator bindings document Sachin Kamat
2013-06-24 15:25 ` Mark Brown
@ 2013-07-03 9:55 ` Mark Brown
1 sibling, 0 replies; 6+ messages in thread
From: Mark Brown @ 2013-07-03 9:55 UTC (permalink / raw)
To: Sachin Kamat
Cc: devicetree-discuss, linux-doc, linux-kernel, grant.likely, rob,
patches
[-- Attachment #1: Type: text/plain, Size: 299 bytes --]
On Mon, Jun 24, 2013 at 03:06:57PM +0530, Sachin Kamat wrote:
> s5m8767 regulator is used on Exynos platforms which use pin controller
> to configure GPIOs. Update the example accordingly.
Applied, thanks. Please use subject lines that match the subsystem and
try to make your changelogs clearer.
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2013-07-03 9:55 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-06-24 9:36 [PATCH 1/1] of/documentation: Update s5m8767-regulator bindings document Sachin Kamat
2013-06-24 15:25 ` Mark Brown
2013-06-25 6:26 ` Sachin Kamat
2013-06-28 11:35 ` Mark Brown
2013-06-30 5:01 ` Sachin Kamat
2013-07-03 9:55 ` Mark Brown
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).