From mboxrd@z Thu Jan 1 00:00:00 1970 From: Laurent Pinchart Date: Wed, 29 Apr 2015 11:37:50 +0000 Subject: Re: [PATCH 01/10] drm: rcar-du: Document the rcar_du_crtc structure Message-Id: <2580958.azi1aY8JmC@avalon> List-Id: References: <1430267884-20560-1-git-send-email-laurent.pinchart+renesas@ideasonboard.com> <1430267884-20560-2-git-send-email-laurent.pinchart+renesas@ideasonboard.com> <5540C18B.4080104@cogentembedded.com> In-Reply-To: <5540C18B.4080104@cogentembedded.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Sergei Shtylyov Cc: Laurent Pinchart , dri-devel@lists.freedesktop.org, linux-sh@vger.kernel.org Hi Sergei, Thank you for the review. On Wednesday 29 April 2015 14:33:31 Sergei Shtylyov wrote: > On 4/29/2015 3:37 AM, Laurent Pinchart wrote: > > Document the structure fields using kerneldoc. > > > > Signed-off-by: Laurent Pinchart > > > > --- > > > > drivers/gpu/drm/rcar-du/rcar_du_crtc.h | 14 ++++++++++++++ > > 1 file changed, 14 insertions(+) > > > > diff --git a/drivers/gpu/drm/rcar-du/rcar_du_crtc.h > > b/drivers/gpu/drm/rcar-du/rcar_du_crtc.h index 5d9aa9b33769..e72f15c8c706 > > 100644 > > --- a/drivers/gpu/drm/rcar-du/rcar_du_crtc.h > > +++ b/drivers/gpu/drm/rcar-du/rcar_du_crtc.h > > @@ -22,6 +22,20 @@ > > > > struct rcar_du_group; > > > > +/* > > Kerneldoc needs /** at the start of comment. I have mixed feelings about that when documenting a driver, as generating documentation for the driver internals might not be very useful. On the other hand, the kernel documentation generation process doesn't include drivers randomly, so I suppose it can't hurt. I'll change it. > > + * rcar_du_crtc - the CRTC, representing a DU superposition processor > > Kerneldoc needs *struct* before the name. Oops, my bad, I'll fix that. Same for patch 02/10. -- Regards, Laurent Pinchart