On Tue, Aug 28, 2018 at 4:57 AM Rob Herring <robh@xxxxxxxxxx> 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 _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
