Hello Wolfram, On 09/29/2014 10:59 PM, Wolfram Sang wrote: > On Fri, Sep 19, 2014 at 10:08:12AM +0200, Sjoerd Simons wrote: >> To enable the cros-ec-tunnel driver to be auto-loaded when build as a >> module add an of match table (and export it) to match the modalias >> information passed on to userspace as the Cros EC MFD driver registers >> the MFD subdevices with an of_compatibility string. >> >> Signed-off-by: Sjoerd Simons <sjoerd.simons@xxxxxxxxxxxxxxx> >> Reviewed-by: Javier Martinez Canillas <javier.martinez@xxxxxxxxxxxxxxx> > > Applied to for-next, thanks! > > However, since you are working on this driver: sparse rightfully > complains about: > > drivers/i2c/busses/i2c-cros-ec-tunnel.c:97:44: warning: cast truncates bits from constant value (10000 becomes 0) > > I don't know the EC protocol, can you check? > Doug posted a patch [0] to fix the issue you are reporting but it was not picked. I tested that the patch applies on top of linux-next + the latest cros_ec cleanups (that includes a patch for this i2c driver and was taken by Lee. So I think is safe for you take Doug's patch through your tree since it seems to not cause any conflict with the mfd tree. Or if you prefer, I can re-post that patch for 3.19. Best regards, Javier [0]: https://lkml.org/lkml/2014/6/23/830 -- 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
