Date: Tue, 18 Jul 2017 07:09:02 -0600
From: Jonathan Corbet <corbet@lwn.net>
To: Daniel Vetter <daniel@ffwll.ch>
Cc: linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org,
        dri-devel@lists.freedesktop.org
Subject: Re: [PATCH 6/7] docs: Do not include from
 include/drm/drm_color_mgmt.h
Message-ID: <20170718070902.4e38f233@lwn.net>
In-Reply-To: <20170718064241.kzzvtcww4mkqaaqq@phenom.ffwll.local>
References: <20170717210048.16400-1-corbet@lwn.net>
        <20170717210048.16400-7-corbet@lwn.net>
        <20170718064241.kzzvtcww4mkqaaqq@phenom.ffwll.local>
Organization: LWN.net
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 8bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 943
Lines: 27

On Tue, 18 Jul 2017 08:42:42 +0200
Daniel Vetter <daniel@ffwll.ch> wrote:

> > Commit 8f2e045ec878 (drm/color: un-inline drm_color_lut_extract()) moved
> > the only kerneldoc comment out of include/drm/drm_color_mgmt.h, leading to
> > this warning:
> > 
> >     ./include/drm/drm_color_mgmt.h:1: warning: no structured comments found
> > 
> > That comment is already picked up in drm_color_mgmt.c, so just delete the
> > directive.
> > 
> > CC: dri-devel@lists.freedesktop.org
> > Signed-off-by: Jonathan Corbet <corbet@lwn.net>
> > ---  
> 
> We have something like this already queued for 4.14, but if you want to
> fix all the warnings in 4.13 already I think that makes sense. But I'll
> leave that to you and merging through docs-fixes.

4.14 was my plan too; I think I've sent Linus enough churn for this cycle
already...:)  Since you have it queued, I'll just drop mine.  Plenty of
other warnings to beat my head against.

Thanks,

jon