Thanks Liming. I forgot the freeze time.
From my point, this is not critical and it is a feature.

Thanks,
Zhichao

> -----Original Message-----
> From: [email protected] <[email protected]> On Behalf Of
> gaoliming via groups.io
> Sent: Friday, November 18, 2022 10:23 AM
> To: Gao, Zhichao <[email protected]>; Wu, Jiaxin
> <[email protected]>; Zhang, JuX <[email protected]>;
> [email protected]
> Cc: Ni, Ray <[email protected]>
> Subject: [edk2-devel] 回复: [PATCH] ShellPkg: Update
> ProcessorUpgradeTable REF:
> https://bugzilla.tianocore.org/show_bug.cgi?id=4156
> 
> Zhichao:
>   Please see the announce mail
> https://edk2.groups.io/g/devel/message/96175
> 
>   Now, we are in hard feature freeze, only the critical bug fix is allowed.
> If the patch wants to catch this
> stable tag, please highlight it and get approval from Stewards.
> 
> Thanks
> Liming
> > -----邮件原件-----
> > 发件人: Gao, Zhichao <[email protected]>
> > 发送时间: 2022年11月18日 10:02
> > 收件人: Wu, Jiaxin <[email protected]>; Zhang, JuX
> <[email protected]>;
> > [email protected]; Gao, Liming <[email protected]>
> > 抄送: Ni, Ray <[email protected]>
> > 主题: RE: [PATCH] ShellPkg: Update ProcessorUpgradeTable REF:
> > https://bugzilla.tianocore.org/show_bug.cgi?id=4156
> >
> > Hi Liming,
> >
> > I have created the PR: https://github.com/tianocore/edk2/pull/3647
> > But I am not able to add the push label. Can you help to add it? And
> > why I
> lost
> > the right, or is the progress changed?
> >
> > Thanks,
> > Zhichao
> >
> > > -----Original Message-----
> > > From: Wu, Jiaxin <[email protected]>
> > > Sent: Thursday, November 17, 2022 9:49 AM
> > > To: Gao, Zhichao <[email protected]>; Zhang, JuX
> > > <[email protected]>; [email protected]
> > > Cc: Ni, Ray <[email protected]>
> > > Subject: RE: [PATCH] ShellPkg: Update ProcessorUpgradeTable REF:
> > > https://bugzilla.tianocore.org/show_bug.cgi?id=4156
> > >
> > > Reviewed-by: Jiaxin Wu <[email protected]>
> > >
> > > > -----Original Message-----
> > > > From: Gao, Zhichao <[email protected]>
> > > > Sent: Thursday, November 17, 2022 9:27 AM
> > > > To: Zhang, JuX <[email protected]>; [email protected]
> > > > Cc: Wu, Jiaxin <[email protected]>; Ni, Ray <[email protected]>
> > > > Subject: RE: [PATCH] ShellPkg: Update ProcessorUpgradeTable REF:
> > > > https://bugzilla.tianocore.org/show_bug.cgi?id=4156
> > > >
> > > > The code change looks fine to me. But I would adjust the commit
> > > > message when I send the PR.
> > > > Reviewed-by: Zhichao Gao <[email protected]>
> > > >
> > > > Thanks,
> > > > Zhichao
> > > >
> > > > > -----Original Message-----
> > > > > From: Zhang, JuX <[email protected]>
> > > > > Sent: Wednesday, November 16, 2022 3:13 PM
> > > > > To: [email protected]
> > > > > Cc: Gao, Zhichao <[email protected]>; Wu, Jiaxin
> > > > > <[email protected]>
> > > > > Subject: [PATCH] ShellPkg: Update ProcessorUpgradeTable REF:
> > > > > https://bugzilla.tianocore.org/show_bug.cgi?id=4156
> > > > >
> > > > > Refer to the latest SMBIOS v3.6, SMBIOS Socket not completely
> defined.
> > > > >
> > > > > SMBIOS v3.6:
> > > > >
> > >
> >
> https://www.dmtf.org/sites/default/files/standards/documents/DSP0134_3.
> > > > > 6.0.pdf
> > > > >
> > > > > Signed-off-by: JuX Zhang <[email protected]>
> > > > > Cc: Zhichao Gao <[email protected]>
> > > > > Cc: Jiaxin Wu <[email protected]>
> > > > > ---
> > > > >  .../SmbiosView/QueryTable.c                   | 36
> > +++++++++++++++++++
> > > > >  1 file changed, 36 insertions(+)
> > > > >
> > > > > diff --git
> > > > >
> > >
> >
> a/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/QueryTable.
> > > > > c
> > > > >
> > >
> >
> b/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/QueryTable.
> > > > > c
> > > > > index 2e62ae3a34f8..edb46817e422 100644
> > > > > ---
> > > > >
> > >
> >
> a/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/QueryTable.
> > > > > c
> > > > > +++
> > > > >
> > >
> >
> b/ShellPkg/Library/UefiShellDebug1CommandsLib/SmbiosView/QueryTable.
> > > > > c
> > > > > @@ -601,6 +601,42 @@ TABLE_ITEM  ProcessorUpgradeTable[] = {
> > > > >    {
> > > > >
> > > > >      0x3F,
> > > > >
> > > > >      L"Socket LGA4677"
> > > > >
> > > > > +  },
> > > > >
> > > > > +  {
> > > > >
> > > > > +    0x40,
> > > > >
> > > > > +    L"Socket LGA1700"
> > > > >
> > > > > +  },
> > > > >
> > > > > +  {
> > > > >
> > > > > +    0x41,
> > > > >
> > > > > +    L"Socket BGA1744"
> > > > >
> > > > > +  },
> > > > >
> > > > > +  {
> > > > >
> > > > > +    0x42,
> > > > >
> > > > > +    L"Socket BGA1781"
> > > > >
> > > > > +  },
> > > > >
> > > > > +  {
> > > > >
> > > > > +    0x43,
> > > > >
> > > > > +    L"Socket BGA1211"
> > > > >
> > > > > +  },
> > > > >
> > > > > +  {
> > > > >
> > > > > +    0x44,
> > > > >
> > > > > +    L"Socket BGA2422"
> > > > >
> > > > > +  },
> > > > >
> > > > > +  {
> > > > >
> > > > > +    0x45,
> > > > >
> > > > > +    L"Socket LGA1211"
> > > > >
> > > > > +  },
> > > > >
> > > > > +  {
> > > > >
> > > > > +    0x46,
> > > > >
> > > > > +    L"Socket LGA2422"
> > > > >
> > > > > +  },
> > > > >
> > > > > +  {
> > > > >
> > > > > +    0x47,
> > > > >
> > > > > +    L"Socket LGA5773"
> > > > >
> > > > > +  },
> > > > >
> > > > > +  {
> > > > >
> > > > > +    0x48,
> > > > >
> > > > > +    L"Socket BGA5773"
> > > > >
> > > > >    }
> > > > >
> > > > >  };
> > > > >
> > > > >
> > > > >
> > > > > --
> > > > > 2.26.2.windows.1
> 
> 
> 
> 
> 
> 
> 



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#96479): https://edk2.groups.io/g/devel/message/96479
Mute This Topic: https://groups.io/mt/95104806/21656
Group Owner: [email protected]
Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to