mbox series

[v1,0/2] MT8192 pinctrl properties adjustments

Message ID 20220525155714.1837360-1-nfraprado@collabora.com (mailing list archive)
Headers show
Series MT8192 pinctrl properties adjustments | expand

Message

Nícolas F. R. A. Prado May 25, 2022, 3:57 p.m. UTC
The two patches in this series substitute properties in the mt8192
pinctrl dt-binding for ones which have a clearer meaning and are more
standardized. At this point there's no DT using the mt8192 pinctrl
binding, so if such changes are to be made, they need to happen now.


Nícolas F. R. A. Prado (2):
  dt-bindings: pinctrl: mt8192: Add drive-strength-microamp
  dt-bindings: pinctrl: mt8192: Use generic bias instead of pull-*-adv

 .../bindings/pinctrl/pinctrl-mt8192.yaml      | 58 ++++++-------------
 1 file changed, 18 insertions(+), 40 deletions(-)

Comments

AngeloGioacchino Del Regno May 30, 2022, 8:34 a.m. UTC | #1
Il 25/05/22 17:57, Nícolas F. R. A. Prado ha scritto:
> 
> The two patches in this series substitute properties in the mt8192
> pinctrl dt-binding for ones which have a clearer meaning and are more
> standardized. At this point there's no DT using the mt8192 pinctrl
> binding, so if such changes are to be made, they need to happen now.
> 
> 
> Nícolas F. R. A. Prado (2):
>    dt-bindings: pinctrl: mt8192: Add drive-strength-microamp
>    dt-bindings: pinctrl: mt8192: Use generic bias instead of pull-*-adv
> 
>   .../bindings/pinctrl/pinctrl-mt8192.yaml      | 58 ++++++-------------
>   1 file changed, 18 insertions(+), 40 deletions(-)
> 

Hey Nic,

As you know, I definitely agree with these changes, but they don't deserve
to get any Fixes tag: backporting should be useless in this case... and you
anyway aren't fixing a faulty binding.

Cheers!
Angelo
Nícolas F. R. A. Prado May 31, 2022, 9:19 p.m. UTC | #2
On Mon, May 30, 2022 at 10:34:19AM +0200, AngeloGioacchino Del Regno wrote:
> Il 25/05/22 17:57, Nícolas F. R. A. Prado ha scritto:
> > 
> > The two patches in this series substitute properties in the mt8192
> > pinctrl dt-binding for ones which have a clearer meaning and are more
> > standardized. At this point there's no DT using the mt8192 pinctrl
> > binding, so if such changes are to be made, they need to happen now.
> > 
> > 
> > Nícolas F. R. A. Prado (2):
> >    dt-bindings: pinctrl: mt8192: Add drive-strength-microamp
> >    dt-bindings: pinctrl: mt8192: Use generic bias instead of pull-*-adv
> > 
> >   .../bindings/pinctrl/pinctrl-mt8192.yaml      | 58 ++++++-------------
> >   1 file changed, 18 insertions(+), 40 deletions(-)
> > 
> 
> Hey Nic,
> 
> As you know, I definitely agree with these changes, but they don't deserve
> to get any Fixes tag: backporting should be useless in this case... and you
> anyway aren't fixing a faulty binding.

Alright, I was on the fence on this one, but I can drop the tags for v2.

Thanks,
Nícolas

> 
> Cheers!
> Angelo