> -----Original Message----- > From: Jerin Jacob Kollanukkaran <jer...@marvell.com> > Sent: Tuesday, July 23, 2019 3:10 PM > To: Phil Yang (Arm Technology China) <phil.y...@arm.com>; dev@dpdk.org > Cc: tho...@monjalon.net; gage.e...@intel.com; > hemant.agra...@nxp.com; Honnappa Nagarahalli > <honnappa.nagaraha...@arm.com>; Gavin Hu (Arm Technology China) > <gavin...@arm.com>; nd <n...@arm.com>; nd <n...@arm.com> > Subject: RE: [EXT] [PATCH v6 1/3] eal/arm64: add 128-bit atomic compare > exchange > > > > > > > [master]dell[dpdk.org] $ ./devtools/checkpatches.sh > > > > Yes. I did this check before upstream this patch. > > I ignored ' WARNING:MACRO_WITH_FLOW_CONTROL: Macros with flow > > control statements should be avoided ' warnings. > > However, I didn't see any LONG_LINE warnings in the CI report here. > > Yes. But Not sure why?
Not sure. It might because DPDK CI changed tabstop width or indent width which is different than Kernel checkpatch.pl. > > I sent v8 with "WARNING:LONG_LINE: line over 80 characters" warning fixes > with latest kernel > checkpatch.pl. Please review. +1 I really appreciate your help. > > > http://mails.dpdk.org/archives/test-report/2019-July/090889.html > > BTW, I think these 3 warnings in the report might be ineluctable.