[PATCH 02/13] ARM: dts: imx6qdl: SolidRun: move AR8035 into microsom

[Russell King - ARM Linux]

On Mon, Nov 27, 2017 at 06:12:38PM -0200, Fabio Estevam wrote:
> On Mon, Nov 27, 2017 at 2:52 PM, Russell King
> <rmk+kernel at armlinux.org.uk> wrote:
> 
> > +               pinctrl_microsom_enet_ar8035: microsom-enet-ar8035 {
> > +                       fsl,pins = <
> > +                               MX6QDL_PAD_ENET_MDIO__ENET_MDIO         0x1b8b0
> > +                               MX6QDL_PAD_ENET_MDC__ENET_MDC           0x1b0b0
> > +                               /* AR8035 reset */
> > +                               MX6QDL_PAD_KEY_ROW4__GPIO4_IO15         0x130b0
> > +                               /* AR8035 interrupt */
> > +                               MX6QDL_PAD_DI0_PIN2__GPIO4_IO18         0x80000000
> 
> Please avoid  0x80000000 and use the real IOMUX value instead.
> 
> > +                               /* GPIO16 -> AR8035 25MHz */
> > +                               MX6QDL_PAD_GPIO_16__ENET_REF_CLK        0xc0000000
> 
> Ditto.
> 
> > +                               MX6QDL_PAD_RGMII_TXC__RGMII_TXC         0x80000000
> 
> Ditto.

This isn't new, this is merely moving the definition from an existing
file, and when moving definitions, the golden rule is you don't
modify them.  If changes are required they should be a separate
patch.

I'll try to read the correct values from the registers, but I see
this as an entirely separate thing from this series, and not
something that this series should be concerned with.

-- 
RMK's Patch system: http://www.armlinux.org.uk/developer/patches/
FTTC broadband for 0.8mile line in suburbia: sync at 8.8Mbps down 630kbps up
According to speedtest.net: 8.21Mbps down 510kbps up