4.9-stable patches

added patches:
	x86-olpc-fix-logical-not-is-only-applied-to-the-left-hand-side.patch

diff --git a/queue-4.9/series b/queue-4.9/series
index 4124e910efcabd39a3fe2ce675d45d7239981cba..e55b0578d50a3521f4ee34322853e8b5fdfd7756 100644 (file)
--- a/queue-4.9/series
+++ b/queue-4.9/series
@@ -46,3 +46,4 @@ usb-hcd-fix-urb-giveback-issue-in-tasklet-function.patch
 usb-gadget-fix-use-after-free-read-in-usb_udc_uevent.patch
 netfilter-nf_tables-fix-null-deref-due-to-zeroed-list-head.patch
 scsi-zfcp-fix-missing-auto-port-scan-and-thus-missing-target-ports.patch
+x86-olpc-fix-logical-not-is-only-applied-to-the-left-hand-side.patch

diff --git a/queue-4.9/x86-olpc-fix-logical-not-is-only-applied-to-the-left-hand-side.patch b/queue-4.9/x86-olpc-fix-logical-not-is-only-applied-to-the-left-hand-side.patch
new file mode 100644 (file)
index 0000000..0923db5
--- /dev/null
+++ b/queue-4.9/x86-olpc-fix-logical-not-is-only-applied-to-the-left-hand-side.patch
@@ -0,0 +1,49 @@
+From 3a2ba42cbd0b669ce3837ba400905f93dd06c79f Mon Sep 17 00:00:00 2001
+From: Alexander Lobakin <alexandr.lobakin@intel.com>
+Date: Fri, 15 Jul 2022 17:15:36 +0200
+Subject: x86/olpc: fix 'logical not is only applied to the left hand side'
+
+From: Alexander Lobakin <alexandr.lobakin@intel.com>
+
+commit 3a2ba42cbd0b669ce3837ba400905f93dd06c79f upstream.
+
+The bitops compile-time optimization series revealed one more
+problem in olpc-xo1-sci.c:send_ebook_state(), resulted in GCC
+warnings:
+
+arch/x86/platform/olpc/olpc-xo1-sci.c: In function 'send_ebook_state':
+arch/x86/platform/olpc/olpc-xo1-sci.c:83:63: warning: logical not is only applied to the left hand side of comparison [-Wlogical-not-parentheses]
+   83 |         if (!!test_bit(SW_TABLET_MODE, ebook_switch_idev->sw) == state)
+      |                                                               ^~
+arch/x86/platform/olpc/olpc-xo1-sci.c:83:13: note: add parentheses around left hand side expression to silence this warning
+
+Despite this code working as intended, this redundant double
+negation of boolean value, together with comparing to `char`
+with no explicit conversion to bool, makes compilers think
+the author made some unintentional logical mistakes here.
+Make it the other way around and negate the char instead
+to silence the warnings.
+
+Fixes: d2aa37411b8e ("x86/olpc/xo1/sci: Produce wakeup events for buttons and switches")
+Cc: stable@vger.kernel.org # 3.5+
+Reported-by: Guenter Roeck <linux@roeck-us.net>
+Reported-by: kernel test robot <lkp@intel.com>
+Reviewed-and-tested-by: Guenter Roeck <linux@roeck-us.net>
+Signed-off-by: Alexander Lobakin <alexandr.lobakin@intel.com>
+Signed-off-by: Yury Norov <yury.norov@gmail.com>
+Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
+---
+ arch/x86/platform/olpc/olpc-xo1-sci.c |    2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+--- a/arch/x86/platform/olpc/olpc-xo1-sci.c
++++ b/arch/x86/platform/olpc/olpc-xo1-sci.c
+@@ -85,7 +85,7 @@ static void send_ebook_state(void)
+               return;
+       }
+ 
+-      if (!!test_bit(SW_TABLET_MODE, ebook_switch_idev->sw) == state)
++      if (test_bit(SW_TABLET_MODE, ebook_switch_idev->sw) == !!state)
+               return; /* Nothing new to report. */
+ 
+       input_report_switch(ebook_switch_idev, SW_TABLET_MODE, state);