[U-Boot] [PATCH 3/5] usb: s3c-otg: Split out PHY control
Pavel Machek
pavel at denx.de
Tue Nov 4 20:34:21 CET 2014
On Tue 2014-11-04 06:07:32, Marek Vasut wrote:
> Split the Samsung specific PHY control into a separate file
> and compile this into the S3C OTG driver only if used on a
> Samsung system.
>
> Signed-off-by: Marek Vasut <marex at denx.de>
> Cc: Chin Liang See <clsee at altera.com>
> Cc: Dinh Nguyen <dinguyen at opensource.altera.com>
> Cc: Vince Bridgers <vbridger at altera.com>
Acked-by: Pavel Machek <pavel at denx.de>
I know you are just moving the code, but...
> +void otg_phy_init(struct s3c_udc *dev)
> +{
> + unsigned int usb_phy_ctrl = dev->pdata->usb_phy_ctrl;
> + struct s3c_usbotg_phy *phy =
> + (struct s3c_usbotg_phy *)dev->pdata->regs_phy;
> +
> + dev->pdata->phy_control(1);
> +
> + /*USB PHY0 Enable */
Wrong comment style.
> + printf("USB PHY0 Enable\n");
> +
> + /* Enable PHY */
> + writel(readl(usb_phy_ctrl) | USB_PHY_CTRL_EN0, usb_phy_ctrl);
We have helpers for setting/clearing bits, right?
--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
More information about the U-Boot
mailing list