diff options
author | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2020-09-09 09:37:40 +0200 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2023-08-11 11:54:02 +0200 |
commit | 9e5374875f96192099cd773c1b45079a986f8aa1 (patch) | |
tree | ed98531b59b0effbebccde4b19bee6f8db548aa4 /ipc/msgutil.c | |
parent | ca33c070415166b094c2750a6190846473c95802 (diff) | |
download | linux-stable-9e5374875f96192099cd773c1b45079a986f8aa1.tar.gz linux-stable-9e5374875f96192099cd773c1b45079a986f8aa1.tar.bz2 linux-stable-9e5374875f96192099cd773c1b45079a986f8aa1.zip |
Revert "driver core: Annotate dev_err_probe() with __must_check"
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@canb.auug.org.au>
Reported-by: Krzysztof Kozlowski <krzk@kernel.org>
Fixes: e1f82a0dcf38 ("driver core: Annotate dev_err_probe() with __must_check")
Cc: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'ipc/msgutil.c')
0 files changed, 0 insertions, 0 deletions