[U-Boot] some USB cleanup on EfikaMX

Wolfgang Denk wd at denx.de
Fri Sep 9 00:32:02 CEST 2011


Dear Jana Rapava,

In message <CAB+7RbE2MXooOuN3pniuBw7+K71Q0avCfxzuZXuBFfCZpvdctA at mail.gmail.com> you wrote:
> --===============1315613369==
> Content-Type: multipart/alternative; boundary=00151744109ca7dd4e04ac7470e6
> 
> --00151744109ca7dd4e04ac7470e6
> Content-Type: text/plain; charset=ISO-8859-1

Please resubmit as a proper patch.
See http://www.denx.de/wiki/U-Boot/Patches for instructions.

Most important: your signed-off-by: line is missing, as is a
descriptionof which specific problem you are fixing.

> --- a/board/efikamx/efikamx-usb.c
> +++ b/board/efikamx/efikamx-usb.c
> @@ -154,6 +154,7 @@ void efika_usb_phy_reset(void)
>         gpio_set_value(IOMUX_TO_GPIO(MX51_PIN_EIM_D27), 1);
>  }
> 
> +

Please do not add random (and excessive) white space.

> +       tmp = readl(OTG_BASE_ADDR + MX51_USB_CTRL_OFFSET);
> +       tmp &= ~(MX51_OTG_WUE_BIT | MX51_OTG_PM_BIT | MX51_H1_ULPI_IE_BIT |
> MX51_H1_WUE_BIT);

Your mailer line-wrapped your patch and thus corrupted it.  Please
make sure to configure your mailer not to do that. Even better, use
"git format-patch" to create hte patch and "git send-email" to submit
it.

> -       tmp = readl(OTG_BASE_ADDR + 0x808);
> -       tmp &= ~(1 << 8);
> -       tmp |= 1 << 5;
> -       writel(tmp, OTG_BASE_ADDR + 0x808);
> +       tmp = readl(OTG_BASE_ADDR + MX51_PHY_CTRL0_OFFSET);
> +       tmp &= ~MX51_OTG_OVERCURD_BIT;
> +       tmp |= MX51_EHCI_POWERPINSE_BIT;
> +       writel(tmp, OTG_BASE_ADDR + MX51_PHY_CTRL0_OFFSET);

Please consider using clrsetbits_le32() here instead.

Please fix globally.

Thanks.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
This cultural mystique surrounding the  biological  function  --  you
realize humans are overly preoccupied with the subject.
	-- Kelinda the Kelvan, "By Any Other Name", stardate 4658.9


More information about the U-Boot mailing list