Messages in this thread | | | From | Tang Yuantian-B29983 <> | Subject | RE: [PATCH] i2c/busses: (mpc) Add support for SMBUS_READ_BLOCK_DATA | Date | Fri, 18 Nov 2011 03:15:39 +0000 |
| |
I have tested this patch. It works on p1022ds platform. Thanks.
Regards, Yuantian
> -----Original Message----- > From: Guenter Roeck [mailto:guenter.roeck@ericsson.com] > Sent: 2011年11月18日 2:19 > To: sun york-R58495 > Cc: Jean Delvare; Tabi Timur-B04825; linux-i2c@vger.kernel.org; linux- > kernel@vger.kernel.org; Tang Yuantian-B29983 > Subject: Re: [PATCH] i2c/busses: (mpc) Add support for > SMBUS_READ_BLOCK_DATA > > If you look closely, you may realize that the mpc_read also doesn't > validate of any of the other parameters. Are you going to request that > such validations be added as well ? How about the other functions in this > driver ? Should each function also validate each of its parameters ? If > not, where do you draw the line ? > > Besides, the caller is the SMBus block read function, which does know > that SMBus block reads are limited to 32 bytes (plus length). > > Guenter > >
| |