| 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
| ||
|
Date: Fri, 1 Apr 2016 18:35:56 +0530
From: Kishon Vijay Abraham I <kishon@...com>
To: Vivek Gautam <gautam.vivek@...sung.com>, <vireshk@...nel.org>,
<shiraz.linux.kernel@...il.com>, <heiko@...ech.de>,
<maxime.ripard@...e-electrons.com>, <wens@...e.org>
CC: <linux@....linux.org.uk>, <linux-arm-kernel@...ts.infradead.org>,
<spear-devel@...t.st.com>, <linux-kernel@...r.kernel.org>,
<linux-rockchip@...ts.infradead.org>, <linux-usb@...r.kernel.org>,
<linux-samsung-soc@...r.kernel.org>, <linux-omap@...r.kernel.org>,
<cpgs@...sung.com>
Subject: Re: [PATCH 1/2] phy: Group vendor specific phy drivers
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.
> 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.
Also not all of them are platform drivers (e.g ULPI PHY drivers).
Thanks
Kishon
Powered by blists - more mailing lists