Hi Simon,
On Fri, May 25, 2018 at 4:41 AM, Simon Glass <s...@chromium.org> wrote:
> On 24 May 2018 at 02:42, Mario Six <mario....@gdsys.cc> wrote:
>> Add some tests for sysreset_get_status.
>>
>> Signed-off-by: Mario Six <mario....@gdsys.cc>
>>
>> ---
>>
>> v2 -> v3:
>> New in v3.
>>
>> ---
>> drivers/sysreset/sysreset_sandbox.c | 16 ++++++++++++++++
>> test/dm/sysreset.c | 19 +++++++++++++++++++
>> 2 files changed, 35 insertions(+)
>
> Reviewed-by: Simon Glass <s...@chromium.org>
>
>>
>> diff --git a/drivers/sysreset/sysreset_sandbox.c
>> b/drivers/sysreset/sysreset_sandbox.c
>> index f12c4e84198..e03b5c122fe 100644
>> --- a/drivers/sysreset/sysreset_sandbox.c
>> +++ b/drivers/sysreset/sysreset_sandbox.c
>> @@ -29,6 +29,13 @@ static int sandbox_warm_sysreset_request(struct udevice
>> *dev,
>> return -EINPROGRESS;
>> }
>>
>> +int sandbox_warm_sysreset_get_status(struct udevice *dev, char *buf, int
>> size)
>> +{
>> + strncpy(buf, "Reset Status: WARM", size);
>
> strlcpy() might be better.
>
OK, will use strlcpy() in v4.
Best regards,
Mario
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot
