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 Regards, Bin