Change return type of function sysc_check_one_child() from int to void as it always returns 0. Remove the now-unnecessary return statement as well. Accordingly, at its call site, delete the error variable that previously stored the return value.
Signed-off-by: Nishka Dasgupta <[email protected]> --- Changes in v3: - Remove return statement in sysc_check_one_child(). - Remove braces at call site. Changes in v2: - Remove error variable entirely. - Change return type of sysc_check_one_child(). drivers/bus/ti-sysc.c | 14 ++++---------- 1 file changed, 4 insertions(+), 10 deletions(-) diff --git a/drivers/bus/ti-sysc.c b/drivers/bus/ti-sysc.c index e6deabd8305d..9c6d3e121d37 100644 --- a/drivers/bus/ti-sysc.c +++ b/drivers/bus/ti-sysc.c @@ -615,8 +615,8 @@ static void sysc_check_quirk_stdout(struct sysc *ddata, * node but children have "ti,hwmods". These belong to the interconnect * target node and are managed by this driver. */ -static int sysc_check_one_child(struct sysc *ddata, - struct device_node *np) +static void sysc_check_one_child(struct sysc *ddata, + struct device_node *np) { const char *name; @@ -626,20 +626,14 @@ static int sysc_check_one_child(struct sysc *ddata, sysc_check_quirk_stdout(ddata, np); sysc_parse_dts_quirks(ddata, np, true); - - return 0; } static int sysc_check_children(struct sysc *ddata) { struct device_node *child; - int error; - for_each_child_of_node(ddata->dev->of_node, child) { - error = sysc_check_one_child(ddata, child); - if (error) - return error; - } + for_each_child_of_node(ddata->dev->of_node, child) + sysc_check_one_child(ddata, child); return 0; } -- 2.19.1
