Change return type of function sysc_check_one_child() from int to void as it always returns 0. Accordingly, at its callsite, delete the variable that previously stored the return value.
Signed-off-by: Nishka Dasgupta <nishkadg.li...@gmail.com> --- Changes in v2: - Remove error variable entirely. - Change return type of sysc_check_one_child(). drivers/bus/ti-sysc.c | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/drivers/bus/ti-sysc.c b/drivers/bus/ti-sysc.c index e6deabd8305d..1c30fa58d70c 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; @@ -633,12 +633,9 @@ static int sysc_check_one_child(struct sysc *ddata, 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; + sysc_check_one_child(ddata, child); } return 0; -- 2.19.1