Re: [PATCH v2 1/2] i2c: wmt: split out i2c-viai2c-common.{c,h}

[Wolfram Sang <wsa@kernel.org>]

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

Hi!

first of all, thank you for reworking your driver to share common code
with the WMT driver. This effort is much appreciated and we are close to
go, I think. But some remarks.

> +config I2C_VIAI2C_COMMON
> +	tristate
> +
>  config I2C_WMT
>  	tristate "Wondermedia WM8xxx SoC I2C bus support"
>  	depends on ARCH_VT8500 || COMPILE_TEST
> +	select I2C_VIAI2C_COMMON
>  	help
>  	  Say yes if you want to support the I2C bus on Wondermedia 8xxx-series
>  	  SoCs.
> diff --git a/drivers/i2c/busses/Makefile b/drivers/i2c/busses/Makefile
> index af56fe2c75c0..b7e20c3531b5 100644
> --- a/drivers/i2c/busses/Makefile
> +++ b/drivers/i2c/busses/Makefile
> @@ -120,6 +120,7 @@ obj-$(CONFIG_I2C_TEGRA_BPMP)	+= i2c-tegra-bpmp.o
>  obj-$(CONFIG_I2C_UNIPHIER)	+= i2c-uniphier.o
>  obj-$(CONFIG_I2C_UNIPHIER_F)	+= i2c-uniphier-f.o
>  obj-$(CONFIG_I2C_VERSATILE)	+= i2c-versatile.o
> +obj-$(CONFIG_I2C_VIAI2C_COMMON)	+= i2c-viai2c-common.o
>  obj-$(CONFIG_I2C_WMT)		+= i2c-wmt.o
>  i2c-octeon-objs := i2c-octeon-core.o i2c-octeon-platdrv.o
>  obj-$(CONFIG_I2C_OCTEON)	+= i2c-octeon.o

I'd prefer to link the core object to the driver object like pasemi or
octeon do. Or is there an argument for having a seperate module?

Regards!


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