diff mbox

[V5,1/6] i2c: Add i2c support to Freescale Layerscape platforms

Message ID 1445255774-32887-1-git-send-email-B48286@freescale.com (mailing list archive)
State New, archived
Headers show

Commit Message

Hou Zhiqiang Oct. 19, 2015, 11:56 a.m. UTC
From: Shaohui Xie <Shaohui.Xie@freescale.com>

Modify the I2C_IMX config to support to Layerscape platforms.

Signed-off-by: Mingkai Hu <Mingkai.Hu@freescale.com>
Signed-off-by: Wenbin Song <Wenbin.Song@freescale.com>
Signed-off-by: Hou Zhiqiang <B48286@freescale.com>
---
V5: V4 V3 V2
 - No change.

 drivers/i2c/busses/Kconfig | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Wolfram Sang Oct. 20, 2015, 3:52 p.m. UTC | #1
On Mon, Oct 19, 2015 at 07:56:09PM +0800, Zhiqiang Hou wrote:
> From: Shaohui Xie <Shaohui.Xie@freescale.com>
> 
> Modify the I2C_IMX config to support to Layerscape platforms.
> 
> Signed-off-by: Mingkai Hu <Mingkai.Hu@freescale.com>
> Signed-off-by: Wenbin Song <Wenbin.Song@freescale.com>
> Signed-off-by: Hou Zhiqiang <B48286@freescale.com>

Three signed offs for a simple Kconfig change? And the person doing
the patch has no signed off?? That looks wrong. Maybe the above signed
offs should be acked by?

> ---
> V5: V4 V3 V2
>  - No change.
> 
>  drivers/i2c/busses/Kconfig | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/i2c/busses/Kconfig b/drivers/i2c/busses/Kconfig
> index 08b8617..14147ec 100644
> --- a/drivers/i2c/busses/Kconfig
> +++ b/drivers/i2c/busses/Kconfig
> @@ -582,10 +582,10 @@ config I2C_IMG
>  
>  config I2C_IMX
>  	tristate "IMX I2C interface"
> -	depends on ARCH_MXC
> +	depends on ARCH_MXC || ARCH_LAYERSCAPE
>  	help
>  	  Say Y here if you want to use the IIC bus controller on
> -	  the Freescale i.MX/MXC processors.
> +	  the Freescale i.MX/MXC or Layerscape processors.
>  
>  	  This driver can also be built as a module.  If so, the module
>  	  will be called i2c-imx.
> -- 
> 2.1.0.27.g96db324
>
Hou Zhiqiang Oct. 21, 2015, 4:10 a.m. UTC | #2
> -----Original Message-----

> From: Wolfram Sang [mailto:wsa@the-dreams.de]

> Sent: 2015?10?20? 23:53

> To: Hou Zhiqiang-B48286

> Cc: linux-arm-kernel@lists.infradead.org; catalin.marinas@arm.com;

> will.deacon@arm.com; linux-i2c@vger.kernel.org; linux-

> watchdog@vger.kernel.org; linux-doc@vger.kernel.org; linux-

> clk@vger.kernel.org; mark.rutland@arm.com; linux@roeck-us.net;

> wim@iguana.be; corbet@lwn.net; mturquette@baylibre.com;

> sboyd@codeaurora.org; Hu Mingkai-B21284; Xie Shaohui-B21989; Wood Scott-

> B07421; Sharma Bhupesh-B45370; Song Wenbin-B53747

> Subject: Re: [PATCH V5 1/6] i2c: Add i2c support to Freescale Layerscape

> platforms

> 

> On Mon, Oct 19, 2015 at 07:56:09PM +0800, Zhiqiang Hou wrote:

> > From: Shaohui Xie <Shaohui.Xie@freescale.com>

> >

> > Modify the I2C_IMX config to support to Layerscape platforms.

> >

> > Signed-off-by: Mingkai Hu <Mingkai.Hu@freescale.com>

> > Signed-off-by: Wenbin Song <Wenbin.Song@freescale.com>

> > Signed-off-by: Hou Zhiqiang <B48286@freescale.com>

> 

> Three signed offs for a simple Kconfig change? And the person doing the

> patch has no signed off?? That looks wrong. Maybe the above signed offs

> should be acked by?

> 


Will add the signed-off of the author.

Thanks,
Zhiqiang
diff mbox

Patch

diff --git a/drivers/i2c/busses/Kconfig b/drivers/i2c/busses/Kconfig
index 08b8617..14147ec 100644
--- a/drivers/i2c/busses/Kconfig
+++ b/drivers/i2c/busses/Kconfig
@@ -582,10 +582,10 @@  config I2C_IMG
 
 config I2C_IMX
 	tristate "IMX I2C interface"
-	depends on ARCH_MXC
+	depends on ARCH_MXC || ARCH_LAYERSCAPE
 	help
 	  Say Y here if you want to use the IIC bus controller on
-	  the Freescale i.MX/MXC processors.
+	  the Freescale i.MX/MXC or Layerscape processors.
 
 	  This driver can also be built as a module.  If so, the module
 	  will be called i2c-imx.