[PATCH 1/2] ARM: dts: imx6: Add support for Toradex Apalis SoM

[Petr Štetiar]

Joshua Clayton <stillcompiling at gmail.com> [2015-11-19 08:59:34]:

> On Thursday, November 19, 2015 03:12:05 PM Lucas Stach wrote:
> > > +
> > > +/* (PAD_CTL_PUS_22K_UP | PAD_CTL_PUE | PAD_CTL_PKE | PAD_CTL_SPEED_LOW | PAD_CTL_DSE_80ohm) */
> > > +#define PAD_CTRL_PU_22k 0x0f058
> > > +
> > I don't particularly like those defines. Maybe just replace by the hex
> > values?
>
> Seems like a genuine improvement to me,
> except that they should not be in the dts itself.
> Rather in arch/arm/boot/dts/imx6<num>-pinfunc.h, or another include file

Ok, so how should it look like? Define bit fields and then register content?
Something like:

  #define PAD_CTL_PUS_22K_UP              (3 << 14)
  #define PAD_CTL_PUE                     (1 << 13)
  #define PAD_CTL_PKE                     (1 << 12)
  #define PAD_CTL_SPEED_LOW               (1 << 6)
  #define PAD_CTL_DSE_80ohm               (3 << 3)

  #define PAD_CTRL_PU_22k PAD_CTL_PUS_22K_UP | PAD_CTL_PUE | PAD_CTL_PKE | 
                   PAD_CTL_SPEED_LOW | PAD_CTL_DSE_80ohm

or just register value with a comment:

  /* (PAD_CTL_PUS_22K_UP | PAD_CTL_PUE | PAD_CTL_PKE | PAD_CTL_SPEED_LOW | PAD_CTL_DSE_80ohm) */
  #define PAD_CTRL_PU_22k 0x0f058

-- ynezz