Hi,
On Mon, Jul 09, 2018 at 06:14:43PM +0200, Wolfram Sang wrote:
> > static int ov772x_read(struct i2c_client *client, u8 addr)
> > {
> > - int ret;
> > - u8 val;
> > -
> > - ret = i2c_master_send(client, &addr, 1);
> > - if (ret < 0)
> > - return ret;
> > - ret = i2c_master_recv(client, &val, 1);
> > - if (ret < 0)
> > - return ret;
> > -
> > - return val;
> > + return sccb_read_byte(client, addr);
> > }
> >
> > static inline int ov772x_write(struct i2c_client *client, u8 addr, u8
> > value)
> > {
> > - return i2c_smbus_write_byte_data(client, addr, value);
> > + return sccb_write_byte(client, addr, value);
> > }Reviewed-by: Sebastian Reichel <sebastian.reic...@collabora.co.uk> > Minor nit: I'd rather drop these two functions and use the > sccb-accessors directly. > > However, I really like how this looks here: It is totally clear we are > doing SCCB and hide away all the details. I think it would be even better to introduce a SSCB regmap layer and use that. -- Sebastian
signature.asc
Description: PGP signature
