From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752270AbdHPOVP (ORCPT ); Wed, 16 Aug 2017 10:21:15 -0400 Received: from mail-it0-f67.google.com ([209.85.214.67]:37230 "EHLO mail-it0-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752167AbdHPOVN (ORCPT ); Wed, 16 Aug 2017 10:21:13 -0400 MIME-Version: 1.0 X-Originating-IP: [90.77.100.34] In-Reply-To: <20170615185418.13980-4-javier@dowhile0.org> References: <20170615185418.13980-1-javier@dowhile0.org> <20170615185418.13980-4-javier@dowhile0.org> From: Javier Martinez Canillas Date: Wed, 16 Aug 2017 16:21:11 +0200 Message-ID: Subject: Re: [RESEND PATCH v5 03/16] ARM: dts: efm32: Add generic compatible string for I2C EEPROM To: Linux Kernel Cc: Wolfram Sang , Rob Herring , Javier Martinez Canillas , "devicetree@vger.kernel.org" , Rob Herring , =?UTF-8?Q?Uwe_Kleine=2DK=C3=B6nig?= , Mark Rutland , Russell King , "linux-arm-kernel@lists.infradead.org" Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Jun 15, 2017 at 8:54 PM, Javier Martinez Canillas wrote: > The at24 driver allows to register I2C EEPROM chips using different vendor > and devices, but the I2C subsystem does not take the vendor into account > when matching using the I2C table since it only has device entries. > > But when matching using an OF table, both the vendor and device has to be > taken into account so the driver defines only a set of compatible strings > using the "atmel" vendor as a generic fallback for compatible I2C devices. > > So add this generic fallback to the device node compatible string to make > the device to match the driver using the OF device ID table. > > Signed-off-by: Javier Martinez Canillas > Any comment on this? Wolfram, maybe you can ack the pending DTS patches so I can post them to the arm-soc maintainers directly for they to pick them? Best regards, Javier From mboxrd@z Thu Jan 1 00:00:00 1970 From: Javier Martinez Canillas Subject: Re: [RESEND PATCH v5 03/16] ARM: dts: efm32: Add generic compatible string for I2C EEPROM Date: Wed, 16 Aug 2017 16:21:11 +0200 Message-ID: References: <20170615185418.13980-1-javier@dowhile0.org> <20170615185418.13980-4-javier@dowhile0.org> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Return-path: In-Reply-To: <20170615185418.13980-4-javier-0uQlZySMnqxg9hUCZPvPmw@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Linux Kernel Cc: Wolfram Sang , Rob Herring , Javier Martinez Canillas , "devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , Rob Herring , =?UTF-8?Q?Uwe_Kleine=2DK=C3=B6nig?= , Mark Rutland , Russell King , "linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org" List-Id: devicetree@vger.kernel.org On Thu, Jun 15, 2017 at 8:54 PM, Javier Martinez Canillas wrote: > The at24 driver allows to register I2C EEPROM chips using different vendor > and devices, but the I2C subsystem does not take the vendor into account > when matching using the I2C table since it only has device entries. > > But when matching using an OF table, both the vendor and device has to be > taken into account so the driver defines only a set of compatible strings > using the "atmel" vendor as a generic fallback for compatible I2C devices. > > So add this generic fallback to the device node compatible string to make > the device to match the driver using the OF device ID table. > > Signed-off-by: Javier Martinez Canillas > Any comment on this? Wolfram, maybe you can ack the pending DTS patches so I can post them to the arm-soc maintainers directly for they to pick them? Best regards, Javier -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html From mboxrd@z Thu Jan 1 00:00:00 1970 From: javier@dowhile0.org (Javier Martinez Canillas) Date: Wed, 16 Aug 2017 16:21:11 +0200 Subject: [RESEND PATCH v5 03/16] ARM: dts: efm32: Add generic compatible string for I2C EEPROM In-Reply-To: <20170615185418.13980-4-javier@dowhile0.org> References: <20170615185418.13980-1-javier@dowhile0.org> <20170615185418.13980-4-javier@dowhile0.org> Message-ID: To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Thu, Jun 15, 2017 at 8:54 PM, Javier Martinez Canillas wrote: > The at24 driver allows to register I2C EEPROM chips using different vendor > and devices, but the I2C subsystem does not take the vendor into account > when matching using the I2C table since it only has device entries. > > But when matching using an OF table, both the vendor and device has to be > taken into account so the driver defines only a set of compatible strings > using the "atmel" vendor as a generic fallback for compatible I2C devices. > > So add this generic fallback to the device node compatible string to make > the device to match the driver using the OF device ID table. > > Signed-off-by: Javier Martinez Canillas > Any comment on this? Wolfram, maybe you can ack the pending DTS patches so I can post them to the arm-soc maintainers directly for they to pick them? Best regards, Javier