diff mbox

[RESEND,1/2] i2c: versatile: Allow compile test build

Message ID 1468042283.21863.2.camel@ingics.com
State Accepted
Headers show

Commit Message

Axel Lin July 9, 2016, 5:31 a.m. UTC
There is no build dependency for this driver, so enable COMPILE_TEST to get
better build coverage.

Signed-off-by: Axel Lin <axel.lin@ingics.com>
---
Resend with CC VE maintainers.
 drivers/i2c/busses/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Liviu Dudau July 11, 2016, 8:44 a.m. UTC | #1
On Sat, Jul 09, 2016 at 01:31:23PM +0800, Axel Lin wrote:
> There is no build dependency for this driver, so enable COMPILE_TEST to get
> better build coverage.
> 
> Signed-off-by: Axel Lin <axel.lin@ingics.com>

For the whole series:

Tested-and-acked-by: Liviu Dudau <Liviu.Dudau@arm.com>

Thanks,
Liviu

> ---
> Resend with CC VE maintainers.
>  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 faa8e68..6e0bba0 100644
> --- a/drivers/i2c/busses/Kconfig
> +++ b/drivers/i2c/busses/Kconfig
> @@ -924,7 +924,7 @@ config I2C_UNIPHIER_F
>  
>  config I2C_VERSATILE
>  	tristate "ARM Versatile/Realview I2C bus support"
> -	depends on ARCH_VERSATILE || ARCH_REALVIEW || ARCH_VEXPRESS
> +	depends on ARCH_VERSATILE || ARCH_REALVIEW || ARCH_VEXPRESS || COMPILE_TEST
>  	select I2C_ALGOBIT
>  	help
>  	  Say yes if you want to support the I2C serial bus on ARMs Versatile
> -- 
> 2.5.0
>
Sudeep Holla July 14, 2016, 10:51 a.m. UTC | #2
On 11/07/16 09:44, Liviu Dudau wrote:
> On Sat, Jul 09, 2016 at 01:31:23PM +0800, Axel Lin wrote:
>> There is no build dependency for this driver, so enable COMPILE_TEST to get
>> better build coverage.
>>
>> Signed-off-by: Axel Lin <axel.lin@ingics.com>
>
> For the whole series:
>
> Tested-and-acked-by: Liviu Dudau <Liviu.Dudau@arm.com>

Thanks Liviu for testing them. Since these are cleanup patches, they
need to wait for v4.9, PR for v4.8 already sent and pulled.
Wolfram Sang July 14, 2016, 12:08 p.m. UTC | #3
> Thanks Liviu for testing them. Since these are cleanup patches, they
> need to wait for v4.9, PR for v4.8 already sent and pulled.

I'd think they should go via my I2C tree anyhow.
Wolfram Sang July 14, 2016, 12:23 p.m. UTC | #4
On Sat, Jul 09, 2016 at 01:31:23PM +0800, Axel Lin wrote:
> There is no build dependency for this driver, so enable COMPILE_TEST to get
> better build coverage.
> 
> Signed-off-by: Axel Lin <axel.lin@ingics.com>

Applied to for-next, thanks!
Sudeep Holla July 14, 2016, 12:33 p.m. UTC | #5
On 14/07/16 13:08, Wolfram Sang wrote:
>> Thanks Liviu for testing them. Since these are cleanup patches, they
>> need to wait for v4.9, PR for v4.8 already sent and pulled.
>
> I'd think they should go via my I2C tree anyhow.
>

Sorry, I was wrong, I misread the path. Thanks for pulling.
diff mbox

Patch

diff --git a/drivers/i2c/busses/Kconfig b/drivers/i2c/busses/Kconfig
index faa8e68..6e0bba0 100644
--- a/drivers/i2c/busses/Kconfig
+++ b/drivers/i2c/busses/Kconfig
@@ -924,7 +924,7 @@  config I2C_UNIPHIER_F
 
 config I2C_VERSATILE
 	tristate "ARM Versatile/Realview I2C bus support"
-	depends on ARCH_VERSATILE || ARCH_REALVIEW || ARCH_VEXPRESS
+	depends on ARCH_VERSATILE || ARCH_REALVIEW || ARCH_VEXPRESS || COMPILE_TEST
 	select I2C_ALGOBIT
 	help
 	  Say yes if you want to support the I2C serial bus on ARMs Versatile