From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail05.groups.io (mail05.groups.io [45.79.224.7]) by spool.mail.gandi.net (Postfix) with ESMTPS id 3035BAC0B54 for ; Mon, 24 Feb 2025 05:31:09 +0000 (UTC) DKIM-Signature: a=rsa-sha256; bh=G9d/Ko/3sgJ0E0rT0JOtmv9716pv57ToXzwlfZPIaYE=; c=relaxed/simple; d=groups.io; h=From:To:CC:Subject:Thread-Topic:Thread-Index:Date:Message-ID:References:In-Reply-To:Accept-Language:msip_labels:MIME-Version:Precedence:List-Subscribe:List-Help:Sender:List-Id:Mailing-List:Delivered-To:Resent-Date:Resent-From:Reply-To:List-Unsubscribe-Post:List-Unsubscribe:Content-Language:Content-Type; s=20240830; t=1740375069; v=1; x=1740634268; b=mPkRU0NQMy1INFKYDIiv4n4UfsRs9WLeGuycW77NgQTIEf+pH9+fEkc9HYmpGy6kfu0MvZRw 1PgVbgrRTDXMnPhxgsRcOIiz+BpUtuOFyISi6NvKKOS5XiR/O9dyzSKNlenDdfpiJ7l8/XSBUC2 mjgSik1iPHE2uHGFxINJvoxVgHDat5KuHpVmG0p0LYxIooLRsNBsRulsYkRyoVuWjth4HTtuIqR /6ZmBZH4C9VPq5/WPYxCXV9WSQ2lP98MV81mFNJZ56KUzlAu41Thq3TOtqaCFi2mqrFS1nElZOa /VpkX1kxUaDS1ZU+IP8G53xwIuKx6xmfKOKr+sx6e4uMw== X-Received: by 127.0.0.2 with SMTP id 8xs8YY7687511xjifWN0Q3fj; Sun, 23 Feb 2025 21:31:08 -0800 X-Received: from mgamail.intel.com (mgamail.intel.com [198.175.65.11]) by mx.groups.io with SMTP id smtpd.web10.82620.1740375067410733397 for ; Sun, 23 Feb 2025 21:31:07 -0800 X-CSE-ConnectionGUID: Z6aS3NxtT++96qkXoD3AdQ== X-CSE-MsgGUID: AXMa8gkCTbiLR3rR9ufvRQ== X-IronPort-AV: E=McAfee;i="6700,10204,11354"; a="51332900" X-IronPort-AV: E=Sophos;i="6.13,309,1732608000"; d="scan'208,217";a="51332900" X-Received: from fmviesa007.fm.intel.com ([10.60.135.147]) by orvoesa103.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 23 Feb 2025 21:31:05 -0800 X-CSE-ConnectionGUID: rkA/ANoAQx+S7qoAIFrXJA== X-CSE-MsgGUID: b282/g3RRu+Os0L7RWIMVw== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.13,309,1732608000"; d="scan'208,217";a="115977279" X-Received: from orsmsx903.amr.corp.intel.com ([10.22.229.25]) by fmviesa007.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 23 Feb 2025 21:31:04 -0800 X-Received: from ORSMSX901.amr.corp.intel.com (10.22.229.23) by ORSMSX903.amr.corp.intel.com (10.22.229.25) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.1544.14; Sun, 23 Feb 2025 21:31:02 -0800 X-Received: from orsedg603.ED.cps.intel.com (10.7.248.4) by ORSMSX901.amr.corp.intel.com (10.22.229.23) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.1544.14 via Frontend Transport; Sun, 23 Feb 2025 21:31:02 -0800 X-Received: from NAM11-CO1-obe.outbound.protection.outlook.com (104.47.56.168) by edgegateway.intel.com (134.134.137.100) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.1.2507.44; Sun, 23 Feb 2025 21:31:02 -0800 X-Received: from DM4PR11MB8226.namprd11.prod.outlook.com (2603:10b6:8:182::12) by SA3PR11MB7626.namprd11.prod.outlook.com (2603:10b6:806:307::8) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8466.20; Mon, 24 Feb 2025 05:30:56 +0000 X-Received: from DM4PR11MB8226.namprd11.prod.outlook.com ([fe80::aa83:b533:5b08:5390]) by DM4PR11MB8226.namprd11.prod.outlook.com ([fe80::aa83:b533:5b08:5390%5]) with mapi id 15.20.8466.016; Mon, 24 Feb 2025 05:30:56 +0000 From: "Ni, Ray via groups.io" To: Sean Rhodes CC: "Dong, Guo" , "Tong, Justin" , "devel@edk2.groups.io" Subject: Re: [edk2-devel] [tianocore/edk2] 83a86f: UefiPayloadPkg: Allow boot timeout to be configured Thread-Topic: [tianocore/edk2] 83a86f: UefiPayloadPkg: Allow boot timeout to be configured Thread-Index: AQHbhjquMCpof/6nKUCroW0NmWpM7bNV7B+i Date: Mon, 24 Feb 2025 05:30:56 +0000 Message-ID: References: In-Reply-To: Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: msip_labels: x-ms-publictraffictype: Email x-ms-traffictypediagnostic: DM4PR11MB8226:EE_|SA3PR11MB7626:EE_ x-ms-office365-filtering-correlation-id: fbcf8428-068d-43bf-57a8-08dd549469be x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam-message-info: =?iso-8859-1?Q?RTDDke9EhoAq7RFjHtvvddZHBUYLE7XeBkhooHZhVZiNTeBxOWLTDXOxlv?= =?iso-8859-1?Q?MwOziyWzD7ZDjRoDOFXhgakR5xF6zd5Eo5hAVreqkR8TNSd+YNd5xKWDhl?= =?iso-8859-1?Q?FyPVRzsZp7rV0uuxXMPvAbzi36qoHoTqsFOkaiLmcBF21yN94JruIgdtaP?= =?iso-8859-1?Q?17OLjQ8ntbKG4qXy+mnVE/saGBJW0pHl5MvOPsj2I8b8uQxOt4lcB16Wa6?= =?iso-8859-1?Q?lUuwjE0lH0su5L59pu39X/y015Olc7IK2zo+FnMvVlrswhZg7l9ssVXSDE?= =?iso-8859-1?Q?8VNK954TP7rHsvHeKI5VcvkKS+ijYBiX4zeU8goPkkXgmsxtI3HTtb90Nj?= =?iso-8859-1?Q?rne97yMBgsMsgPNLTyXBBnwMSjXyVCCgdDKfu6kedAJGCswJvIgoWTXxqw?= =?iso-8859-1?Q?zCyXlrQWQKsWkUaMeZg/IhNoQ8HnDPvxALExac5lLefYuv2fu3iphDJq7f?= =?iso-8859-1?Q?1JWiGYrmNWzMAn81tVmh+qMAVhecTlXGoZCsqswM2JopPrpXesOobefuTR?= =?iso-8859-1?Q?mC3p6n3m+44RUw/tEGNb7CTEf9Q5yEMqVIqMoaSeOR4pbdHSG5ryI8gMLz?= =?iso-8859-1?Q?7HcxLty4+huhtpZBCCoD8a0BvZY/B9PdrWQcQwVAswdaYh968qT0nOZwYs?= =?iso-8859-1?Q?/Jb3Kswc1paR/AiZGUCb6pn6elv9R13bugSi8xaz3xo7Z2gske314QIGSk?= =?iso-8859-1?Q?cWLaIQyKBmiwiCeF6inaQcuuN8N2eiJDM9tiZ/1gAXWo+ecIMX6pABVEnp?= =?iso-8859-1?Q?cL/g8S1rHAZWQ+rLwo3cKYBz8+i8zshzFdUBDl5VDKacUe9M6Jtno6e/xA?= =?iso-8859-1?Q?YkiWdr/0sqvHRjePCI5uOih/N8l8NS7jtJgBrxWiZ2pRxdYRV4fhUYTRR1?= =?iso-8859-1?Q?WXKH9eJZJR/ZzKdtJRt6y1N/RfENdTQELBkWc7HSsbvm5eXBNMOSBgcIYb?= =?iso-8859-1?Q?vqa/Wjsr2smfPcMdbsrlIq7cJR6gSk0s3+Ug079fZDp8gSTOmuOJ4Xkfnf?= =?iso-8859-1?Q?Ivp+7XecltnRztepdawvajec0qCPEEbqKMYynR4PVstFyz26VwWYX9PhKn?= =?iso-8859-1?Q?eoRTXrToAbxgkrE8u3I8XFY37l0nIKxkI8HWNSl1WbAr62egEfAcWi6XIk?= =?iso-8859-1?Q?OHk4kE1S8LS08Xkvi9BidCLEjWMCusg6G97sllbibs7sTwPqHv3pZv3Adm?= =?iso-8859-1?Q?So1Q57Z8hmzXsXhdcemFPO9knJtnLiNSDRSrVukRGSVQgwvsv+qzQ7aZ+Q?= =?iso-8859-1?Q?a/GR625BYB3EJBdVlcorLkj31t5giiCJUG0Dhe2OwX9wCq/5rc0ysCwaB7?= =?iso-8859-1?Q?FwnKa07VE5fajfLjDFFJpI752hU0v8ZX3x9A1x6pHQuF56UFOUj1BraR6C?= =?iso-8859-1?Q?WixZfR6Fm43LB9MGmZjovojW5i5lHpHnOfn9GAc+e0Oo6v9QeWCADhVf0N?= =?iso-8859-1?Q?2M4ykiKT9DhxbMcuFNv12yqcD99JDEF4UqXGig=3D=3D?= x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?iso-8859-1?Q?Q26ooYKBBiSxWLqnSRW+Cdf2rNGChoygsJ1tlfoXs1Z58S5ObesOsUkKu7?= =?iso-8859-1?Q?f8vjwpXNcgbmpQucCHDJyp6k+DBvAGsIzB+QHpQHOj3Z0sPWXDIDiETeiD?= =?iso-8859-1?Q?HSNN/ETeohdkyg887Zz7C8HmqFoLdm4BYzjP4PVZFKP83zn6P29Sdj4/Pi?= =?iso-8859-1?Q?h4B338A8xZs0Ao8sGvvEA3o5TBccSmG0z6dA5WjV+rELLGEZRe6s6fCuIu?= =?iso-8859-1?Q?ak/dchU541M9WfOVGiP4Eit2DSU+zt941xy8bT+NHaclkAUSHCR5gi5mSE?= =?iso-8859-1?Q?J2HE4MPn1NXRr+1jS9ZMkVnPEAfG9FfKDQ+rHEgay88KqOu+HmeFYoC0u0?= =?iso-8859-1?Q?mZ7W5qil6rGoGoPJH5gay4XS+8DQKMwBmXjV4gE4bQt/HWjMk0LX/WFM47?= =?iso-8859-1?Q?csK2PLpHgYVwZ56O6rosYyIgIyB3kkSXeHz+fkC2K3tlapWK7yeEN2+uuj?= =?iso-8859-1?Q?xec8x4gPsCcwHD09mmbOnON+X8bYsPt+oLeby/pM5Ay9bzUG00oMC2XujI?= =?iso-8859-1?Q?a5uqUWDMwaHjbMBMM6dTt2mXkrvXw6Lv6gZ08//8NPOyDGi3iqN6hXKJd4?= =?iso-8859-1?Q?VIFlfmIqMaand927BBMd1lbf6dO8Z5C323lTCT3ie7hj5TkU5lvr7NACrL?= =?iso-8859-1?Q?HkWcdnxBFv86nH6ch5L20TCJhjnXmHJhdVm6Bcghg7HWgVGsESTkFkz3WC?= =?iso-8859-1?Q?ROTMRLQ6bivXhcXeEnvD1N2AYePhTTj3GaQGg4Xx//sqxpvUL+hXXkJd5j?= =?iso-8859-1?Q?KjfUtEIFXdMQEKHfQLDzmMT7UGG1iNfoLUUpjbpN9rvqTxZZFP3XMzd+L3?= =?iso-8859-1?Q?r0fGZs2rywdgU7Be+qneA8jt4KEUbgqnvV0uZ34rM4HxwmHWbD1lijEKzH?= =?iso-8859-1?Q?2rt8HbxoVDsg5w+bWK6cXD3VEDFov/6ey9K+TIvUFnzcMSvXsiXH9KLAi7?= =?iso-8859-1?Q?wdd3bKSXaM9W6s01OdisncjHuyIZoXgMaNtxMeZ/ekXhVnVVJUZ7tVwwhz?= =?iso-8859-1?Q?xeBvx5H5/vmNVZ1nI7SWNB6gIIl+kbj5rHaHlOs0EMH8aaAzDFuqo0iLWr?= =?iso-8859-1?Q?L+Yt16mircDoYEEBzw/4K2nB7OaRKH02oP9NeU927uXMbcSsIcWxk1Oj+Y?= =?iso-8859-1?Q?4UzqQT83lpzRb0WGalT9vt/5t8whultYOk1GZWSsp72PPkdXFkfAQzJcn5?= =?iso-8859-1?Q?z0HYQjDrG/cj9FfqGaGWHAFbZTzFnllMjoC8tZA9vV21HlC7j1m4eZ4h5l?= =?iso-8859-1?Q?xcrllZxEdKDbPi/7ADFznci+pItoAqhwscWXn1YM6jo/2wC93zuOFDvM7Y?= =?iso-8859-1?Q?6l61RLCpqjKppNJIVipRRDjJ2UHAmWeS7jU/t7geWLptcD6vAKF5bf41iq?= =?iso-8859-1?Q?rI/X0UvBwlsrRjoV/Dm1R+W9rS0VKJSDfCzcsERC2pcj74EuropozjBSgW?= =?iso-8859-1?Q?qH/Yv/R17/5Yi/P8wEZg8c3tyDOinGGFthXLtboja2/yggpxzqOYS+ZeTE?= =?iso-8859-1?Q?ktqeus6/zYEQSaP/h9Lj3cJDSvAZqzyy+PwtVpEMdKatNMxsV7Ed1DRJKp?= =?iso-8859-1?Q?kZUcZN46WcgbHxx6XswKEV4pBVat9rUthH3SvQGljbPWQxtyWZmJ4YFx2s?= =?iso-8859-1?Q?UgBv9Yr19xdKM=3D?= MIME-Version: 1.0 X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: DM4PR11MB8226.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: fbcf8428-068d-43bf-57a8-08dd549469be X-MS-Exchange-CrossTenant-originalarrivaltime: 24 Feb 2025 05:30:56.2089 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 46c98d88-e344-4ed4-8496-4ed7712e255d X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: YEi56qZNHvaVIzSw8PiHluVQPj6jeC9F/zLwSsXdJG2nq8uvG7svnAYskROyQ+++MAOH5ZpjZS5Esq0baZX+Sg== X-MS-Exchange-Transport-CrossTenantHeadersStamped: SA3PR11MB7626 X-OriginatorOrg: intel.com Precedence: Bulk List-Subscribe: List-Help: Sender: devel@edk2.groups.io List-Id: Mailing-List: list devel@edk2.groups.io; contact devel+owner@edk2.groups.io Resent-Date: Sun, 23 Feb 2025 21:31:07 -0800 Resent-From: ray.ni@intel.com Reply-To: devel@edk2.groups.io,ray.ni@intel.com List-Unsubscribe-Post: List-Unsubscribe=One-Click List-Unsubscribe: X-Gm-Message-State: pLipfS5kYMKVYjkNq0NohA4Jx7686176AA= Content-Language: en-US Content-Type: multipart/alternative; boundary="_000_DM4PR11MB8226F5DCD5C603F82B54F0F28CC02DM4PR11MB8226namp_" X-GND-Status: LEGIT Authentication-Results: spool.mail.gandi.net; dkim=pass header.d=groups.io header.s=20240830 header.b=mPkRU0NQ; dmarc=pass (policy=none) header.from=groups.io; spf=pass (spool.mail.gandi.net: domain of bounce@groups.io designates 45.79.224.7 as permitted sender) smtp.mailfrom=bounce@groups.io --_000_DM4PR11MB8226F5DCD5C603F82B54F0F28CC02DM4PR11MB8226namp_ Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Sean, I do not think the change that makes PcdPlatformBootTimeOut as DynamicHii P= CD in UefiPayloadPkg is correct. When VARIABLE_SUPPORT is not "EMU" or "SPI", UPL does not include any varia= ble manipulation logic. Defining a PCD to a variable seems not make sense. Thanks, Ray ________________________________ From: Sean Rhodes via edk2-commits Sent: Monday, February 24, 2025 5:32 To: edk2-commits@lists.sourceforge.net Subject: [tianocore/edk2] 83a86f: UefiPayloadPkg: Allow boot timeout to be = configured Branch: refs/heads/master Home: https://github.com/tianocore/edk2 Commit: 83a86f465ccb1a792f5c55e721e39bb97377fd2d https://github.com/tianocore/edk2/commit/83a86f465ccb1a792f5c55e721e3= 9bb97377fd2d Author: Sean Rhodes Date: 2025-02-23 (Sun, 23 Feb 2025) Changed paths: M UefiPayloadPkg/UefiPayloadPkg.dsc Log Message: ----------- UefiPayloadPkg: Allow boot timeout to be configured This patch moves `PcdPlatformBootTimeOut` from `[PcdsDynamicExDefault]` to `[PcdsDynamicHii]`, allowing the boot timeout to be configured through HII, rather than being a compile-time setting. Signed-off-by: Sean Rhodes To unsubscribe from these emails, change your notification settings at http= s://github.com/tianocore/edk2/settings/notifications _______________________________________________ edk2-commits mailing list edk2-commits@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/edk2-commits -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#121138): https://edk2.groups.io/g/devel/message/121138 Mute This Topic: https://groups.io/mt/111353434/7686176 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io] -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- --_000_DM4PR11MB8226F5DCD5C603F82B54F0F28CC02DM4PR11MB8226namp_ Content-Type: text/html; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable
Sean,
I do not think the change that makes PcdPlatformBootTimeOut as DynamicHii P= CD in UefiPayloadPkg is correct.

When VARIABLE_SUPPORT is not "EMU" or "SPI", UPL does n= ot include any variable manipulation logic.
Defining a PCD to a variable seems not make sense.

Thanks,
Ray

From: Sean Rhodes via edk2-= commits <edk2-commits@lists.sourceforge.net>
Sent: Monday, February 24, 2025 5:32
To: edk2-commits@lists.sourceforge.net <edk2-commits@lists.source= forge.net>
Subject: [tianocore/edk2] 83a86f: UefiPayloadPkg: Allow boot timeout= to be configured
 
  Branch: refs/heads/master
  Home:   https://github.com/tianocore/edk2
  Commit: 83a86f465ccb1a792f5c55e721e39bb97377fd2d
      https://github.com/tianocore/edk2/commit/83a86f465ccb1a792f5c55e721e39bb973= 77fd2d
  Author: Sean Rhodes <sean@starlabs.systems>
  Date:   2025-02-23 (Sun, 23 Feb 2025)

  Changed paths:
    M UefiPayloadPkg/UefiPayloadPkg.dsc

  Log Message:
  -----------
  UefiPayloadPkg: Allow boot timeout to be configured

This patch moves `PcdPlatformBootTimeOut` from `[PcdsDynamicExDefault]`
to `[PcdsDynamicHii]`, allowing the boot timeout to be configured through HII, rather than being a compile-time setting.

Signed-off-by: Sean Rhodes <sean@starlabs.systems>



To unsubscribe from these emails, change your notification settings at https://github.com/tianocore/edk2/settings/notifications


_______________________________________________
edk2-commits mailing list
edk2-commits@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/edk2-co= mmits
_._,_._,_
Groups.io Links:

=20 You receive all messages sent to this group. =20 =20

View/Reply Online (#121138) | =20 | Mute= This Topic | New Topic
Your Subscriptio= n | Contact Group Owner | Unsubscribe [rebecca@openfw.io]

_._,_._,_
--_000_DM4PR11MB8226F5DCD5C603F82B54F0F28CC02DM4PR11MB8226namp_--