20.09.2021 09:43, Neil Armstrong wrote:
Hi,

On 19/09/2021 17:52, Vyacheslav Bocharov wrote:
Add support for new home automation devices.

While applying I had a few warnings:

+       if (!env_get("serial")) {
+               len = meson_sm_read_efuse(EFUSE_SN_OFFSET, serial,

ERROR: trailing whitespace
#777: FILE: board/amlogic/jethub-j80/jethub-j80.c:48:
+^I^Ilen = meson_sm_read_efuse(EFUSE_SN_OFFSET, serial, $

CHECK: Alignment should match open parenthesis
#778: FILE: board/amlogic/jethub-j80/jethub-j80.c:49:
+               len = meson_sm_read_efuse(EFUSE_SN_OFFSET, serial,
+                       EFUSE_SN_SIZE);

CHECK: Alignment should match open parenthesis
#785: FILE: board/amlogic/jethub-j80/jethub-j80.c:56:
+               len = meson_sm_read_efuse(EFUSE_USID_OFFSET, usid,
+                       EFUSE_USID_SIZE);

I'll double-check everything and resend a new version of the patches.


This file isn't part of any MAINTAINERS file, either add it to the s400 one or 
the j80 one.

I'll add it to jethub-j80/MAINTAINERS.

Thanks!


Reply via email to