From mboxrd@z Thu Jan 1 00:00:00 1970 From: Laurent Pinchart Date: Thu, 18 Dec 2014 19:23:46 +0000 Subject: Re: [PATCH] input: adxl34x: Add OF match support Message-Id: <10011053.aypz0IuTJG@avalon> List-Id: References: <1418868923-13411-1-git-send-email-laurent.pinchart+renesas@ideasonboard.com> <1492007.8cQN1myWLD@avalon> In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Geert Uytterhoeven Cc: Wolfram Sang , Laurent Pinchart , "linux-input-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , Linux-sh list , Linux I2C Hi Geert, 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 ? -- Regards, Laurent Pinchart