[U-Boot] [PATCH v2 2/5] x86: tangier: Add a sysreset driver

[Bin Meng]

Hi Andy,

On Thu, Jul 19, 2018 at 6:38 PM, Andy Shevchenko
<andriy.shevchenko at linux.intel.com> wrote:
> On Thu, 2018-07-19 at 13:29 +0300, Andy Shevchenko wrote:
>> On Thu, 2018-07-19 at 18:04 +0800, Bin Meng wrote:
>> > Hi Andy,
>> >
>> > On Thu, Jul 19, 2018 at 6:07 PM, Bin Meng <bmeng.cn at gmail.com>
>> > wrote:
>> > > This adds a reset driver for tangier processor.
>> > >
>> > > Signed-off-by: Bin Meng <bmeng.cn at gmail.com>
>> > >
>> > > ---
>> > >
>> > > Changes in v2:
>> > > - new patch to add a reset driver for tangier processor
>> > >
>> > >  arch/x86/cpu/tangier/Makefile   |  2 +-
>> > >  arch/x86/cpu/tangier/sysreset.c | 48
>> > > +++++++++++++++++++++++++++++++++++++++++
>> > >  2 files changed, 49 insertions(+), 1 deletion(-)
>> > >  create mode 100644 arch/x86/cpu/tangier/sysreset.c
>> > >
>> >
>> > Could you please help test this driver? This series is at
>> > u-boot-x86/reset branch. Thanks!
>>
>> Can you point me out to what exact steps I have to perform to test
>> this
>> functionality?
>>
>> Just boot to U-boot console, run 'reset' and see if everything works
>> as
>> before? Or something more?
>
> So, I merged the mentioned branch on top of my testing setup for ACPI
> bits. After, I boot to U-boot console and type reset (done 2 times with
> different delay after boot: few seconds vs. few dozen of seconds). After
> I tried to boot my ACPI kernel.
>
> If it's enough, you may take mine
> Tested-by: Andy Shevchenko <andriy.shevchenko at linux.intel.com>
>

This is enough. Thank you for the testing!

> for u-boot-x86/reset branch.
>

Regards,
Bin