Received: by 2002:a25:8b12:0:0:0:0:0 with SMTP id i18csp2363847ybl; Mon, 19 Aug 2019 00:30:27 -0700 (PDT) X-Google-Smtp-Source: APXvYqyJdO0hq/U90wVJQlgv5T3M4jJEQ6b/8XeGaPbhYyxffcCDPTOudzJEY4Ws5IjMAXSotZLr X-Received: by 2002:aa7:9516:: with SMTP id b22mr22633421pfp.106.1566199827323; Mon, 19 Aug 2019 00:30:27 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1566199827; cv=none; d=google.com; s=arc-20160816; b=YmIlYlyMshkE14SXXspHqA6a5Yf/R4uUug/1lQR/5tHEuZemtTBB93Egn3vVvc1zLx 5OBb9XXAgSwjx62U4qPstXSGKl4+lgEEWuubzJ0lzTKT+wkO4vz0vuk7Sdz6bAEHqbiY 9Zlgy5AWODaHbu3VE8HHRZ+IgnHjyD0IuDrtJKyToNOc10Rh2nCJDl3Y/7hTmdu93lHa LVkeqW9TTr3QYlzyeZtu4fFJmLG+92gxPARLlvHGAu7GOq99NzKzyEwHIuCAnjHHbr4O A9j2VxbP9dirk33TcnZuOGgSfD0dLII9nIZc67Se0rlHNFFSLZ0L057XxqEg8GKQJ1H0 27Dg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:message-id:subject:cc :to:from:date; bh=FVjlfakXvV0KR1Th/BSjuNjxH4p1Zktr5rUavKUK3Kw=; b=ZVT3SHQ1G47jApGdiTRqKSrCoPprlqc7Px+kIkvB+xb5LFYiOAbRLLs48ag1Rs5Ts+ m8Qm2/EvCPg+VOHyscCd2Te17soZDrnWDyHeSWH4hgtNEDrqm+AFW2ibsSoIRGhAAGvs RCNMt36cpwBhmWYu0XWS1gjfrOGA520pjBqLkEHL2HDch56Im599FVspJ49GwbYCcTpW jgdLaWkSJuIgSm5q5ZIsOwNpMsOTetYIi5cc60QVxCaplA7zgRZOwPM9lU0H3lmc8Szy Tw2X/NEswzpmowz10ULBsqg7lyOZCq+sDekij9GHgQW0koMLRmlsPsRfip5obs7nqKYE tm/g== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=intel.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id x3si9348827plv.422.2019.08.19.00.30.12; Mon, 19 Aug 2019 00:30:27 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=intel.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726937AbfHSH3O (ORCPT + 99 others); Mon, 19 Aug 2019 03:29:14 -0400 Received: from mga18.intel.com ([134.134.136.126]:62493 "EHLO mga18.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726261AbfHSH3O (ORCPT ); Mon, 19 Aug 2019 03:29:14 -0400 X-Amp-Result: UNKNOWN X-Amp-Original-Verdict: FILE UNKNOWN X-Amp-File-Uploaded: False Received: from fmsmga008.fm.intel.com ([10.253.24.58]) by orsmga106.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 19 Aug 2019 00:27:59 -0700 X-IronPort-AV: E=Sophos;i="5.64,403,1559545200"; d="scan'208";a="177797511" Received: from paasikivi.fi.intel.com ([10.237.72.42]) by fmsmga008-auth.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 19 Aug 2019 00:27:52 -0700 Received: by paasikivi.fi.intel.com (Postfix, from userid 1000) id 09D1F202FC; Mon, 19 Aug 2019 10:26:22 +0300 (EEST) Date: Mon, 19 Aug 2019 10:26:21 +0300 From: Sakari Ailus To: Hugues FRUCHET Cc: Alexandre TORGUE , Mauro Carvalho Chehab , Hans Verkuil , "linux-media@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "linux-kernel@vger.kernel.org" , "linux-stm32@st-md-mailman.stormreply.com" , Benjamin Gaignard , Yannick FERTRE , Philippe CORNU , Mickael GUENE Subject: Re: [PATCH v6 2/4] media: stm32-dcmi: trace the supported fourcc/mbus_code Message-ID: <20190819072621.GZ6133@paasikivi.fi.intel.com> References: <1565790533-10043-1-git-send-email-hugues.fruchet@st.com> <1565790533-10043-3-git-send-email-hugues.fruchet@st.com> <20190816081514.GU6133@paasikivi.fi.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.10.1 (2018-07-13) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Hugues, On Mon, Aug 19, 2019 at 07:23:17AM +0000, Hugues FRUCHET wrote: > Hi Sakari, > > On 8/16/19 10:15 AM, Sakari Ailus wrote: > > Hi Hugues, > > > > On Wed, Aug 14, 2019 at 03:48:51PM +0200, Hugues Fruchet wrote: > >> Add a trace of the set of supported fourcc/mbus_code which > >> intersect between DCMI and source sub-device. > >> > >> Signed-off-by: Hugues Fruchet > >> --- > >> drivers/media/platform/stm32/stm32-dcmi.c | 12 ++++++++++-- > >> 1 file changed, 10 insertions(+), 2 deletions(-) > >> > >> diff --git a/drivers/media/platform/stm32/stm32-dcmi.c b/drivers/media/platform/stm32/stm32-dcmi.c > >> index b462f71..18acecf 100644 > >> --- a/drivers/media/platform/stm32/stm32-dcmi.c > >> +++ b/drivers/media/platform/stm32/stm32-dcmi.c > >> @@ -1447,12 +1447,20 @@ static int dcmi_formats_init(struct stm32_dcmi *dcmi) > >> /* Code supported, have we got this fourcc yet? */ > >> for (j = 0; j < num_fmts; j++) > >> if (sd_fmts[j]->fourcc == > >> - dcmi_formats[i].fourcc) > >> + dcmi_formats[i].fourcc) { > >> /* Already available */ > >> + dev_dbg(dcmi->dev, "Skipping fourcc/code: %4.4s/0x%x\n", > >> + (char *)&sd_fmts[j]->fourcc, > >> + mbus_code.code); > >> break; > >> - if (j == num_fmts) > >> + } > >> + if (j == num_fmts) { > >> /* New */ > >> sd_fmts[num_fmts++] = dcmi_formats + i; > >> + dev_dbg(dcmi->dev, "Supported fourcc/code: %4.4s/0x%x\n", > > > > Over 80 characters per line. > > > > This an exception of the "80 chars" in order to be able to grep in > kernel messages: > https://www.kernel.org/doc/html/v4.10/process/coding-style.html > "However, never break user-visible strings such as printk messages, > because that breaks the ability to grep for them." > > This exception is managed in checkpatch.pl (--strict). This exception is for cases where wrapping the line in the usual way, e.g. at argument boundaries, does not prevent it exceeding 80 characters. But it is not the case here. -- Sakari Ailus sakari.ailus@linux.intel.com