Hi Bin, On Sun, 26 Apr 2020 at 23:58, Bin Meng <bmeng...@gmail.com> wrote: > > Hi Simon, > > On Thu, Apr 23, 2020 at 5:58 PM Bin Meng <bmeng...@gmail.com> wrote: > > > > Hi Simon, > > > > On Thu, Apr 23, 2020 at 5:46 PM Bin Meng <bmeng...@gmail.com> wrote: > > > > > > Hi Simon, > > > > > > On Wed, Apr 22, 2020 at 5:37 AM Simon Glass <s...@chromium.org> wrote: > > > > > > > > Hi Andy, > > > > > > > > On Tue, 21 Apr 2020 at 11:43, Andy Shevchenko > > > > <andriy.shevche...@linux.intel.com> wrote: > > > > > > > > > > On Sun, Apr 19, 2020 at 02:36:48PM -0600, Simon Glass wrote: > > > > > > This is split from the original series in an attempt to get things > > > > > > applied > > > > > > in chunks. > > > > > > > > > > > > v7 is just a rebase as requested > > > > > > > > > > So, overall I have no concerns, except that compatible string in ACPI > > > > > seems > > > > > half-baked solution and, due to PRP0001 awareness (my understanding > > > > > that it > > > > > should not be part of production devices), seems not needed right now. > > > > > It means we should require to have _HID and/or _CID. > > > > > > > > OK I am really not sure what to say about this. Let's deal with it > > > > when we see the use of it later. > > > > > > > > > > > > > > Code looks fine to me, though. > > > > > > > > > > > > > OK good. I will send part B soon. > > > > > > > > > > v7 has been applied to u-boot-x86, thanks! > > > > > > > Unfortunately this series breaks sandbox_spl, please take a look. > > > > https://gitlab.denx.de/u-boot/custodians/u-boot-x86/-/jobs/81615 > > > > Did you send out a fix for this series so that I can squash in?
Oops, no, I sent out a new series. I will send a fixup patch. Regards, Simon