DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 6E6B320827
MIME-Version: 1.0
References: <1540456194-26031-1-git-send-email-jaganath.kanakkassery@intel.com>
 <CAJzH+brrSKy-oBLcpctz=oHg9a5Yb33ChBrgAWbgVeSuJgjCog@mail.gmail.com>
 <CABBYNZJR6tZVOShTyS-EoNsXxHpHxczc94GLmtgs6m+rrmFAmg@mail.gmail.com> <CAJzH+bqzaGzjPS=cW_-+aaUFNLom4f_Se8T7kcnpPQtgXd3B2g@mail.gmail.com>
In-Reply-To: <CAJzH+bqzaGzjPS=cW_-+aaUFNLom4f_Se8T7kcnpPQtgXd3B2g@mail.gmail.com>
From:   Luiz Augusto von Dentz <luiz.dentz@gmail.com>
Date:   Wed, 7 Nov 2018 18:19:19 +0200
Message-ID: <CABBYNZLmT3QQqfwd0ukj4h0iaEp1B_-D6qPO4RgPp7j_KR6z3w@mail.gmail.com>
Subject: Re: [PATCH v1] btmgmt: Add BREDR PHYs in PHY Configuration commands
To:     Jaganath Kanakkassery <jaganath.k.os@gmail.com>
Cc:     "linux-bluetooth@vger.kernel.org" <linux-bluetooth@vger.kernel.org>,
        Jaganath Kanakkassery <jaganath.kanakkassery@intel.com>
Content-Type: text/plain; charset="UTF-8"
Sender: linux-bluetooth-owner@vger.kernel.org
Precedence: bulk

Hi Jaganath,
On Wed, Nov 7, 2018 at 1:55 PM Jaganath K <jaganath.k.os@gmail.com> wrote:
>
> On Wed, Nov 7, 2018 at 5:10 PM Luiz Augusto von Dentz
> <luiz.dentz@gmail.com> wrote:
> >
> > Hi Jaganath,
> > On Tue, Nov 6, 2018 at 11:38 AM Jaganath K <jaganath.k.os@gmail.com> wrote:
> > >
> > > Hi,
> > > On Thu, Oct 25, 2018 at 2:06 PM Jaganath Kanakkassery
> > > <jaganath.k.os@gmail.com> wrote:
> > > >
> > > > This also adds LE prefix for LE phys to make it more
> > > > descriptive
> > > > ---
> > > >  lib/mgmt.h     | 32 ++++++++++++++++---------
> > > >  tools/btmgmt.c | 75 ++++++++++++++++++++++++++++++++++------------------------
> > > >  2 files changed, 65 insertions(+), 42 deletions(-)
> > > >
> > > > diff --git a/lib/mgmt.h b/lib/mgmt.h
> > > > index ec6a380..570dec9 100644
> > > > --- a/lib/mgmt.h
> > > > +++ b/lib/mgmt.h
> > > > @@ -552,16 +552,26 @@ struct mgmt_cp_set_appearance {
> > > >
> > > >  #define MGMT_OP_GET_PHY_CONFIGURATION  0x0044
> > > >  struct mgmt_rp_get_phy_confguration {
> > > > -       uint16_t        supported_phys;
> > > > -       uint16_t        selected_phys;
> > > > -} __packed;
> > > > -
> > > > -#define MGMT_PHY_LE_1M_TX              0x0001
> > > > -#define MGMT_PHY_LE_1M_RX              0x0002
> > > > -#define MGMT_PHY_LE_2M_TX              0x0004
> > > > -#define MGMT_PHY_LE_2M_RX              0x0008
> > > > -#define MGMT_PHY_LE_CODED_TX           0x0010
> > > > -#define MGMT_PHY_LE_CODED_RX           0x0020
> > > > +       uint32_t        supported_phys;
> > > > +       uint32_t        configurable_phys;
> > > > +       uint32_t        selected_phys;
> > > > +} __packed;
> >
> > Is this a change in the protocol? It looks like the phys are now 32
> > bits, or is this an actual fix and that the way the kernel communicate
> > then? Either way this should probably be split from the changes to the
> > PHY strings.
> >
> Actually we added BREDR packet types also to PHYs and along with that
> made phy types 32 bit.
> So this change is mainly for that. One more change is renaming of LE
> PHY strings which i have
> mentioned in the commit description. You want that to be in a separate patch?

Right, well we can go with a single patch since you would have to
change the btmgmt anyway, though I would add the change to 32 bits in
the patch description.

> Thanks,
> Jaganath


-- 
Luiz Augusto von Dentz