Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1389789

Re: [PATCH v2 2/2] phy: Group vendor specific phy drivers

From Vivek Gautam <gautamvivek1987@gmail.com>
Newsgroups linux.kernel
Subject Re: [PATCH v2 2/2] phy: Group vendor specific phy drivers
Date 2016-04-28 07:30 +0200
Message-ID <rsMGt-68v-1@gated-at.bofh.it> (permalink)
References <rkWjD-52m-7@gated-at.bofh.it> <rkWjD-52m-5@gated-at.bofh.it> <rs4m6-2DE-9@gated-at.bofh.it> <rsMn7-5ZM-1@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


Hi Kishon,


On Thu, Apr 28, 2016 at 10:28 AM, Kishon Vijay Abraham I <kishon@ti.com> wrote:
> Hi,
>
> On Tuesday 26 April 2016 11:36 AM, Vivek Gautam wrote:
>> Hi Kishon,
>>
>>
>> On Wed, Apr 6, 2016 at 7:37 PM, Vivek Gautam <gautam.vivek@samsung.com> wrote:
>>> Adding vendor specific directories in phy to group
>>> phy drivers under their respective vendor umbrella.
>>>
>>> Also updated the MAINTAINERS file to reflect the correct
>>> directory structure for phy drivers.
>>>
>>> Signed-off-by: Vivek Gautam <gautam.vivek@samsung.com>
>>> Acked-by: Heiko Stuebner <heiko@sntech.de>
>>> Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
>>> ---
>>>
>>> Changes from v1:
>>>  - Updated the MAINTAINERS file to reflect the same change
>>>    in directory structure.
>>>  - Removed PHY_PLAT config as pointed out by Kishon.
>>>    So we don't require the second patch in the v1 of this series:
>>>    [PATCH 2/2] arm: mach-spear: Enable PHY_PLAT to meet dependency
>>>  - Renamed sunxi --> allwinner; rcar --> renesas.
>>>  - Fixed error coming with qcom Makefile.
>>
>> Does this change looks okay now ?
>> I think you must be planning to take this in after 4.7 rc1 is out, isn't it ?
>
> I'm not sure. I'm thinking which should be the right way to group the phy
> drivers, by vendors or by type (usb, pcie, etc..). I'm more inclined to group
> it by type.

I guess there are atleast couple of phy drivers (PIPE and ST's MiPHY) that
have multiple roles - USB or PCIe or SATA.

So grouping such drivers in one of the phy-types will be tricky, WDUT

>
> Thanks
> Kishon



-- 
Best Regards
Vivek Gautam
Samsung R&D Institute, Bangalore
India

Back to linux.kernel | Previous | NextPrevious in thread | Find similar | Unroll thread


Thread

Re: [PATCH v2 2/2] phy: Group vendor specific phy drivers Vivek Gautam <gautamvivek1987@gmail.com> - 2016-04-26 08:10 +0200
  Re: [PATCH v2 2/2] phy: Group vendor specific phy drivers Kishon Vijay Abraham I <kishon@ti.com> - 2016-04-28 07:10 +0200
    Re: [PATCH v2 2/2] phy: Group vendor specific phy drivers Vivek Gautam <gautamvivek1987@gmail.com> - 2016-04-28 07:30 +0200

csiph-web