On Tue, Aug 28, 2018 at 4:57 AM Rob Herring <r...@kernel.org> wrote: > In preparation to remove the node name pointer from struct device_node, > convert printf users to use the %pOFn format specifier.
> + snprintf((char *)dai->platform_name, DAI_NAME_SIZE, "%llx.%pOFn", > + (unsigned long long) res.start, dma_channel_np); I understand that is not the part of the change, but wouldn't be good to replace that ugly casting with proper specifier as well? I mean %pa ... &res.start > + dev_err(&pdev->dev, "%pOFn sync own reference\n", > + np); One line? -- With Best Regards, Andy Shevchenko