| Message ID | 1465368713-17866-1-git-send-email-m.szyprowski@samsung.com |
|---|---|
| State | Accepted |
| Commit | 9f5a802b1d51dc80a27d828a5f7dcc8ec4a72f03 |
| Headers | show |
On Wed, Jun 8, 2016 at 1:51 AM, Marek Szyprowski <m.szyprowski@samsung.com> wrote: > Change return value back to -ENODEV when no region is defined for given > device. This restores old behavior of this function, as some drivers rely > on such error code. > > Reported-by: Liviu Dudau <liviu.dudau@arm.com> > Fixes: 59ce4039727ef40 ("of: reserved_mem: add support for using more than > one region for given device") > Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com> > --- > drivers/of/of_reserved_mem.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Acked-by: Rob Herring <robh@kernel.org>
On 8 June 2016 at 18:35, Rob Herring <robh@kernel.org> wrote: > On Wed, Jun 8, 2016 at 1:51 AM, Marek Szyprowski > <m.szyprowski@samsung.com> wrote: >> Change return value back to -ENODEV when no region is defined for given >> device. This restores old behavior of this function, as some drivers rely >> on such error code. >> >> Reported-by: Liviu Dudau <liviu.dudau@arm.com> >> Fixes: 59ce4039727ef40 ("of: reserved_mem: add support for using more than >> one region for given device") >> Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com> >> --- >> drivers/of/of_reserved_mem.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) > Looks reasonable; FWIW Reviewed-by: Sumit Semwal <sumit.semwal@linaro.org> > Acked-by: Rob Herring <robh@kernel.org> > _______________________________________________ > dri-devel mailing list > dri-devel@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel
diff --git a/drivers/of/of_reserved_mem.c b/drivers/of/of_reserved_mem.c index 3cf129f..06af99f 100644 --- a/drivers/of/of_reserved_mem.c +++ b/drivers/of/of_reserved_mem.c @@ -334,7 +334,7 @@ int of_reserved_mem_device_init_by_idx(struct device *dev, target = of_parse_phandle(np, "memory-region", idx); if (!target) - return -EINVAL; + return -ENODEV; rmem = __find_rmem(target); of_node_put(target);
Change return value back to -ENODEV when no region is defined for given device. This restores old behavior of this function, as some drivers rely on such error code. Reported-by: Liviu Dudau <liviu.dudau@arm.com> Fixes: 59ce4039727ef40 ("of: reserved_mem: add support for using more than one region for given device") Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com> --- drivers/of/of_reserved_mem.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)