[U-Boot] [PATCH 2/4] dm: ofnode: add ofnode_device_is_compatible() helper

Masahiro Yamada yamada.masahiro at socionext.com
Mon Apr 23 04:50:49 UTC 2018


2018-04-23 5:11 GMT+09:00 Simon Glass <sjg at chromium.org>:
> Hi Masahiro,
>
> On 17 April 2018 at 20:38, Masahiro Yamada <yamada.masahiro at socionext.com>
> wrote:
>> device_is_compatible() takes udevice, but there is no such a helper
>> that takes ofnode.
>>
>> Signed-off-by: Masahiro Yamada <yamada.masahiro at socionext.com>
>> ---
>>
>>  drivers/core/device.c |  8 +-------
>>  drivers/core/ofnode.c | 11 +++++++++++
>>  include/dm/ofnode.h   | 11 +++++++++++
>>  3 files changed, 23 insertions(+), 7 deletions(-)
>
> Please can you add a call to this to a test?
>

No.

I do not see any ofnode helper test in test/dm/.

You are requesting additional work beyond this patch.
It is unfair.


This helper is tested indirectly by other tests.

Of course, you (and anybody) are free to add per-helper grained tests,
but this is not a good reason to block this patch.




-- 
Best Regards
Masahiro Yamada


More information about the U-Boot mailing list