Re: [PATCH] [RFC] OMAP4: clock: shrink clock data utilizing preprocessor.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, May 13, 2011 at 2:30 PM, Tony Lindgren <tony@xxxxxxxxxxx> wrote:
> * Vladimir Zapolskiy <vzapolskiy@xxxxxxxxx> [110512 17:35]:
>> This mass change reduces homogeneous data chunks along clock
>> definitions. No semantical difference is added to the change,
>> and still it could be introduced easily.
>>
>> Signed-off-by: Vladimir Zapolskiy <vzapolskiy@xxxxxxxxx>
>> ---
>> Âarch/arm/mach-omap2/clock44xx_data.c | 3313 +++++++++-------------------------
>> Â1 files changed, 848 insertions(+), 2465 deletions(-)
>
> Hehehe, the diffstat certainly looks good :)
>
> This would be best done for all of them using some
> perl/sed/python script. And could be done for the hwmod data
> too. Do you already have something like that available?
>
Unfortunately I don't have an automated tool, but that would be great
to have such a script. For this time I've checked the correctness of the
change comparing the preprocessed output.

In fact if nobody has serious objections about the nature of the change
itself, then I'd like to continue with similar modifications for other OMAP
specific massive data files, hopefully there is a lot of redundant data to
eliminate :)

I can try to do that starting from writing the script for automation,
though I don't have a strong feeling that it's a straightforward task,
and preprocessed code check and/or runtime check might be preferable.

> Paul and & Benoit, what's your take on doing something like
> this? I'd assume updating your data generation scripts would
> be trivial?
>

With best wishes,
Vladimir
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux