[RFC PATCH] OMAP: UART: Enable tx wakeup bit in wer

[Paul Walmsley]

Hi

On Tue, 6 Mar 2012, Raja, Govindraj wrote:

> I see two ways of adding this tx wake-up flag.
> 
> 1.) passing a feature flag from pdata and populating the feature value to wer.
> 
> 2.) Duplicating the uart1/2/3 hwmod for omap3430 without dev attrib
> and uart1/2/3/4
>      hwmod for omap3630 with dev attribs.
> 
> Approach-2 adds unnecessary code duplication in hwmod file with and without
> dev_attribs, So right now here is the patch to follow approach [1],
> 
> If its strongly recommended to use approach-2 will add it.
> (My concern is the diffstat it might produce with approach-2)

Both 1 and 2 are needed.

The current way that errata are handled in arch/arm/mach-omap2/serial.c 
needs to be changed.  Those should be flags that are passed from hwmod 
dev_attr data.  The presence of the TX wakeup bit should simply be another 
dev_attr flag.  These flags should be passed to the driver via the 
omap_up.errata field for the time being.

The exception to this is if the IP block's revision register was changed 
when TX wakeups became possible.  If that's the case, then that mechanism 
can be used in lieu of dev_attr and platform_data in this situation.


- Paul