Received-SPF: pass (google.com: domain of linux-bluetooth-owner@vger.kernel.org designates 2620:137:e000::1:18 as permitted sender) client-ip=2620:137:e000::1:18;
MIME-Version: 1.0
References: <20220428030534.3220410-1-aahringo@redhat.com> <06164dbe-3c11-c627-0929-a399414c01bf@datenfreihafen.org>
 <CABBYNZKCoj+W1MygU4eZZxGVSrWuRfEWx-fLOXvV-vXXY72vmw@mail.gmail.com> <3b5d28c4-1877-ba8f-dcb3-d5c3cbc06a1f@datenfreihafen.org>
In-Reply-To: <3b5d28c4-1877-ba8f-dcb3-d5c3cbc06a1f@datenfreihafen.org>
From:   Luiz Augusto von Dentz <luiz.dentz@gmail.com>
Date:   Wed, 1 Jun 2022 14:58:47 -0700
Message-ID: <CABBYNZK1uWjxmvJUBPaUdMvj8B-p=TtRKTUwQ0szCgLhh+7CcA@mail.gmail.com>
Subject: Re: [PATCH bluetooth-next 0/3] net: 6lowpan: simplify lookup by nhc id
To:     Stefan Schmidt <stefan@datenfreihafen.org>
Cc:     Alexander Aring <aahringo@redhat.com>,
        Jukka Rissanen <jukka.rissanen@linux.intel.com>,
        "linux-bluetooth@vger.kernel.org" <linux-bluetooth@vger.kernel.org>,
        linux-wpan - ML <linux-wpan@vger.kernel.org>,
        Linus Torvalds <torvalds@linuxfoundation.org>
Content-Type: text/plain; charset="UTF-8"
Precedence: bulk

Hi Stefan,

On Wed, Jun 1, 2022 at 2:56 PM Stefan Schmidt <stefan@datenfreihafen.org> wrote:
>
> Hello Luiz.
>
> On 01.06.22 23:48, Luiz Augusto von Dentz wrote:
> > Hi Stefan,
> >
> > On Wed, Jun 1, 2022 at 2:37 PM Stefan Schmidt <stefan@datenfreihafen.org> wrote:
> >>
> >> Hello.
> >>
> >> On 28.04.22 05:05, Alexander Aring wrote:
> >>> Hi,
> >>>
> >>> this patch series removes the rb data structure for looking up a nhc by
> >>> nhc id. Instead we using the existing nexthdr lookup array by iterating
> >>> over it and find the right nhc by nhc id. It's simply not worth it to
> >>> use such complex handling for such small amount of nhc. As we only
> >>> support nhc ids which fits into 1 byte and there are not two byte nhc
> >>> ids values specified yet, we let the nhc layer only handle 1 byte values.
> >>> If there is the need for 2 byte nhc values we can add support for it.
> >>>
> >>> - Alex
> >>>
> >>> Alexander Aring (3):
> >>>     net: 6lowpan: remove const from scalars
> >>>     net: 6lowpan: use array for find nhc id
> >>>     net: 6lowpan: constify lowpan_nhc structures
> >>
> >> Marcel, Luiz, are you still picking up generic 6lowpan patches or only
> >> the ones for bluetooth?
> >>
> >> These three have been around for over a month and acked by me and Jukka,
> >> but I can't find them anywhere in bluetooth-next or Linus tree.
> >>
> >> If 6lowpan is of less concern for you I can route them through my
> >> ieee80254 tree as well.
> >
> > Up to you, I was not aware they normally were merged thru
> > bluetooth-next but I would be fine merging those as well.
>
> If you and Marcel don't mind I would like to switch this over to being
> merged through ieee802154. Mostly because 6lowpan is vital for the 15.4
> work we have.
>
> With Jukka stepping down from co-maintaining this part (thanks a lot for
> your work!) this will most likely be reviewed by Alex and me anyway.
>
> We still need to ensure that patches are hitting wpan as well as
> bluetooth mailing list, but the MAINTAINERS file handles this already.
>
> Let me know if you are unhappy with the switch. If not I will start with
> merging these three as well as the MAINTAINERS update from Jukka.

I have no problem with that.

-- 
Luiz Augusto von Dentz