From mboxrd@z Thu Jan 1 00:00:00 1970 Content-Type: multipart/mixed; boundary="===============3554942619300004309==" MIME-Version: 1.0 From: kernel test robot Subject: [PATCH] FROMLIST: HID: nintendo: fix flexible_array.cocci warnings Date: Fri, 14 May 2021 00:00:51 +0800 Message-ID: <20210513160051.GA121471@6b55adf73326> In-Reply-To: <202105140036.iyfEnAkz-lkp@intel.com> List-Id: To: kbuild@lists.01.org --===============3554942619300004309== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable CC: kbuild-all(a)lists.01.org TO: Lee Jones From: kernel test robot drivers/hid/hid-nintendo.c:155:4-8: WARNING use flexible-array member inste= ad (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-len= gth-and-one-element-arrays) drivers/hid/hid-nintendo.c:161:4-8: WARNING use flexible-array member inste= ad (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-len= gth-and-one-element-arrays) Zero-length and one-element arrays are deprecated, see Documentation/process/deprecated.rst Flexible-array members should be used instead. Generated by: scripts/coccinelle/misc/flexible_array.cocci Fixes: f4dc6237a840 ("FROMLIST: HID: nintendo: add nintendo switch controll= er driver") CC: Daniel J. Ogorchock Reported-by: kernel test robot Signed-off-by: kernel test robot --- tree: https://git.linaro.org/people/lee.jones/linux.git aosp-android-main= line-tracking head: 9b2a8deb96b1f9d4cc52497c4559f187db0408e1 commit: f4dc6237a840e52e0ecd3c46393af5b98029f650 [307/599] FROMLIST: HID: n= intendo: add nintendo switch controller driver :::::: branch date: 5 hours ago :::::: commit date: 5 hours ago Please take the patch only if it's a positive warning. Thanks! hid-nintendo.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) --- a/drivers/hid/hid-nintendo.c +++ b/drivers/hid/hid-nintendo.c @@ -152,13 +152,13 @@ struct joycon_subcmd_request { u8 packet_num; /* incremented every send */ u8 rumble_data[8]; u8 subcmd_id; - u8 data[0]; /* length depends on the subcommand */ + u8 data[]; /* length depends on the subcommand */ } __packed; = struct joycon_subcmd_reply { u8 ack; /* MSB 1 for ACK, 0 for NACK */ u8 id; /* id of requested subcmd */ - u8 data[0]; /* will be at most 35 bytes */ + u8 data[]; /* will be at most 35 bytes */ } __packed; = struct joycon_input_report { --===============3554942619300004309==--