Hi, On Thursday, August 14, 2014 08:07:40 PM Vivek Gautam wrote: > Hi Bartlomiej Zolnierkiewicz, > > > On Thursday, August 14, 2014 7:55 PM, Bartlomiej Zolnierkiewicz > <b.zolnierkie@xxxxxxxxxxx> wrote > > Subject: [PATCH 3/5] usb: phy: samsung: remove old USB 2.0 PHY driver > > I would say it co-incidence that both of us sent the patch-series for > similar cleanup at the same time. ;) :) > There's one thing that I would want to comment here, since we don't have any > new usb-phy driver for S3C64XX, > so we can't simply remove this entire driver. > I have posted my patch-series [1], which does cleanup while keeping the > support for S3C64XX. AFAIK S3C64XX code from drivers/usb/phy/phy-samsung-usb2.c has never been used as this platform still uses its own code from arch/arm/mach-s3c64xx/setup-usb-phy.c (there are no users in the kernel tree of either s3c64xx-usb2phy platform device or "samsung,s3c64xx-usb2phy" DT compatible) . Therefore I think that the entire drivers/usb/phy/phy-samsung-usb2.c driver should be removed (somebody with the hardware can as well add S3C64XX support to the new drivers/phy/phy-samsung-usb2.c driver and port the platform to use it). > Please let me know what you think of that series. > > [1] [PATCH 0/7] usb-phy: samsung: Cleanup the unused drivers > https://lkml.org/lkml/2014/8/14/235 Looks fine to me except the above mentioned issue with S3C64XX platform support. > > drivers/usb/phy/phy-samsung-usb2 driver got replaced by > > drivers/phy/phy-samsung-usb2 one and is no longer used. > > > > Cc: Kamil Debski <k.debski@xxxxxxxxxxx> > > Cc: Vivek Gautam <gautam.vivek@xxxxxxxxxxx> > > Signed-off-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@xxxxxxxxxxx> > > Acked-by: Kyungmin Park <kyungmin.park@xxxxxxxxxxx> Best regards, -- Bartlomiej Zolnierkiewicz Samsung R&D Institute Poland Samsung Electronics -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
 |