[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <f9f63e49-41f4-4598-3554-ac53ba4dcd55@linaro.org>
Date: Wed, 16 Jun 2021 15:56:48 +0200
From: Daniel Lezcano <daniel.lezcano@...aro.org>
To: 周琰杰 (Zhou Yanjie)
<zhouyanjie@...yeetech.com>, tglx@...utronix.de
Cc: linux-mips@...r.kernel.org, linux-kernel@...r.kernel.org,
dongsheng.qiu@...enic.com, aric.pzqi@...enic.com,
rick.tyliu@...enic.com, sihui.liu@...enic.com,
jun.jiang@...enic.com, sernia.zhou@...mail.com,
paul@...pouillou.net
Subject: Re: [PATCH v2 1/2] clocksource: Ingenic: Rename unreasonable array
names.
On 04/06/2021 18:31, 周琰杰 (Zhou Yanjie) wrote:
> 1.Rename the "ingenic_ost_clk_info[]" to "x1000_ost_clk_info[]"
> to facilitate the addition of OST support for X2000 SoC in a
> later commit.
> 2.When the OST support for X2000 SoC is added, there will be two
> compatible strings, so renaming "ingenic_ost_of_match[]" to
> "ingenic_ost_of_matches[]" is more reasonable.
> 3.Remove the unnecessary comma in "ingenic_ost_of_matches[]" to
> reduce code size as much as possible.
>
> Signed-off-by: 周琰杰 (Zhou Yanjie) <zhouyanjie@...yeetech.com>
> ---
I have applied this patch but not the 2/2.
Thanks
-- Daniel
--
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs
Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog
Powered by blists - more mailing lists