[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-Id: <20220218000922.134857-1-j.neuschaefer@gmx.net>
Date: Fri, 18 Feb 2022 01:09:16 +0100
From: Jonathan Neuschäfer <j.neuschaefer@....net>
To: linux-clk@...r.kernel.org
Cc: linux-kernel@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
linux-mips@...r.kernel.org,
Philippe Mathieu-Daudé <f4bug@...at.org>,
Jonathan Neuschäfer <j.neuschaefer@....net>
Subject: [PATCH v4 0/5] clk drivers: Terminate clk_div_table with sentinel element
I noticed that some of the clk_div_tables in different drivers are not
terminated with a sentinel element. This may cause the code in
clk-divider.c to read garbage that happens to be beyond the end.
This patchset fixed all instances of this bug that I could find, except
for a case in drivers/phy/ti/phy-j721e-wiz.c that is already fixed in
linux-next:
https://lore.kernel.org/lkml/20220117110108.4117-1-kishon@ti.com/
In this version of the patchset, I addressed Philippe Mathieu-Daudé's
style comments. As far as I found it reasonable, I simply updated the
existing patches, but in the case of the Actions Semiconductor drivers,
I made a bigger cleanup in a separate patch.
v4:
- Various style fixes
v3:
- Change the Actions patch according to Mani's comment
v2:
- Add Fixes tags
Jonathan Neuschäfer (5):
clk: actions: Terminate clk_div_table with sentinel element
clk: loongson1: Terminate clk_div_table with sentinel element
clk: hisilicon: Terminate clk_div_table with sentinel element
clk: clps711x: Terminate clk_div_table with sentinel element
clk: actions: Make sentinel elements more obvious
drivers/clk/actions/owl-s500.c | 16 ++++++++--------
drivers/clk/actions/owl-s700.c | 17 ++++++++++-------
drivers/clk/actions/owl-s900.c | 26 +++++++++++++-------------
drivers/clk/clk-clps711x.c | 2 ++
drivers/clk/hisilicon/clk-hi3559a.c | 4 ++--
drivers/clk/loongson1/clk-loongson1c.c | 1 +
6 files changed, 36 insertions(+), 30 deletions(-)
--
2.34.1
Powered by blists - more mailing lists