Re: [PATCH] Add OMAP Support for Generic PWM Devices using Dual-mode Timers

[Tony Lindgren <tony@atomide.com>]

* Kevin Hilman <khilman@deeprootsystems.com> [101116 11:43]:
> Grant Erickson <marathon96@gmail.com> writes:
> 
> [...]
> 
> >> Before something like this can merge, I would rather see
> >> 
> >> 1) generic PWM framework pushed along and merged
> >> 2) the dmtimer hwmod conversion finished
> >> 
> >> Yes, I know it's a lot more work to fix the core/framework code before
> >> having a feature included, but having something more generic that can
> >> actually support multiple PWM sources is clearly needed.
> >
> > No disagreement on the long-term architectural and design goals. All good
> > stuff.
> >
> > However, patches have to be submitted against the repository and branch we
> > have today, not those we might have tomorrow.
> 
> This comment is where we diverge. 
> 
> An alternative to waiting for the generic framework cleanup/refactor
> work is to contribute to it and help it along.

Yeah I agree, with Kevin. We should not export any more omap specific
functions in arch/arm/*omap*/ as they always lead into maintenance
nightmares later on. We still have several merge cycles of work just
to get rid of the existing exported functions.

Regards,

Tony