Re: [PATCH v2 00/10] Kconfig vs. default devices

2023-05-02 Thread Alex Bennée
Fabiano Rosas writes: > v2: > Applying the feedback received, all small tweaks. > > Patch 6 still needs consensus on whether to apply the fix to Kconfig > or elsewhere. Link to the previous version: > https://lore.kernel.org/r/461ba038-31bf-49c4-758b-94ece36f1...@redhat.com > > changelog: > > -

Re: [PATCH v2 00/10] Kconfig vs. default devices

2023-05-02 Thread Alex Bennée
Thomas Huth writes: > On 08/02/2023 20.43, Philippe Mathieu-Daudé wrote: >> On 8/2/23 20:26, Fabiano Rosas wrote: >> >>> We currently have a situation where disabling a Kconfig might result >>> in a runtime error when QEMU selects the corresponding device as a >>> default value for an option.

Re: [PATCH v2 00/10] Kconfig vs. default devices

2023-02-08 Thread Thomas Huth
On 08/02/2023 20.43, Philippe Mathieu-Daudé wrote: On 8/2/23 20:26, Fabiano Rosas wrote: We currently have a situation where disabling a Kconfig might result in a runtime error when QEMU selects the corresponding device as a default value for an option. But first a disambiguation: Kconfig defa

Re: [PATCH v2 00/10] Kconfig vs. default devices

2023-02-08 Thread Fabiano Rosas
Philippe Mathieu-Daudé writes: > On 8/2/23 20:26, Fabiano Rosas wrote: > >> We currently have a situation where disabling a Kconfig might result >> in a runtime error when QEMU selects the corresponding device as a >> default value for an option. But first a disambiguation: >> >> Kconfig default

Re: [PATCH v2 00/10] Kconfig vs. default devices

2023-02-08 Thread Philippe Mathieu-Daudé
On 8/2/23 20:26, Fabiano Rosas wrote: We currently have a situation where disabling a Kconfig might result in a runtime error when QEMU selects the corresponding device as a default value for an option. But first a disambiguation: Kconfig default:: a device "Foo" for which there's "config FO

[PATCH v2 00/10] Kconfig vs. default devices

2023-02-08 Thread Fabiano Rosas
v2: Applying the feedback received, all small tweaks. Patch 6 still needs consensus on whether to apply the fix to Kconfig or elsewhere. Link to the previous version: https://lore.kernel.org/r/461ba038-31bf-49c4-758b-94ece36f1...@redhat.com changelog: - patch 1: moved isa-parallel to a build tim