mbox series

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

Message ID 20200907211712.9697-1-chris.packham@alliedtelesis.co.nz
Headers show
Series 98dx3236 i2c related fixes | expand

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

>


-- 
Gregory Clement, Bootlin
Embedded Linux and Kernel engineering
http://bootlin.com