Re: [PATCH] mfd: intel_quark_i2c_gpio: Add support for SIMATIC IOT2000 platform

[Lee Jones]

On Tue, 09 May 2017, Andy Shevchenko wrote:

> On Tue, 2017-05-09 at 06:40 +0200, Jan Kiszka wrote:
> > The SIMATIC IOT2000 is derived from the Galileo Gen2 board and shares
> > its I2C frequency.
> > 
> > Signed-off-by: Jan Kiszka <jan.kiszka@xxxxxxxxxxx>
> 
> > Signed-off-by: Sascha Weisenberger <sascha.weisenberger@xxxxxxxxxxx>
> 
> Hmm... I thought that SoB chain goes from first author to last. (Or in
> case of maintainer / submitter their SoB goes at the end, which is
> apparently not the case here).

Yes, that's correct.  I will switch them when I apply the patch.

> Other than above
> 
> Reviewed-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>

> > 
> > ---
> > Âdrivers/mfd/intel_quark_i2c_gpio.c | 4 ++++
> > Â1 file changed, 4 insertions(+)
> > 
> > diff --git a/drivers/mfd/intel_quark_i2c_gpio.c
> > b/drivers/mfd/intel_quark_i2c_gpio.c
> > index 7946d6e38b87..98dddf2831ad 100644
> > --- a/drivers/mfd/intel_quark_i2c_gpio.c
> > +++ b/drivers/mfd/intel_quark_i2c_gpio.c
> > @@ -72,6 +72,10 @@ static const struct i2c_mode_info
> > platform_i2c_mode_info[] = {
> > Â		.name = "GalileoGen2",
> > Â		.i2c_scl_freq = 400000,
> > Â	},
> > +	{
> > +		.name = "SIMATIC IOT2000",
> > +		.i2c_scl_freq = 400000,
> > +	},
> > Â	{}
> > Â};
> > Â
> 

-- 
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org â Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog