[U-Boot] [PATCH] ARM: am3517-evm: Remove undefined pinmuxing

Adam Ford aford173 at gmail.com
Mon Jul 1 13:59:47 UTC 2019


On Wed, Jun 26, 2019 at 12:08 PM Adam Ford <aford173 at gmail.com> wrote:
>
> The pinmux controller on AM3517 is the same as OMAP3, however
> according to the AM3517 TRM, there are no valid entries between
> 48002180 and 48002210, which are supported by other OMAP3 devices.
> This patch removes the references undefined pin muxing.
>

Please do NOT apply this patch.

It seems to work OK on newer hardware, but this patch appears to break
older hardware, but I am not sure why since these registers are not
listed in the TRM.

Thank you,

adam
> Signed-off-by: Adam Ford <aford173 at gmail.com>
>
> diff --git a/board/logicpd/am3517evm/am3517evm.h b/board/logicpd/am3517evm/am3517evm.h
> index 99a08032e0..94d8802b9b 100644
> --- a/board/logicpd/am3517evm/am3517evm.h
> +++ b/board/logicpd/am3517evm/am3517evm.h
> @@ -294,18 +294,6 @@ const omap3_sysinfo sysinfo = {
>         MUX_VAL(CP(HECC1_TXD),          (IEN  | PTU | EN  | M0)) \
>         MUX_VAL(CP(HECC1_RXD),          (IEN  | PTU | EN  | M0)) \
>         /* HSUSB */\
> -       MUX_VAL(CP(HSUSB0_CLK),         (IEN  | PTD | DIS | M0)) \
> -       MUX_VAL(CP(HSUSB0_STP),         (IDIS | PTU | EN  | M0)) \
> -       MUX_VAL(CP(HSUSB0_DIR),         (IEN  | PTD | DIS | M0)) \
> -       MUX_VAL(CP(HSUSB0_NXT),         (IEN  | PTD | DIS | M0)) \
> -       MUX_VAL(CP(HSUSB0_DATA0),       (IEN  | PTD | DIS | M0)) \
> -       MUX_VAL(CP(HSUSB0_DATA1),       (IEN  | PTD | DIS | M0)) \
> -       MUX_VAL(CP(HSUSB0_DATA2),       (IEN  | PTD | DIS | M0)) \
> -       MUX_VAL(CP(HSUSB0_DATA3),       (IEN  | PTD | DIS | M0)) \
> -       MUX_VAL(CP(HSUSB0_DATA4),       (IEN  | PTD | DIS | M0)) \
> -       MUX_VAL(CP(HSUSB0_DATA5),       (IEN  | PTD | DIS | M0)) \
> -       MUX_VAL(CP(HSUSB0_DATA6),       (IEN  | PTD | DIS | M0)) \
> -       MUX_VAL(CP(HSUSB0_DATA7),       (IEN  | PTD | DIS | M0)) \
>         MUX_VAL(CP(USB0_DRVBUS),        (IEN  | PTD | EN  | M0)) \
>         /* HDQ */\
>         MUX_VAL(CP(HDQ_SIO),            (IEN  | PTU | EN  | M0)) \
> --
> 2.17.1
>


More information about the U-Boot mailing list