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

Raja, Govindraj govindraj.raja at ti.com
Fri Mar 9 01:19:51 EST 2012


On Wed, Mar 7, 2012 at 11:27 AM, Paul Walmsley <paul at pwsan.com> wrote:
> 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.

okay.

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

I will check on the MVR reg available for uart-rev on different omap socs,
and will get back by end on next of week with a possible patch.

--
Thanks,
Govindraj.R



More information about the linux-arm-kernel mailing list