Re: [PATCH v2] i2c: busses: remove duplicate dev_err()

From: Markus Elfring
Date: Thu Apr 16 2020 - 14:23:24 EST


> it will print an error message by itself when platform_get_irq()
> goes wrong. so don't need dev_err() in here again.

I suggest to improve the commit message considerably.

Would you like to integrate any information from a response
on the topic âdrm/meson: Delete an error message in meson_dw_hdmi_bind()â by
a contributor like Neil Armstrong?
https://lore.kernel.org/dri-devel/54e0b6e4-4a39-c567-7355-2b2330a95294@xxxxxxxxxxxx/
https://lore.kernel.org/patchwork/comment/1418937/
https://lkml.org/lkml/2020/4/15/186


> v1 -> v2:

Will a version identification be sufficient without the arrow?


> - add 19 similar changes by Markus's suggestion.

* With which search tool did you find the presented update candidates?

* Will a link to the commit âdriver core: platform: Add an error message
to platform_get_irq*()â (by Stephen Boyd from 2019-07-29) become helpful
as background information for such collateral evolution?
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/drivers/base/platform.c?id=7723f4c5ecdb8d832f049f8483beb0d1081cedf6

Regards,
Markus