From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from cloudserver096301.home.net.pl (cloudserver096301.home.net.pl [79.96.179.35]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id A5FB121D49181 for ; Fri, 4 Aug 2017 17:11:44 -0700 (PDT) Received: from 109241231046.gdansk.vectranet.pl (109.241.231.46) (HELO [192.168.0.106]) by serwer1539010.home.pl (79.96.179.35) with SMTP (IdeaSmtpServer 0.82) id 29b645da0f4fdc94; Sat, 5 Aug 2017 02:13:51 +0200 To: Laszlo Ersek , edk2-devel@lists.01.org References: <721bf59b-166b-7a67-a1e7-b3e30d2487d3@3mdeb.com> <5f50924b-c781-2715-2447-a7a94d571efb@3mdeb.com> <6c6655ec-a01d-c2ee-ed38-453e707dd3ac@redhat.com> <4d713a2b-1526-0476-d765-f591fe8cfaf2@3mdeb.com> <6be56212-9e83-7485-076f-e26e71838adf@redhat.com> From: =?UTF-8?Q?Piotr_Kr=c3=b3l?= Message-ID: <04ef8e4a-2935-fe7d-5842-752e0530098e@3mdeb.com> Date: Sat, 5 Aug 2017 02:13:51 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.2.1 MIME-Version: 1.0 In-Reply-To: <6be56212-9e83-7485-076f-e26e71838adf@redhat.com> Subject: Re: CorebootPayloadPkg: redirect UEFI Shell to serial X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.22 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 05 Aug 2017 00:11:45 -0000 Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 8bit -----BEGIN PGP SIGNED MESSAGE----- Hash: SHA256 On 07/25/2017 11:16 AM, Laszlo Ersek wrote: > On 07/15/17 15:00, Piotr Król wrote: Hi Laszlo, > >> 2. There are 2 minor typos like gUEfi instead of gUefi and >> entrhy instead of entry, does it make sense to send patches for >> that ? In various places I saw more typos like that. > > Typo corrections are welcome. Patch sent. > >> 4. I had to move some PCDs to PcdsFixedAtBuild section. > > Why? After reading your post I know I made mistake by taking your reference to PlatformBootManagerLib in ArmVirtPkg without rewriting it correctly. They use FixedAtBuild, but CorebootPayloadPkg provide the same parameters as dynamic PCDs - this is some inconsistency that I wanted to address. I resolve my issue by using const value when mSerialConsole is created, the druing runtime I use PcdGet calls family to get correct values. > >> This seems to makes difference for PcdSet64S. After that I'm >> getting error from that function. Where I can read about >> difference between normal PCD and FixedAtBuild > > As their name says, fixed-at-build PCDs can't be set at runtime. - > The .dec file declares the possible storage types for a given PCD > that any platform can assign to the PCD. The .dec file also sets a > default value. - in the platform .dsc file, the platform can omit > the PCD completely. In this case modules built into the platform > will get the default value from the .dec, and they won't be able to > set the PCD. - the platform .dsc can also spell out the PCD. This > determines the actual storage the platform assigns to the PCD (such > as dynamic, dynamic-ex, dynamic-hii, fixed-at-build) and also lets > the platform set its own default for the PCD. Modules built into > the platform will see the following values for the PCD: - if the > PCD is fixed-at-build, then they'll get the value set in the DSC. > The PCD cannot be set by modules. - if the PCD is dynamic or > dynamic-ex, then the initial value is the one set in the DSC. > Modules can overwrite the PCD, and later "get"s will return that > value. - if the PCD is dynamic-hii, then it is backed by a > non-volatile UEFI variable. > > ... This is quite a bit of hand-waving, but should be good enough > for now. The way fixed-at-build PCDs seem to work for you is by > design; I'd rather try to understand why you had to change a > dynamic PCD into a fixed one. Thanks for this explanation. > > The rest of your questions should be directly addressed to the > maintainers of those packages / modules (see Maintainers.txt). I will send that in coming days. Best Regards, - -- Piotr Król Embedded Systems Consultant https://3mdeb.com | @3mdeb_com -----BEGIN PGP SIGNATURE----- iQIzBAEBCAAdFiEE4DCbLYWmfoRjKeNLsu5x6WeqnkwFAlmFDb8ACgkQsu5x6Weq nkxJ6w/7BWXNDvE/svknuasRds4m8mtask1dkEeq/2VFlivHMffb+s9Cz6um5fOj xjtOdZxtQBfjUHG6dW34v1wThq9qnpr1550WJaXv4gum2zMc2G40ffkq5Hluko8s 0qEGyVknZMu5+Z5rEbxirV38UudjO8+onZb4FQE26RiaLPr0TjmrgZUhWtTdXlq9 TbuGFSVe0u3RcASehreRB11dpCuDEknlBIyLqISlw065uchGMnBBVtkrxv4yf/fk vh+INY7ja/Ap2sKHp4/+7lQ5e+bk0WcdfjCGwbnszqB307sncHIm2Xx33aBktolS NcHuIh3SVbfsHZ2SiwOEDClLd4UFuW5qvwnkXXVytBFHOIHpdAhLwpNM8+rvFw3t 6T/cajO0MyWaBzOcT5wK4N1PxKtgRWHc2iTbjENTdSUFK0CiX0wjVRoPtJdBjxGS WdP7NnXoDP0GLkdvMf+1FvamJHLwgP/3p5rwZ/WXGR8ySWrkOfE8h7O00VlRHdf4 OcXcf3ZmLCSZFpxstv3b/V5pK4EDISAu5VeoUbamwsnZQhaaDALvqNcf80Kp/7Yi eyVYLP+mQWnptzvPc1tqbF/Pb4tDPTGPHZXSDKYbfd0LXwyUaHkwwGr/0VKTD1r/ B2O1H1tVSIWSMW3atEUUCitidGSscxOGDmRJP6g2H6wuuoP9C2k= =kE52 -----END PGP SIGNATURE-----