Hi Xiao, OK. I will check the two empty lines carefully and check the patch by "git am".
Best regards, Xiaojun -----Original Message----- From: Wang, Xiao W [mailto:[email protected]] Sent: Thursday, April 02, 2020 4:56 PM To: Xiaojun Liu; Zhang, Qi Z; Kwan, Ngai-mint; Keller, Jacob E Cc: [email protected] Subject: RE: [PATCH v3 4/5] net/fm10k: add flow interface and switch management Hi, Applying: net/fm10k: add flow interface and switch management .git/rebase-apply/patch:915: new blank line at EOF. + warning: 1 line adds whitespace errors. Besides, I see that you use two empty lines to delimit two functions in some places, please align to 1 empty line. Please understand upstreaming may mean burden of maintenance, so let's make the big patch in better shape. 😊 Best Regards, Xiao > -----Original Message----- > From: Xiaojun Liu <[email protected]> > Sent: Friday, March 20, 2020 2:59 PM > To: Wang, Xiao W <[email protected]>; Zhang, Qi Z > <[email protected]>; Kwan, Ngai-mint <[email protected]>; Keller, > Jacob E <[email protected]> > Cc: [email protected]; Xiaojun Liu <[email protected]> > Subject: [PATCH v3 4/5] net/fm10k: add flow interface and switch management > > Add flow interface to support offload flow into HW. > It supports parse vlan and parse mpls, all these > data will be transferred to ffu data. > Add switch management, includes initialization, > port mapping, epl port link, LED controller, interrupt handler. > It create 3 threads. One for interrupt handler, one for > LED controller, one for statistics. > > Signed-off-by: Xiaojun Liu <[email protected]> > --- > drivers/net/fm10k/Makefile | 2 + > drivers/net/fm10k/base/fm10k_type.h | 1 + > drivers/net/fm10k/switch/fm10k_flow.c | 872 +++++++++++ > drivers/net/fm10k/switch/fm10k_flow.h | 26 + > drivers/net/fm10k/switch/fm10k_stats.c | 128 ++ > drivers/net/fm10k/switch/fm10k_switch.c | 2382 > +++++++++++++++++++++++++++++++ [...]
