On Fri, Sep 25, 2020 at 6:28 AM Rasesh Mody <rm...@marvell.com> wrote: > >From: Ferruh Yigit <ferruh.yi...@intel.com> > >Sent: Monday, September 21, 2020 6:38 AM > > > >Error observed with gcc 11 under development gcc (GCC) 11.0.0 20200920 > >(experimental) > > > >build error: > >../drivers/net/qede/qede_main.c: In function ‘qed_get_current_link’: > >../drivers/net/qede/qede_main.c:587:17: > > warning: this ‘if’ clause does not guard... > > [-Wmisleading-indentation] > > 587 | if (!ptt) > > | ^~ > >../drivers/net/qede/qede_main.c:590:25: > > note: ...this statement, but the latter is misleadingly indented > > as if it were guarded by the ‘if’ > > 590 | qed_fill_link(hwfn, ptt, if_link); > > | ^~~~~~~~~~~~~ > > > >Signed-off-by: Ferruh Yigit <ferruh.yi...@intel.com> > >--- > >I am not clear what the exact intention of the original code is, compiler > >warning is about wrong indentation of 'qed_fill_link(..)', I have updated the > >code to keep the behavior same but fix the indentation. > >Please send a proper fix if the existing behavior needs to be updated. > > The existing behavior needs to be fixed, I'll send a proper fix for 20.11.
I suppose this is fixed with: https://patchwork.dpdk.org/patch/79562/ I'll mark Ferruh patch as rejected. -- David Marchand