Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751073AbdCMFXD (ORCPT ); Mon, 13 Mar 2017 01:23:03 -0400 Received: from fllnx209.ext.ti.com ([198.47.19.16]:51748 "EHLO fllnx209.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751009AbdCMFWx (ORCPT ); Mon, 13 Mar 2017 01:22:53 -0400 Subject: Re: [RFC PATCH] phy: samsung: move the Samsung specific phy files to "samsung" directory To: Vivek Gautam References: <20170309113311.15345-1-jh80.chung@samsung.com> <58C142F6.2030708@ti.com> CC: Jaehoon Chung , "linux-kernel@vger.kernel.org" , Kukjin Kim , , , Sylwester Nawrocki , , From: Kishon Vijay Abraham I Message-ID: <58C62C5F.4060307@ti.com> Date: Mon, 13 Mar 2017 10:51:35 +0530 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.7.2 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3623 Lines: 87 Hi, On Sunday 12 March 2017 02:48 PM, Vivek Gautam wrote: > Hi Kishon, > > > On Thu, Mar 9, 2017 at 5:26 PM, Kishon Vijay Abraham I wrote: >> Hi, >> >> On Thursday 09 March 2017 05:03 PM, Jaehoon Chung wrote: >>> Make the "samsung" directory and move the Samsung specific files to >>> there for maintaining the files relevant to Samsung. >> >> The number of phy drivers in drivers/phy is getting unmanageable. I think this >> is a good step to make it a little better. Can you also add a MAINTAINER for >> drivers/phy/samsung? > > I remember making a similar attempt in past [1], but that time we couldn't > reach an agreement as to whether group the phy drivers based on > vendors or based on the type of phy. > > If you are fine with grouping the drivers for each vendor, I hope you can > consider picking that patch (I can respin the patch based on linux-phy/next). > Other driver maintainers were also cool with that older patch. Sure, you can re-spin the patch. At that point of time I didn't think grouping phy drivers for each vendor is required. But especially after [1] where I failed to notice an existing phy driver can be reused and later has to be reverted. This could have been easily identified by MAINTAINERS of that particular platform. That's why now I feel grouping phy drivers and having a MAINTAINER for every vendor directory will help to identify such issues. Thanks Kishon [1] https://git.kernel.org/pub/scm/linux/kernel/git/kishon/linux-phy.git/commit/?h=fixes&id=9200c6f177638909dbbaded8aeeeccbd48744400 > > Let me know your comments. > > [1] https://patchwork.kernel.org/patch/8762561/ > > Regards > Vivek > >> >> Thanks >> Kishon >>> >>> Signed-off-by: Jaehoon Chung >>> --- >>> drivers/phy/Kconfig | 96 +---------------------- >>> drivers/phy/Makefile | 14 +--- >>> drivers/phy/samsung/Kconfig | 92 ++++++++++++++++++++++ >>> drivers/phy/samsung/Makefile | 11 +++ >>> drivers/phy/{ => samsung}/phy-exynos-dp-video.c | 0 >>> drivers/phy/{ => samsung}/phy-exynos-mipi-video.c | 0 >>> drivers/phy/{ => samsung}/phy-exynos-pcie.c | 0 >>> drivers/phy/{ => samsung}/phy-exynos4210-usb2.c | 0 >>> drivers/phy/{ => samsung}/phy-exynos4x12-usb2.c | 0 >>> drivers/phy/{ => samsung}/phy-exynos5-usbdrd.c | 0 >>> drivers/phy/{ => samsung}/phy-exynos5250-sata.c | 0 >>> drivers/phy/{ => samsung}/phy-exynos5250-usb2.c | 0 >>> drivers/phy/{ => samsung}/phy-s5pv210-usb2.c | 0 >>> drivers/phy/{ => samsung}/phy-samsung-usb2.c | 0 >>> drivers/phy/{ => samsung}/phy-samsung-usb2.h | 0 >>> 15 files changed, 108 insertions(+), 105 deletions(-) >>> create mode 100644 drivers/phy/samsung/Kconfig >>> create mode 100644 drivers/phy/samsung/Makefile >>> rename drivers/phy/{ => samsung}/phy-exynos-dp-video.c (100%) >>> rename drivers/phy/{ => samsung}/phy-exynos-mipi-video.c (100%) >>> rename drivers/phy/{ => samsung}/phy-exynos-pcie.c (100%) >>> rename drivers/phy/{ => samsung}/phy-exynos4210-usb2.c (100%) >>> rename drivers/phy/{ => samsung}/phy-exynos4x12-usb2.c (100%) >>> rename drivers/phy/{ => samsung}/phy-exynos5-usbdrd.c (100%) >>> rename drivers/phy/{ => samsung}/phy-exynos5250-sata.c (100%) >>> rename drivers/phy/{ => samsung}/phy-exynos5250-usb2.c (100%) >>> rename drivers/phy/{ => samsung}/phy-s5pv210-usb2.c (100%) >>> rename drivers/phy/{ => samsung}/phy-samsung-usb2.c (100%) >>> rename drivers/phy/{ => samsung}/phy-samsung-usb2.h (100%) > > [snip] > > >