Re: [PATCH 10/11] include/exec: fix kerneldoc definition

2023-03-13 Thread Peter Maydell
On Mon, 13 Mar 2023 at 17:30, Peter Maydell wrote: > So I think the problem here is not with Sphinx, but with the > kernel-doc script. That script has an option "-Werror" which > turns its warnings into errors, but our Sphinx extension > docs/sphinx/kerneldoc.py does not set it. I think we need to

Re: [PATCH 10/11] include/exec: fix kerneldoc definition

2023-03-13 Thread Peter Maydell
On Mon, 13 Mar 2023 at 17:14, Thomas Huth wrote: > > On 13/03/2023 18.03, Peter Maydell wrote: > > On Mon, 13 Mar 2023 at 17:00, Thomas Huth wrote: > >> I also keep running into this problem ... I wonder whether we should run > >> sphinx with "-W" to turn warnings into errors when configure has b

Re: [PATCH 10/11] include/exec: fix kerneldoc definition

2023-03-13 Thread Thomas Huth
On 13/03/2023 18.03, Peter Maydell wrote: On Mon, 13 Mar 2023 at 17:00, Thomas Huth wrote: On 10/03/2023 11.31, Alex Bennée wrote: The kerneldoc processor complains about the mismatched variable name. Fix it. Signed-off-by: Alex Bennée --- include/exec/memory.h | 2 +- 1 file changed,

Re: [PATCH 10/11] include/exec: fix kerneldoc definition

2023-03-13 Thread Peter Maydell
On Mon, 13 Mar 2023 at 17:00, Thomas Huth wrote: > > On 10/03/2023 11.31, Alex Bennée wrote: > > The kerneldoc processor complains about the mismatched variable name. > > Fix it. > > > > Signed-off-by: Alex Bennée > > --- > > include/exec/memory.h | 2 +- > > 1 file changed, 1 insertion(+), 1

Re: [PATCH 10/11] include/exec: fix kerneldoc definition

2023-03-13 Thread Thomas Huth
On 10/03/2023 11.31, Alex Bennée wrote: The kerneldoc processor complains about the mismatched variable name. Fix it. Signed-off-by: Alex Bennée --- include/exec/memory.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/exec/memory.h b/include/exec/memory.h index 6

Re: [PATCH 10/11] include/exec: fix kerneldoc definition

2023-03-10 Thread Philippe Mathieu-Daudé
On 10/3/23 11:31, Alex Bennée wrote: The kerneldoc processor complains about the mismatched variable name. Fix it. Signed-off-by: Alex Bennée --- include/exec/memory.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) Reviewed-by: Philippe Mathieu-Daudé

[PATCH 10/11] include/exec: fix kerneldoc definition

2023-03-10 Thread Alex Bennée
The kerneldoc processor complains about the mismatched variable name. Fix it. Signed-off-by: Alex Bennée --- include/exec/memory.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/exec/memory.h b/include/exec/memory.h index 6fa0b071f0..15ade918ba 100644 --- a/include/e