[U-Boot] [PATCH] sandbox: add a sandbox timer and basic test
Thomas Chou
thomas at wytron.com.tw
Fri Oct 30 00:27:58 CET 2015
Hi Simon,
On 10/30/2015 01:16 AM, Simon Glass wrote:
> Hi Thomas,
>
> On 22 October 2015 at 07:25, Thomas Chou <thomas at wytron.com.tw> wrote:
>> Add a sandbox timer which get time from host os and a basic
>> test.
>>
>> Signed-off-by: Thomas Chou <thomas at wytron.com.tw>
>> ---
>> arch/sandbox/dts/sandbox.dts | 4 ++
>> board/sandbox/sandbox.c | 2 +
>> common/board_f.c | 2 +-
>> configs/sandbox_defconfig | 2 +
>> doc/device-tree-bindings/timer/sandbox_timer.txt | 7 ++++
>> drivers/timer/Kconfig | 7 ++++
>> drivers/timer/Makefile | 1 +
>> drivers/timer/sandbox_timer.c | 53 ++++++++++++++++++++++++
>> include/configs/sandbox.h | 2 +
>> test/dm/Makefile | 1 +
>> test/dm/timer.c | 27 ++++++++++++
>
> Can you please split out the board_f.c change into a separate commit?
But the sandbox will fail (with seg fault or hang) without the change in
board_f.c . Is this alright?
>> +static const struct udevice_id sandbox_timer_ids[] = {
>> + { .compatible = "sandbox,timer", },
>
> You can drop the penultimate comma.
>
OK.
Thanks a lot for your review.
Best regards,
Thomas
More information about the U-Boot
mailing list