From mboxrd@z Thu Jan 1 00:00:00 1970 From: Viresh Kumar Subject: Re: [PATCH V1 2/7] cpufreq: Loongson1: Update cpufreq of Loongson1B Date: Thu, 7 Apr 2016 12:26:55 +0530 Message-ID: <20160407065655.GH14903@vireshk-i7> References: <1459946095-7637-1-git-send-email-keguang.zhang@gmail.com> <1459946095-7637-3-git-send-email-keguang.zhang@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <1459946095-7637-3-git-send-email-keguang.zhang@gmail.com> Sender: linux-pm-owner@vger.kernel.org To: Keguang Zhang Cc: linux-mips@linux-mips.org, linux-clk@vger.kernel.org, linux-pm@vger.kernel.org, dmaengine@vger.kernel.org, linux-gpio@vger.kernel.org, linux-mtd@lists.infradead.org, Ralf Baechle , Michael Turquette , Stephen Boyd , "Rafael J. Wysocki" , Vinod Koul , Dan Williams , Linus Walleij , Alexandre Courbot , Boris Brezillon , Richard Weinberger , David Woodhouse , Brian Norris List-Id: linux-gpio@vger.kernel.org On 06-04-16, 20:34, Keguang Zhang wrote: > From: Kelvin Cheung > > - Rename the file to loongson1-cpufreq.c > - Use kcalloc() instead of kzalloc() > - Use devm_kzalloc() instead of global structure > - Use dev_get_platdata() to access the platform_data field > instead of referencing it directly > - Remove superfluous error messages > > Signed-off-by: Kelvin Cheung > --- > drivers/cpufreq/Makefile | 2 +- > drivers/cpufreq/loongson1-cpufreq.c | 230 ++++++++++++++++++++++++++++++++++++ > drivers/cpufreq/ls1x-cpufreq.c | 222 ---------------------------------- > 3 files changed, 231 insertions(+), 223 deletions(-) > create mode 100644 drivers/cpufreq/loongson1-cpufreq.c > delete mode 100644 drivers/cpufreq/ls1x-cpufreq.c This can't be reviewed. I am not going to compare them line by line to see what you might have changed in between. Please use '-C -M' options of git format-patch to generate this, it will give a review-able output of this. -- viresh