mbox series

[0/6] reduce number of warnings being disabled

Message ID 20241030173830.881368-1-bruce.richardson@intel.com (mailing list archive)
Headers
Series reduce number of warnings being disabled |

Message

Bruce Richardson Oct. 30, 2024, 5:38 p.m. UTC
for historical reasons, many drivers had extra warnings disabled,
especially in their base code directory. Many, but not all, of these
warning disabling flags are unnecessary so remove as many as we can.

Bruce Richardson (6):
  net/i40e/base: remove warning disable flags
  net/ice/base: remove warning disable flag
  net/e1000/base: remove warning disable flags
  net/iavf: remove warning disable flag
  net/fm10k/base: remove warning disable flags
  common/idpf/base: remove warning disable flags

 drivers/common/idpf/base/meson.build | 13 ++++++++-----
 drivers/net/e1000/base/meson.build   |  7 ++++---
 drivers/net/fm10k/base/meson.build   |  6 +++---
 drivers/net/i40e/base/meson.build    |  9 +++++----
 drivers/net/iavf/meson.build         |  2 --
 drivers/net/ice/base/meson.build     |  1 -
 6 files changed, 20 insertions(+), 18 deletions(-)

--
2.43.0
  

Comments

Konstantin Ananyev Oct. 30, 2024, 6:17 p.m. UTC | #1
> -----Original Message-----
> From: Bruce Richardson <bruce.richardson@intel.com>
> Sent: Wednesday, October 30, 2024 5:38 PM
> To: dev@dpdk.org
> Cc: Bruce Richardson <bruce.richardson@intel.com>
> Subject: [PATCH 0/6] reduce number of warnings being disabled
> 
> for historical reasons, many drivers had extra warnings disabled,
> especially in their base code directory. Many, but not all, of these
> warning disabling flags are unnecessary so remove as many as we can.
> 
> Bruce Richardson (6):
>   net/i40e/base: remove warning disable flags
>   net/ice/base: remove warning disable flag
>   net/e1000/base: remove warning disable flags
>   net/iavf: remove warning disable flag
>   net/fm10k/base: remove warning disable flags
>   common/idpf/base: remove warning disable flags
> 
>  drivers/common/idpf/base/meson.build | 13 ++++++++-----
>  drivers/net/e1000/base/meson.build   |  7 ++++---
>  drivers/net/fm10k/base/meson.build   |  6 +++---
>  drivers/net/i40e/base/meson.build    |  9 +++++----
>  drivers/net/iavf/meson.build         |  2 --
>  drivers/net/ice/base/meson.build     |  1 -
>  6 files changed, 20 insertions(+), 18 deletions(-)
> 
> --

Series-Acked-by: Konstantin Ananyev <konstantin.ananyev@huawei.com>

> 2.43.0
  
Bruce Richardson Oct. 31, 2024, 1:26 p.m. UTC | #2
On Wed, Oct 30, 2024 at 06:17:12PM +0000, Konstantin Ananyev wrote:
> 
> 
> > -----Original Message-----
> > From: Bruce Richardson <bruce.richardson@intel.com>
> > Sent: Wednesday, October 30, 2024 5:38 PM
> > To: dev@dpdk.org
> > Cc: Bruce Richardson <bruce.richardson@intel.com>
> > Subject: [PATCH 0/6] reduce number of warnings being disabled
> > 
> > for historical reasons, many drivers had extra warnings disabled,
> > especially in their base code directory. Many, but not all, of these
> > warning disabling flags are unnecessary so remove as many as we can.
> > 
> > Bruce Richardson (6):
> >   net/i40e/base: remove warning disable flags
> >   net/ice/base: remove warning disable flag
> >   net/e1000/base: remove warning disable flags
> >   net/iavf: remove warning disable flag
> >   net/fm10k/base: remove warning disable flags
> >   common/idpf/base: remove warning disable flags
> > 
> >  drivers/common/idpf/base/meson.build | 13 ++++++++-----
> >  drivers/net/e1000/base/meson.build   |  7 ++++---
> >  drivers/net/fm10k/base/meson.build   |  6 +++---
> >  drivers/net/i40e/base/meson.build    |  9 +++++----
> >  drivers/net/iavf/meson.build         |  2 --
> >  drivers/net/ice/base/meson.build     |  1 -
> >  6 files changed, 20 insertions(+), 18 deletions(-)
> > 
> > --
> 
> Series-Acked-by: Konstantin Ananyev <konstantin.ananyev@huawei.com>
>
Series applied to dpdk-next-net-intel.

@Konstantin: FYI, when running checks on the patches, I get warnings that the
address you used to ack the patches is not your main email address in
.mailmap. Not sure if you want to update your primary email in that file?

/Bruce