Re: [PATCH v2 net-next 0/6] net sched actions: code style cleanup and fixes

[Davide Caratti <dcaratti@redhat.com>]

On Wed, 2018-06-27 at 13:33 -0400, Roman Mashak wrote:
> The patchset fixes a few code stylistic issues and typos, as well as one
> detected by sparse semantic checker tool.
> 
> No functional changes introduced.
> 
> Patch 1 & 2 fix coding style bits caught by the checkpatch.pl script
> Patch 3 fixes an issue with a shadowed variable
> Patch 4 adds sizeof() operator instead of magic number for buffer length
> Patch 5 fixes typos in diagnostics messages
> Patch 6 explicitly sets unsigned char for bitwise operation
> 
> v2:
>    - submit for net-next
>    - added Reviewed-by tags
>    - use u8* instead of char* as per Davide Caratti suggestion
> 
> Roman Mashak (6):
>   net sched actions: fix coding style in pedit action
>   net sched actions: fix coding style in pedit headers
>   net sched actions: fix sparse warning
>   net sched actions: use sizeof operator for buffer length
>   net sched actions: fix misleading text strings in pedit action
>   net sched actions: avoid bitwise operation on signed value in pedit
> 
>  include/net/tc_act/tc_pedit.h        |  1 +
>  include/uapi/linux/tc_act/tc_pedit.h |  9 ++++++--
>  net/sched/act_pedit.c                | 43 +++++++++++++++++++-----------------
>  3 files changed, 31 insertions(+), 22 deletions(-)
> 
for the series:

Acked-by: Davide Caratti <dcaratti@redhat.com>

regards,
-- 
davide