From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751368AbcCRClj (ORCPT ); Thu, 17 Mar 2016 22:41:39 -0400 Received: from lists.s-osg.org ([54.187.51.154]:54393 "EHLO lists.s-osg.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751241AbcCRClg (ORCPT ); Thu, 17 Mar 2016 22:41:36 -0400 Subject: Re: [PATCH] regulator: Rename files for Maxim PMIC drivers To: Krzysztof Kozlowski , linux-kernel@vger.kernel.org References: <1458237294-13730-1-git-send-email-javier@osg.samsung.com> <56EB46D7.5030807@samsung.com> Cc: Liam Girdwood , Chanwoo Choi , Mark Brown From: Javier Martinez Canillas Message-ID: <56EB6AD6.7050407@osg.samsung.com> Date: Thu, 17 Mar 2016 23:41:26 -0300 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.6.0 MIME-Version: 1.0 In-Reply-To: <56EB46D7.5030807@samsung.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hello Krzysztof, Thanks a lot for your review. On 03/17/2016 09:07 PM, Krzysztof Kozlowski wrote: > On 18.03.2016 02:54, Javier Martinez Canillas wrote: >> Most Maxim PMIC regulator drivers are for sub-devices of Multi-Function >> Devices with drivers under drivers/mfd. But for many of these, the same >> object file name was used for both the MFD and the regulator drivers. >> >> Having 2 different drivers with the same name causes a lot of confusion >> to Kbuild, specially if these are built as module since only one module >> will be installed and also exported symbols will be undefined due being >> overwritten by the other module during modpost. > > These regulator drivers do not export symbols. In case of max14577 only > main MFD driver exports symbols so what do you mean by "overwriting by > other module"? > That's correct, what I meant is that if only the MFD driver is built, then Kbuild / modpost are able to obtain the exported symbols and add it to the Module.symvers file. But if the regulator driver is also built, then the build system isn't able to handle that case and the exported symbols from Module.symvers disappear. So IIUC what happens is that the build system gets the exported symbols from the max14755 MFD module but then finds another module that has the same name (with no exported symbols) and so discards the list of symbols that previously had for that module. That's why I used the "overwriting by the other module". > Beside that comment the patch itself is okay: > > Reviewed-by: Krzysztof Kozlowski > > > > Best regards, > > Krzysztof > Best regards, -- Javier Martinez Canillas Open Source Group Samsung Research America