linux-kernel - Re: [PATCH V5 0/4] cpufreq: exynos5440: support for cpufreq driver

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] [thread-next>] [day] [month] [year] [list]

Date:	Mon, 8 Apr 2013 14:06:38 +0530
From:	amit daniel kachhap <amit.daniel@...sung.com>
To:	"Rafael J. Wysocki" <rjw@...k.pl>,
	Kukjin Kim <kgene.kim@...sung.com>
Cc:	linux-pm@...r.kernel.org, linux-kernel@...r.kernel.org,
	linux-samsung-soc@...r.kernel.org,
	linux-arm-kernel@...ts.infradead.org,
	Thomas Abraham <thomas.abraham@...aro.org>,
	cpufreq@...r.kernel.org,
	Inderpal Singh <inderpal.singh@...aro.org>,
	Viresh Kumar <viresh.kumar@...aro.org>,
	Sylwester Nawrocki <s.nawrocki@...sung.com>,
	Russell King - ARM Linux <linux@....linux.org.uk>
Subject: Re: [PATCH V5 0/4] cpufreq: exynos5440: support for cpufreq driver

On Fri, Apr 5, 2013 at 6:08 PM, Rafael J. Wysocki <rjw@...k.pl> wrote:
> On Thursday, March 28, 2013 01:35:18 PM Amit Daniel Kachhap wrote:
>> Submitting the V5 version of exynos5440 cpufreq driver. This patchset addresses
>> all the coding and design concerns raised especially by Viresh.
>>
>> Changes in V5:
>> * Removed the unnecessary DT look up entry from mach-exynos5-dt.c.
>> * Fixed all coding and comments issue raised by Viresh.
>>
>> Changes in V4:
>> * Added dev_err logs instead of pr_err.
>> * Used the devm_ioremap_resource API.
>> * Implemented several coding guidelines and minor error comments from Sylwester,
>>   Russell and Viresh.
>>
>> Changes in V3:
>> * Converted the driver to probe based as suggested by Viresh. This is also
>>   beneficial for multiplatform kernel.
>> * Other coding guidelines related changes.
>> * Moved the DT node outside cpu0 node as the driver is now a platform
>>   driver.
>>
>> Changes in V2:
>> * Added OPP library support to parse DT parameters.
>> * Removed a hack to handle interrupts in bootup.
>> * Implemented other review comments from Viresh and Inder.
>>
>> All these patches are dependent on Thomas Abraham common clock patches.
>> (http://www.mail-archive.com/linux-samsung-soc@vger.kernel.org/msg15860.html)
>> This whole patch series is based on 3.9-rc4.
>>
>> Amit Daniel Kachhap (4):
>>   cpufreq: exynos: Add cpufreq driver for exynos5440
>>   cpufreq: exynos: Remove error return even if no soc is found
>>   arm: exynos: Enable OPP library support for exynos5440
>>   arm: dts: Add cpufreq controller node for Exynos5440 SoC
>>
>>  .../bindings/cpufreq/cpufreq-exynos5440.txt        |   28 ++
>>  arch/arm/boot/dts/exynos5440.dtsi                  |   12 +
>>  arch/arm/mach-exynos/Kconfig                       |    2 +
>>  drivers/cpufreq/Kconfig.arm                        |    9 +
>>  drivers/cpufreq/Makefile                           |    1 +
>>  drivers/cpufreq/exynos-cpufreq.c                   |    2 +-
>>  drivers/cpufreq/exynos5440-cpufreq.c               |  474 ++++++++++++++++++++
>>  7 files changed, 527 insertions(+), 1 deletions(-)
>>  create mode 100644 Documentation/devicetree/bindings/cpufreq/cpufreq-exynos5440.txt
>>  create mode 100644 drivers/cpufreq/exynos5440-cpufreq.c
>
> Is this for me or the Samsung tree?
>
> If it's for me, I need ACKs from the Samsung tree maintainers.  Also, please
> address the Viresh's comments.
>
> Thanks,
> Rafael
>
Hi Rafael,

I submitted the V6 version with all comments from Viresh addressed.

Hi Kukjin Kim,

Any suggestion on how this patch should be merged. In my opinion it
should go via Rafael's tree as this patch uses some fixes present in
Rafael's tree like modified cpufreq_notify_transition API.

Thanks,
Amit Daniel
>
> --
> I speak only for myself.
> Rafael J. Wysocki, Intel Open Source Technology Center.
> --
> 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/