On Thursday, July 24, 2014 09:04:02 AM Lukasz Majewski wrote:

Hi Rafael,

quoted

On Monday, July 21, 2014 09:02:34 AM Lukasz Majewski wrote:

quoted

This commit adds first regression test "cpufreq_freq_test.sh" for
the cpufreq subsystem.

First of all, I'm not seeing any explanation why this script should be
shipped with the kernel.

OK.

quoted

What regressions it tests against in particular and 

Do you require SHA's/commit messages of commits which were developed to
fix issues spotted with this test script?

No.  I want information about what kind of bugs can be catched with the
help of this script.

quoted

how it does that.

Is this information required in the commit message or can it stay in
the README file created in the same commit?

There should be *some* information in the changelog too.

-- 
I speak only for myself.
Rafael J. Wysocki, Intel Open Source Technology Center.