* Re: [edk2-devel] [PATCH] [edk2-stable201908] MdePkg/BluetoothLeConfig.h: Add type EfiBluetoothSmpPeerAddressList [not found] <15BE8BCCADEA547C.18314@groups.io> @ 2019-08-26 18:19 ` Ni, Ray 2019-08-26 23:56 ` Liming Gao 0 siblings, 1 reply; 8+ messages in thread From: Ni, Ray @ 2019-08-26 18:19 UTC (permalink / raw) To: devel@edk2.groups.io, Ni, Ray; +Cc: Kinney, Michael D, Gao, Liming I understand it's now in code freeze phase. Just make a try to see whether this very clean/simple header file change can be included in the stable tag. Any platform that uses BLE stack can get benefit from this definition update. > -----Original Message----- > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Ni, Ray > Sent: Monday, August 26, 2019 11:16 AM > To: devel@edk2.groups.io > Cc: Kinney, Michael D <michael.d.kinney@intel.com>; Gao, Liming <liming.gao@intel.com> > Subject: [edk2-devel] [PATCH] [edk2-stable201908] MdePkg/BluetoothLeConfig.h: Add type EfiBluetoothSmpPeerAddressList > > To support auto-connection, EFI_BLUETOOTH_LE_SMP_DATA_TYPE needs to > add a new data type EfiBluetoothSmpPeerAddressList which associates > with a list of Bluetooth per address connected before. > > This new data type was added in UEFI spec 2.7b. > > Signed-off-by: Ray Ni <ray.ni@intel.com> > Cc: Michael D Kinney <michael.d.kinney@intel.com> > Cc: Liming Gao <liming.gao@intel.com> > --- > MdePkg/Include/Protocol/BluetoothLeConfig.h | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/MdePkg/Include/Protocol/BluetoothLeConfig.h b/MdePkg/Include/Protocol/BluetoothLeConfig.h > index 8c0f881f85..8726a58b15 100644 > --- a/MdePkg/Include/Protocol/BluetoothLeConfig.h > +++ b/MdePkg/Include/Protocol/BluetoothLeConfig.h > @@ -2,7 +2,7 @@ > EFI Bluetooth LE Config Protocol as defined in UEFI 2.7. > This protocol abstracts user interface configuration for BluetoothLe device. > > - Copyright (c) 2017, Intel Corporation. All rights reserved.<BR> > + Copyright (c) 2017 - 2019, Intel Corporation. All rights reserved.<BR> > SPDX-License-Identifier: BSD-2-Clause-Patent > > @par Revision Reference: > @@ -451,6 +451,8 @@ typedef enum { > EfiBluetoothSmpLocalCSRK, /* If Key hierarchy not supported */ > EfiBluetoothSmpLocalSignCounter, > EfiBluetoothSmpLocalDIV, > + EfiBluetoothSmpPeerAddressList, > + EfiBluetoothSmpMax, > } EFI_BLUETOOTH_LE_SMP_DATA_TYPE; > > /** > -- > 2.21.0.windows.1 > > > ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [edk2-devel] [PATCH] [edk2-stable201908] MdePkg/BluetoothLeConfig.h: Add type EfiBluetoothSmpPeerAddressList 2019-08-26 18:19 ` [edk2-devel] [PATCH] [edk2-stable201908] MdePkg/BluetoothLeConfig.h: Add type EfiBluetoothSmpPeerAddressList Ni, Ray @ 2019-08-26 23:56 ` Liming Gao 2019-08-27 0:20 ` Michael D Kinney 0 siblings, 1 reply; 8+ messages in thread From: Liming Gao @ 2019-08-26 23:56 UTC (permalink / raw) To: Ni, Ray, devel@edk2.groups.io Cc: Kinney, Michael D, afish@apple.com, leif.lindholm@linaro.org, Laszlo Ersek (lersek@redhat.com) Cc to all Stewards. >-----Original Message----- >From: Ni, Ray >Sent: Tuesday, August 27, 2019 2:20 AM >To: devel@edk2.groups.io; Ni, Ray <ray.ni@intel.com> >Cc: Kinney, Michael D <michael.d.kinney@intel.com>; Gao, Liming ><liming.gao@intel.com> >Subject: RE: [edk2-devel] [PATCH] [edk2-stable201908] >MdePkg/BluetoothLeConfig.h: Add type EfiBluetoothSmpPeerAddressList > >I understand it's now in code freeze phase. >Just make a try to see whether this very clean/simple header file change can >be included in the stable tag. >Any platform that uses BLE stack can get benefit from this definition update. > >> -----Original Message----- >> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Ni, Ray >> Sent: Monday, August 26, 2019 11:16 AM >> To: devel@edk2.groups.io >> Cc: Kinney, Michael D <michael.d.kinney@intel.com>; Gao, Liming ><liming.gao@intel.com> >> Subject: [edk2-devel] [PATCH] [edk2-stable201908] >MdePkg/BluetoothLeConfig.h: Add type EfiBluetoothSmpPeerAddressList >> >> To support auto-connection, EFI_BLUETOOTH_LE_SMP_DATA_TYPE needs >to >> add a new data type EfiBluetoothSmpPeerAddressList which associates >> with a list of Bluetooth per address connected before. >> >> This new data type was added in UEFI spec 2.7b. >> >> Signed-off-by: Ray Ni <ray.ni@intel.com> >> Cc: Michael D Kinney <michael.d.kinney@intel.com> >> Cc: Liming Gao <liming.gao@intel.com> >> --- >> MdePkg/Include/Protocol/BluetoothLeConfig.h | 4 +++- >> 1 file changed, 3 insertions(+), 1 deletion(-) >> >> diff --git a/MdePkg/Include/Protocol/BluetoothLeConfig.h >b/MdePkg/Include/Protocol/BluetoothLeConfig.h >> index 8c0f881f85..8726a58b15 100644 >> --- a/MdePkg/Include/Protocol/BluetoothLeConfig.h >> +++ b/MdePkg/Include/Protocol/BluetoothLeConfig.h >> @@ -2,7 +2,7 @@ >> EFI Bluetooth LE Config Protocol as defined in UEFI 2.7. >> This protocol abstracts user interface configuration for BluetoothLe device. >> >> - Copyright (c) 2017, Intel Corporation. All rights reserved.<BR> >> + Copyright (c) 2017 - 2019, Intel Corporation. All rights reserved.<BR> >> SPDX-License-Identifier: BSD-2-Clause-Patent >> >> @par Revision Reference: >> @@ -451,6 +451,8 @@ typedef enum { >> EfiBluetoothSmpLocalCSRK, /* If Key hierarchy not supported */ >> EfiBluetoothSmpLocalSignCounter, >> EfiBluetoothSmpLocalDIV, >> + EfiBluetoothSmpPeerAddressList, >> + EfiBluetoothSmpMax, >> } EFI_BLUETOOTH_LE_SMP_DATA_TYPE; >> >> /** >> -- >> 2.21.0.windows.1 >> >> >> ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [edk2-devel] [PATCH] [edk2-stable201908] MdePkg/BluetoothLeConfig.h: Add type EfiBluetoothSmpPeerAddressList 2019-08-26 23:56 ` Liming Gao @ 2019-08-27 0:20 ` Michael D Kinney 2019-08-27 0:45 ` Ni, Ray 0 siblings, 1 reply; 8+ messages in thread From: Michael D Kinney @ 2019-08-27 0:20 UTC (permalink / raw) To: Gao, Liming, Ni, Ray, devel@edk2.groups.io, Kinney, Michael D Cc: afish@apple.com, leif.lindholm@linaro.org, Laszlo Ersek (lersek@redhat.com) Ray, Are there any components that need these new enum values? The patch looks very straightforward and is from the UEFI Spec, so I think the risk is low. But if consumers will not show up until after the stable tag, perhaps this should be one of the first patches after the stable tag. Thanks, Mike > -----Original Message----- > From: Gao, Liming > Sent: Monday, August 26, 2019 4:57 PM > To: Ni, Ray <ray.ni@intel.com>; devel@edk2.groups.io > Cc: Kinney, Michael D <michael.d.kinney@intel.com>; > afish@apple.com; leif.lindholm@linaro.org; Laszlo Ersek > (lersek@redhat.com) <lersek@redhat.com> > Subject: RE: [edk2-devel] [PATCH] [edk2-stable201908] > MdePkg/BluetoothLeConfig.h: Add type > EfiBluetoothSmpPeerAddressList > > Cc to all Stewards. > > >-----Original Message----- > >From: Ni, Ray > >Sent: Tuesday, August 27, 2019 2:20 AM > >To: devel@edk2.groups.io; Ni, Ray <ray.ni@intel.com> > >Cc: Kinney, Michael D <michael.d.kinney@intel.com>; > Gao, Liming > ><liming.gao@intel.com> > >Subject: RE: [edk2-devel] [PATCH] [edk2-stable201908] > >MdePkg/BluetoothLeConfig.h: Add type > EfiBluetoothSmpPeerAddressList > > > >I understand it's now in code freeze phase. > >Just make a try to see whether this very clean/simple > header file > >change can be included in the stable tag. > >Any platform that uses BLE stack can get benefit from > this definition update. > > > >> -----Original Message----- > >> From: devel@edk2.groups.io <devel@edk2.groups.io> On > Behalf Of Ni, > >> Ray > >> Sent: Monday, August 26, 2019 11:16 AM > >> To: devel@edk2.groups.io > >> Cc: Kinney, Michael D <michael.d.kinney@intel.com>; > Gao, Liming > ><liming.gao@intel.com> > >> Subject: [edk2-devel] [PATCH] [edk2-stable201908] > >MdePkg/BluetoothLeConfig.h: Add type > EfiBluetoothSmpPeerAddressList > >> > >> To support auto-connection, > EFI_BLUETOOTH_LE_SMP_DATA_TYPE needs > >to > >> add a new data type EfiBluetoothSmpPeerAddressList > which associates > >> with a list of Bluetooth per address connected > before. > >> > >> This new data type was added in UEFI spec 2.7b. > >> > >> Signed-off-by: Ray Ni <ray.ni@intel.com> > >> Cc: Michael D Kinney <michael.d.kinney@intel.com> > >> Cc: Liming Gao <liming.gao@intel.com> > >> --- > >> MdePkg/Include/Protocol/BluetoothLeConfig.h | 4 +++- > >> 1 file changed, 3 insertions(+), 1 deletion(-) > >> > >> diff --git > a/MdePkg/Include/Protocol/BluetoothLeConfig.h > >b/MdePkg/Include/Protocol/BluetoothLeConfig.h > >> index 8c0f881f85..8726a58b15 100644 > >> --- a/MdePkg/Include/Protocol/BluetoothLeConfig.h > >> +++ b/MdePkg/Include/Protocol/BluetoothLeConfig.h > >> @@ -2,7 +2,7 @@ > >> EFI Bluetooth LE Config Protocol as defined in > UEFI 2.7. > >> This protocol abstracts user interface > configuration for BluetoothLe device. > >> > >> - Copyright (c) 2017, Intel Corporation. All rights > reserved.<BR> > >> + Copyright (c) 2017 - 2019, Intel Corporation. All > rights > >> + reserved.<BR> > >> SPDX-License-Identifier: BSD-2-Clause-Patent > >> > >> @par Revision Reference: > >> @@ -451,6 +451,8 @@ typedef enum { > >> EfiBluetoothSmpLocalCSRK, /* If Key hierarchy not > supported */ > >> EfiBluetoothSmpLocalSignCounter, > >> EfiBluetoothSmpLocalDIV, > >> + EfiBluetoothSmpPeerAddressList, > >> + EfiBluetoothSmpMax, > >> } EFI_BLUETOOTH_LE_SMP_DATA_TYPE; > >> > >> /** > >> -- > >> 2.21.0.windows.1 > >> > >> > >> ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [edk2-devel] [PATCH] [edk2-stable201908] MdePkg/BluetoothLeConfig.h: Add type EfiBluetoothSmpPeerAddressList 2019-08-27 0:20 ` Michael D Kinney @ 2019-08-27 0:45 ` Ni, Ray 2019-08-27 10:18 ` Leif Lindholm 2019-08-27 12:47 ` Laszlo Ersek 0 siblings, 2 replies; 8+ messages in thread From: Ni, Ray @ 2019-08-27 0:45 UTC (permalink / raw) To: Kinney, Michael D, Gao, Liming, devel@edk2.groups.io Cc: afish@apple.com, leif.lindholm@linaro.org, Laszlo Ersek (lersek@redhat.com) The consumers are in close source and won't be in open source in near future. But consumers depend on open source repo. > -----Original Message----- > From: Kinney, Michael D > Sent: Monday, August 26, 2019 5:20 PM > To: Gao, Liming <liming.gao@intel.com>; Ni, Ray <ray.ni@intel.com>; devel@edk2.groups.io; Kinney, Michael D > <michael.d.kinney@intel.com> > Cc: afish@apple.com; leif.lindholm@linaro.org; Laszlo Ersek (lersek@redhat.com) <lersek@redhat.com> > Subject: RE: [edk2-devel] [PATCH] [edk2-stable201908] MdePkg/BluetoothLeConfig.h: Add type > EfiBluetoothSmpPeerAddressList > > Ray, > > Are there any components that need these new enum values? > > The patch looks very straightforward and is from the > UEFI Spec, so I think the risk is low. But if consumers > will not show up until after the stable tag, perhaps this > should be one of the first patches after the stable tag. > > Thanks, > > Mike > > > -----Original Message----- > > From: Gao, Liming > > Sent: Monday, August 26, 2019 4:57 PM > > To: Ni, Ray <ray.ni@intel.com>; devel@edk2.groups.io > > Cc: Kinney, Michael D <michael.d.kinney@intel.com>; > > afish@apple.com; leif.lindholm@linaro.org; Laszlo Ersek > > (lersek@redhat.com) <lersek@redhat.com> > > Subject: RE: [edk2-devel] [PATCH] [edk2-stable201908] > > MdePkg/BluetoothLeConfig.h: Add type > > EfiBluetoothSmpPeerAddressList > > > > Cc to all Stewards. > > > > >-----Original Message----- > > >From: Ni, Ray > > >Sent: Tuesday, August 27, 2019 2:20 AM > > >To: devel@edk2.groups.io; Ni, Ray <ray.ni@intel.com> > > >Cc: Kinney, Michael D <michael.d.kinney@intel.com>; > > Gao, Liming > > ><liming.gao@intel.com> > > >Subject: RE: [edk2-devel] [PATCH] [edk2-stable201908] > > >MdePkg/BluetoothLeConfig.h: Add type > > EfiBluetoothSmpPeerAddressList > > > > > >I understand it's now in code freeze phase. > > >Just make a try to see whether this very clean/simple > > header file > > >change can be included in the stable tag. > > >Any platform that uses BLE stack can get benefit from > > this definition update. > > > > > >> -----Original Message----- > > >> From: devel@edk2.groups.io <devel@edk2.groups.io> On > > Behalf Of Ni, > > >> Ray > > >> Sent: Monday, August 26, 2019 11:16 AM > > >> To: devel@edk2.groups.io > > >> Cc: Kinney, Michael D <michael.d.kinney@intel.com>; > > Gao, Liming > > ><liming.gao@intel.com> > > >> Subject: [edk2-devel] [PATCH] [edk2-stable201908] > > >MdePkg/BluetoothLeConfig.h: Add type > > EfiBluetoothSmpPeerAddressList > > >> > > >> To support auto-connection, > > EFI_BLUETOOTH_LE_SMP_DATA_TYPE needs > > >to > > >> add a new data type EfiBluetoothSmpPeerAddressList > > which associates > > >> with a list of Bluetooth per address connected > > before. > > >> > > >> This new data type was added in UEFI spec 2.7b. > > >> > > >> Signed-off-by: Ray Ni <ray.ni@intel.com> > > >> Cc: Michael D Kinney <michael.d.kinney@intel.com> > > >> Cc: Liming Gao <liming.gao@intel.com> > > >> --- > > >> MdePkg/Include/Protocol/BluetoothLeConfig.h | 4 +++- > > >> 1 file changed, 3 insertions(+), 1 deletion(-) > > >> > > >> diff --git > > a/MdePkg/Include/Protocol/BluetoothLeConfig.h > > >b/MdePkg/Include/Protocol/BluetoothLeConfig.h > > >> index 8c0f881f85..8726a58b15 100644 > > >> --- a/MdePkg/Include/Protocol/BluetoothLeConfig.h > > >> +++ b/MdePkg/Include/Protocol/BluetoothLeConfig.h > > >> @@ -2,7 +2,7 @@ > > >> EFI Bluetooth LE Config Protocol as defined in > > UEFI 2.7. > > >> This protocol abstracts user interface > > configuration for BluetoothLe device. > > >> > > >> - Copyright (c) 2017, Intel Corporation. All rights > > reserved.<BR> > > >> + Copyright (c) 2017 - 2019, Intel Corporation. All > > rights > > >> + reserved.<BR> > > >> SPDX-License-Identifier: BSD-2-Clause-Patent > > >> > > >> @par Revision Reference: > > >> @@ -451,6 +451,8 @@ typedef enum { > > >> EfiBluetoothSmpLocalCSRK, /* If Key hierarchy not > > supported */ > > >> EfiBluetoothSmpLocalSignCounter, > > >> EfiBluetoothSmpLocalDIV, > > >> + EfiBluetoothSmpPeerAddressList, > > >> + EfiBluetoothSmpMax, > > >> } EFI_BLUETOOTH_LE_SMP_DATA_TYPE; > > >> > > >> /** > > >> -- > > >> 2.21.0.windows.1 > > >> > > >> > > >> ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [edk2-devel] [PATCH] [edk2-stable201908] MdePkg/BluetoothLeConfig.h: Add type EfiBluetoothSmpPeerAddressList 2019-08-27 0:45 ` Ni, Ray @ 2019-08-27 10:18 ` Leif Lindholm 2019-08-27 12:47 ` Laszlo Ersek 1 sibling, 0 replies; 8+ messages in thread From: Leif Lindholm @ 2019-08-27 10:18 UTC (permalink / raw) To: Ni, Ray Cc: Kinney, Michael D, Gao, Liming, devel@edk2.groups.io, afish@apple.com, Laszlo Ersek (lersek@redhat.com) Understood. But that also means they depend on a well tested open source repo. So I agree with Mike - the value of getting it into the stable tag would be defeated by getting it in too late for useful testing. I would prefer for it to go into after the stable tag. However, that is my procedural opinion. The diff is trivial. So I am not not saying "no", only that my preference would be to merge it after the tag. Best Regards, Leif On Tue, Aug 27, 2019 at 12:45:54AM +0000, Ni, Ray wrote: > The consumers are in close source and won't be in open source in near future. > But consumers depend on open source repo. > > > -----Original Message----- > > From: Kinney, Michael D > > Sent: Monday, August 26, 2019 5:20 PM > > To: Gao, Liming <liming.gao@intel.com>; Ni, Ray <ray.ni@intel.com>; devel@edk2.groups.io; Kinney, Michael D > > <michael.d.kinney@intel.com> > > Cc: afish@apple.com; leif.lindholm@linaro.org; Laszlo Ersek (lersek@redhat.com) <lersek@redhat.com> > > Subject: RE: [edk2-devel] [PATCH] [edk2-stable201908] MdePkg/BluetoothLeConfig.h: Add type > > EfiBluetoothSmpPeerAddressList > > > > Ray, > > > > Are there any components that need these new enum values? > > > > The patch looks very straightforward and is from the > > UEFI Spec, so I think the risk is low. But if consumers > > will not show up until after the stable tag, perhaps this > > should be one of the first patches after the stable tag. > > > > Thanks, > > > > Mike > > > > > -----Original Message----- > > > From: Gao, Liming > > > Sent: Monday, August 26, 2019 4:57 PM > > > To: Ni, Ray <ray.ni@intel.com>; devel@edk2.groups.io > > > Cc: Kinney, Michael D <michael.d.kinney@intel.com>; > > > afish@apple.com; leif.lindholm@linaro.org; Laszlo Ersek > > > (lersek@redhat.com) <lersek@redhat.com> > > > Subject: RE: [edk2-devel] [PATCH] [edk2-stable201908] > > > MdePkg/BluetoothLeConfig.h: Add type > > > EfiBluetoothSmpPeerAddressList > > > > > > Cc to all Stewards. > > > > > > >-----Original Message----- > > > >From: Ni, Ray > > > >Sent: Tuesday, August 27, 2019 2:20 AM > > > >To: devel@edk2.groups.io; Ni, Ray <ray.ni@intel.com> > > > >Cc: Kinney, Michael D <michael.d.kinney@intel.com>; > > > Gao, Liming > > > ><liming.gao@intel.com> > > > >Subject: RE: [edk2-devel] [PATCH] [edk2-stable201908] > > > >MdePkg/BluetoothLeConfig.h: Add type > > > EfiBluetoothSmpPeerAddressList > > > > > > > >I understand it's now in code freeze phase. > > > >Just make a try to see whether this very clean/simple > > > header file > > > >change can be included in the stable tag. > > > >Any platform that uses BLE stack can get benefit from > > > this definition update. > > > > > > > >> -----Original Message----- > > > >> From: devel@edk2.groups.io <devel@edk2.groups.io> On > > > Behalf Of Ni, > > > >> Ray > > > >> Sent: Monday, August 26, 2019 11:16 AM > > > >> To: devel@edk2.groups.io > > > >> Cc: Kinney, Michael D <michael.d.kinney@intel.com>; > > > Gao, Liming > > > ><liming.gao@intel.com> > > > >> Subject: [edk2-devel] [PATCH] [edk2-stable201908] > > > >MdePkg/BluetoothLeConfig.h: Add type > > > EfiBluetoothSmpPeerAddressList > > > >> > > > >> To support auto-connection, > > > EFI_BLUETOOTH_LE_SMP_DATA_TYPE needs > > > >to > > > >> add a new data type EfiBluetoothSmpPeerAddressList > > > which associates > > > >> with a list of Bluetooth per address connected > > > before. > > > >> > > > >> This new data type was added in UEFI spec 2.7b. > > > >> > > > >> Signed-off-by: Ray Ni <ray.ni@intel.com> > > > >> Cc: Michael D Kinney <michael.d.kinney@intel.com> > > > >> Cc: Liming Gao <liming.gao@intel.com> > > > >> --- > > > >> MdePkg/Include/Protocol/BluetoothLeConfig.h | 4 +++- > > > >> 1 file changed, 3 insertions(+), 1 deletion(-) > > > >> > > > >> diff --git > > > a/MdePkg/Include/Protocol/BluetoothLeConfig.h > > > >b/MdePkg/Include/Protocol/BluetoothLeConfig.h > > > >> index 8c0f881f85..8726a58b15 100644 > > > >> --- a/MdePkg/Include/Protocol/BluetoothLeConfig.h > > > >> +++ b/MdePkg/Include/Protocol/BluetoothLeConfig.h > > > >> @@ -2,7 +2,7 @@ > > > >> EFI Bluetooth LE Config Protocol as defined in > > > UEFI 2.7. > > > >> This protocol abstracts user interface > > > configuration for BluetoothLe device. > > > >> > > > >> - Copyright (c) 2017, Intel Corporation. All rights > > > reserved.<BR> > > > >> + Copyright (c) 2017 - 2019, Intel Corporation. All > > > rights > > > >> + reserved.<BR> > > > >> SPDX-License-Identifier: BSD-2-Clause-Patent > > > >> > > > >> @par Revision Reference: > > > >> @@ -451,6 +451,8 @@ typedef enum { > > > >> EfiBluetoothSmpLocalCSRK, /* If Key hierarchy not > > > supported */ > > > >> EfiBluetoothSmpLocalSignCounter, > > > >> EfiBluetoothSmpLocalDIV, > > > >> + EfiBluetoothSmpPeerAddressList, > > > >> + EfiBluetoothSmpMax, > > > >> } EFI_BLUETOOTH_LE_SMP_DATA_TYPE; > > > >> > > > >> /** > > > >> -- > > > >> 2.21.0.windows.1 > > > >> > > > >> > > > >> > ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [edk2-devel] [PATCH] [edk2-stable201908] MdePkg/BluetoothLeConfig.h: Add type EfiBluetoothSmpPeerAddressList 2019-08-27 0:45 ` Ni, Ray 2019-08-27 10:18 ` Leif Lindholm @ 2019-08-27 12:47 ` Laszlo Ersek 2019-08-27 16:16 ` Ni, Ray 1 sibling, 1 reply; 8+ messages in thread From: Laszlo Ersek @ 2019-08-27 12:47 UTC (permalink / raw) To: Ni, Ray, Kinney, Michael D, Gao, Liming, devel@edk2.groups.io Cc: afish@apple.com, leif.lindholm@linaro.org On 08/27/19 02:45, Ni, Ray wrote: > The consumers are in close source and won't be in open source in near future. > But consumers depend on open source repo. In my interpretation, this patch does not qualify as a bugfix (suitable for merging during the hard feature freeze), and should be postponed to the next development cycle. Thanks Laszlo > >> -----Original Message----- >> From: Kinney, Michael D >> Sent: Monday, August 26, 2019 5:20 PM >> To: Gao, Liming <liming.gao@intel.com>; Ni, Ray <ray.ni@intel.com>; devel@edk2.groups.io; Kinney, Michael D >> <michael.d.kinney@intel.com> >> Cc: afish@apple.com; leif.lindholm@linaro.org; Laszlo Ersek (lersek@redhat.com) <lersek@redhat.com> >> Subject: RE: [edk2-devel] [PATCH] [edk2-stable201908] MdePkg/BluetoothLeConfig.h: Add type >> EfiBluetoothSmpPeerAddressList >> >> Ray, >> >> Are there any components that need these new enum values? >> >> The patch looks very straightforward and is from the >> UEFI Spec, so I think the risk is low. But if consumers >> will not show up until after the stable tag, perhaps this >> should be one of the first patches after the stable tag. >> >> Thanks, >> >> Mike >> >>> -----Original Message----- >>> From: Gao, Liming >>> Sent: Monday, August 26, 2019 4:57 PM >>> To: Ni, Ray <ray.ni@intel.com>; devel@edk2.groups.io >>> Cc: Kinney, Michael D <michael.d.kinney@intel.com>; >>> afish@apple.com; leif.lindholm@linaro.org; Laszlo Ersek >>> (lersek@redhat.com) <lersek@redhat.com> >>> Subject: RE: [edk2-devel] [PATCH] [edk2-stable201908] >>> MdePkg/BluetoothLeConfig.h: Add type >>> EfiBluetoothSmpPeerAddressList >>> >>> Cc to all Stewards. >>> >>>> -----Original Message----- >>>> From: Ni, Ray >>>> Sent: Tuesday, August 27, 2019 2:20 AM >>>> To: devel@edk2.groups.io; Ni, Ray <ray.ni@intel.com> >>>> Cc: Kinney, Michael D <michael.d.kinney@intel.com>; >>> Gao, Liming >>>> <liming.gao@intel.com> >>>> Subject: RE: [edk2-devel] [PATCH] [edk2-stable201908] >>>> MdePkg/BluetoothLeConfig.h: Add type >>> EfiBluetoothSmpPeerAddressList >>>> >>>> I understand it's now in code freeze phase. >>>> Just make a try to see whether this very clean/simple >>> header file >>>> change can be included in the stable tag. >>>> Any platform that uses BLE stack can get benefit from >>> this definition update. >>>> >>>>> -----Original Message----- >>>>> From: devel@edk2.groups.io <devel@edk2.groups.io> On >>> Behalf Of Ni, >>>>> Ray >>>>> Sent: Monday, August 26, 2019 11:16 AM >>>>> To: devel@edk2.groups.io >>>>> Cc: Kinney, Michael D <michael.d.kinney@intel.com>; >>> Gao, Liming >>>> <liming.gao@intel.com> >>>>> Subject: [edk2-devel] [PATCH] [edk2-stable201908] >>>> MdePkg/BluetoothLeConfig.h: Add type >>> EfiBluetoothSmpPeerAddressList >>>>> >>>>> To support auto-connection, >>> EFI_BLUETOOTH_LE_SMP_DATA_TYPE needs >>>> to >>>>> add a new data type EfiBluetoothSmpPeerAddressList >>> which associates >>>>> with a list of Bluetooth per address connected >>> before. >>>>> >>>>> This new data type was added in UEFI spec 2.7b. >>>>> >>>>> Signed-off-by: Ray Ni <ray.ni@intel.com> >>>>> Cc: Michael D Kinney <michael.d.kinney@intel.com> >>>>> Cc: Liming Gao <liming.gao@intel.com> >>>>> --- >>>>> MdePkg/Include/Protocol/BluetoothLeConfig.h | 4 +++- >>>>> 1 file changed, 3 insertions(+), 1 deletion(-) >>>>> >>>>> diff --git >>> a/MdePkg/Include/Protocol/BluetoothLeConfig.h >>>> b/MdePkg/Include/Protocol/BluetoothLeConfig.h >>>>> index 8c0f881f85..8726a58b15 100644 >>>>> --- a/MdePkg/Include/Protocol/BluetoothLeConfig.h >>>>> +++ b/MdePkg/Include/Protocol/BluetoothLeConfig.h >>>>> @@ -2,7 +2,7 @@ >>>>> EFI Bluetooth LE Config Protocol as defined in >>> UEFI 2.7. >>>>> This protocol abstracts user interface >>> configuration for BluetoothLe device. >>>>> >>>>> - Copyright (c) 2017, Intel Corporation. All rights >>> reserved.<BR> >>>>> + Copyright (c) 2017 - 2019, Intel Corporation. All >>> rights >>>>> + reserved.<BR> >>>>> SPDX-License-Identifier: BSD-2-Clause-Patent >>>>> >>>>> @par Revision Reference: >>>>> @@ -451,6 +451,8 @@ typedef enum { >>>>> EfiBluetoothSmpLocalCSRK, /* If Key hierarchy not >>> supported */ >>>>> EfiBluetoothSmpLocalSignCounter, >>>>> EfiBluetoothSmpLocalDIV, >>>>> + EfiBluetoothSmpPeerAddressList, >>>>> + EfiBluetoothSmpMax, >>>>> } EFI_BLUETOOTH_LE_SMP_DATA_TYPE; >>>>> >>>>> /** >>>>> -- >>>>> 2.21.0.windows.1 >>>>> >>>>> >>>>> > ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [edk2-devel] [PATCH] [edk2-stable201908] MdePkg/BluetoothLeConfig.h: Add type EfiBluetoothSmpPeerAddressList 2019-08-27 12:47 ` Laszlo Ersek @ 2019-08-27 16:16 ` Ni, Ray 0 siblings, 0 replies; 8+ messages in thread From: Ni, Ray @ 2019-08-27 16:16 UTC (permalink / raw) To: devel@edk2.groups.io, lersek@redhat.com, Kinney, Michael D, Gao, Liming Cc: afish@apple.com, leif.lindholm@linaro.org I relegalized the code freeze might be end very soon. I am ok with that. > -----Original Message----- > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Laszlo Ersek > Sent: Tuesday, August 27, 2019 5:48 AM > To: Ni, Ray <ray.ni@intel.com>; Kinney, Michael D <michael.d.kinney@intel.com>; Gao, Liming <liming.gao@intel.com>; > devel@edk2.groups.io > Cc: afish@apple.com; leif.lindholm@linaro.org > Subject: Re: [edk2-devel] [PATCH] [edk2-stable201908] MdePkg/BluetoothLeConfig.h: Add type > EfiBluetoothSmpPeerAddressList > > On 08/27/19 02:45, Ni, Ray wrote: > > The consumers are in close source and won't be in open source in near future. > > But consumers depend on open source repo. > > In my interpretation, this patch does not qualify as a bugfix (suitable > for merging during the hard feature freeze), and should be postponed to > the next development cycle. > > Thanks > Laszlo > > > > >> -----Original Message----- > >> From: Kinney, Michael D > >> Sent: Monday, August 26, 2019 5:20 PM > >> To: Gao, Liming <liming.gao@intel.com>; Ni, Ray <ray.ni@intel.com>; devel@edk2.groups.io; Kinney, Michael D > >> <michael.d.kinney@intel.com> > >> Cc: afish@apple.com; leif.lindholm@linaro.org; Laszlo Ersek (lersek@redhat.com) <lersek@redhat.com> > >> Subject: RE: [edk2-devel] [PATCH] [edk2-stable201908] MdePkg/BluetoothLeConfig.h: Add type > >> EfiBluetoothSmpPeerAddressList > >> > >> Ray, > >> > >> Are there any components that need these new enum values? > >> > >> The patch looks very straightforward and is from the > >> UEFI Spec, so I think the risk is low. But if consumers > >> will not show up until after the stable tag, perhaps this > >> should be one of the first patches after the stable tag. > >> > >> Thanks, > >> > >> Mike > >> > >>> -----Original Message----- > >>> From: Gao, Liming > >>> Sent: Monday, August 26, 2019 4:57 PM > >>> To: Ni, Ray <ray.ni@intel.com>; devel@edk2.groups.io > >>> Cc: Kinney, Michael D <michael.d.kinney@intel.com>; > >>> afish@apple.com; leif.lindholm@linaro.org; Laszlo Ersek > >>> (lersek@redhat.com) <lersek@redhat.com> > >>> Subject: RE: [edk2-devel] [PATCH] [edk2-stable201908] > >>> MdePkg/BluetoothLeConfig.h: Add type > >>> EfiBluetoothSmpPeerAddressList > >>> > >>> Cc to all Stewards. > >>> > >>>> -----Original Message----- > >>>> From: Ni, Ray > >>>> Sent: Tuesday, August 27, 2019 2:20 AM > >>>> To: devel@edk2.groups.io; Ni, Ray <ray.ni@intel.com> > >>>> Cc: Kinney, Michael D <michael.d.kinney@intel.com>; > >>> Gao, Liming > >>>> <liming.gao@intel.com> > >>>> Subject: RE: [edk2-devel] [PATCH] [edk2-stable201908] > >>>> MdePkg/BluetoothLeConfig.h: Add type > >>> EfiBluetoothSmpPeerAddressList > >>>> > >>>> I understand it's now in code freeze phase. > >>>> Just make a try to see whether this very clean/simple > >>> header file > >>>> change can be included in the stable tag. > >>>> Any platform that uses BLE stack can get benefit from > >>> this definition update. > >>>> > >>>>> -----Original Message----- > >>>>> From: devel@edk2.groups.io <devel@edk2.groups.io> On > >>> Behalf Of Ni, > >>>>> Ray > >>>>> Sent: Monday, August 26, 2019 11:16 AM > >>>>> To: devel@edk2.groups.io > >>>>> Cc: Kinney, Michael D <michael.d.kinney@intel.com>; > >>> Gao, Liming > >>>> <liming.gao@intel.com> > >>>>> Subject: [edk2-devel] [PATCH] [edk2-stable201908] > >>>> MdePkg/BluetoothLeConfig.h: Add type > >>> EfiBluetoothSmpPeerAddressList > >>>>> > >>>>> To support auto-connection, > >>> EFI_BLUETOOTH_LE_SMP_DATA_TYPE needs > >>>> to > >>>>> add a new data type EfiBluetoothSmpPeerAddressList > >>> which associates > >>>>> with a list of Bluetooth per address connected > >>> before. > >>>>> > >>>>> This new data type was added in UEFI spec 2.7b. > >>>>> > >>>>> Signed-off-by: Ray Ni <ray.ni@intel.com> > >>>>> Cc: Michael D Kinney <michael.d.kinney@intel.com> > >>>>> Cc: Liming Gao <liming.gao@intel.com> > >>>>> --- > >>>>> MdePkg/Include/Protocol/BluetoothLeConfig.h | 4 +++- > >>>>> 1 file changed, 3 insertions(+), 1 deletion(-) > >>>>> > >>>>> diff --git > >>> a/MdePkg/Include/Protocol/BluetoothLeConfig.h > >>>> b/MdePkg/Include/Protocol/BluetoothLeConfig.h > >>>>> index 8c0f881f85..8726a58b15 100644 > >>>>> --- a/MdePkg/Include/Protocol/BluetoothLeConfig.h > >>>>> +++ b/MdePkg/Include/Protocol/BluetoothLeConfig.h > >>>>> @@ -2,7 +2,7 @@ > >>>>> EFI Bluetooth LE Config Protocol as defined in > >>> UEFI 2.7. > >>>>> This protocol abstracts user interface > >>> configuration for BluetoothLe device. > >>>>> > >>>>> - Copyright (c) 2017, Intel Corporation. All rights > >>> reserved.<BR> > >>>>> + Copyright (c) 2017 - 2019, Intel Corporation. All > >>> rights > >>>>> + reserved.<BR> > >>>>> SPDX-License-Identifier: BSD-2-Clause-Patent > >>>>> > >>>>> @par Revision Reference: > >>>>> @@ -451,6 +451,8 @@ typedef enum { > >>>>> EfiBluetoothSmpLocalCSRK, /* If Key hierarchy not > >>> supported */ > >>>>> EfiBluetoothSmpLocalSignCounter, > >>>>> EfiBluetoothSmpLocalDIV, > >>>>> + EfiBluetoothSmpPeerAddressList, > >>>>> + EfiBluetoothSmpMax, > >>>>> } EFI_BLUETOOTH_LE_SMP_DATA_TYPE; > >>>>> > >>>>> /** > >>>>> -- > >>>>> 2.21.0.windows.1 > >>>>> > >>>>> > >>>>> > > > > > ^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH] [edk2-stable201908] MdePkg/BluetoothLeConfig.h: Add type EfiBluetoothSmpPeerAddressList @ 2019-08-26 18:16 Ni, Ray 2019-09-04 0:05 ` [edk2-devel] " Liming Gao 0 siblings, 1 reply; 8+ messages in thread From: Ni, Ray @ 2019-08-26 18:16 UTC (permalink / raw) To: devel; +Cc: Michael D Kinney, Liming Gao To support auto-connection, EFI_BLUETOOTH_LE_SMP_DATA_TYPE needs to add a new data type EfiBluetoothSmpPeerAddressList which associates with a list of Bluetooth per address connected before. This new data type was added in UEFI spec 2.7b. Signed-off-by: Ray Ni <ray.ni@intel.com> Cc: Michael D Kinney <michael.d.kinney@intel.com> Cc: Liming Gao <liming.gao@intel.com> --- MdePkg/Include/Protocol/BluetoothLeConfig.h | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/MdePkg/Include/Protocol/BluetoothLeConfig.h b/MdePkg/Include/Protocol/BluetoothLeConfig.h index 8c0f881f85..8726a58b15 100644 --- a/MdePkg/Include/Protocol/BluetoothLeConfig.h +++ b/MdePkg/Include/Protocol/BluetoothLeConfig.h @@ -2,7 +2,7 @@ EFI Bluetooth LE Config Protocol as defined in UEFI 2.7. This protocol abstracts user interface configuration for BluetoothLe device. - Copyright (c) 2017, Intel Corporation. All rights reserved.<BR> + Copyright (c) 2017 - 2019, Intel Corporation. All rights reserved.<BR> SPDX-License-Identifier: BSD-2-Clause-Patent @par Revision Reference: @@ -451,6 +451,8 @@ typedef enum { EfiBluetoothSmpLocalCSRK, /* If Key hierarchy not supported */ EfiBluetoothSmpLocalSignCounter, EfiBluetoothSmpLocalDIV, + EfiBluetoothSmpPeerAddressList, + EfiBluetoothSmpMax, } EFI_BLUETOOTH_LE_SMP_DATA_TYPE; /** -- 2.21.0.windows.1 ^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [edk2-devel] [PATCH] [edk2-stable201908] MdePkg/BluetoothLeConfig.h: Add type EfiBluetoothSmpPeerAddressList 2019-08-26 18:16 Ni, Ray @ 2019-09-04 0:05 ` Liming Gao 0 siblings, 0 replies; 8+ messages in thread From: Liming Gao @ 2019-09-04 0:05 UTC (permalink / raw) To: devel@edk2.groups.io, Ni, Ray; +Cc: Kinney, Michael D Reviewed-by: Liming Gao <liming.gao@intel.com> >-----Original Message----- >From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of Ni, >Ray >Sent: Tuesday, August 27, 2019 2:16 AM >To: devel@edk2.groups.io >Cc: Kinney, Michael D <michael.d.kinney@intel.com>; Gao, Liming ><liming.gao@intel.com> >Subject: [edk2-devel] [PATCH] [edk2-stable201908] >MdePkg/BluetoothLeConfig.h: Add type EfiBluetoothSmpPeerAddressList > >To support auto-connection, EFI_BLUETOOTH_LE_SMP_DATA_TYPE needs to >add a new data type EfiBluetoothSmpPeerAddressList which associates >with a list of Bluetooth per address connected before. > >This new data type was added in UEFI spec 2.7b. > >Signed-off-by: Ray Ni <ray.ni@intel.com> >Cc: Michael D Kinney <michael.d.kinney@intel.com> >Cc: Liming Gao <liming.gao@intel.com> >--- > MdePkg/Include/Protocol/BluetoothLeConfig.h | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > >diff --git a/MdePkg/Include/Protocol/BluetoothLeConfig.h >b/MdePkg/Include/Protocol/BluetoothLeConfig.h >index 8c0f881f85..8726a58b15 100644 >--- a/MdePkg/Include/Protocol/BluetoothLeConfig.h >+++ b/MdePkg/Include/Protocol/BluetoothLeConfig.h >@@ -2,7 +2,7 @@ > EFI Bluetooth LE Config Protocol as defined in UEFI 2.7. > This protocol abstracts user interface configuration for BluetoothLe device. > >- Copyright (c) 2017, Intel Corporation. All rights reserved.<BR> >+ Copyright (c) 2017 - 2019, Intel Corporation. All rights reserved.<BR> > SPDX-License-Identifier: BSD-2-Clause-Patent > > @par Revision Reference: >@@ -451,6 +451,8 @@ typedef enum { > EfiBluetoothSmpLocalCSRK, /* If Key hierarchy not supported */ > EfiBluetoothSmpLocalSignCounter, > EfiBluetoothSmpLocalDIV, >+ EfiBluetoothSmpPeerAddressList, >+ EfiBluetoothSmpMax, > } EFI_BLUETOOTH_LE_SMP_DATA_TYPE; > > /** >-- >2.21.0.windows.1 > > > ^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2019-09-04 0:05 UTC | newest] Thread overview: 8+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- [not found] <15BE8BCCADEA547C.18314@groups.io> 2019-08-26 18:19 ` [edk2-devel] [PATCH] [edk2-stable201908] MdePkg/BluetoothLeConfig.h: Add type EfiBluetoothSmpPeerAddressList Ni, Ray 2019-08-26 23:56 ` Liming Gao 2019-08-27 0:20 ` Michael D Kinney 2019-08-27 0:45 ` Ni, Ray 2019-08-27 10:18 ` Leif Lindholm 2019-08-27 12:47 ` Laszlo Ersek 2019-08-27 16:16 ` Ni, Ray 2019-08-26 18:16 Ni, Ray 2019-09-04 0:05 ` [edk2-devel] " Liming Gao
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox