Re: kernel-lintdoc parser - was: Re: [PATCH v2 0/3] doc-rst:c-domain: fix some issues in the c-domain

2016-09-22 Thread Markus Heiser
Am 22.09.2016 um 14:35 schrieb Mauro Carvalho Chehab : > Hi Markus, > 3) this is actually a more complex problem: how to represent returned values > from the function callbacks. Maybe we'll need to patch kernel-doc. This might be a solution for dense kernel-doc comments where you want to have pa

Re: kernel-lintdoc parser - was: Re: [PATCH v2 0/3] doc-rst:c-domain: fix some issues in the c-domain

2016-09-22 Thread Markus Heiser
Hi Mauro, thanks a lot for your tests and inspirations ... Am 22.09.2016 um 14:35 schrieb Mauro Carvalho Chehab : > Hi Markus, > > Em Thu, 22 Sep 2016 09:08:50 -0300 > Mauro Carvalho Chehab escreveu: > >> Em Tue, 20 Sep 2016 20:56:35 +0200 >> Markus Heiser escreveu: >> > >> The new parser

kernel-lintdoc parser - was: Re: [PATCH v2 0/3] doc-rst:c-domain: fix some issues in the c-domain

2016-09-22 Thread Mauro Carvalho Chehab
Hi Markus, Em Thu, 22 Sep 2016 09:08:50 -0300 Mauro Carvalho Chehab escreveu: > Em Tue, 20 Sep 2016 20:56:35 +0200 > Markus Heiser escreveu: > > The new parser seems to have some bugs, like those: > > $ kernel-lintdoc include/media/v4l2-ctrls.h > include/media/v4l2-ctrls.h:106 :WARN: typedef

Re: [PATCH v2 0/3] doc-rst:c-domain: fix some issues in the c-domain

2016-09-22 Thread Mauro Carvalho Chehab
Em Tue, 20 Sep 2016 20:56:35 +0200 Markus Heiser escreveu: > > If I understood it right, I could do something like: > > > > diff --git a/Documentation/media/conf_nitpick.py > > b/Documentation/media/conf_nitpick.py > > index 480d548af670..2de603871536 100644 > > --- a/Documentation/media/conf_n

Re: [PATCH v2 0/3] doc-rst:c-domain: fix some issues in the c-domain

2016-09-20 Thread Mauro Carvalho Chehab
Em Tue, 20 Sep 2016 13:00:33 -0600 Jonathan Corbet escreveu: > On Tue, 20 Sep 2016 20:56:35 +0200 > Markus Heiser wrote: > > > > I submitted one patch fixing it. Not sure if it got merged by Jon > > > or not. > > > > Ups, I might have overseen this patch .. as Jon said, its hard to > > follo

Re: [PATCH v2 0/3] doc-rst:c-domain: fix some issues in the c-domain

2016-09-20 Thread Jonathan Corbet
On Tue, 20 Sep 2016 20:56:35 +0200 Markus Heiser wrote: > > I submitted one patch fixing it. Not sure if it got merged by Jon > > or not. > > Ups, I might have overseen this patch .. as Jon said, its hard to > follow you ;) > > I tested the above with Jon's docs-next, so it seems your patch i

Re: [PATCH v2 0/3] doc-rst:c-domain: fix some issues in the c-domain

2016-09-20 Thread Markus Heiser
Am 19.09.2016 um 17:00 schrieb Mauro Carvalho Chehab : >> Hmm, as far as I see, the output is not correct ... The output of >> functions with a function pointer argument are missing the >> leading parenthesis in the function definition: >> >> .. c:function:: struct v4l2_m2m_ctx * v4l2_m2m_ctx_

Re: [PATCH v2 0/3] doc-rst:c-domain: fix some issues in the c-domain

2016-09-19 Thread Mauro Carvalho Chehab
Em Mon, 19 Sep 2016 13:36:55 +0200 Markus Heiser escreveu: > Hi Mauro, > > sorry for my late reply (so much work to do) .. > > Am 09.09.2016 um 14:25 schrieb Markus Heiser : > > >> Using either this approach or my kernel-doc patch, I'm now getting > >> only two warnings: > >> > >> 1) at medi

Re: [PATCH v2 0/3] doc-rst:c-domain: fix some issues in the c-domain

2016-09-19 Thread Markus Heiser
Hi Mauro, sorry for my late reply (so much work to do) .. Am 09.09.2016 um 14:25 schrieb Markus Heiser : >> Using either this approach or my kernel-doc patch, I'm now getting >> only two warnings: >> >> 1) at media-entity.h, even without nitpick mode: >> >> ./include/media/media-entity.h:1053

Re: [PATCH v2 0/3] doc-rst:c-domain: fix some issues in the c-domain

2016-09-17 Thread Markus Heiser
Am 16.09.2016 um 18:02 schrieb Jonathan Corbet : > On Wed, 7 Sep 2016 09:12:55 +0200 > Markus Heiser wrote: > >> according to your remarks I fixed the first and second patch. The third >> patch is >> resend unchanged; > > OK, I've applied the first two, finally. ... > Information hiding is

Re: [PATCH v2 0/3] doc-rst:c-domain: fix some issues in the c-domain

2016-09-16 Thread Jonathan Corbet
On Wed, 7 Sep 2016 09:12:55 +0200 Markus Heiser wrote: > according to your remarks I fixed the first and second patch. The third patch > is > resend unchanged; OK, I've applied the first two, finally. > > Am 06.09.2016 um 14:28 schrieb Jonathan Corbet : > > > > As others have pointed out, we

Re: [PATCH v2 0/3] doc-rst:c-domain: fix some issues in the c-domain

2016-09-09 Thread Markus Heiser
Am 09.09.2016 um 14:08 schrieb Mauro Carvalho Chehab : > Em Wed, 7 Sep 2016 09:12:55 +0200 > Markus Heiser escreveu: > >> From: Markus Heiser >> >> Hi Jon, >> >> according to your remarks I fixed the first and second patch. The third >> patch is >> resend unchanged; >> >>> Am 06.09.2016 u

Re: [PATCH v2 0/3] doc-rst:c-domain: fix some issues in the c-domain

2016-09-09 Thread Mauro Carvalho Chehab
Em Wed, 7 Sep 2016 09:12:55 +0200 Markus Heiser escreveu: > From: Markus Heiser > > Hi Jon, > > according to your remarks I fixed the first and second patch. The third patch > is > resend unchanged; > > > Am 06.09.2016 um 14:28 schrieb Jonathan Corbet : > > > > As others have pointed out, w

[PATCH v2 0/3] doc-rst:c-domain: fix some issues in the c-domain

2016-09-07 Thread Markus Heiser
From: Markus Heiser Hi Jon, according to your remarks I fixed the first and second patch. The third patch is resend unchanged; > Am 06.09.2016 um 14:28 schrieb Jonathan Corbet : > > As others have pointed out, we generally want to hide the difference > between functions and macros, so this is p