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

[Adam Ford]

On Mon, Jul 1, 2019 at 8:59 AM Adam Ford <aford173 at gmail.com> wrote:
>
> 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.
> >

Tom,

I noticed this was in your ti branch.  I was hoping you could avoid
applying it because it breaks older hardware.

adam

>
> 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
> >