Re: [PATCH] riscv: propagate insert_resource result from add_resource

[Ben Dooks <ben.dooks@codethink.co.uk>]

On 12/05/2026 13:46, Thorsten Blum wrote:
> Currently, add_resource() returns 1 on success, even though its callers
> only check for negative values. Instead, propagate the insert_resource()
> result from add_resource() to align with standard kernel return-value
> conventions (0 on success, negative errno on failure).
> 
> Signed-off-by: Thorsten Blum <thorsten.blum@linux.dev>
> ---
>   arch/riscv/kernel/setup.c | 8 +++-----
>   1 file changed, 3 insertions(+), 5 deletions(-)
> 
> diff --git a/arch/riscv/kernel/setup.c b/arch/riscv/kernel/setup.c
> index c89cc272440b..6e3d8acae71a 100644
> --- a/arch/riscv/kernel/setup.c
> +++ b/arch/riscv/kernel/setup.c
> @@ -71,16 +71,14 @@ static struct resource *standard_resources;
>   static int __init add_resource(struct resource *parent,
>   				struct resource *res)
>   {
> -	int ret = 0;
> +	int ret;
>   
>   	ret = insert_resource(parent, res);
> -	if (ret < 0) {
> +	if (ret < 0)
>   		pr_err("Failed to add a %s resource at %llx\n",
>   			res->name, (unsigned long long) res->start);


Maybe just use %pR for the resource start/end here and avoid
the casting.


-- 
Ben Dooks				http://www.codethink.co.uk/
Senior Engineer				Codethink - Providing Genius

https://www.codethink.co.uk/privacy.html