On Sun, Jul 19, 2020 at 05:35:04AM -0400, B K Karthik wrote: > On Sun, Jul 19, 2020 at 5:26 AM Greg Kroah-Hartman > <gregkh@xxxxxxxxxxxxxxxxxxx> wrote: > > > > On Sat, Jul 18, 2020 at 05:16:26AM -0400, B K Karthik wrote: > > > fixed multiple parentheses coding style issues reported by checkpatch. > > > > > > Signed-off-by: B K Karthik <karthik.bk2000@xxxxxxxx> > > > --- > > > drivers/staging/rtl8188eu/include/ieee80211.h | 4 +-- > > > .../staging/rtl8188eu/include/osdep_service.h | 4 +-- > > > drivers/staging/rtl8188eu/include/wifi.h | 34 +++++++++---------- > > > 3 files changed, 21 insertions(+), 21 deletions(-) > > > > You can never add warnings to the build, like this patch did. Always > > test-build your patches at the very least... > > understood, I will definitely do that. > but these warnings are (mostly) [-Wunused-value] and [-Wformat]. > that should not have (?) occurred due to the usage of parentheses. They are not there _before_ your patch, and only show up _after_ your patch, so perhaps your patch is wrong? :) thanks, greg k-h _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel