lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Date:	Mon, 22 Apr 2013 09:07:58 -0400
From:	Eduardo Valentin <eduardo.valentin@...com>
To:	amit daniel kachhap <amit.daniel@...sung.com>
CC:	Eduardo Valentin <eduardo.valentin@...com>,
	"Zhang, Rui" <rui.zhang@...el.com>,
	"linux-pm@...r.kernel.org" <linux-pm@...r.kernel.org>,
	"linux-kernel@...r.kernel.org >> \"linux-kernel@...r.kernel.org\"" 
	<linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 00/30] thermal: cpu_cooling: code improvements

Amit,

On 21-04-2013 22:34, amit daniel kachhap wrote:
> Hi Eduardo,
>
> I looked at your whole series and they certainly looks useful, so for
> the entire series,
> Acked-by: Amit Daniel Kachhap <amit.daniel@...sung.com>
>

Ok, thanks.

Side note: Your Linaro email address is bouncing. Could you please send 
a patch updating the files you have signed?

> Thanks,
> Amit Daniel Kachhap
>
> On Thu, Apr 18, 2013 at 2:02 AM, Eduardo Valentin
> <eduardo.valentin@...com> wrote:
>> On 17-04-2013 13:11, Eduardo Valentin wrote:
>>>
>>> Rui,
>>
>>
>>
>> Looks like I forgot to add the lists on patch 00. Doing so..
>>>
>>>
>>> This is a series of patches to improve the code for cpu_cooling.
>>> Majority of these changes are actually kernel-doc updates, as
>>> they required to be revised. A part from these, there are some
>>> changes regarding code style and readiness.
>>>
>>> Some may complain that these are a matter of taste, indeed.
>>> But I tried to just be unbiased and just keep the changes related
>>> to kernel-doc and checkpatch.pl. After this series, theses
>>> tools should be generating decent output ;-).
>>>
>>> The remaining changes are actually a couple of fixes for defensive
>>> coding. A part from updating the MAINTAINERS entry to cover cpu_cooling.
>>>
>>> This series has been tested on TI OMAP4430, together with the
>>> thermal driver under staging tree.
>>>
>>> All these patches are based on current linux-next/master.
>>>
>>> They are also available here:
>>>
>>> All best,
>>>
>>> Eduardo Valentin (30):
>>>     thermal: cpu_cooling: remove unused headers
>>>     thermal: cpu_cooling: remove trailing white spaces
>>>     thermal: cpu_cooling: fix kernel_doc for cpufreq_cooling_device
>>>     thermal: cpu_cooling: use EXPORT_SYMBOL_GPL
>>>     thermal: cpu_cooling: remove compiler warning
>>>     thermal: cpu_cooling: standardize end of function
>>>     thermal: cpu_cooling: fix kernel doc for is_cpufreq_valid
>>>     thermal: cpu_cooling: add documentation for get_property
>>>     thermal: cpu_cooling: standardize comment style
>>>     thermal: cpu_cooling: align on open parenthesis
>>>     thermal: cpu_cooling: remove trailing blank line
>>>     thermal: cpu_cooling: document cpufreq_get_cooling_level
>>>     thermal: cpu_cooling: remove checkpatch.pl warning
>>>     thermal: cpu_cooling: improve documentation for get_cpu_frequency
>>>     thermal: cpu_cooling: update documentation for cpufreq_apply_cooling
>>>     thermal: cpu_cooling: update documentation for
>>>       cpufreq_thermal_notifier
>>>     thermal: cpu_cooling: update Kconfig entry
>>>     thermal: cpu_cooling: alignment improvements
>>>     thermal: cpu_cooling: update documentation for cpufreq_get_max_state
>>>     thermal: cpu_cooling: update documentation for cpufreq_get_cur_state
>>>     thermal: cpu_cooling: update documentation for cpufreq_set_cur_state
>>>     thermal: cpu_cooling: update kernel-doc for cpufreq_cooling_register
>>>     thermal: cpu_cooling: update kernel-doc comment for
>>>       cpufreq_cooling_unregister
>>>     thermal: cpu_cooling: use snprintf instead of sprintf
>>>     thermal: cpu_cooling: remove not needed curl brackets
>>>     thermal: cpu_cooling: remove unused symbols
>>>     thermal: cpu_cooling: add needed header for cpu_cooling.h
>>>     MAINTAINERS: update thermal entry by adding file cpu_cooling.h
>>>     thermal: cpu_cooling: remove unused symbols on cpu_cooling.h
>>>     thermal: cpu_cooling: improve line breaking
>>>
>>>    MAINTAINERS                   |   1 +
>>>    drivers/thermal/Kconfig       |   5 +-
>>>    drivers/thermal/cpu_cooling.c | 156
>>> ++++++++++++++++++++++++++++++------------
>>>    include/linux/cpu_cooling.h   |  20 +++---
>>>    4 files changed, 125 insertions(+), 57 deletions(-)
>>>
>>
>> --
>> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
>> the body of a message to majordomo@...r.kernel.org
>> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>> Please read the FAQ at  http://www.tux.org/lkml/
>
>

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ