On 3/28/20 5:21 PM, Dan Jessie wrote:
This patch fixes the checkpatch.pl warning:
WARNING: braces {} are not necessary for single statement blocks
This is the only instance of the problem noted by checkpatch.pl in staging: greybus.
Signed-off-by: Dan Jessie dtjessie@gmail.com
Looks good, thanks.
Reviewed-by: Alex Elder elder@linaro.org
v3: - Reworded patch message - Added patch version history v2: - Noted this was the only place the issue is reported, as suggested by Alex Elder
drivers/staging/greybus/hid.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/staging/greybus/hid.c b/drivers/staging/greybus/hid.c index 04bfd9110502..ed706f39e87a 100644 --- a/drivers/staging/greybus/hid.c +++ b/drivers/staging/greybus/hid.c @@ -290,9 +290,8 @@ static int gb_hid_parse(struct hid_device *hid) } rdesc = kzalloc(rsize, GFP_KERNEL);
- if (!rdesc) {
- if (!rdesc) return -ENOMEM;
- }
ret = gb_hid_get_report_desc(ghid, rdesc); if (ret) {