Messages in this thread | | | From | Richard Genoud <> | Date | Wed, 16 Apr 2014 15:33:04 +0200 | Subject | Re: [PATCH V4 5/5] usb-phy: samsung-usb3: Remove older phy-samsung-usb3 driver |
| |
Hi Vivek,
2014-04-09 13:34 GMT+02:00 Vivek Gautam <gautam.vivek@samsung.com>: > Hi Tomasz, > ' > > On Wed, Apr 9, 2014 at 4:43 PM, Tomasz Figa <t.figa@samsung.com> wrote: >> Hi Vivek, >> > Thanks for reviewing the patch series. > >> >> On 08.04.2014 16:36, Vivek Gautam wrote: >>> >>> Removing this older USB 3.0 DRD controller PHY driver, since >>> a new driver based on generic phy framework is now available. >>> >>> Also removing the dt node for older driver from Exynos5250 >>> device tree and updating the dt node for DWC3 controller. >>> >>> Signed-off-by: Vivek Gautam <gautam.vivek@samsung.com> >>> --- >>> >>> NOTE: This patch should be merged only when the new USB 3.0 >>> DRD phy controller driver is available in the tree from the >>> patches: >>> phy: Add new Exynos5 USB 3.0 PHY driver; and >>> dt: exynos5250: Enable support for generic USB DRD phy >>> >>> arch/arm/boot/dts/exynos5250.dtsi | 17 +- >>> drivers/usb/phy/phy-samsung-usb.h | 83 --------- >>> drivers/usb/phy/phy-samsung-usb3.c | 350 >>> ------------------------------------ >>> 3 files changed, 2 insertions(+), 448 deletions(-) >>> delete mode 100644 drivers/usb/phy/phy-samsung-usb3.c >>> >>> diff --git a/arch/arm/boot/dts/exynos5250.dtsi >>> b/arch/arm/boot/dts/exynos5250.dtsi >>> index 92c6fcd..1cb1e91 100644 >>> --- a/arch/arm/boot/dts/exynos5250.dtsi >>> +++ b/arch/arm/boot/dts/exynos5250.dtsi >> >> >> IMHO driver and dts changes should be separated into two patches, first >> updating device tree to use the new driver and second removing the driver. > > Sure will separate the patch into driver and dts change. > >> >> After fixing this issue, >> >> Reviewed-by: Tomasz Figa <t.figa@samsung.com> >> I guess you should also remove phy-samsung-usb3.c references in the Makefile, Kconfig and exynos_defconfig
Richard.
| |