Change return type of of_clk_get_parent_count() to unsigned
Get series with b4
b4 am [email protected]Versions newest first
- v1 Latest9 patches 2 repliesIn review
Patches in v1 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/9 | clk: Make of_clk_get_parent_count() return unsigned intsdrivers/ include/ | +15 | 0 replies | |
| 2/9 | clk: at91: Remove impossible checks for of_clk_get_parent_count()drivers/ drivers/ drivers/ +4 more | +15 | 0 replies | |
| 3/9 | clk: gpio: Remove impossible check for of_clk_get_parent_count() < 0drivers/ | +2 | 0 replies | |
| 4/9 | clk: h8300: Remove impossible check for of_clk_get_parent_count()drivers/ drivers/ | +4 | 0 replies | |
| 5/9 | clk: st: Remove impossible check for of_clk_get_parent_count() < 0drivers/ drivers/ | +4 | 0 replies | |
| 6/9 | clk: sunxi: Use proper type for of_clk_get_parent_count() return valuedrivers/ | +1 | 0 replies | |
| 7/9 | clk: ti: Update for of_clk_get_parent_count() returning unsigned intdrivers/ drivers/ drivers/ +1 more | +8 | 0 replies | |
| 8/9 | usb: dwc3: Remove impossible check for of_clk_get_parent_count() < 0drivers/ | +5 | 1 reply | |
| 9/9 | simplefb: Remove impossible check for of_clk_get_parent_count() < 0drivers/ | +2 | 1 reply |
Reviews trailers per patch
2 of 9 patches have a Reviewed-by or Acked-by.
| Patch | Felipe Balbi | Tomi Valkeinen |
|---|---|---|
| 1/9 | ||
| 2/9 | ||
| 3/9 | ||
| 4/9 | ||
| 5/9 | ||
| 6/9 | ||
| 7/9 | ||
| 8/9 | A-b | |
| 9/9 | A-b |
Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.
Cover letter [PATCH 0/9] Change return type of of_clk_get_parent_count() to unsigned
Russell King recently pointed out that of_clk_get_parent_count() can return a negative number if the "clocks" property isn't present in a node. Upon investigating all the callers of this API, none of the them seem to care about that case, and furthermore some callers are assigning the return value to an unsigned int which could cause very large numbers of parents to be used.
This series changes the API to return an unsigned int from 0 (no parents) to N, thereby avoiding any problems with negative return values.
Stephen Boyd (9):
clk: Make of_clk_get_parent_count() return unsigned ints
clk: at91: Remove impossible checks for of_clk_get_parent_count()
clk: gpio: Remove impossible check for of_clk_get_parent_count() < 0
clk: h8300: Remove impossible check for of_clk_get_parent_count()
clk: st: Remove impossible check for of_clk_get_parent_count() < 0
clk: sunxi: Use proper type for of_clk_get_parent_count() return value
clk: ti: Update for of_clk_get_parent_count() returning unsigned int
usb: dwc3: Remove impossible check for of_clk_get_parent_count() < 0
simplefb: Remove impossible check for of_clk_get_parent_count() < 0
drivers/clk/at91/clk-generated.c | 4 ++-- drivers/clk/at91/clk-main.c | 4 ++-- drivers/clk/at91/clk-master.c | 4 ++-- drivers/clk/at91/clk-programmable.c | 4 ++-- drivers/clk/at91/clk-slow.c | 6 +++--- drivers/clk/at91/clk-smd.c | 4 ++-- drivers/clk/at91/clk-usb.c | 4 ++-- drivers/clk/clk-gpio.c | 6 ++---- drivers/clk/clk.c | 16 ++++++++++++++-- drivers/clk/h8300/clk-div.c | 4 ++-- drivers/clk/h8300/clk-h8s2678.c | 4 ++-- drivers/clk/st/clk-flexgen.c | 4 ++-- drivers/clk/st/clkgen-mux.c | 4 ++-- drivers/clk/sunxi/clk-sun6i-ar100.c | 2 +- drivers/clk/ti/clockdomain.c | 2 +- drivers/clk/ti/composite.c | 8 ++++---- drivers/clk/ti/dpll.c | 2 +- drivers/clk/ti/mux.c | 4 ++-- drivers/usb/dwc3/dwc3-of-simple.c | 9 +++++---- drivers/video/fbdev/simplefb.c | 4 ++-- include/linux/clk-provider.h | 2 +- 21 files changed, 56 insertions(+), 45 deletions(-)
Signature · 2 lines
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project