mbox series

[0/4] docs/interop/firmware.json: scripts/qapi-gen.py compatibility

Message ID 20240306-qapi-firmware-json-v1-0-619f7122a249@linutronix.de (mailing list archive)
Headers show
Series docs/interop/firmware.json: scripts/qapi-gen.py compatibility | expand

Message

Thomas Weißschuh March 6, 2024, 10:31 a.m. UTC 
docs/interop/firmware.json is currently not usable with qapi-gen.py due
to various non-functional issues.
Fix those issue to provide compatibility.

Signed-off-by: Thomas Weißschuh <thomas.weissschuh@linutronix.de>
---
Thomas Weißschuh (4):
      docs/interop/firmware.json: Align examples
      docs/interop/firmware.json: Fix doc for FirmwareFlashMode
      docs/interop/firmware.json: Use full include paths
      docs/interop/firmware.json: Include pragma.json

 docs/interop/firmware.json | 392 +++++++++++++++++++++++----------------------
 1 file changed, 197 insertions(+), 195 deletions(-)
---
base-commit: db596ae19040574e41d086e78469014191d7d7fc
change-id: 20240306-qapi-firmware-json-6fb1213936dd

Best regards,

Comments

Philippe Mathieu-Daudé March 6, 2024, 4:15 p.m. UTC | #1 
On 6/3/24 11:31, Thomas Weißschuh wrote:
> docs/interop/firmware.json is currently not usable with qapi-gen.py due
> to various non-functional issues.
> Fix those issue to provide compatibility.

Could we add some lines in docs/meson.build to cover this files
during our CI tests?

> 
> Signed-off-by: Thomas Weißschuh <thomas.weissschuh@linutronix.de>
> ---
> Thomas Weißschuh (4):
>        docs/interop/firmware.json: Align examples
>        docs/interop/firmware.json: Fix doc for FirmwareFlashMode
>        docs/interop/firmware.json: Use full include paths
>        docs/interop/firmware.json: Include pragma.json
> 
>   docs/interop/firmware.json | 392 +++++++++++++++++++++++----------------------
>   1 file changed, 197 insertions(+), 195 deletions(-)
> ---
> base-commit: db596ae19040574e41d086e78469014191d7d7fc
> change-id: 20240306-qapi-firmware-json-6fb1213936dd
> 
> Best regards,
Thomas Weißschuh March 6, 2024, 4:35 p.m. UTC | #2 
On Wed, Mar 06, 2024 at 05:15:34PM +0100, Philippe Mathieu-Daudé wrote:
> On 6/3/24 11:31, Thomas Weißschuh wrote:
> > docs/interop/firmware.json is currently not usable with qapi-gen.py due
> > to various non-functional issues.
> > Fix those issue to provide compatibility.
> 
> Could we add some lines in docs/meson.build to cover this files
> during our CI tests?

Sounds good.

I am also developing a new tool which would use the generated QAPI
files which I plan to submit soonish.

> > 
> > Signed-off-by: Thomas Weißschuh <thomas.weissschuh@linutronix.de>
> > ---
> > Thomas Weißschuh (4):
> >        docs/interop/firmware.json: Align examples
> >        docs/interop/firmware.json: Fix doc for FirmwareFlashMode
> >        docs/interop/firmware.json: Use full include paths
> >        docs/interop/firmware.json: Include pragma.json
> > 
> >   docs/interop/firmware.json | 392 +++++++++++++++++++++++----------------------
> >   1 file changed, 197 insertions(+), 195 deletions(-)
> > ---
> > base-commit: db596ae19040574e41d086e78469014191d7d7fc
> > change-id: 20240306-qapi-firmware-json-6fb1213936dd
> > 
> > Best regards,