* [PATCH RESEND][next] nfp: Fix fall-through warnings for Clang
@ 2021-03-05 9:49 Gustavo A. R. Silva
2021-03-05 12:19 ` Simon Horman
0 siblings, 1 reply; 5+ messages in thread
From: Gustavo A. R. Silva @ 2021-03-05 9:49 UTC (permalink / raw)
To: Simon Horman, Jakub Kicinski, David S. Miller
Cc: oss-drivers, netdev, linux-kernel, Gustavo A. R. Silva, linux-hardening
In preparation to enable -Wimplicit-fallthrough for Clang, fix a warning
by explicitly adding a break statement instead of letting the code fall
through to the next case.
Link: https://github.com/KSPP/linux/issues/115
Signed-off-by: Gustavo A. R. Silva <gustavoars@kernel.org>
---
drivers/net/ethernet/netronome/nfp/nfp_net_repr.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/net/ethernet/netronome/nfp/nfp_net_repr.c b/drivers/net/ethernet/netronome/nfp/nfp_net_repr.c
index b3cabc274121..3b8e675087de 100644
--- a/drivers/net/ethernet/netronome/nfp/nfp_net_repr.c
+++ b/drivers/net/ethernet/netronome/nfp/nfp_net_repr.c
@@ -103,6 +103,7 @@ nfp_repr_get_stats64(struct net_device *netdev, struct rtnl_link_stats64 *stats)
case NFP_PORT_PF_PORT:
case NFP_PORT_VF_PORT:
nfp_repr_vnic_get_stats64(repr->port, stats);
+ break;
default:
break;
}
--
2.27.0
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH RESEND][next] nfp: Fix fall-through warnings for Clang
2021-03-05 9:49 [PATCH RESEND][next] nfp: Fix fall-through warnings for Clang Gustavo A. R. Silva
@ 2021-03-05 12:19 ` Simon Horman
2021-04-20 20:23 ` Gustavo A. R. Silva
0 siblings, 1 reply; 5+ messages in thread
From: Simon Horman @ 2021-03-05 12:19 UTC (permalink / raw)
To: Gustavo A. R. Silva
Cc: Jakub Kicinski, David S. Miller, oss-drivers, netdev,
linux-kernel, linux-hardening
On Fri, Mar 05, 2021 at 03:49:37AM -0600, Gustavo A. R. Silva wrote:
> In preparation to enable -Wimplicit-fallthrough for Clang, fix a warning
> by explicitly adding a break statement instead of letting the code fall
> through to the next case.
>
> Link: https://github.com/KSPP/linux/issues/115
> Signed-off-by: Gustavo A. R. Silva <gustavoars@kernel.org>
Thanks Gustavo,
this looks good to me.
Acked-by: Simon Horman <simon.horman@netronome.com>
> ---
> drivers/net/ethernet/netronome/nfp/nfp_net_repr.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/net/ethernet/netronome/nfp/nfp_net_repr.c b/drivers/net/ethernet/netronome/nfp/nfp_net_repr.c
> index b3cabc274121..3b8e675087de 100644
> --- a/drivers/net/ethernet/netronome/nfp/nfp_net_repr.c
> +++ b/drivers/net/ethernet/netronome/nfp/nfp_net_repr.c
> @@ -103,6 +103,7 @@ nfp_repr_get_stats64(struct net_device *netdev, struct rtnl_link_stats64 *stats)
> case NFP_PORT_PF_PORT:
> case NFP_PORT_VF_PORT:
> nfp_repr_vnic_get_stats64(repr->port, stats);
> + break;
> default:
> break;
> }
> --
> 2.27.0
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH RESEND][next] nfp: Fix fall-through warnings for Clang
2021-03-05 12:19 ` Simon Horman
@ 2021-04-20 20:23 ` Gustavo A. R. Silva
2021-04-21 9:39 ` [oss-drivers] " Simon Horman
0 siblings, 1 reply; 5+ messages in thread
From: Gustavo A. R. Silva @ 2021-04-20 20:23 UTC (permalink / raw)
To: Simon Horman, Gustavo A. R. Silva
Cc: Jakub Kicinski, David S. Miller, oss-drivers, netdev,
linux-kernel, linux-hardening
Hi all,
Friendly ping: who can take this, please?
Thanks
--
Gustavo
On 3/5/21 06:19, Simon Horman wrote:
> On Fri, Mar 05, 2021 at 03:49:37AM -0600, Gustavo A. R. Silva wrote:
>> In preparation to enable -Wimplicit-fallthrough for Clang, fix a warning
>> by explicitly adding a break statement instead of letting the code fall
>> through to the next case.
>>
>> Link: https://github.com/KSPP/linux/issues/115
>> Signed-off-by: Gustavo A. R. Silva <gustavoars@kernel.org>
>
> Thanks Gustavo,
>
> this looks good to me.
>
> Acked-by: Simon Horman <simon.horman@netronome.com>
>
>> ---
>> drivers/net/ethernet/netronome/nfp/nfp_net_repr.c | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/drivers/net/ethernet/netronome/nfp/nfp_net_repr.c b/drivers/net/ethernet/netronome/nfp/nfp_net_repr.c
>> index b3cabc274121..3b8e675087de 100644
>> --- a/drivers/net/ethernet/netronome/nfp/nfp_net_repr.c
>> +++ b/drivers/net/ethernet/netronome/nfp/nfp_net_repr.c
>> @@ -103,6 +103,7 @@ nfp_repr_get_stats64(struct net_device *netdev, struct rtnl_link_stats64 *stats)
>> case NFP_PORT_PF_PORT:
>> case NFP_PORT_VF_PORT:
>> nfp_repr_vnic_get_stats64(repr->port, stats);
>> + break;
>> default:
>> break;
>> }
>> --
>> 2.27.0
>>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [oss-drivers] Re: [PATCH RESEND][next] nfp: Fix fall-through warnings for Clang
2021-04-20 20:23 ` Gustavo A. R. Silva
@ 2021-04-21 9:39 ` Simon Horman
2021-05-18 0:07 ` Gustavo A. R. Silva
0 siblings, 1 reply; 5+ messages in thread
From: Simon Horman @ 2021-04-21 9:39 UTC (permalink / raw)
To: Gustavo A. R. Silva
Cc: Gustavo A. R. Silva, Jakub Kicinski, David S. Miller,
oss-drivers, netdev, linux-kernel, linux-hardening
On Tue, Apr 20, 2021 at 03:23:39PM -0500, Gustavo A. R. Silva wrote:
> On 3/5/21 06:19, Simon Horman wrote:
> > On Fri, Mar 05, 2021 at 03:49:37AM -0600, Gustavo A. R. Silva wrote:
> >> In preparation to enable -Wimplicit-fallthrough for Clang, fix a warning
> >> by explicitly adding a break statement instead of letting the code fall
> >> through to the next case.
> >>
> >> Link: https://github.com/KSPP/linux/issues/115
> >> Signed-off-by: Gustavo A. R. Silva <gustavoars@kernel.org>
> >
> > Thanks Gustavo,
> >
> > this looks good to me.
> >
> > Acked-by: Simon Horman <simon.horman@netronome.com>
>
> Hi all,
>
> Friendly ping: who can take this, please?
Hi Jakub, Hi David,
I'm happy to repost this patch if it would help with your processes.
> >> ---
> >> drivers/net/ethernet/netronome/nfp/nfp_net_repr.c | 1 +
> >> 1 file changed, 1 insertion(+)
> >>
> >> diff --git a/drivers/net/ethernet/netronome/nfp/nfp_net_repr.c b/drivers/net/ethernet/netronome/nfp/nfp_net_repr.c
> >> index b3cabc274121..3b8e675087de 100644
> >> --- a/drivers/net/ethernet/netronome/nfp/nfp_net_repr.c
> >> +++ b/drivers/net/ethernet/netronome/nfp/nfp_net_repr.c
> >> @@ -103,6 +103,7 @@ nfp_repr_get_stats64(struct net_device *netdev, struct rtnl_link_stats64 *stats)
> >> case NFP_PORT_PF_PORT:
> >> case NFP_PORT_VF_PORT:
> >> nfp_repr_vnic_get_stats64(repr->port, stats);
> >> + break;
> >> default:
> >> break;
> >> }
> >> --
> >> 2.27.0
> >>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [oss-drivers] Re: [PATCH RESEND][next] nfp: Fix fall-through warnings for Clang
2021-04-21 9:39 ` [oss-drivers] " Simon Horman
@ 2021-05-18 0:07 ` Gustavo A. R. Silva
0 siblings, 0 replies; 5+ messages in thread
From: Gustavo A. R. Silva @ 2021-05-18 0:07 UTC (permalink / raw)
To: Simon Horman
Cc: Gustavo A. R. Silva, Jakub Kicinski, David S. Miller,
oss-drivers, netdev, linux-kernel, linux-hardening
Hi all,
I'm taking this in my -next[1] branch for v5.14.
Thanks
--
Gustavo
[1] https://git.kernel.org/pub/scm/linux/kernel/git/gustavoars/linux.git/log/?h=for-next/kspp
On 4/21/21 04:39, Simon Horman wrote:
> On Tue, Apr 20, 2021 at 03:23:39PM -0500, Gustavo A. R. Silva wrote:
>> On 3/5/21 06:19, Simon Horman wrote:
>>> On Fri, Mar 05, 2021 at 03:49:37AM -0600, Gustavo A. R. Silva wrote:
>>>> In preparation to enable -Wimplicit-fallthrough for Clang, fix a warning
>>>> by explicitly adding a break statement instead of letting the code fall
>>>> through to the next case.
>>>>
>>>> Link: https://github.com/KSPP/linux/issues/115
>>>> Signed-off-by: Gustavo A. R. Silva <gustavoars@kernel.org>
>>>
>>> Thanks Gustavo,
>>>
>>> this looks good to me.
>>>
>>> Acked-by: Simon Horman <simon.horman@netronome.com>
>>
>> Hi all,
>>
>> Friendly ping: who can take this, please?
>
> Hi Jakub, Hi David,
>
> I'm happy to repost this patch if it would help with your processes.
>
>>>> ---
>>>> drivers/net/ethernet/netronome/nfp/nfp_net_repr.c | 1 +
>>>> 1 file changed, 1 insertion(+)
>>>>
>>>> diff --git a/drivers/net/ethernet/netronome/nfp/nfp_net_repr.c b/drivers/net/ethernet/netronome/nfp/nfp_net_repr.c
>>>> index b3cabc274121..3b8e675087de 100644
>>>> --- a/drivers/net/ethernet/netronome/nfp/nfp_net_repr.c
>>>> +++ b/drivers/net/ethernet/netronome/nfp/nfp_net_repr.c
>>>> @@ -103,6 +103,7 @@ nfp_repr_get_stats64(struct net_device *netdev, struct rtnl_link_stats64 *stats)
>>>> case NFP_PORT_PF_PORT:
>>>> case NFP_PORT_VF_PORT:
>>>> nfp_repr_vnic_get_stats64(repr->port, stats);
>>>> + break;
>>>> default:
>>>> break;
>>>> }
>>>> --
>>>> 2.27.0
>>>>
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2021-05-18 0:06 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-03-05 9:49 [PATCH RESEND][next] nfp: Fix fall-through warnings for Clang Gustavo A. R. Silva
2021-03-05 12:19 ` Simon Horman
2021-04-20 20:23 ` Gustavo A. R. Silva
2021-04-21 9:39 ` [oss-drivers] " Simon Horman
2021-05-18 0:07 ` Gustavo A. R. Silva
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.