From 7c822e139a4c7faabbc0141547f4f76adae70a67 Mon Sep 17 00:00:00 2001 From: Greg Kroah-Hartman Date: Fri, 26 Jan 2024 17:33:19 -0800 Subject: [PATCH] 4.19-stable patches added patches: driver-code-print-symbolic-error-code.patch driver-core-annotate-dev_err_probe-with-__must_check.patch drivers-core-fix-kernel-doc-markup-for-dev_err_probe.patch --- ...river-code-print-symbolic-error-code.patch | 37 ++++++++++++++++ ...tate-dev_err_probe-with-__must_check.patch | 34 +++++++++++++++ ...-kernel-doc-markup-for-dev_err_probe.patch | 43 +++++++++++++++++++ queue-4.19/series | 3 ++ 4 files changed, 117 insertions(+) create mode 100644 queue-4.19/driver-code-print-symbolic-error-code.patch create mode 100644 queue-4.19/driver-core-annotate-dev_err_probe-with-__must_check.patch create mode 100644 queue-4.19/drivers-core-fix-kernel-doc-markup-for-dev_err_probe.patch diff --git a/queue-4.19/driver-code-print-symbolic-error-code.patch b/queue-4.19/driver-code-print-symbolic-error-code.patch new file mode 100644 index 00000000000..30b3fd054a0 --- /dev/null +++ b/queue-4.19/driver-code-print-symbolic-error-code.patch @@ -0,0 +1,37 @@ +From 04ece65d9bad8638c9b38f3edabc6e516335edf8 Mon Sep 17 00:00:00 2001 +From: =?UTF-8?q?Micha=C5=82=20Miros=C5=82aw?= +Date: Fri, 28 Aug 2020 18:14:35 +0200 +Subject: driver code: print symbolic error code +MIME-Version: 1.0 +Content-Type: text/plain; charset=UTF-8 +Content-Transfer-Encoding: 8bit + +From: Michał Mirosław + +commit 693a8e936590f93451e6f5a3d748616f5a59c80b upstream. + +dev_err_probe() prepends the message with an error code. Let's make it +more readable by translating the code to a more recognisable symbol. + +Fixes: a787e5400a1c ("driver core: add device probe log helper") +Signed-off-by: Michał Mirosław +Link: https://lore.kernel.org/r/ea3f973e4708919573026fdce52c264db147626d.1598630856.git.mirq-linux@rere.qmqm.pl +Signed-off-by: Greg Kroah-Hartman +--- + drivers/base/core.c | 4 ++-- + 1 file changed, 2 insertions(+), 2 deletions(-) + +--- a/drivers/base/core.c ++++ b/drivers/base/core.c +@@ -3366,9 +3366,9 @@ int dev_err_probe(const struct device *d + vaf.va = &args; + + if (err != -EPROBE_DEFER) +- dev_err(dev, "error %d: %pV", err, &vaf); ++ dev_err(dev, "error %pe: %pV", ERR_PTR(err), &vaf); + else +- dev_dbg(dev, "error %d: %pV", err, &vaf); ++ dev_dbg(dev, "error %pe: %pV", ERR_PTR(err), &vaf); + + va_end(args); + diff --git a/queue-4.19/driver-core-annotate-dev_err_probe-with-__must_check.patch b/queue-4.19/driver-core-annotate-dev_err_probe-with-__must_check.patch new file mode 100644 index 00000000000..4c916cff360 --- /dev/null +++ b/queue-4.19/driver-core-annotate-dev_err_probe-with-__must_check.patch @@ -0,0 +1,34 @@ +From e1f82a0dcf388d98bcc7ad195c03bd812405e6b2 Mon Sep 17 00:00:00 2001 +From: Andy Shevchenko +Date: Wed, 26 Aug 2020 13:44:59 +0300 +Subject: driver core: Annotate dev_err_probe() with __must_check + +From: Andy Shevchenko + +commit e1f82a0dcf388d98bcc7ad195c03bd812405e6b2 upstream. + +We have got already new users of this API which interpret it differently +and miss the opportunity to optimize their code. + +In order to avoid similar cases in the future, annotate dev_err_probe() +with __must_check. + +Fixes: a787e5400a1c ("driver core: add device probe log helper") +Signed-off-by: Andy Shevchenko +Link: https://lore.kernel.org/r/20200826104459.81979-1-andriy.shevchenko@linux.intel.com +Signed-off-by: Greg Kroah-Hartman +--- + include/linux/device.h | 2 +- + 1 file changed, 1 insertion(+), 1 deletion(-) + +--- a/include/linux/device.h ++++ b/include/linux/device.h +@@ -1582,7 +1582,7 @@ do { \ + dev_driver_string(dev), dev_name(dev), ## arg) + + extern __printf(3, 4) +-int dev_err_probe(const struct device *dev, int err, const char *fmt, ...); ++int __must_check dev_err_probe(const struct device *dev, int err, const char *fmt, ...); + + /* Create alias, so I can be autoloaded. */ + #define MODULE_ALIAS_CHARDEV(major,minor) \ diff --git a/queue-4.19/drivers-core-fix-kernel-doc-markup-for-dev_err_probe.patch b/queue-4.19/drivers-core-fix-kernel-doc-markup-for-dev_err_probe.patch new file mode 100644 index 00000000000..89290b72353 --- /dev/null +++ b/queue-4.19/drivers-core-fix-kernel-doc-markup-for-dev_err_probe.patch @@ -0,0 +1,43 @@ +From ca33c070415166b094c2750a6190846473c95802 Mon Sep 17 00:00:00 2001 +From: Mauro Carvalho Chehab +Date: Wed, 9 Sep 2020 11:53:43 +0200 +Subject: drivers: core: fix kernel-doc markup for dev_err_probe() + +From: Mauro Carvalho Chehab + +commit 074b3aad307de6126fbac1fff4996d1034b48fee upstream. + +There are two literal blocks there. Fix the markups, in order +to produce the right html output and solve those warnings: + + ./drivers/base/core.c:4218: WARNING: Unexpected indentation. + ./drivers/base/core.c:4222: WARNING: Definition list ends without a blank line; unexpected unindent. + ./drivers/base/core.c:4223: WARNING: Block quote ends without a blank line; unexpected unindent. + +Fixes: a787e5400a1c ("driver core: add device probe log helper") +Signed-off-by: Mauro Carvalho Chehab +Signed-off-by: Greg Kroah-Hartman +--- + drivers/base/core.c | 6 ++++-- + 1 file changed, 4 insertions(+), 2 deletions(-) + +--- a/drivers/base/core.c ++++ b/drivers/base/core.c +@@ -3344,13 +3344,15 @@ define_dev_printk_level(_dev_info, KERN_ + * This helper implements common pattern present in probe functions for error + * checking: print debug or error message depending if the error value is + * -EPROBE_DEFER and propagate error upwards. +- * It replaces code sequence: ++ * It replaces code sequence:: + * if (err != -EPROBE_DEFER) + * dev_err(dev, ...); + * else + * dev_dbg(dev, ...); + * return err; +- * with ++ * ++ * with:: ++ * + * return dev_err_probe(dev, err, ...); + * + * Returns @err. diff --git a/queue-4.19/series b/queue-4.19/series index 95c05be6064..b49ba89bd5e 100644 --- a/queue-4.19/series +++ b/queue-4.19/series @@ -17,3 +17,6 @@ nouveau-vmm-don-t-set-addr-on-the-fail-path-to-avoid-warning.patch block-remove-special-casing-of-compound-pages.patch powerpc-use-always-instead-of-always-y-in-for-crtsavres.o.patch x86-cpu-amd-fix-disabling-xsaves-on-amd-family-0x17-due-to-erratum.patch +driver-core-annotate-dev_err_probe-with-__must_check.patch +driver-code-print-symbolic-error-code.patch +drivers-core-fix-kernel-doc-markup-for-dev_err_probe.patch -- 2.47.3