From mboxrd@z Thu Jan 1 00:00:00 1970 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.subspace.kernel.org (Postfix) with ESMTPS id A8E311755E; Mon, 5 Feb 2024 10:40:09 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=198.137.202.133 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1707129613; cv=none; b=gTQqE+a3ZqgEGB2YkhJQMOY3hk6mlHBBc5W7Q9TFqkYdM7k4SSaBO0Oq3so0UieIH/bua87FxeNqTSYz6FoH5rdxilLsj0gR7f+saTErDrFyHqYduAP9fElUagKUezkkwjx1dYEuUjWuWMEFG6w0s+rYoqHToFEjQtazivuWYw8= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1707129613; c=relaxed/simple; bh=Pi6euJLSnj2co6H/IooWtWX35oWGVFQDRFlDcv+0sck=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=E0WHV88G4rwESDwxbuFdZ9HdNb5SwlpRJoV4LTnIywWUssbQByHbR6Ath3cHw7T9kgh0RUJJrsrppyjcR7rsKrnGDKrdiFpu7l7ZL8lPULFiHrvK6E53ad9DAf2LPxd+gFHDo8nVtuqwhZiXhQ3xmLEoDlrbgjfMY55ElqiqMcI= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=infradead.org; spf=none smtp.mailfrom=infradead.org; dkim=pass (2048-bit key) header.d=infradead.org header.i=@infradead.org header.b=pOXBKu2U; arc=none smtp.client-ip=198.137.202.133 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=infradead.org Authentication-Results: smtp.subspace.kernel.org; spf=none smtp.mailfrom=infradead.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=infradead.org header.i=@infradead.org header.b="pOXBKu2U" DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=bombadil.20210309; h=Content-Transfer-Encoding: Content-Type:In-Reply-To:From:References:Cc:To:Subject:MIME-Version:Date: Message-ID:Sender:Reply-To:Content-ID:Content-Description; bh=/XPiUugopTvJyLAW30gfJ0WGytmJrayjsJOj85rr9tA=; b=pOXBKu2U2s9ahudf5rB1rX1L9T nCn5+N8MrXtz+PeaZKjGCZ4EJUpy6oQZKCXwx4AqGkqmEDpNN381D05iUCB2y98Ec2or/6s8vz5YN nIIEvx9xSdl7F/zlMQnA5orfDEtXrAwmOCs+jjhU/Nhgmd8gV3ASuOGqX/YGpVTSB7jkUmSanOUVp hpLhLoNp8xCeqNs9lANdusVHZs0I7Dm1/px7bnjQnHJddXouvQpSqAOg/HHw28yZKeUlkKAAIqNbo Myh4c8TE3eYlndJLUJogiJMUiXBgwLxDaxMXevigd1m1K+eMySJ3khsrqapGoNzNF/9v5vPAZnxRd 9aOKIeaw==; Received: from [50.53.50.0] (helo=[192.168.254.15]) by bombadil.infradead.org with esmtpsa (Exim 4.97.1 #2 (Red Hat Linux)) id 1rWwOJ-00000002rJF-3GlM; Mon, 05 Feb 2024 10:39:55 +0000 Message-ID: <8f3bab1f-8697-40c0-91f2-de934b4b9ddb@infradead.org> Date: Mon, 5 Feb 2024 02:39:53 -0800 Precedence: bulk X-Mailing-List: linux-arm-msm@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH 17/17] linux: v4l2-vp9.h: Fix kerneldoc Content-Language: en-US To: Hans Verkuil , Ricardo Ribalda , Jonathan Corbet Cc: Tiffany Lin , Andrew-CT Chen , Yunfei Dong , Mauro Carvalho Chehab , Matthias Brugger , AngeloGioacchino Del Regno , Sakari Ailus , Laurent Pinchart , Hans Verkuil , Kieran Bingham , Bin Liu , Ezequiel Garcia , Philipp Zabel , Stanimir Varbanov , Vikash Garodia , Bryan O'Donoghue , Bjorn Andersson , Konrad Dybcio , Sylwester Nawrocki , Krzysztof Kozlowski , Alim Akhtar , Marek Szyprowski , Andrzej Hajda , Bingbu Cao , Tianshu Qiu , Greg Kroah-Hartman , Neil Armstrong , Kevin Hilman , Jerome Brunet , Martin Blumenstingl , linux-media@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org, linux-rockchip@lists.infradead.org, linux-arm-msm@vger.kernel.org, linux-samsung-soc@vger.kernel.org, linux-staging@lists.linux.dev, linux-amlogic@lists.infradead.org, Sakari Ailus References: <20240126-gix-mtk-warnings-v1-0-eed7865fce18@chromium.org> <20240126-gix-mtk-warnings-v1-17-eed7865fce18@chromium.org> <201ae1d1-1e03-40e2-9cc4-49df70abb8da@xs4all.nl> From: Randy Dunlap In-Reply-To: <201ae1d1-1e03-40e2-9cc4-49df70abb8da@xs4all.nl> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit On 2/5/24 02:29, Hans Verkuil wrote: > On 27/01/2024 10:57, Sakari Ailus wrote: >> Hi Ricardo, >> >> On Fri, Jan 26, 2024 at 11:16:16PM +0000, Ricardo Ribalda wrote: >>> Kerneldoc cannot understand arrays defined like >>> v4l2_frame_symbol_counts. >>> >>> Adding an asterisk to the name does do the trick. >>> >>> Disable the kerneldoc notation for now, it is already ignored: >>> https://docs.kernel.org/search.html?q=v4l2_vp9_frame_symbol_counts >> >> Wouldn't it be nicer to fix kerneldoc instead? It might not be difficult at >> all. >> >> Feel free to, but I can also give it a try. >> > > It would be nice to have this fixed in kerneldoc itself. I'm holding this > patch back for two weeks to see if someone wants to work on kerneldoc. > > If not, then I'll take this anyway to fix the noise in our build. > > Note that while this header is indeed ignored in the documentation, that > is really more a bug and it would be nice to actually include this header > somewhere in our documentation. So fixing these kerneldoc warnings one way > or another is something that we should do. > It's just waiting for Jon to apply it: (from Sakari) https://lore.kernel.org/all/20240131084934.191226-1-sakari.ailus@linux.intel.com/ Thanks. -- #Randy