Hi Geert, On Thursday 18 December 2014 21:23:46 Laurent Pinchart wrote: > On Thursday 18 December 2014 14:03:18 Geert Uytterhoeven wrote: > > On Thu, Dec 18, 2014 at 1:49 PM, Laurent Pinchart wrote: > > > There are three compatible strings defined for the ADXL345 and ADXL346 > > > in Documentation/devicetree/bindings/i2c/trivial-devices.txt: > > > "adi,adxl345", "adi,adxl346", "adi,adxl34x". Given that the last one is > > > a fallback for the first two I don't see a need to add the specific > > > compatible strings to the driver for now. If a new totally incompatible > > > chip named ADXL347 comes out we will need a new driver which won't be > > > allowed to use the "adi,adxl34x" compatible string. > > > > FWIW, I'm the evil person who added the adxl entries to that file... > > git-blame had already reported you. > > Do you think we should remove that compatible string ? Ping ? -- Regards, Laurent Pinchart -- 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
