Thread (3 messages) 3 messages, 2 authors, 2019-08-29

Re: [PATCH] [RFC] tty/serial: imx: make use of format specifier %dE

From: Andy Shevchenko <hidden>
Date: 2019-08-29 13:43:38
Also in: linux-doc, linux-serial, lkml

On Thu, Aug 29, 2019 at 7:40 AM Uwe Kleine-König [off-list ref] wrote:
I created a patch that teaches printk et al to emit a symbolic error
name for an error valued integer[1]. With that applied

        dev_err(&pdev->dev, "failed to get ipg clk: %dE\n", ret);

emits

        ... failed to get ipg clk: EPROBE_DEFER

if ret is -EPROBE_DEFER. Petr Mladek (i.e. one of the printk
maintainers) had concerns if this would be well received and worth the
effort. He asked to present it to a few subsystems. So for now, this
patch converting the imx UART driver shouldn't be applied yet but it
would be great to get some feedback about if you think that being able
to easily printk (for example) "EIO" instead of "-5" is a good idea.
Would it help you? Do you think it helps your users?
No, it makes sense only for debug where the user is supposed to be
developer and thus needs anyway to know code base better than average.


-- 
With Best Regards,
Andy Shevchenko

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help