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...@intel.com>; Kinney, Michael D
> <michael.d.kin...@intel.com>; Gao, Liming <liming....@intel.com>;
> devel@edk2.groups.io
> Cc: af...@apple.com; leif.lindh...@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....@intel.com>; Ni, Ray <ray...@intel.com>;
> >> devel@edk2.groups.io; Kinney, Michael D
> >> <michael.d.kin...@intel.com>
> >> Cc: af...@apple.com; leif.lindh...@linaro.org; Laszlo Ersek
> >> (ler...@redhat.com) <ler...@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...@intel.com>; devel@edk2.groups.io
> >>> Cc: Kinney, Michael D <michael.d.kin...@intel.com>;
> >>> af...@apple.com; leif.lindh...@linaro.org; Laszlo Ersek
> >>> (ler...@redhat.com) <ler...@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...@intel.com>
> >>>> Cc: Kinney, Michael D <michael.d.kin...@intel.com>;
> >>> Gao, Liming
> >>>> <liming....@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.kin...@intel.com>;
> >>> Gao, Liming
> >>>> <liming....@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...@intel.com>
> >>>>> Cc: Michael D Kinney <michael.d.kin...@intel.com>
> >>>>> Cc: Liming Gao <liming....@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
> >>>>>
> >>>>>
> >>>>>
> >
>
>
>
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#46481): https://edk2.groups.io/g/devel/message/46481
Mute This Topic: https://groups.io/mt/33037003/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
