Hi Kiran, On Wed, Jun 28, 2023 at 4:57 AM K, Kiran <kiran.k@xxxxxxxxx> wrote: > > Hi Paul, > > Thanks for the comments. > > > > > Dear Kiran, > > > > > > Thank you for your patch. > > > > Am 28.06.23 um 13:39 schrieb Kiran K: > > > Hardware variant for GalePeak core (CNVi) is added. > > > > The code names are officially spelled with a space, I believe: Gale Peak. > > > > Maybe also mention 0x1c. > > I will fix it in V2 version of patch. > > > > > Signed-off-by: Kiran K <kiran.k@xxxxxxxxx> > > > --- > > > drivers/bluetooth/btintel.c | 3 +++ > > > 1 file changed, 3 insertions(+) > > > > > > diff --git a/drivers/bluetooth/btintel.c b/drivers/bluetooth/btintel.c > > > index dd1e48808ee2..f4d096639080 100644 > > > --- a/drivers/bluetooth/btintel.c > > > +++ b/drivers/bluetooth/btintel.c > > > @@ -480,6 +480,7 @@ static int btintel_version_info_tlv(struct hci_dev > > *hdev, > > > case 0x18: /* Slr */ > > > case 0x19: /* Slr-F */ > > > case 0x1b: /* Mgr */ > > > + case 0x1c: /* GaP */ > > > > Why not add the full name in the comment? Maybe: Gale Peak (GaP). > Ack. I think it is about time to introduce proper definitions, or use an enum, for these values. It would be also great to have btmon decoding these values so you can include a HCI trace in the patch description. > > > case 0x1b: > > > + case 0x1c: > > > /* Display version information of TLV type */ > > > btintel_version_info_tlv(hdev, &ver_tlv); > > > > > > > > > Kind regards, > > > > Paul > > Regards, > Kiran -- Luiz Augusto von Dentz
