On Tue, 2012-10-09 at 11:42 +0100, Sudeep KarkadaNagesha wrote:
On 09/10/12 11:33, Amit Kucheria wrote:
On Tue, Oct 9, 2012 at 2:13 PM, Jon Medhurst (Tixy) tixy@linaro.org wrote:
On Tue, 2012-10-09 at 09:27 +0100, Sudeep KarkadaNagesha wrote:
On 09/10/12 08:10, Viresh Kumar wrote:
Following is the declaration of name field in struct cpufreq_driver: char name[CPUFREQ_NAME_LEN];
where CPUFREQ_NAME_LEN is 16.
So, length of drivers name must be <=15 (one position for '\0'). Current name is crossing this limit and so name doesn't get printed properly when we do:
$ cat /sys/devices/system/cpu/cpu0/cpufreq/scaling_driver
This patch renames driver-name to fix this issue.
Signed-off-by: Viresh Kumar viresh.kumar@linaro.org
Acked-by: Sudeep KarkadaNagesha sudeep.karkadanagesha@arm.com
Shall I apply this to the ARM Landing Team vexpress tree then?
I'd say yes, the driver lives in your tree. I'm not sure how you coordinate the code drops with ARM since it should be in their tree as well for the future.
Robin raised a valid point that this driver is TC2 specific and bL is not appropriate name for it including the file name(my bad choice at first).
Considering the confusion over what the correct naming should be, I don't think it's worth me apply the renaming patch at the moment ;-)