Re: [PATCH] omap: dmtimer: Enable autoidle

[Kevin Hilman <khilman@ti.com>]

"DebBarma, Tarun Kanti" <tarun.kanti@ti.com> writes:

> [...]
>> >> >> >
>> >> >> > Should this be changed to apply only on that one? I don't know
>> >about
>> >> >> > OMAP4, but I think this should work on OMAP2 at least based on
>> >spec.
>> >> >> It works on OMAP2 and OMAP4.
>> >> >Ok, on OMAP4 I should have clarified further.
>> >> >AUTOIDLE is present only for millisecond timers: 1,2,10.
>> >> >The reset of the timers do not have this bit.
>> >>
>> >> Does it matter if the bit is set on the timers that don't have it or
>> >does
>> >> the HW just ignore it? Or should we enumerate the timers that support
>> >it
>> >> on omap4?
>> >For other timers it happens to be SOFTRESET bit. So HW would not ignore.
>> 
>> Ok, so this one should only be implemented for omap2/3 or added logic is
>> needed for omap4.
> YES.
>

Tarun is autoidle feature handled in your hwmod conversion series?

If not, this should be implemented as a patch on top of your series
since we will know which timers have this capability.

Kevin