| Message ID | 20170718214339.7774-22-robh@kernel.org |
|---|---|
| State | Accepted |
| Commit | 13bf35b571950f664ac48cd7b593177f1ba6dafd |
| Headers | show |
On Tue, Jul 18, 2017 at 04:43:02PM -0500, Rob Herring wrote: > Now that we have a custom printf format specifier, convert users of > full_name to use %pOF instead. This is preparation to remove storing > of the full path string for each node. > > Signed-off-by: Rob Herring <robh@kernel.org> > Cc: Alan Tull <atull@kernel.org> > Cc: Moritz Fischer <moritz.fischer@ettus.com> > Cc: linux-fpga@vger.kernel.org Acked-by: Moritz Fischer <mdf@kernel.org> > --- > drivers/fpga/fpga-region.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/fpga/fpga-region.c b/drivers/fpga/fpga-region.c > index 3b6b2f4182a1..d9ab7c75b14f 100644 > --- a/drivers/fpga/fpga-region.c > +++ b/drivers/fpga/fpga-region.c > @@ -319,8 +319,8 @@ static int child_regions_with_firmware(struct device_node *overlay) > of_node_put(child_region); > > if (ret) > - pr_err("firmware-name not allowed in child FPGA region: %s", > - child_region->full_name); > + pr_err("firmware-name not allowed in child FPGA region: %pOF", > + child_region); > > return ret; > } > -- > 2.11.0 >
On Tue, Jul 18, 2017 at 11:42 PM, Moritz Fischer <mdf@kernel.org> wrote: > On Tue, Jul 18, 2017 at 04:43:02PM -0500, Rob Herring wrote: >> Now that we have a custom printf format specifier, convert users of >> full_name to use %pOF instead. This is preparation to remove storing >> of the full path string for each node. >> >> Signed-off-by: Rob Herring <robh@kernel.org> >> Cc: Alan Tull <atull@kernel.org> >> Cc: Moritz Fischer <moritz.fischer@ettus.com> >> Cc: linux-fpga@vger.kernel.org > Acked-by: Moritz Fischer <mdf@kernel.org> Acked-by: Alan Tull <atull@kernel.org> >> --- >> drivers/fpga/fpga-region.c | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/drivers/fpga/fpga-region.c b/drivers/fpga/fpga-region.c >> index 3b6b2f4182a1..d9ab7c75b14f 100644 >> --- a/drivers/fpga/fpga-region.c >> +++ b/drivers/fpga/fpga-region.c >> @@ -319,8 +319,8 @@ static int child_regions_with_firmware(struct device_node *overlay) >> of_node_put(child_region); >> >> if (ret) >> - pr_err("firmware-name not allowed in child FPGA region: %s", >> - child_region->full_name); >> + pr_err("firmware-name not allowed in child FPGA region: %pOF", >> + child_region); >> >> return ret; >> } >> -- >> 2.11.0 >>
diff --git a/drivers/fpga/fpga-region.c b/drivers/fpga/fpga-region.c index 3b6b2f4182a1..d9ab7c75b14f 100644 --- a/drivers/fpga/fpga-region.c +++ b/drivers/fpga/fpga-region.c @@ -319,8 +319,8 @@ static int child_regions_with_firmware(struct device_node *overlay) of_node_put(child_region); if (ret) - pr_err("firmware-name not allowed in child FPGA region: %s", - child_region->full_name); + pr_err("firmware-name not allowed in child FPGA region: %pOF", + child_region); return ret; }
Now that we have a custom printf format specifier, convert users of full_name to use %pOF instead. This is preparation to remove storing of the full path string for each node. Signed-off-by: Rob Herring <robh@kernel.org> Cc: Alan Tull <atull@kernel.org> Cc: Moritz Fischer <moritz.fischer@ettus.com> Cc: linux-fpga@vger.kernel.org --- drivers/fpga/fpga-region.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) -- 2.11.0