riscv: propagate insert_resource result from add_resource

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).

Use %pR to print the full resource range while at it.

Signed-off-by: Thorsten Blum <thorsten.blum@linux.dev>
Link: https://patch.msgid.link/20260512172034.328405-4-thorsten.blum@linux.dev
Signed-off-by: Paul Walmsley <pjw@kernel.org>
This commit is contained in:
Thorsten Blum 2026-06-06 20:17:52 -06:00 committed by Paul Walmsley
parent 7d0dec57e5
commit 13fe217f64

View File

@ -71,16 +71,13 @@ 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) {
pr_err("Failed to add a %s resource at %llx\n",
res->name, (unsigned long long) res->start);
return ret;
}
if (ret < 0)
pr_err("Failed to add resource %s %pR\n", res->name, res);
return 1;
return ret;
}
static int __init add_kernel_resources(void)