From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 8133EC433EF for ; Fri, 22 Oct 2021 14:21:15 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 6529C61056 for ; Fri, 22 Oct 2021 14:21:15 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233041AbhJVOXb (ORCPT ); Fri, 22 Oct 2021 10:23:31 -0400 Received: from relay8-d.mail.gandi.net ([217.70.183.201]:40559 "EHLO relay8-d.mail.gandi.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232825AbhJVOXb (ORCPT ); Fri, 22 Oct 2021 10:23:31 -0400 Received: (Authenticated sender: thomas.perrot@bootlin.com) by relay8-d.mail.gandi.net (Postfix) with ESMTPSA id A203F1BF214; Fri, 22 Oct 2021 14:21:12 +0000 (UTC) From: Thomas Perrot To: linux-spi@vger.kernel.org Cc: broonie@kernel.org, linus.walleij@linaro.org, stable@vger.kernel.org, Thomas Perrot Subject: [PATCH] spi: spl022: fix Microwire full duplex mode Date: Fri, 22 Oct 2021 16:21:04 +0200 Message-Id: <20211022142104.1386379-1-thomas.perrot@bootlin.com> X-Mailer: git-send-email 2.31.1 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: stable@vger.kernel.org There are missing braces in the function that verify controller parameters, then an error is always returned when the parameter to select Microwire frames operation is used on devices allowing it. Signed-off-by: Thomas Perrot --- drivers/spi/spi-pl022.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/spi/spi-pl022.c b/drivers/spi/spi-pl022.c index feebda66f56e..e4484ace584e 100644 --- a/drivers/spi/spi-pl022.c +++ b/drivers/spi/spi-pl022.c @@ -1716,12 +1716,13 @@ static int verify_controller_parameters(struct pl022 *pl022, return -EINVAL; } } else { - if (chip_info->duplex != SSP_MICROWIRE_CHANNEL_FULL_DUPLEX) + if (chip_info->duplex != SSP_MICROWIRE_CHANNEL_FULL_DUPLEX) { dev_err(&pl022->adev->dev, "Microwire half duplex mode requested," " but this is only available in the" " ST version of PL022\n"); - return -EINVAL; + return -EINVAL; + } } } return 0; -- 2.31.1