mbox series

[wpan-next,0/2] ieee802154: Handle imited devices

Message ID 20230324110558.90707-1-miquel.raynal@bootlin.com (mailing list archive)
Headers show
Series ieee802154: Handle imited devices | expand

Message

Miquel Raynal March 24, 2023, 11:05 a.m. UTC
As rightly pointed out by Alexander a few months ago, ca8210 devices
will not support sending frames which are not pure datagrams (hardMAC
wired to the softMAC layer). In order to not confuse users and clarify
that scanning and beaconing is not supported on these devices, let's add
a flag to prevent them to be used with the new APIs.

Miquel Raynal (2):
  net: ieee802154: Handle limited devices with only datagram support
  ieee802154: ca8210: Flag the driver as being limited

 drivers/net/ieee802154/ca8210.c |  3 ++-
 include/net/cfg802154.h         |  3 +++
 net/ieee802154/nl802154.c       | 10 ++++++++++
 3 files changed, 15 insertions(+), 1 deletion(-)

Comments

Alexander Aring March 24, 2023, 1:56 p.m. UTC | #1
Hi,

On Fri, Mar 24, 2023 at 7:07 AM Miquel Raynal <miquel.raynal@bootlin.com> wrote:
>
> As rightly pointed out by Alexander a few months ago, ca8210 devices
> will not support sending frames which are not pure datagrams (hardMAC
> wired to the softMAC layer). In order to not confuse users and clarify
> that scanning and beaconing is not supported on these devices, let's add
> a flag to prevent them to be used with the new APIs.
>

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

I appreciate that you care about driver specific quirks which need to
be done here. The users of this driver are getting aware now there is
a lack of support here.

Thanks.

- Alex
Stefan Schmidt March 24, 2023, 4:21 p.m. UTC | #2
Hello.

On 24.03.23 14:56, Alexander Aring wrote:
> Hi,
> 
> On Fri, Mar 24, 2023 at 7:07 AM Miquel Raynal <miquel.raynal@bootlin.com> wrote:
>>
>> As rightly pointed out by Alexander a few months ago, ca8210 devices
>> will not support sending frames which are not pure datagrams (hardMAC
>> wired to the softMAC layer). In order to not confuse users and clarify
>> that scanning and beaconing is not supported on these devices, let's add
>> a flag to prevent them to be used with the new APIs.
>>
> 
> Acked-by: Alexander Aring <aahringo@redhat.com>

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