Hi Andy, Thanks for the patch! On 11/15/19 4:30 PM, Andy Shevchenko wrote: > Replace explicit use of members of struct resource by %pR to print > the resource. > > Signed-off-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> > --- > drivers/media/platform/qcom/venus/firmware.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) Acked-by: Stanimir Varbanov <stanimir.varbanov@xxxxxxxxxx> > > diff --git a/drivers/media/platform/qcom/venus/firmware.c b/drivers/media/platform/qcom/venus/firmware.c > index d3d1748a7ef6..de6812fb55f4 100644 > --- a/drivers/media/platform/qcom/venus/firmware.c > +++ b/drivers/media/platform/qcom/venus/firmware.c > @@ -100,8 +100,7 @@ static int venus_load_fw(struct venus_core *core, const char *fwname, > > mem_va = memremap(r.start, *mem_size, MEMREMAP_WC); > if (!mem_va) { > - dev_err(dev, "unable to map memory region: %pa+%zx\n", > - &r.start, *mem_size); > + dev_err(dev, "unable to map memory region: %pR\n", &r); > ret = -ENOMEM; > goto err_release_fw; > } > -- regards, Stan