[2/2] dt-bindings: input: Update documentation for clps711x-keypad
diff mbox series

Message ID 20181222155520.8156-1-shc_work@mail.ru
State New
Headers show
Series
  • [1/2] input: clps711x-keypad: Switch to use syscon_regmap_lookup_by_phandle()
Related show

Commit Message

Alexander Shiyan Dec. 22, 2018, 3:55 p.m. UTC
Since the driver switched to using the syscon_regmap_lookup_by_phandle()
function, we need to specify the phandle for the driver. This patch
updates the driver binding documentation for these changes.

Signed-off-by: Alexander Shiyan <shc_work@mail.ru>
---
 Documentation/devicetree/bindings/input/clps711x-keypad.txt | 2 ++
 1 file changed, 2 insertions(+)

Comments

Rob Herring Jan. 3, 2019, 8:59 p.m. UTC | #1
On Sat, Dec 22, 2018 at 06:55:20PM +0300, Alexander Shiyan wrote:
> Since the driver switched to using the syscon_regmap_lookup_by_phandle()
> function, we need to specify the phandle for the driver. This patch
> updates the driver binding documentation for these changes.

That's not a backwards compatible change. Is that okay for all users of 
this platform(s)? If so, say why.

> 
> Signed-off-by: Alexander Shiyan <shc_work@mail.ru>
> ---
>  Documentation/devicetree/bindings/input/clps711x-keypad.txt | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/input/clps711x-keypad.txt b/Documentation/devicetree/bindings/input/clps711x-keypad.txt
> index 3eed881..aebedc3 100644
> --- a/Documentation/devicetree/bindings/input/clps711x-keypad.txt
> +++ b/Documentation/devicetree/bindings/input/clps711x-keypad.txt
> @@ -6,6 +6,7 @@ Required Properties:
>  - poll-interval: Poll interval time in milliseconds.
>  - linux,keymap:  The definition can be found at
>                   bindings/input/matrix-keymap.txt.
> +- syscon:        Phandle that points to the CPU syscon1 node.

What does this provide? If the keypad registers are a sub-block of the 
syscon1, then you should make this node a child node.

>  
>  Optional Properties:
>  - autorepeat:    Enable autorepeat feature.
> @@ -17,6 +18,7 @@ Example:
>  		poll-interval = <120>;
>  		row-gpios = <&porta 0 0>,
>  			    <&porta 1 0>;
> +		syscon = <&syscon1>;
>  
>  		linux,keymap = <
>  			MATRIX_KEY(0, 0, KEY_UP)
> -- 
> 2.10.2
>
Alexander Shiyan Jan. 16, 2019, 12:26 p.m. UTC | #2
>Четверг,  3 января 2019, 23:59 +03:00 от Rob Herring <robh@kernel.org>:
>
>On Sat, Dec 22, 2018 at 06:55:20PM +0300, Alexander Shiyan wrote:
>> Since the driver switched to using the syscon_regmap_lookup_by_phandle()
>> function, we need to specify the phandle for the driver. This patch
>> updates the driver binding documentation for these changes.
...
>> diff --git a/Documentation/devicetree/bindings/input/clps711x-keypad.txt b/Documentation/devicetree/bindings/input/clps711x-keypad.txt
>> index 3eed881..aebedc3 100644
>> --- a/Documentation/devicetree/bindings/input/clps711x-keypad.txt
>> +++ b/Documentation/devicetree/bindings/input/clps711x-keypad.txt
>> @@ -6,6 +6,7 @@ Required Properties:
>>  - poll-interval: Poll interval time in milliseconds.
>>  - linux,keymap:  The definition can be found at
>>                   bindings/input/matrix-keymap.txt.
>> +- syscon:        Phandle that points to the CPU syscon1 node.
>
>What does this provide? If the keypad registers are a sub-block of the 
>syscon1, then you should make this node a child node.

Syscon does not populate child devices.

---

Patch
diff mbox series

diff --git a/Documentation/devicetree/bindings/input/clps711x-keypad.txt b/Documentation/devicetree/bindings/input/clps711x-keypad.txt
index 3eed881..aebedc3 100644
--- a/Documentation/devicetree/bindings/input/clps711x-keypad.txt
+++ b/Documentation/devicetree/bindings/input/clps711x-keypad.txt
@@ -6,6 +6,7 @@  Required Properties:
 - poll-interval: Poll interval time in milliseconds.
 - linux,keymap:  The definition can be found at
                  bindings/input/matrix-keymap.txt.
+- syscon:        Phandle that points to the CPU syscon1 node.
 
 Optional Properties:
 - autorepeat:    Enable autorepeat feature.
@@ -17,6 +18,7 @@  Example:
 		poll-interval = <120>;
 		row-gpios = <&porta 0 0>,
 			    <&porta 1 0>;
+		syscon = <&syscon1>;
 
 		linux,keymap = <
 			MATRIX_KEY(0, 0, KEY_UP)