From mboxrd@z Thu Jan 1 00:00:00 1970 From: dmitry.torokhov@gmail.com (Dmitry Torokhov) Date: Tue, 1 May 2018 15:49:20 -0700 Subject: [PATCH v2] Input: atmel_mxt_ts - add missing compatible strings to OF device table In-Reply-To: References: <20180410095340.24074-1-javierm@redhat.com> <20180501184853.GB52077@dtor-ws> <20180501205713.GA130399@dtor-ws> Message-ID: <20180501224920.GB130399@dtor-ws> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Tue, May 01, 2018 at 04:33:07PM -0500, Rob Herring wrote: > On Tue, May 1, 2018 at 3:57 PM, Dmitry Torokhov > wrote: > > From: Javier Martinez Canillas > > > > Commit af503716ac14 ("i2c: core: report OF style module alias for devices > > registered via OF") fixed how the I2C core reports the module alias when > > devices are registered via OF. > > > > But the atmel_mxt_ts driver only has an "atmel,maxtouch" compatible in its > > OF device ID table, so if a Device Tree is using a different one, autoload > > won't be working for the module (the matching works because the I2C device > > ID table is used as a fallback). > > > > So add compatible strings for each of the entries in the I2C device table. > > > > Fixes: af503716ac14 ("i2c: core: report OF style module alias for devices registered via OF") > > Reported-by: Enric Balletbo i Serra > > Signed-off-by: Javier Martinez Canillas > > Tested-by: Enric Balletbo i Serra > > [dtor: document which compatibles are deprecated and should not be used] > > Signed-off-by: Dmitry Torokhov > > --- > > .../devicetree/bindings/input/atmel,maxtouch.txt | 7 +++++++ > > drivers/input/touchscreen/atmel_mxt_ts.c | 5 +++++ > > 2 files changed, 12 insertions(+) > > Reviewed-by: Rob Herring > > Someone is going to make the dts updates, right? I sent out 2 patches... -- Dmitry