| Message ID | 20220202153528.707185-1-wenst@chromium.org |
|---|---|
| State | Accepted |
| Commit | c76eeb14ec4e645a23ed8d627c7e38eca048c527 |
| Headers | show |
| Series | dt-bindings: pinctrl: mt8195: fix bias-pull-{up,down} checks | expand |
On Wed, 02 Feb 2022 23:35:28 +0800, Chen-Yu Tsai wrote: > When the constraints and description for bias-pull-{up,down} were added, > the constraints were not indented correctly, resulting in them being > parsed as part of the description. This effectively nullified their > purpose. > > Move the constraints out of the description block, make each description > part of the same associative array as the enum its describing, and > reindent them correctly so they take effect. > > Also add "type: boolean" to the list of valid values. This corresponds > to having bias-pull-{up,down} without any arguments. > > Fixes: 91e7edceda96 ("dt-bindings: pinctrl: mt8195: change pull up/down description") > Signed-off-by: Chen-Yu Tsai <wenst@chromium.org> > --- > .../bindings/pinctrl/pinctrl-mt8195.yaml | 30 ++++++++++--------- > 1 file changed, 16 insertions(+), 14 deletions(-) > Reviewed-by: Rob Herring <robh@kernel.org>
On Wed, Feb 2, 2022 at 4:35 PM Chen-Yu Tsai <wenst@chromium.org> wrote: > When the constraints and description for bias-pull-{up,down} were added, > the constraints were not indented correctly, resulting in them being > parsed as part of the description. This effectively nullified their > purpose. > > Move the constraints out of the description block, make each description > part of the same associative array as the enum its describing, and > reindent them correctly so they take effect. > > Also add "type: boolean" to the list of valid values. This corresponds > to having bias-pull-{up,down} without any arguments. > > Fixes: 91e7edceda96 ("dt-bindings: pinctrl: mt8195: change pull up/down description") > Signed-off-by: Chen-Yu Tsai <wenst@chromium.org> Patch applied! Yours, Linus Walleij
diff --git a/Documentation/devicetree/bindings/pinctrl/pinctrl-mt8195.yaml b/Documentation/devicetree/bindings/pinctrl/pinctrl-mt8195.yaml index 328ea59c5466..8299662c2c09 100644 --- a/Documentation/devicetree/bindings/pinctrl/pinctrl-mt8195.yaml +++ b/Documentation/devicetree/bindings/pinctrl/pinctrl-mt8195.yaml @@ -99,6 +99,14 @@ patternProperties: enum: [2, 4, 6, 8, 10, 12, 14, 16] bias-pull-down: + oneOf: + - type: boolean + - enum: [100, 101, 102, 103] + description: mt8195 pull down PUPD/R0/R1 type define value. + - enum: [200, 201, 202, 203, 204, 205, 206, 207] + description: mt8195 pull down RSEL type define value. + - enum: [75000, 5000] + description: mt8195 pull down RSEL type si unit value(ohm). description: | For pull down type is normal, it don't need add RSEL & R1R0 define and resistance value. @@ -115,13 +123,6 @@ patternProperties: & "MTK_PULL_SET_RSEL_110" & "MTK_PULL_SET_RSEL_111" define in mt8195. It can also support resistance value(ohm) "75000" & "5000" in mt8195. - oneOf: - - enum: [100, 101, 102, 103] - - description: mt8195 pull down PUPD/R0/R1 type define value. - - enum: [200, 201, 202, 203, 204, 205, 206, 207] - - description: mt8195 pull down RSEL type define value. - - enum: [75000, 5000] - - description: mt8195 pull down RSEL type si unit value(ohm). An example of using RSEL define: pincontroller { @@ -146,6 +147,14 @@ patternProperties: }; bias-pull-up: + oneOf: + - type: boolean + - enum: [100, 101, 102, 103] + description: mt8195 pull up PUPD/R0/R1 type define value. + - enum: [200, 201, 202, 203, 204, 205, 206, 207] + description: mt8195 pull up RSEL type define value. + - enum: [1000, 1500, 2000, 3000, 4000, 5000, 10000, 75000] + description: mt8195 pull up RSEL type si unit value(ohm). description: | For pull up type is normal, it don't need add RSEL & R1R0 define and resistance value. @@ -163,13 +172,6 @@ patternProperties: define in mt8195. It can also support resistance value(ohm) "1000" & "1500" & "2000" & "3000" & "4000" & "5000" & "10000" & "75000" in mt8195. - oneOf: - - enum: [100, 101, 102, 103] - - description: mt8195 pull up PUPD/R0/R1 type define value. - - enum: [200, 201, 202, 203, 204, 205, 206, 207] - - description: mt8195 pull up RSEL type define value. - - enum: [1000, 1500, 2000, 3000, 4000, 5000, 10000, 75000] - - description: mt8195 pull up RSEL type si unit value(ohm). An example of using RSEL define: pincontroller { i2c0-pins {
When the constraints and description for bias-pull-{up,down} were added, the constraints were not indented correctly, resulting in them being parsed as part of the description. This effectively nullified their purpose. Move the constraints out of the description block, make each description part of the same associative array as the enum its describing, and reindent them correctly so they take effect. Also add "type: boolean" to the list of valid values. This corresponds to having bias-pull-{up,down} without any arguments. Fixes: 91e7edceda96 ("dt-bindings: pinctrl: mt8195: change pull up/down description") Signed-off-by: Chen-Yu Tsai <wenst@chromium.org> --- .../bindings/pinctrl/pinctrl-mt8195.yaml | 30 ++++++++++--------- 1 file changed, 16 insertions(+), 14 deletions(-)