mbox series

[0/4] Add I2C controller support for MT6797 SoC

Message ID 20200222162444.11590-1-manivannan.sadhasivam@linaro.org (mailing list archive)
Headers show
Series Add I2C controller support for MT6797 SoC | expand

Message

Manivannan Sadhasivam Feb. 22, 2020, 4:24 p.m. UTC
Hello,

This patchset adds I2C controller support for Mediatek MT6797 SoC. There
are a total of 8 I2C controllers in this SoC (2 being shared) and they are
same as the controllers present in MT6577 SoC. Hence, the driver support is
added with DT fallback method.

As per the datasheet, there are controllers with _imm prefix like i2c2_imm
and i2c3_imm. These appears to be in different memory regions but sharing
the same pins with i2c2 and i2c3 respectively. Since there is no clear
evidence of what they really are, I've adapted the numbering/naming scheme
from the downstream code by Mediatek.

This patchset has been tested on 96Boards X20 development board.

Thanks,
Mani

Manivannan Sadhasivam (4):
  dt-bindings: i2c: Document I2C controller binding for MT6797 SoC
  arm64: dts: mediatek: Add I2C support for MT6797 SoC
  arm64: dts: mediatek: Enable I2C support for 96Boards X20 Development
    board
  arm64: dts: mediatek: Switch to SPDX license identifier for MT6797 SoC

 .../devicetree/bindings/i2c/i2c-mt65xx.txt    |   1 +
 .../boot/dts/mediatek/mt6797-x20-dev.dts      |  49 ++++
 arch/arm64/boot/dts/mediatek/mt6797.dtsi      | 229 +++++++++++++++++-
 3 files changed, 271 insertions(+), 8 deletions(-)

Comments

Manivannan Sadhasivam May 12, 2020, 2:12 p.m. UTC | #1
Hi Matthias,

On Sat, Feb 22, 2020 at 09:54:40PM +0530, Manivannan Sadhasivam wrote:
> Hello,
> 
> This patchset adds I2C controller support for Mediatek MT6797 SoC. There
> are a total of 8 I2C controllers in this SoC (2 being shared) and they are
> same as the controllers present in MT6577 SoC. Hence, the driver support is
> added with DT fallback method.
> 
> As per the datasheet, there are controllers with _imm prefix like i2c2_imm
> and i2c3_imm. These appears to be in different memory regions but sharing
> the same pins with i2c2 and i2c3 respectively. Since there is no clear
> evidence of what they really are, I've adapted the numbering/naming scheme
> from the downstream code by Mediatek.
> 
> This patchset has been tested on 96Boards X20 development board.
> 

Looks like this series has slipped through the cracks...

Thanks,
Mani

> Thanks,
> Mani
> 
> Manivannan Sadhasivam (4):
>   dt-bindings: i2c: Document I2C controller binding for MT6797 SoC
>   arm64: dts: mediatek: Add I2C support for MT6797 SoC
>   arm64: dts: mediatek: Enable I2C support for 96Boards X20 Development
>     board
>   arm64: dts: mediatek: Switch to SPDX license identifier for MT6797 SoC
> 
>  .../devicetree/bindings/i2c/i2c-mt65xx.txt    |   1 +
>  .../boot/dts/mediatek/mt6797-x20-dev.dts      |  49 ++++
>  arch/arm64/boot/dts/mediatek/mt6797.dtsi      | 229 +++++++++++++++++-
>  3 files changed, 271 insertions(+), 8 deletions(-)
> 
> -- 
> 2.17.1
>
Matthias Brugger May 16, 2020, 3:05 p.m. UTC | #2
On 22/02/2020 17:24, Manivannan Sadhasivam wrote:
> Hello,
> 
> This patchset adds I2C controller support for Mediatek MT6797 SoC. There
> are a total of 8 I2C controllers in this SoC (2 being shared) and they are
> same as the controllers present in MT6577 SoC. Hence, the driver support is
> added with DT fallback method.
> 
> As per the datasheet, there are controllers with _imm prefix like i2c2_imm
> and i2c3_imm. These appears to be in different memory regions but sharing
> the same pins with i2c2 and i2c3 respectively. Since there is no clear
> evidence of what they really are, I've adapted the numbering/naming scheme
> from the downstream code by Mediatek.
> 
> This patchset has been tested on 96Boards X20 development board.
> 
> Thanks,
> Mani
> 
> Manivannan Sadhasivam (4):
>   dt-bindings: i2c: Document I2C controller binding for MT6797 SoC
>   arm64: dts: mediatek: Add I2C support for MT6797 SoC
>   arm64: dts: mediatek: Enable I2C support for 96Boards X20 Development
>     board
>   arm64: dts: mediatek: Switch to SPDX license identifier for MT6797 SoC

All four queued now in v5.7-next/dts64

Thanks!

> 
>  .../devicetree/bindings/i2c/i2c-mt65xx.txt    |   1 +
>  .../boot/dts/mediatek/mt6797-x20-dev.dts      |  49 ++++
>  arch/arm64/boot/dts/mediatek/mt6797.dtsi      | 229 +++++++++++++++++-
>  3 files changed, 271 insertions(+), 8 deletions(-)
>