This is a note to let you know that I've just added the patch titled Revert "driver core: Annotate dev_err_probe() with __must_check" to the 5.4-stable tree which can be found at: http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary The filename of the patch is: revert-driver-core-annotate-dev_err_probe-with-__must_check.patch and it can be found in the queue-5.4 subdirectory. If you, or anyone else, feels it should not be added to the stable tree, please let <stable@xxxxxxxxxxxxxxx> know about it. >From f601e8f37c2c1c52f2923fffc48204a7f7dc023d Mon Sep 17 00:00:00 2001 From: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> Date: Wed, 9 Sep 2020 09:37:40 +0200 Subject: Revert "driver core: Annotate dev_err_probe() with __must_check" From: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> commit f601e8f37c2c1c52f2923fffc48204a7f7dc023d upstream. This reverts commit e1f82a0dcf388d98bcc7ad195c03bd812405e6b2 as it's already starting to cause build warnings in linux-next for things that are "obviously correct". It's up to driver authors do "do the right thing" here with this function, and if they don't want to call it as the last line of a function, that's up to them, otherwise code that looks like: ret = dev_err_probe(..., ret, ...); does look really "odd". Reported-by: Stephen Rothwell <sfr@xxxxxxxxxxxxxxxx> Reported-by: Krzysztof Kozlowski <krzk@xxxxxxxxxx> Fixes: e1f82a0dcf38 ("driver core: Annotate dev_err_probe() with __must_check") Cc: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> Signed-off-by: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> Signed-off-by: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> --- include/linux/device.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) --- a/include/linux/device.h +++ b/include/linux/device.h @@ -1872,7 +1872,7 @@ do { \ dev_driver_string(dev), dev_name(dev), ## arg) extern __printf(3, 4) -int __must_check dev_err_probe(const struct device *dev, int err, const char *fmt, ...); +int 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) \ Patches currently in stable-queue which might be from gregkh@xxxxxxxxxxxxxxxxxxx are queue-5.4/fs-protect-reconfiguration-of-sb-read-write-from-racing-writes.patch queue-5.4/documentation-security-bugs.rst-update-preferences-when-dealing-with-the-linux-distros-group.patch queue-5.4/usb-dwc3-don-t-reset-device-side-if-dwc3-was-configured-as-host-only.patch queue-5.4/btrfs-qgroup-remove-one-time-use-variables-for-quota_root-checks.patch queue-5.4/loop-select-i-o-scheduler-none-from-inside-add_disk.patch queue-5.4/usb-serial-option-add-quectel-ec200a-module-support.patch queue-5.4/revert-usb-dwc3-core-enable-autoretry-feature-in-the-controller.patch queue-5.4/revert-usb-xhci-tegra-fix-error-check.patch queue-5.4/mtd-spinand-toshiba-fix-ecc_get_status.patch queue-5.4/acpi-processor-perflib-use-the-no-limit-frequency-qos.patch queue-5.4/mtd-rawnand-meson-fix-oob-available-bytes-for-ecc.patch queue-5.4/net-tap_open-set-sk_uid-from-current_fsuid.patch queue-5.4/usb-quirks-add-quirk-for-focusrite-scarlett.patch queue-5.4/usb-dwc3-pci-skip-byt-gpio-lookup-table-for-hardwired-phy.patch queue-5.4/net-usbnet-fix-warning-in-usbnet_start_xmit-usb_submit_urb.patch queue-5.4/driver-core-add-device-probe-log-helper.patch queue-5.4/documentation-security-bugs.rst-clarify-cve-handling.patch queue-5.4/asoc-cs42l51-fix-driver-to-properly-autoload-with-automatic-module-loading.patch queue-5.4/test_firmware-prevent-race-conditions-by-a-correct-implementation-of-locking.patch queue-5.4/net-tun_chr_open-set-sk_uid-from-current_fsuid.patch queue-5.4/acpi-processor-perflib-avoid-updating-frequency-qos-unnecessarily.patch queue-5.4/usb-serial-simple-sort-driver-entries.patch queue-5.4/serial-8250_dw-preserve-original-value-of-dlf-register.patch queue-5.4/btrfs-check-for-commit-error-at-btrfs_attach_transaction_barrier.patch queue-5.4/ext2-drop-fragment-support.patch queue-5.4/dm-cache-policy-smq-ensure-io-doesn-t-prevent-cleaner-policy-progress.patch queue-5.4/cpufreq-intel_pstate-drop-acpi-_pss-states-table-patching.patch queue-5.4/libceph-fix-potential-hang-in-ceph_osdc_notify.patch queue-5.4/fs-sysv-null-check-to-prevent-null-ptr-deref-bug.patch queue-5.4/tpm_tis-explicitly-check-for-error-code.patch queue-5.4/drivers-core-fix-kernel-doc-markup-for-dev_err_probe.patch queue-5.4/btrfs-qgroup-return-enotconn-instead-of-einval-when-quotas-are-not-enabled.patch queue-5.4/revert-driver-core-annotate-dev_err_probe-with-__must_check.patch queue-5.4/test_firmware-return-enomem-instead-of-enospc-on-failed-memory-allocation.patch queue-5.4/perf-fix-function-pointer-case.patch queue-5.4/virtio-net-fix-race-between-set-queues-and-probe.patch queue-5.4/hwmon-nct7802-fix-for-temp6-peci1-processed-even-if-peci1-disabled.patch queue-5.4/arm64-add-ampere1-to-the-spectre-bhb-affected-list.patch queue-5.4/net-sched-sch_qfq-account-for-stab-overhead-in-qfq_enqueue.patch queue-5.4/usb-xhci-mtk-set-the-dma-max_seg_size.patch queue-5.4/btrfs-fix-race-between-quota-disable-and-quota-assign-ioctls.patch queue-5.4/usb-serial-option-support-quectel-em060k_128.patch queue-5.4/driver-code-print-symbolic-error-code.patch queue-5.4/can-gs_usb-gs_can_close-add-missing-set-of-can-state-to-can_state_stopped.patch queue-5.4/usb-zaurus-add-id-for-a-300-b-500-c-700.patch queue-5.4/asoc-wm8904-fill-the-cache-for-wm8904_adc_test_0-register.patch queue-5.4/arm64-fix-bit-shifting-ub-in-the-midr_cpu_model-macro.patch queue-5.4/serial-sifive-fix-sifive_serial_console_setup-section.patch queue-5.4/staging-ks7010-potential-buffer-overflow-in-ks_wlan_set_encode_ext.patch queue-5.4/bluetooth-l2cap-fix-use-after-free-in-l2cap_sock_ready_cb.patch queue-5.4/usb-ohci-at91-fix-the-unhandle-interrupt-when-resume.patch queue-5.4/driver-core-annotate-dev_err_probe-with-__must_check.patch queue-5.4/usb-serial-simple-add-kaufmann-rks-can-vcp.patch queue-5.4/s390-dasd-fix-hanging-device-after-quiesce-resume.patch queue-5.4/scsi-zfcp-defer-fc_rport-blocking-until-after-adisc-response.patch