Re: [PATCH] video: imxfb: correct the bitmask for DMACR_HM/_TM

From: Martin Kaiser
Date: Fri Nov 25 2016 - 03:44:01 EST


Thus wrote Uwe Kleine-König (u.kleine-koenig@xxxxxxxxxxxxxx):

> > ok, understood. I wasn't able to dig up an imx1 specification. Do you
> > know if it's publicly available?

> http://www.nxp.com/assets/documents/data/en/reference-manuals/MC9328MX1RM.pdf

Thanks.

> So you put the values to use in the device tree? Then the right thing to
> do is to check the device type in the driver and mask accordingly when
> the values are written to the hardware.

Device tree and platform data contain the entire register, not the
individual components. The macros are provided to build the register
value from the components, but nobody's using them.

> IMHO dropping the macros is the right thing to do.

Ok, I'll submit a patch for this.

Best regards,
Martin