[v2,0/3] 98dx3236 i2c related fixes

Message

Chris Packham Sept. 7, 2020, 9:17 p.m. UTC

I noticed these while adding support for i2c recovery for a couple of our
boards. They date back to when I initially added support for the 98dx3236. They
probably haven't been causing a problem because the HW defaults are correct and
unless you attempt to use the specific pinctrl functions there won't be a
problem.

Change in v2:
- Fix grammo in patch 2/3
- Add r-by from Andrew

Chris Packham (3):
  pinctrl: mvebu: Fix i2c sda definition for 98DX3236
  ARM: dts: Remove non-existent i2c1 from 98dx3236
  ARM: dts: Add i2c0 pinctrl information for 98dx3236

 arch/arm/boot/dts/armada-xp-98dx3236.dtsi | 12 +++++++-----
 drivers/pinctrl/mvebu/pinctrl-armada-xp.c |  2 +-
 2 files changed, 8 insertions(+), 6 deletions(-)

Comments

Gregory CLEMENT Sept. 23, 2020, 2:38 p.m. UTC | #1

Hi Chris,

> I noticed these while adding support for i2c recovery for a couple of our
> boards. They date back to when I initially added support for the 98dx3236. They
> probably haven't been causing a problem because the HW defaults are correct and
> unless you attempt to use the specific pinctrl functions there won't be a
> problem.
>
> Change in v2:
> - Fix grammo in patch 2/3
> - Add r-by from Andrew
>
> Chris Packham (3):
>   pinctrl: mvebu: Fix i2c sda definition for 98DX3236
>   ARM: dts: Remove non-existent i2c1 from 98dx3236
>   ARM: dts: Add i2c0 pinctrl information for 98dx3236

Applied the 2 dts pacthes on mvebu/dt

Thanks,

Gregory


>
>  arch/arm/boot/dts/armada-xp-98dx3236.dtsi | 12 +++++++-----
>  drivers/pinctrl/mvebu/pinctrl-armada-xp.c |  2 +-
>  2 files changed, 8 insertions(+), 6 deletions(-)
>
> -- 
> 2.28.0
>