From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752542AbbHTIiS (ORCPT ); Thu, 20 Aug 2015 04:38:18 -0400 Received: from ozlabs.org ([103.22.144.67]:38378 "EHLO ozlabs.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751635AbbHTIiP (ORCPT ); Thu, 20 Aug 2015 04:38:15 -0400 X-powerpc-patch-notification: thanks X-powerpc-patch-commit: a1a42b7920ba57f287d742d42f113402fbb7c85e In-Reply-To: <1438273132-20926-23-git-send-email-javier@osg.samsung.com> To: Javier Martinez Canillas , linux-kernel@vger.kernel.org From: Michael Ellerman Cc: Javier Martinez Canillas , linuxppc-dev@lists.ozlabs.org Subject: Re: [22/27] macintosh: therm_windtunnel: Export OF module alias information Message-Id: <20150820083813.D508D1402A6@ozlabs.org> Date: Thu, 20 Aug 2015 18:38:13 +1000 (AEST) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, 2015-30-07 at 16:18:47 UTC, Javier Martinez Canillas wrote: > The I2C core always reports the MODALIAS uevent as "i2c: regardless if the driver was matched using the I2C id_table or the > of_match_table. So technically there's no need for a driver to export > the OF table since currently it's not used. > > In fact, the I2C device ID table is mandatory for I2C drivers since > a i2c_device_id is passed to the driver's probe function even if the > I2C core used the OF table to match the driver. > > And since the I2C core uses different tables, OF-only drivers needs to > have duplicated data that has to be kept in sync and also the dev node > compatible manufacturer prefix is stripped when reporting the MODALIAS. > > To avoid the above, the I2C core behavior may be changed in the future > to not require an I2C device table for OF-only drivers and report the > OF module alias. So, it's better to also export the OF table to prevent > breaking module autoloading if that happens. > > Signed-off-by: Javier Martinez Canillas Applied to powerpc next, thanks. https://git.kernel.org/powerpc/c/a1a42b7920ba57f287d7 cheers