mbox series

[wpan-next,v3,0/3] ieee802154: A bunch of light changes

Message ID 20220125122540.855604-1-miquel.raynal@bootlin.com (mailing list archive)
Headers show
Series ieee802154: A bunch of light changes | expand

Message

Miquel Raynal Jan. 25, 2022, 12:25 p.m. UTC
Here are a few small cleanups and improvements in preparation of a wider
series bringing a lot of features. These are aside changes, hence they
have their own small series.

Changes in v3:
* Split the v2 into two series: fixes for the wpan branch and cleanups
  for wpan-next. Here are random "cleanups".
* Reworded the ieee802154_wake/stop_queue helpers kdoc as discussed
  with Alexander.

Miquel Raynal (3):
  net: ieee802154: hwsim: Ensure frame checksum are valid
  net: ieee802154: Use the IEEE802154_MAX_PAGE define when relevant
  net: mac802154: Explain the use of ieee802154_wake/stop_queue()

 drivers/net/ieee802154/mac802154_hwsim.c |  2 +-
 include/net/mac802154.h                  | 12 ++++++++++++
 net/ieee802154/nl-phy.c                  |  4 ++--
 3 files changed, 15 insertions(+), 3 deletions(-)

Comments

Stefan Schmidt Jan. 27, 2022, 3:54 p.m. UTC | #1
Hello.

On 25.01.22 13:25, Miquel Raynal wrote:
> Here are a few small cleanups and improvements in preparation of a wider
> series bringing a lot of features. These are aside changes, hence they
> have their own small series.
> 
> Changes in v3:
> * Split the v2 into two series: fixes for the wpan branch and cleanups
>    for wpan-next. Here are random "cleanups".
> * Reworded the ieee802154_wake/stop_queue helpers kdoc as discussed
>    with Alexander.
> 
> Miquel Raynal (3):
>    net: ieee802154: hwsim: Ensure frame checksum are valid
>    net: ieee802154: Use the IEEE802154_MAX_PAGE define when relevant
>    net: mac802154: Explain the use of ieee802154_wake/stop_queue()
> 
>   drivers/net/ieee802154/mac802154_hwsim.c |  2 +-
>   include/net/mac802154.h                  | 12 ++++++++++++
>   net/ieee802154/nl-phy.c                  |  4 ++--
>   3 files changed, 15 insertions(+), 3 deletions(-)
> 

I am happy with all three of them now. Alex, let me know if there is 
anything else you want to be adressed and ack if not so I can pull these in.

regards
Stefan Schmidt
Alexander Aring Jan. 28, 2022, 12:34 a.m. UTC | #2
Hi,

On Thu, Jan 27, 2022 at 10:54 AM Stefan Schmidt
<stefan@datenfreihafen.org> wrote:
>
> Hello.
>
> On 25.01.22 13:25, Miquel Raynal wrote:
> > Here are a few small cleanups and improvements in preparation of a wider
> > series bringing a lot of features. These are aside changes, hence they
> > have their own small series.
> >
> > Changes in v3:
> > * Split the v2 into two series: fixes for the wpan branch and cleanups
> >    for wpan-next. Here are random "cleanups".
> > * Reworded the ieee802154_wake/stop_queue helpers kdoc as discussed
> >    with Alexander.
> >
> > Miquel Raynal (3):
> >    net: ieee802154: hwsim: Ensure frame checksum are valid
> >    net: ieee802154: Use the IEEE802154_MAX_PAGE define when relevant
> >    net: mac802154: Explain the use of ieee802154_wake/stop_queue()
> >
> >   drivers/net/ieee802154/mac802154_hwsim.c |  2 +-
> >   include/net/mac802154.h                  | 12 ++++++++++++
> >   net/ieee802154/nl-phy.c                  |  4 ++--
> >   3 files changed, 15 insertions(+), 3 deletions(-)
> >
>
> I am happy with all three of them now. Alex, let me know if there is
> anything else you want to be adressed and ack if not so I can pull these in.

Everything is fine.

Acked-by: Alexander Aring <aahringo@redhat.com>

Thanks.

- Alex
Stefan Schmidt Jan. 28, 2022, 10:27 a.m. UTC | #3
Hello.

On 25.01.22 13:25, Miquel Raynal wrote:
> Here are a few small cleanups and improvements in preparation of a wider
> series bringing a lot of features. These are aside changes, hence they
> have their own small series.
> 
> Changes in v3:
> * Split the v2 into two series: fixes for the wpan branch and cleanups
>    for wpan-next. Here are random "cleanups".
> * Reworded the ieee802154_wake/stop_queue helpers kdoc as discussed
>    with Alexander.
> 
> Miquel Raynal (3):
>    net: ieee802154: hwsim: Ensure frame checksum are valid
>    net: ieee802154: Use the IEEE802154_MAX_PAGE define when relevant
>    net: mac802154: Explain the use of ieee802154_wake/stop_queue()
> 
>   drivers/net/ieee802154/mac802154_hwsim.c |  2 +-
>   include/net/mac802154.h                  | 12 ++++++++++++
>   net/ieee802154/nl-phy.c                  |  4 ++--
>   3 files changed, 15 insertions(+), 3 deletions(-)
> 

This patch has been applied to the wpan-next tree and will be
part of the next pull request to net-next. Thanks!

regards
Stefan Schmidt