]> git.kernelconcepts.de Git - karo-tx-uboot.git/commitdiff
ehci-atmel: fix compiler warning
authorAndreas Bießmann <andreas.devel@googlemail.com>
Thu, 28 Jun 2012 02:50:37 +0000 (02:50 +0000)
committerAlbert ARIBAUD <albert.u.boot@aribaud.net>
Sat, 7 Jul 2012 12:07:30 +0000 (14:07 +0200)
Commit 7a101e946cba55e32d3d1265e30456c810046da3 introduced following warning:
---8<---
ehci-atmel.c: In function 'ehci_hcd_init':
ehci-atmel.c:49:2: warning: suggest parentheses around comparison in operand of '&' [-Wparentheses]
ehci-atmel.c: In function 'ehci_hcd_stop':
ehci-atmel.c:79:2: warning: suggest parentheses around comparison in operand of '&' [-Wparentheses]
--->8---

This patch fixes it.

Signed-off-by: Andreas Bießmann <andreas.devel@googlemail.com>
cc: Bo Shen <voice.shen@atmel.com>
cc: Marek Vasut <marex@denx.de>

drivers/usb/host/ehci-atmel.c

index 532db2289f7d42f4f10717bab5960b20471d689a..15b9b60eb44c369511abcf7eadfb97dce7627f22 100644 (file)
@@ -46,7 +46,7 @@ int ehci_hcd_init(void)
        start_time = get_timer(0);
        /* Enable UTMI PLL */
        writel(AT91_PMC_UPLLEN | AT91_PMC_BIASEN, &pmc->uckr);
        start_time = get_timer(0);
        /* Enable UTMI PLL */
        writel(AT91_PMC_UPLLEN | AT91_PMC_BIASEN, &pmc->uckr);
-       while (readl(&pmc->sr) & AT91_PMC_LOCKU != AT91_PMC_LOCKU) {
+       while ((readl(&pmc->sr) & AT91_PMC_LOCKU) != AT91_PMC_LOCKU) {
                WATCHDOG_RESET();
                tmp_time = get_timer(0);
                if ((tmp_time - start_time) > EN_UPLL_TIMEOUT) {
                WATCHDOG_RESET();
                tmp_time = get_timer(0);
                if ((tmp_time - start_time) > EN_UPLL_TIMEOUT) {
@@ -76,7 +76,7 @@ int ehci_hcd_stop(void)
        start_time = get_timer(0);
        /* Disable UTMI PLL */
        writel(readl(&pmc->uckr) & ~AT91_PMC_UPLLEN, &pmc->uckr);
        start_time = get_timer(0);
        /* Disable UTMI PLL */
        writel(readl(&pmc->uckr) & ~AT91_PMC_UPLLEN, &pmc->uckr);
-       while (readl(&pmc->sr) & AT91_PMC_LOCKU == AT91_PMC_LOCKU) {
+       while ((readl(&pmc->sr) & AT91_PMC_LOCKU) == AT91_PMC_LOCKU) {
                WATCHDOG_RESET();
                tmp_time = get_timer(0);
                if ((tmp_time - start_time) > EN_UPLL_TIMEOUT) {
                WATCHDOG_RESET();
                tmp_time = get_timer(0);
                if ((tmp_time - start_time) > EN_UPLL_TIMEOUT) {