| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <CAFp+6iEyY0F46W+5aGZkDieYgSdsVogO4LWhkKnOZbNVH+hUmA@mail.gmail.com>
Date: Wed, 6 Apr 2016 06:48:00 -0700
From: Vivek Gautam <gautamvivek1987@...il.com>
To: Kishon Vijay Abraham I <kishon@...com>
Cc: Vivek Gautam <gautam.vivek@...sung.com>, vireshk@...nel.org,
Shiraz Hashim <shiraz.linux.kernel@...il.com>,
Heiko Stübner <heiko@...ech.de>,
Maxime Ripard <maxime.ripard@...e-electrons.com>,
Chen-Yu Tsai <wens@...e.org>,
Russell King - ARM Linux <linux@....linux.org.uk>,
"linux-arm-kernel@...ts.infradead.org"
<linux-arm-kernel@...ts.infradead.org>, spear-devel@...t.st.com,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
linux-rockchip@...ts.infradead.org,
Linux USB Mailing List <linux-usb@...r.kernel.org>,
"linux-samsung-soc@...r.kernel.org"
<linux-samsung-soc@...r.kernel.org>,
"linux-omap@...r.kernel.org" <linux-omap@...r.kernel.org>,
CPGS <cpgs@...sung.com>
Subject: Re: [PATCH 1/2] phy: Group vendor specific phy drivers
Hi Kishon,
On Wed, Apr 6, 2016 at 5:51 AM, Kishon Vijay Abraham I <kishon@...com> wrote:
> Hi,
>
> On Friday 01 April 2016 07:05 PM, Vivek Gautam wrote:
>> Hi,
>>
>>
>> On Fri, Apr 1, 2016 at 6:05 AM, Kishon Vijay Abraham I <kishon@...com> wrote:
>>> Hi,
>>>
>>> On Friday 01 April 2016 04:59 PM, Vivek Gautam wrote:
>>>> Adding vendor specific directories in phy to group
>>>> phy drivers under their respective vendor umbrella.
>>>>
>>>> Signed-off-by: Vivek Gautam <gautam.vivek@...sung.com>
>>>> ---
>>>>
>>>> With growing number of phy drivers, it makes sense to
>>>> group these drivers under their respective vendor/platform
>>>> umbrella directory.
>>>>
>>>> Build-tested 'multi_v7_defconfig'.
>>>>
>>>> drivers/phy/Kconfig | 386 +---------------------
>>>> drivers/phy/Makefile | 57 +---
>>>> drivers/phy/bcom/Kconfig | 27 ++
>>>> drivers/phy/bcom/Makefile | 3 +
>>>> drivers/phy/{ => bcom}/phy-bcm-cygnus-pcie.c | 0
>>>> drivers/phy/{ => bcom}/phy-bcm-kona-usb2.c | 0
>>>> drivers/phy/{ => bcom}/phy-brcmstb-sata.c | 0
>>>> drivers/phy/hisi/Kconfig | 20 ++
>>>> drivers/phy/hisi/Makefile | 2 +
>>>> drivers/phy/{ => hisi}/phy-hi6220-usb.c | 0
>>>> drivers/phy/{ => hisi}/phy-hix5hd2-sata.c | 0
>>>> drivers/phy/marvell/Kconfig | 50 +++
>>>> drivers/phy/marvell/Makefile | 6 +
>>>> drivers/phy/{ => marvell}/phy-armada375-usb2.c | 0
>>>> drivers/phy/{ => marvell}/phy-berlin-sata.c | 0
>>>> drivers/phy/{ => marvell}/phy-berlin-usb.c | 0
>>>> drivers/phy/{ => marvell}/phy-mvebu-sata.c | 0
>>>> drivers/phy/{ => marvell}/phy-pxa-28nm-hsic.c | 0
>>>> drivers/phy/{ => marvell}/phy-pxa-28nm-usb2.c | 0
>>>> drivers/phy/qcom/Kconfig | 23 ++
>>>> drivers/phy/qcom/Makefile | 5 +
>>>> drivers/phy/{ => qcom}/phy-qcom-apq8064-sata.c | 0
>>>> drivers/phy/{ => qcom}/phy-qcom-ipq806x-sata.c | 0
>>>> drivers/phy/{ => qcom}/phy-qcom-ufs-i.h | 0
>>>> drivers/phy/{ => qcom}/phy-qcom-ufs-qmp-14nm.c | 0
>>>> drivers/phy/{ => qcom}/phy-qcom-ufs-qmp-14nm.h | 0
>>>> drivers/phy/{ => qcom}/phy-qcom-ufs-qmp-20nm.c | 0
>>>> drivers/phy/{ => qcom}/phy-qcom-ufs-qmp-20nm.h | 0
>>>> drivers/phy/{ => qcom}/phy-qcom-ufs.c | 0
>>>> drivers/phy/rcar/Kconfig | 16 +
>>>> drivers/phy/rcar/Makefile | 2 +
>>>> drivers/phy/{ => rcar}/phy-rcar-gen2.c | 0
>>>> drivers/phy/{ => rcar}/phy-rcar-gen3-usb2.c | 0
>>>> drivers/phy/rockchip/Kconfig | 23 ++
>>>> drivers/phy/rockchip/Makefile | 3 +
>>>> drivers/phy/{ => rockchip}/phy-rockchip-dp.c | 0
>>>> drivers/phy/{ => rockchip}/phy-rockchip-emmc.c | 0
>>>> drivers/phy/{ => rockchip}/phy-rockchip-usb.c | 0
>>>> drivers/phy/samsung/Kconfig | 87 +++++
>>>> drivers/phy/samsung/Makefile | 10 +
>>>> drivers/phy/{ => samsung}/phy-exynos-dp-video.c | 0
>>>> drivers/phy/{ => samsung}/phy-exynos-mipi-video.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
>>>> drivers/phy/st/Kconfig | 51 +++
>>>> drivers/phy/st/Makefile | 6 +
>>>> drivers/phy/{ => st}/phy-miphy28lp.c | 0
>>>> drivers/phy/{ => st}/phy-miphy365x.c | 0
>>>> drivers/phy/{ => st}/phy-spear1310-miphy.c | 0
>>>> drivers/phy/{ => st}/phy-spear1340-miphy.c | 0
>>>> drivers/phy/{ => st}/phy-stih407-usb.c | 0
>>>> drivers/phy/{ => st}/phy-stih41x-usb.c | 0
>>>> drivers/phy/sunxi/Kconfig | 28 ++
>>>> drivers/phy/sunxi/Makefile | 2 +
>>>> drivers/phy/{ => sunxi}/phy-sun4i-usb.c | 0
>>>> drivers/phy/{ => sunxi}/phy-sun9i-usb.c | 0
>>>> drivers/phy/ti/Kconfig | 67 ++++
>>>> drivers/phy/ti/Makefile | 6 +
>>>> drivers/phy/{ => ti}/phy-dm816x-usb.c | 0
>>>> drivers/phy/{ => ti}/phy-omap-control.c | 0
>>>> drivers/phy/{ => ti}/phy-omap-usb2.c | 0
>>>> drivers/phy/{ => ti}/phy-ti-pipe3.c | 0
>>>> drivers/phy/{ => ti}/phy-tusb1210.c | 0
>>>> drivers/phy/{ => ti}/phy-twl4030-usb.c | 0
>>>> drivers/phy/{ => ti}/ulpi_phy.h | 0
>>>
>>> ulpi_phy.h is not specific to TI though at this point tusb1210 is the only ULPI
>>> driver.
>>
>> Yes, you are right. But i was hesitant to add a multi-level header
>> file include in the phy driver
>> which uses it.
>>
>> Does it makes sense to move this ulpi_phy.h to include/linux/phy ?
>
> yeah.. that should be okay.
Alright, i will move this file there then.
>>
>>>> 71 files changed, 464 insertions(+), 416 deletions(-)
>>>> create mode 100644 drivers/phy/bcom/Kconfig
>>>> create mode 100644 drivers/phy/bcom/Makefile
>>>> rename drivers/phy/{ => bcom}/phy-bcm-cygnus-pcie.c (100%)
>>>> rename drivers/phy/{ => bcom}/phy-bcm-kona-usb2.c (100%)
>>>> rename drivers/phy/{ => bcom}/phy-brcmstb-sata.c (100%)
>>>> create mode 100644 drivers/phy/hisi/Kconfig
>>>> create mode 100644 drivers/phy/hisi/Makefile
>>>> rename drivers/phy/{ => hisi}/phy-hi6220-usb.c (100%)
>>>> rename drivers/phy/{ => hisi}/phy-hix5hd2-sata.c (100%)
>>>> create mode 100644 drivers/phy/marvell/Kconfig
>>>> create mode 100644 drivers/phy/marvell/Makefile
>>>> rename drivers/phy/{ => marvell}/phy-armada375-usb2.c (100%)
>>>> rename drivers/phy/{ => marvell}/phy-berlin-sata.c (100%)
>>>> rename drivers/phy/{ => marvell}/phy-berlin-usb.c (100%)
>>>> rename drivers/phy/{ => marvell}/phy-mvebu-sata.c (100%)
>>>> rename drivers/phy/{ => marvell}/phy-pxa-28nm-hsic.c (100%)
>>>> rename drivers/phy/{ => marvell}/phy-pxa-28nm-usb2.c (100%)
>>>> create mode 100644 drivers/phy/qcom/Kconfig
>>>> create mode 100644 drivers/phy/qcom/Makefile
>>>> rename drivers/phy/{ => qcom}/phy-qcom-apq8064-sata.c (100%)
>>>> rename drivers/phy/{ => qcom}/phy-qcom-ipq806x-sata.c (100%)
>>>> rename drivers/phy/{ => qcom}/phy-qcom-ufs-i.h (100%)
>>>> rename drivers/phy/{ => qcom}/phy-qcom-ufs-qmp-14nm.c (100%)
>>>> rename drivers/phy/{ => qcom}/phy-qcom-ufs-qmp-14nm.h (100%)
>>>> rename drivers/phy/{ => qcom}/phy-qcom-ufs-qmp-20nm.c (100%)
>>>> rename drivers/phy/{ => qcom}/phy-qcom-ufs-qmp-20nm.h (100%)
>>>> rename drivers/phy/{ => qcom}/phy-qcom-ufs.c (100%)
>>>> create mode 100644 drivers/phy/rcar/Kconfig
>>>> create mode 100644 drivers/phy/rcar/Makefile
>>>> rename drivers/phy/{ => rcar}/phy-rcar-gen2.c (100%)
>>>> rename drivers/phy/{ => rcar}/phy-rcar-gen3-usb2.c (100%)
>>>> create mode 100644 drivers/phy/rockchip/Kconfig
>>>> create mode 100644 drivers/phy/rockchip/Makefile
>>>> rename drivers/phy/{ => rockchip}/phy-rockchip-dp.c (100%)
>>>> rename drivers/phy/{ => rockchip}/phy-rockchip-emmc.c (100%)
>>>> rename drivers/phy/{ => rockchip}/phy-rockchip-usb.c (100%)
>>>> 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-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%)
>>>> create mode 100644 drivers/phy/st/Kconfig
>>>> create mode 100644 drivers/phy/st/Makefile
>>>> rename drivers/phy/{ => st}/phy-miphy28lp.c (100%)
>>>> rename drivers/phy/{ => st}/phy-miphy365x.c (100%)
>>>> rename drivers/phy/{ => st}/phy-spear1310-miphy.c (100%)
>>>> rename drivers/phy/{ => st}/phy-spear1340-miphy.c (100%)
>>>> rename drivers/phy/{ => st}/phy-stih407-usb.c (100%)
>>>> rename drivers/phy/{ => st}/phy-stih41x-usb.c (100%)
>>>> create mode 100644 drivers/phy/sunxi/Kconfig
>>>> create mode 100644 drivers/phy/sunxi/Makefile
>>>> rename drivers/phy/{ => sunxi}/phy-sun4i-usb.c (100%)
>>>> rename drivers/phy/{ => sunxi}/phy-sun9i-usb.c (100%)
>>>> create mode 100644 drivers/phy/ti/Kconfig
>>>> create mode 100644 drivers/phy/ti/Makefile
>>>> rename drivers/phy/{ => ti}/phy-dm816x-usb.c (100%)
>>>> rename drivers/phy/{ => ti}/phy-omap-control.c (100%)
>>>> rename drivers/phy/{ => ti}/phy-omap-usb2.c (100%)
>>>> rename drivers/phy/{ => ti}/phy-ti-pipe3.c (100%)
>>>> rename drivers/phy/{ => ti}/phy-tusb1210.c (100%)
>>>> rename drivers/phy/{ => ti}/phy-twl4030-usb.c (100%)
>>>> rename drivers/phy/{ => ti}/ulpi_phy.h (100%)
>>>>
>>>> diff --git a/drivers/phy/Kconfig b/drivers/phy/Kconfig
>>>> index 26566db..80c8c68 100644
>>>> --- a/drivers/phy/Kconfig
>>>> +++ b/drivers/phy/Kconfig
>>>> @@ -15,45 +15,6 @@ config GENERIC_PHY
>>>> phy users can obtain reference to the PHY. All the users of this
>>>> framework should select this config.
>>>>
>>>
>>>> +menuconfig PHY_PLAT
>>>> + bool "Platform Phy drivers"
>>>
>>> Not really convinced about adding a new config here. This will create new
>>> dependencies and will also start breaking users of oldconfig.
>>
>> This serves including all the directory paths in the 'make' for
>> driver/phy/Makefile.
>>
>> Otherwise each directory will have to be built as per particular platform
>> And, not all drivers of a vendor are built for one-kind of platform,
>> for example, Samsung uses ARCH_EXYNOS, as well as ARCH_S5PV210
>> Similarly, Broadcom phy drivers are based on multiple platforms, and so
>> does Marvell's.
>>
>> Please suggest a concise way to include the vendor directories in the build path
>> in drivers/phy/Makefile.
>
> obj-y += vendor/?
Yea, correct. Thanks
Will respin the patch with all the comments.
>
> -Kishon
>
>>
>>>
>>> Also not all of them are platform drivers (e.g ULPI PHY drivers).
>> I Think this applies to ulpi_phy only at this moment, which we can resolve
>> by moving that to include/linux path.
>>
>>>
>>> Thanks
>>> Kishon
>>> --
>>> To unsubscribe from this list: send the line "unsubscribe linux-usb" in
>>> the body of a message to majordomo@...r.kernel.org
>>> More majordomo info at http://vger.kernel.org/majordomo-info.html
>>
>>
>>
--
Best Regards
Vivek Gautam
Samsung R&D Institute, Bangalore
India
Powered by blists - more mailing lists