From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752209AbbIKKKp (ORCPT ); Fri, 11 Sep 2015 06:10:45 -0400 Received: from mailout4.w1.samsung.com ([210.118.77.14]:32935 "EHLO mailout4.w1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751273AbbIKKKn (ORCPT ); Fri, 11 Sep 2015 06:10:43 -0400 X-AuditID: cbfec7f5-f794b6d000001495-0a-55f2a8a05394 Subject: Re: [PATCH 08/18] [media] s5c73m3: Export OF module alias information To: Javier Martinez Canillas , linux-kernel@vger.kernel.org References: <1440054451-1223-1-git-send-email-javier@osg.samsung.com> <1440054451-1223-9-git-send-email-javier@osg.samsung.com> <55F22C24.1070407@osg.samsung.com> Cc: Mauro Carvalho Chehab , Kyungmin Park , linux-media@vger.kernel.org From: Andrzej Hajda Message-id: <55F2A89C.6090102@samsung.com> Date: Fri, 11 Sep 2015 12:10:36 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.2.0 MIME-version: 1.0 In-reply-to: <55F22C24.1070407@osg.samsung.com> Content-type: text/plain; charset=windows-1252 Content-transfer-encoding: 7bit X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFjrCLMWRmVeSWpSXmKPExsVy+t/xK7oLVnwKNZjaZGvx5u0aJouzTW/Y LS7vmsNm0bNhK6vF6mcVDqweW/rvsnv0bVnF6PF5k1wAcxSXTUpqTmZZapG+XQJXRselLvaC V/wVT7bbNTC+4Oli5OSQEDCRaL19kx3CFpO4cG89WxcjF4eQwFJGiaf75rCCJIQEnjNK/L5Z CmILC/hJTDu4jAnEFhEIlfh38TYjRMMyRok/E/rZQBLMArUSd/5OZgGx2QQ0Jf5uvgkW5xXQ krj6oAtsG4uAqsS67rtgtqhAhMSps2+hagQlfky+B9bLKaAvsfvwWSCbA2imnsT9i1oQ4+Ul Nq95yzyBUWAWko5ZCFWzkFQtYGRexSiaWppcUJyUnmukV5yYW1yal66XnJ+7iREStl93MC49 ZnWIUYCDUYmH10L1U6gQa2JZcWXuIUYJDmYlEV7bDqAQb0piZVVqUX58UWlOavEhRmkOFiVx 3pm73ocICaQnlqRmp6YWpBbBZJk4OKUaGHOvW8/Y2Rr5Yubjxm1SVfmmx+5/vt6mPH2JH98e rvPr3DfPTG7e2/H1wXLFU1PeJJ3l9dz4uIU3ePa8Wy4713NntMy7qfaYqei+oHfpc7fC1w95 VI90cB2KX68mOznp/4rfGxM9E+SuCjS1bTj3k717ktziugUvbdw83DuqF/1doWQ+763sWW8l luKMREMt5qLiRAAEhQimVwIAAA== Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 09/11/2015 03:19 AM, Javier Martinez Canillas wrote: > Hello, > > On 08/20/2015 09:07 AM, Javier Martinez Canillas wrote: >> The SPI core always reports the MODALIAS uevent as "spi:" >> regardless of the mechanism that was used to register the device >> (i.e: OF or board code) and the table that is used later to match >> the driver with the device (i.e: SPI id table or OF match table). >> >> So drivers needs to export the SPI id table and this be built into >> the module or udev won't have the necessary information to autoload >> the needed driver module when the device is added. >> >> But this means that OF-only drivers needs to have both OF and SPI id >> tables that have to be kept in sync and also the dev node compatible >> manufacturer prefix is stripped when reporting the MODALIAS. Which can >> lead to issues if two vendors use the same SPI device name for example. >> >> To avoid the above, the SPI core behavior may be changed in the future >> to not require an SPI device table for OF-only drivers and report the >> OF module alias. So, it's better to also export the OF table even when >> is unused now to prevent breaking module loading when the core changes. >> >> Signed-off-by: Javier Martinez Canillas >> --- >> >> drivers/media/i2c/s5c73m3/s5c73m3-spi.c | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/drivers/media/i2c/s5c73m3/s5c73m3-spi.c b/drivers/media/i2c/s5c73m3/s5c73m3-spi.c >> index fa4a5ebda6b2..9983635ec253 100644 >> --- a/drivers/media/i2c/s5c73m3/s5c73m3-spi.c >> +++ b/drivers/media/i2c/s5c73m3/s5c73m3-spi.c >> @@ -31,6 +31,7 @@ static const struct of_device_id s5c73m3_spi_ids[] = { >> { .compatible = "samsung,s5c73m3" }, >> { } >> }; >> +MODULE_DEVICE_TABLE(of, s5c73m3_spi_ids;); Unnecessary semicolon inside parenthesis, I guess it wont compile :) Regards Andrzej >> >> enum spi_direction { >> SPI_DIR_RX, >> > Any comments about this patch? > > Best regards,