Re: [PATCH 6/6] omap: move generic omap3 features to generic

[Venkatraman S <svenkatr@ti.com>]

On Tue, Jun 1, 2010 at 12:48 AM, Nishanth Menon <nm@ti.com> wrote:
> On 05/31/2010 09:46 PM, S, Venkatraman wrote:
>>
>> I understand that you might not have all platforms to test with, but
>> let's not provide a
>> 'generic feature api' without it being available for the supported
>> platforms.
>> It's incomplete without it.
>
> well.. if you did read 0/6
> http://marc.info/?l=linux-omap&m=127458581708411&w=2
> "Caveat: this series just introduces the framework by reorganizing
> the existing data, it does not attempt to define what the features
> for OMAP1,2,3,4 would be. As usual, comments are welcome.
> "

I did see this. My point was the 6/6 claims to do $foo (where $foo=
make it generic)
doesn't fit with a caveat description (My patch doesn't do $foo)

> :) anyways, these are how the three follow on patches will look like (sample
> omap1,2,4 patches are attached), feel free to ack them after testing - but
> this patch 6/6 needs to be done prior to the remaining being added - that is
> one more reason why i stopped where I did.

If you can post them formally as part of the series, I can test and
ack them (with OMAP3, OMAP4)
My original comment was even if these were not implemented due to some
constraints, they should be
mentioned in the code (as TODO / FIXME etc). The caveat description is
not going to show up in
commit logs or printks