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. > > case 0x1b: > > + case 0x1c: > > /* Display version information of TLV type */ > > btintel_version_info_tlv(hdev, &ver_tlv); > > > > > Kind regards, > > Paul Regards, Kiran
