[PATCH v2 0/2] i2c-omap: Enable i2c-omap driver for AM654 SoCs

[PATCH v2 0/2] i2c-omap: Enable i2c-omap driver for AM654 SoCs

[Vignesh R]

Couple of patches to enable i2c-omap driver to be used with TI's new
AM654 platforms.


Vignesh R (2):
  dt-bindings: i2c-omap: Add new compatible for AM654 SoCs
  i2c: busses: Kconfig: Enable I2C_OMAP for ARCH_K3

 Documentation/devicetree/bindings/i2c/i2c-omap.txt | 8 ++++++--
 drivers/i2c/busses/Kconfig                         | 2 +-
 2 files changed, 7 insertions(+), 3 deletions(-)

-- 
2.19.0

[PATCH v2 1/2] dt-bindings: i2c-omap: Add new compatible for AM654 SoCs

[Vignesh R]

AM654 SoCs have same I2C IP as OMAP SoCs. Add new compatible to
handle AM654 SoCs. While at that reformat the existing compatible list
for older SoCs to list one valid compatible per line.

Signed-off-by: Vignesh R <vigneshr@ti.com>
---

v2: Reformat compatible existing compatible list.

 Documentation/devicetree/bindings/i2c/i2c-omap.txt | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/Documentation/devicetree/bindings/i2c/i2c-omap.txt b/Documentation/devicetree/bindings/i2c/i2c-omap.txt
index 7e49839d4124..4b90ba9f31b7 100644
--- a/Documentation/devicetree/bindings/i2c/i2c-omap.txt
+++ b/Documentation/devicetree/bindings/i2c/i2c-omap.txt
@@ -1,8 +1,12 @@
 I2C for OMAP platforms
 
 Required properties :
-- compatible : Must be "ti,omap2420-i2c", "ti,omap2430-i2c", "ti,omap3-i2c"
-  or "ti,omap4-i2c"
+- compatible : Must be
+	"ti,omap2420-i2c" for OMAP2420 SoCs
+	"ti,omap2430-i2c" for OMAP2430 SoCs
+	"ti,omap3-i2c" for OMAP3 SoCs
+	"ti,omap4-i2c" for OMAP4+ SoCs
+	"ti,am654-i2c", "ti,omap4-i2c" for AM654 SoCs
 - ti,hwmods : Must be "i2c<n>", n being the instance number (1-based)
 - #address-cells = <1>;
 - #size-cells = <0>;
-- 
2.19.0

[PATCH v2 2/2] i2c: busses: Kconfig: Enable I2C_OMAP for ARCH_K3

[Vignesh R]

Allow I2C_OMAP to be built for K3 platforms.

Signed-off-by: Vignesh R <vigneshr@ti.com>
---

v2: No changes

 drivers/i2c/busses/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/i2c/busses/Kconfig b/drivers/i2c/busses/Kconfig
index 451d4ae50e66..ac4b09642f63 100644
--- a/drivers/i2c/busses/Kconfig
+++ b/drivers/i2c/busses/Kconfig
@@ -751,7 +751,7 @@ config I2C_OCORES
 
 config I2C_OMAP
 	tristate "OMAP I2C adapter"
-	depends on ARCH_OMAP
+	depends on ARCH_OMAP || ARCH_K3
 	default y if MACH_OMAP_H3 || MACH_OMAP_OSK
 	help
 	  If you say yes to this option, support will be included for the
-- 
2.19.0

Re: [PATCH v2 1/2] dt-bindings: i2c-omap: Add new compatible for AM654 SoCs

[Rob Herring]

On Fri, 28 Sep 2018 10:55:56 +0530, Vignesh R wrote:
> AM654 SoCs have same I2C IP as OMAP SoCs. Add new compatible to
> handle AM654 SoCs. While at that reformat the existing compatible list
> for older SoCs to list one valid compatible per line.
> 
> Signed-off-by: Vignesh R <vigneshr@ti.com>
> ---
> 
> v2: Reformat compatible existing compatible list.
> 
>  Documentation/devicetree/bindings/i2c/i2c-omap.txt | 8 ++++++--
>  1 file changed, 6 insertions(+), 2 deletions(-)
> 

Reviewed-by: Rob Herring <robh@kernel.org>

Re: [PATCH v2 0/2] i2c-omap: Enable i2c-omap driver for AM654 SoCs

[Vignesh R]

On 28-Sep-18 10:55 AM, Vignesh R wrote:
> Couple of patches to enable i2c-omap driver to be used with TI's new
> AM654 platforms.
> 
> 
> Vignesh R (2):
>   dt-bindings: i2c-omap: Add new compatible for AM654 SoCs
>   i2c: busses: Kconfig: Enable I2C_OMAP for ARCH_K3
> 
>  Documentation/devicetree/bindings/i2c/i2c-omap.txt | 8 ++++++--
>  drivers/i2c/busses/Kconfig                         | 2 +-
>  2 files changed, 7 insertions(+), 3 deletions(-)
> 

Gentle ping...

Regards
Vignesh

Re: [PATCH v2 0/2] i2c-omap: Enable i2c-omap driver for AM654 SoCs

[Wolfram Sang]

[-- Attachment #1: Type: text/plain, Size: 729 bytes --]

On Sat, Oct 20, 2018 at 01:23:40PM +0530, Vignesh R wrote:
> 
> 
> On 28-Sep-18 10:55 AM, Vignesh R wrote:
> > Couple of patches to enable i2c-omap driver to be used with TI's new
> > AM654 platforms.
> > 
> > 
> > Vignesh R (2):
> >   dt-bindings: i2c-omap: Add new compatible for AM654 SoCs
> >   i2c: busses: Kconfig: Enable I2C_OMAP for ARCH_K3
> > 
> >  Documentation/devicetree/bindings/i2c/i2c-omap.txt | 8 ++++++--
> >  drivers/i2c/busses/Kconfig                         | 2 +-
> >  2 files changed, 7 insertions(+), 3 deletions(-)
> > 
> 
> Gentle ping...

Maintainer tags are missing. Tony is listed for this driver on OMAP2+.
Is this his realm? Do we need a specific maintainer for this driver?


[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

Re: [PATCH v2 0/2] i2c-omap: Enable i2c-omap driver for AM654 SoCs

[Grygorii Strashko]

On 9/28/18 12:25 AM, Vignesh R wrote:
> Couple of patches to enable i2c-omap driver to be used with TI's new
> AM654 platforms.
> 
> 
> Vignesh R (2):
>    dt-bindings: i2c-omap: Add new compatible for AM654 SoCs
>    i2c: busses: Kconfig: Enable I2C_OMAP for ARCH_K3
> 
>   Documentation/devicetree/bindings/i2c/i2c-omap.txt | 8 ++++++--
>   drivers/i2c/busses/Kconfig                         | 2 +-
>   2 files changed, 7 insertions(+), 3 deletions(-)
> 

Reviewed-by: Grygorii Strashko <grygorii.strashko@ti.com>

-- 
regards,
-grygorii

Re: [PATCH v2 0/2] i2c-omap: Enable i2c-omap driver for AM654 SoCs

[Vignesh R]

On 29/10/18 4:04 AM, Wolfram Sang wrote:
> On Sat, Oct 20, 2018 at 01:23:40PM +0530, Vignesh R wrote:
>>
>>
>> On 28-Sep-18 10:55 AM, Vignesh R wrote:
>>> Couple of patches to enable i2c-omap driver to be used with TI's new
>>> AM654 platforms.
>>>
>>>
>>> Vignesh R (2):
>>>   dt-bindings: i2c-omap: Add new compatible for AM654 SoCs
>>>   i2c: busses: Kconfig: Enable I2C_OMAP for ARCH_K3
>>>
>>>  Documentation/devicetree/bindings/i2c/i2c-omap.txt | 8 ++++++--
>>>  drivers/i2c/busses/Kconfig                         | 2 +-
>>>  2 files changed, 7 insertions(+), 3 deletions(-)
>>>
>>
>> Gentle ping...
> 
> Maintainer tags are missing. Tony is listed for this driver on OMAP2+.
> Is this his realm? Do we need a specific maintainer for this driver?
> 

I think this driver needs a separate entry, will send a patch adding my
name as reviewer for this driver.

-- 
Regards
Vignesh