RE: [PATCH v11 2/4] i2c: Add support for Intel LJCA USB I2C driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Wolfrma,

Thanks for your help

> From: Wolfram Sang <wsa@xxxxxxxxxx>
> 
> On Tue, Aug 29, 2023 at 12:54:06PM +0800, Wentong Wu wrote:
> > Implements the I2C function of Intel USB-I2C/GPIO/SPI adapter device
> > named "La Jolla Cove Adapter" (LJCA). It communicate with LJCA I2C
> > module with specific protocol through interfaces exported by LJCA USB
> > driver.
> >
> > Signed-off-by: Wentong Wu <wentong.wu@xxxxxxxxx>
> > Reviewed-by: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx>
> > Reviewed-by: Andi Shyti <andi.shyti@xxxxxxxxxxxxxxx>
> 
> Basically good...
> 
> > +	if (ret < 0 || ret < sizeof(*r_packet)) {
> > +		dev_err(&ljca_i2c->adap.dev,
> > +			"i2c stop failed ret: %d id: %d\n",
> > +			ret, w_packet->id);
> 
> This was forgotten to be turned into dev_dbg. I could fix it while applying.

Thanks.

> But shall I take this patch via i2c? 

Yes, please help merge this i2c-ljca patch via i2c tree, thanks

> I'd assume the dependencies are hidden away by auxiliary_bus

Exactly, thanks

BR,
Wentong




[Index of Archives]     [Linux GPIO]     [Linux SPI]     [Linux Hardward Monitoring]     [LM Sensors]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux