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 X-Spam-Level: X-Spam-Status: No, score=-6.7 required=3.0 tests=BAYES_00,DATE_IN_PAST_12_24, DKIMWL_WL_HIGH,DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id A217FC433DF for ; Tue, 25 Aug 2020 13:38:19 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 5F59F2071E for ; Tue, 25 Aug 2020 13:38:19 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="I7WK/VBi"; dkim=fail reason="signature verification failed" (2048-bit key) header.d=cerno.tech header.i=@cerno.tech header.b="DH/bvnqJ"; dkim=fail reason="signature verification failed" (2048-bit key) header.d=messagingengine.com header.i=@messagingengine.com header.b="PjL8jOqB" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 5F59F2071E Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=cerno.tech Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=merlin.20170209; h=Sender:Content-Type:Cc: List-Subscribe:List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id: In-Reply-To:MIME-Version:References:Message-ID:Subject:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=F/iNKQcIwpn0VubSgsCfDp3pBN963gZUCHd3vjBbWrs=; b=I7WK/VBi4BCtc1CFRRmMggLeK Z+C2iEtpV5WUcksGPrmAu63yNDjcyuM0iBzOibTY1JGMACY8dxXwwtbi8telHdlVykEE0oSgB8sBA x7zxExqn+4GE6WR9Yhoz3PzP6fvT57VkISmUymzFbeoQ2WwOYzsH5ELss3h+QUamDC5d4O28WTvK5 jXR54R0nSYgu2xLfSwvPWd+X7KV1Wqp2Q8iNHqHukI+6HxPyAOoBli2ofO3clSqLCBkXTEXDxZApR qu2KlHx7T/k9BGbTAysT7hSSDlMrdmfEohK1SnbSTYqVKrcb/nNdg1q2EJr+ytuZANBPqRaPjwcVm kUT9z9vrA==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1kAZ81-0002R0-A1; Tue, 25 Aug 2020 13:36:45 +0000 Received: from new3-smtp.messagingengine.com ([66.111.4.229]) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1kAZ79-000261-QC for linux-arm-kernel@lists.infradead.org; Tue, 25 Aug 2020 13:35:53 +0000 Received: from compute4.internal (compute4.nyi.internal [10.202.2.44]) by mailnew.nyi.internal (Postfix) with ESMTP id 90E6C580434; Tue, 25 Aug 2020 09:35:51 -0400 (EDT) Received: from mailfrontend1 ([10.202.2.162]) by compute4.internal (MEProxy); Tue, 25 Aug 2020 09:35:51 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cerno.tech; h= date:from:to:cc:subject:message-id:references:mime-version :content-type:in-reply-to; s=fm3; bh=heSxZ0v/Qu75xj28+50fhvskv1q 2grCRuH7m6PwDOjk=; b=DH/bvnqJ1QzGZwBII8NxBpiFHWiMB8VukcJblX4AFFu OMc8FJa2a9X5N05NSCKbSb/rnIzDssBW4WPBci8FdiVIDyc2laI6vX4EAwVry+R8 h8oIchfUux44zZhfh2KDS9YOnTGdFXCvb/W4eC/zQT8hQQqq+Klpx7UTsQ8iA++3 y5RBr29dBD1KKyfcmIy7dLySeCnAa3GzQj3HcD5UdBePQ0DCsp0GiCmyERg2xSDV 20dndS7arFJ9SDyRSjpPqiTDQJNpaQFGnjqxMjNCpMJYSnx8hAxUlMDgrT+XKGwN 4EnQkaXy55l7hFISvsX0n2R6YcplFbL7m8boSTnvPfw== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to:x-me-proxy :x-me-proxy:x-me-sender:x-me-sender:x-sasl-enc; s=fm3; bh=heSxZ0 v/Qu75xj28+50fhvskv1q2grCRuH7m6PwDOjk=; b=PjL8jOqBbN2/dyjFswTwf4 lafzm/B4hXrVcRrNGkDYYg+rIRu1qK2VoCq3OsL4g+uVAVtaifdx7Vq7oS8M0tt5 w4IsaXuOvjglMtGAmEVhK21rAfku6z1aOky8UzXbklc5FdpJIq3DZxUZLnF2lHOE YndBGq4Yz3fTRWPBNCO5q/5KGNyPwAAA+Rr205pqOwN4gInJToMugSotPHbaGP8k LN1D6CZI3G0/47X46Yp/mBPCop7utYkv8XRkeRIQQ8xjU9Pual9as157XTbbqgvb fb5S9o4/C3lapGTCyXMDyfKGTSNHGA+P5csFFu35B3bIDqDfYKaCUuA82cv/W3Ew == X-ME-Sender: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgeduiedruddvtddgieelucetufdoteggodetrfdotf fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfqfgfvpdfurfetoffkrfgpnffqhgen uceurghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmne cujfgurhepfffhvffukfhfgggtuggjsehgtderredttddunecuhfhrohhmpeforgigihhm vgcutfhiphgrrhguuceomhgrgihimhgvsegtvghrnhhordhtvggthheqnecuggftrfgrth htvghrnhepuddvudfhkeekhefgffetffelgffftdehffduffegveetffehueeivddvjedv gfevnecukfhppeeltddrkeelrdeikedrjeeinecuvehluhhsthgvrhfuihiivgepudenuc frrghrrghmpehmrghilhhfrhhomhepmhgrgihimhgvsegtvghrnhhordhtvggthh X-ME-Proxy: Received: from localhost (lfbn-tou-1-1502-76.w90-89.abo.wanadoo.fr [90.89.68.76]) by mail.messagingengine.com (Postfix) with ESMTPA id 331FC328005A; Tue, 25 Aug 2020 09:35:51 -0400 (EDT) Date: Mon, 24 Aug 2020 18:55:36 +0200 From: Maxime Ripard To: =?utf-8?B?S8OpdmluIEwnaMO0cGl0YWw=?= Subject: Re: [PATCH 1/7] media: sun6i-csi: Fix the bpp for 10-bit bayer formats Message-ID: <20200824165536.u2yzonoskiqu3c5j@gilmour.lan> References: <20200821145935.20346-1-kevin.lhopital@bootlin.com> <20200821145935.20346-2-kevin.lhopital@bootlin.com> MIME-Version: 1.0 In-Reply-To: <20200821145935.20346-2-kevin.lhopital@bootlin.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20200825_093551_963804_9412BF0E X-CRM114-Status: GOOD ( 12.96 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: mark.rutland@arm.com, devicetree@vger.kernel.org, p.zabel@pengutronix.de, thomas.petazzoni@bootlin.com, linux-kernel@vger.kernel.org, paul.kocialkowski@bootlin.com, wens@csie.org, robh+dt@kernel.org, yong.deng@magewell.com, mchehab@kernel.org, linux-arm-kernel@lists.infradead.org, linux-media@vger.kernel.org Content-Type: multipart/mixed; boundary="===============2263027518442071330==" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org --===============2263027518442071330== Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="bfmdwl5kcym7nka4" Content-Disposition: inline --bfmdwl5kcym7nka4 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Fri, Aug 21, 2020 at 04:59:29PM +0200, K=E9vin L'h=F4pital wrote: > 10-bit bayer formats are aligned to 16 bits in memory, so this is what > needs to be used as bpp for calculating the size of the buffers to > allocate. >=20 > Signed-off-by: K=E9vin L'h=F4pital Generally speaking, you should also explain why it's not an issue for the callers. Depending on what that function is supposed to be doing (returning the padded bits or the padded bits per pixel), your patch could be either right or wrong. Since all the callers are using it to generate the number of bytes per line, your patch is indeed correct. But it should be mentionned in the commit log. Maxime --bfmdwl5kcym7nka4 Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYIAB0WIQRcEzekXsqa64kGDp7j7w1vZxhRxQUCX0PxCAAKCRDj7w1vZxhR xcvsAQCvN3gtT1RS8+Z8WwFFELIkFm7M6jqWf3Z9ZBbmSc0aVQD/SUIIWtkekTpB V+AbU7eYR7tD5BwIbp+/2hTXdMfqogM= =YTpD -----END PGP SIGNATURE----- --bfmdwl5kcym7nka4-- --===============2263027518442071330== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel --===============2263027518442071330==--