Hi Wolfram,
On Wed, Mar 18, 2015 at 10:07:30PM +0100, Wolfram Sang wrote:
> On Mon, Feb 16, 2015 at 03:20:04PM +0200, Baruch Siach wrote:
> > This commit adds a driver for the I2C master controller on the CX92755 SoC. The
> > CX92755 is one SoC in the Conexant Digicolor series. This driver should support
> > other SoCs from that series.
> >
> > Only 7bit slave addresses are currently supported.
> >
> > Tested on the Equinox CX92755 EVK, using i2c-tools utilities to read and write
> > the on-chip (sic) audio codec.
> >
> > Signed-off-by: Baruch Siach <baruch@xxxxxxxxxx>
>
> Looks mostly good.
>
> > +static u32 dc_i2c_func(struct i2c_adapter *adap)
> > +{
> > + return I2C_FUNC_I2C | I2C_FUNC_SMBUS_EMUL;
> > +}
>
> You probably want to advertise I2C_FUNC_NOSTART here, too?
OK.
> And if you could rebase to something after 4.0-rc1, that would be much
> appreciated.
Did that some time ago along with the rest of my queue for that platform. Just
waited for some review before reposting.
Thanks for reviewing,
baruch
--
http://baruch.siach.name/blog/ ~. .~ Tk Open Systems
=}------------------------------------------------ooO--U--Ooo------------{=
- baruch@xxxxxxxxxx - tel: +972.2.679.5364, http://www.tkos.co.il -
--
To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
