[U-Boot] [U-Boot, v3, 3/3] dm: led: add testcase for "default-state" property

Tom Rini trini at konsulko.com
Tue Apr 10 16:22:36 UTC 2018


On Wed, Apr 04, 2018 at 10:01:03AM +0200, linux-kernel-dev at beckhoff.com wrote:

> From: Patrick Bruenn <p.bruenn at beckhoff.com>
> 
> Add two more gpio-leds to sandbox test device tree with default-state
> property set to "on"/"off".
> Add dm_test_led_default_state() to check that these new LED's are set to
> LEDST_ON and LEDST_OFF.
> 
> Signed-off-by: Patrick Bruenn <p.bruenn at beckhoff.com>
> Reviewed-by: Simon Glass <sjg at chromium.org>
> ---
> patman complains about: test/dm/led.c:45: check: Please use a blank line
> after function/struct/union/enum declarations.
> I compared with other DM_TEST() usage and decided to ignore this check.
> Should we fix the macro, patman or keep ignoring this?

This breaks tests actually, please try running 'make tests' after this
(and the rest of your series) is applied, thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20180410/c0bdbaaa/attachment.sig>


More information about the U-Boot mailing list