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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id A16EBC38A2D for ; Tue, 25 Oct 2022 18:09:07 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:References:Message-ID: In-Reply-To:Subject:cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=MjCv8b4p/XqrJ6ZMWK4bS6bvnbN5krXk7w9TULizgKg=; b=hrYPWcU7v0/QIx FFJXEpBcPbsuMYrabV6MVr9Q57qP4OfaKjLvnqXbxoBUTrCjiOT/4XeX8Qpm5nGLW6N9/itVr9L3Q RXDm9lzGYDgQRLvR3dJ9IABEzVcLQm/gCdi4/RFj2JdpfBbwONLfOMmJ6EPmp+YQYyHYkF1nBMfMw uzF3zfMdzuFJUUWrtbt5G/mWyWPVx96TRzaTK39ZxCU7IK1m4sfNkc/O5aVlTskQV9w7IIHrPttPD 1ubG7I6ex/c6FjnTj34BkLRoaIUcP1NK7mJuQkjrDivwEi+m+rpn3asBdx1vDs7dvmONukaC6ON2y MzZTAnAoZMEtT/8MnDtw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1onOLO-006ZmU-QL; Tue, 25 Oct 2022 18:08:06 +0000 Received: from mail3-relais-sop.national.inria.fr ([192.134.164.104]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1onOLL-006Zks-UK for linux-arm-kernel@lists.infradead.org; Tue, 25 Oct 2022 18:08:05 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=inria.fr; s=dc; h=date:from:to:cc:subject:in-reply-to:message-id: references:mime-version; bh=2RpF6v7jb3J8a21THqvWGqERHWXHNn7kIyc4VmrIpDk=; b=Y8f5U39imdcBre/bcuSCHN6VcfZ7BHURtK+YcMjJZJcWLqh+F9oJuZvz Jb+1Y/ON5am8w7SGHXfYjFMl7j2RoH3irDodGjf8UpQRgikKmfsChWMfu 4v+fzyTDGn+OhUsaPdLTUtMy0dwaXeGhcAQjK1U6CuGblFcGeIwCKunkc I=; Authentication-Results: mail3-relais-sop.national.inria.fr; dkim=none (message not signed) header.i=none; spf=SoftFail smtp.mailfrom=julia.lawall@inria.fr; dmarc=fail (p=none dis=none) d=inria.fr X-IronPort-AV: E=Sophos;i="5.95,211,1661810400"; d="scan'208";a="33948386" Received: from 51.123.68.85.rev.sfr.net (HELO hadrien) ([85.68.123.51]) by mail3-relais-sop.national.inria.fr with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 25 Oct 2022 07:01:48 +0200 Date: Tue, 25 Oct 2022 07:01:48 +0200 (CEST) From: Julia Lawall X-X-Sender: jll@hadrien To: Gabriel Lima Luz cc: Rui Miguel Silva , Laurent Pinchart , Steve Longerbeam , Philipp Zabel , Mauro Carvalho Chehab , Greg Kroah-Hartman , Shawn Guo , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam , NXP Linux Team , linux-media@vger.kernel.org, linux-staging@lists.linux.dev, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, outreachy@lists.linux.dev Subject: Re: [PATCH v2] staging: media: imx: imx7-media-csi: Fix parenthesis alignment In-Reply-To: <20221025001239.16345-1-lima.gabriel.luz@gmail.com> Message-ID: References: <20221025001239.16345-1-lima.gabriel.luz@gmail.com> User-Agent: Alpine 2.22 (DEB 394 2020-01-19) MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20221025_110804_575977_7F0DC4A8 X-CRM114-Status: GOOD ( 18.35 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Mon, 24 Oct 2022, Gabriel Lima Luz wrote: > Adhere to Linux kernel coding style. > > Reported by checkpatch: > > CHECK: Alignment should match open parenthesis Here you have made a decision that is quite reasonable, but that does not actually "Fix parenthesis alignment". This highlights an overall problem with your log message. You say what you are "fixing", but you don't give any explanation about why you have done what you have done. Actually the most helpful comment was below the ---, but that will disappear when the patch is applied. julia > > Signed-off-by: Gabriel Lima Luz > --- > Altered changes acording to feedback for a more readable code > > drivers/staging/media/imx/imx7-media-csi.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/staging/media/imx/imx7-media-csi.c b/drivers/staging/media/imx/imx7-media-csi.c > index cbc66ef0eda8..b241b68f456c 100644 > --- a/drivers/staging/media/imx/imx7-media-csi.c > +++ b/drivers/staging/media/imx/imx7-media-csi.c > @@ -1704,7 +1704,8 @@ static int imx7_csi_init_cfg(struct v4l2_subdev *sd, > mf->xfer_func = V4L2_MAP_XFER_FUNC_DEFAULT(mf->colorspace); > mf->ycbcr_enc = V4L2_MAP_YCBCR_ENC_DEFAULT(mf->colorspace); > mf->quantization = V4L2_MAP_QUANTIZATION_DEFAULT(!cc->yuv, > - mf->colorspace, mf->ycbcr_enc); > + mf->colorspace, > + mf->ycbcr_enc); > > csi->cc[i] = cc; > } > -- > 2.34.1 > > > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel