Re: [PATCH 0/2] i2c: ismt: Fix length handling for SMBus block reads

Wolfram Sang
2017-08-29T10:22:29Z

[-- Attachment #1: Type: text/plain, Size: 813 bytes --]

> >> So basically the byte count already precedes the data in the dma_buffer,
> >> then the driver sticks desc->rxbytes in front of this resulting in the
> >> trace above.
> >>
> >> The first patch tackles this.
> >>
> >> The second patch in the series adds a sanity check on the byte count
> >> supplied by the slave device. This might be a nice to have, but is
> >> probably less critical.
> >
> > Both patches look good to me. Seth, Neil, do you agree?
> >
>
> Ping.
>
> Not sure what the usual review time is, let me know if this is premature.

I applied both patches to for-next (v4.14) now to get a broader
audience. for-current (v4.13) might have been also applicable, but I
don't want to apply the patches there without the driver maintainer
acks. I hope this works for you.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]