[Qemu-devel] [PATCH] configs/alpha: Remove unused CONFIG_PARALLEL_ISA switch

Thomas Huth posted 1 patch 7 years ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/1540376314-5727-1-git-send-email-thuth@redhat.com
Test docker-clang@ubuntu passed
Test checkpatch passed
Test asan passed
Test docker-mingw@fedora failed
Test docker-quick@centos7 passed
default-configs/alpha-softmmu.mak | 1 -
1 file changed, 1 deletion(-)
[Qemu-devel] [PATCH] configs/alpha: Remove unused CONFIG_PARALLEL_ISA switch
Posted by Thomas Huth 7 years ago
We don't use CONFIG_PARALLEL_ISA in any of our Makefiles, so this
is just a dead config option which can be removed.

Fixes: a4cb773928e047b137c6998209cf2eec857fac6b
Signed-off-by: Thomas Huth <thuth@redhat.com>
---
 default-configs/alpha-softmmu.mak | 1 -
 1 file changed, 1 deletion(-)

diff --git a/default-configs/alpha-softmmu.mak b/default-configs/alpha-softmmu.mak
index bbe361f..61435cc 100644
--- a/default-configs/alpha-softmmu.mak
+++ b/default-configs/alpha-softmmu.mak
@@ -8,7 +8,6 @@ CONFIG_I82374=y
 CONFIG_I8254=y
 CONFIG_I8257=y
 CONFIG_PARALLEL=y
-CONFIG_PARALLEL_ISA=y
 CONFIG_FDC=y
 CONFIG_PCKBD=y
 CONFIG_VGA_CIRRUS=y
-- 
1.8.3.1


Re: [Qemu-devel] [PATCH] configs/alpha: Remove unused CONFIG_PARALLEL_ISA switch
Posted by Richard Henderson 7 years ago
On 10/24/18 11:18 AM, Thomas Huth wrote:
> We don't use CONFIG_PARALLEL_ISA in any of our Makefiles, so this
> is just a dead config option which can be removed.
> 
> Fixes: a4cb773928e047b137c6998209cf2eec857fac6b
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
>  default-configs/alpha-softmmu.mak | 1 -
>  1 file changed, 1 deletion(-)

Acked-by: Richard Henderson <richard.henderson@linaro.org>


r~

Re: [Qemu-devel] [PATCH] configs/alpha: Remove unused CONFIG_PARALLEL_ISA switch
Posted by Philippe Mathieu-Daudé 7 years ago
On 24/10/18 12:18, Thomas Huth wrote:
> We don't use CONFIG_PARALLEL_ISA in any of our Makefiles, so this
> is just a dead config option which can be removed.
> 
> Fixes: a4cb773928e047b137c6998209cf2eec857fac6b

Oops, this slipped while refactoring the series, this is part of a 
further cleanup that I never finished. Hopefully Kconfig will outdate it.

Thanks for cleaning this!

> Signed-off-by: Thomas Huth <thuth@redhat.com>

Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>

> ---
>   default-configs/alpha-softmmu.mak | 1 -
>   1 file changed, 1 deletion(-)
> 
> diff --git a/default-configs/alpha-softmmu.mak b/default-configs/alpha-softmmu.mak
> index bbe361f..61435cc 100644
> --- a/default-configs/alpha-softmmu.mak
> +++ b/default-configs/alpha-softmmu.mak
> @@ -8,7 +8,6 @@ CONFIG_I82374=y
>   CONFIG_I8254=y
>   CONFIG_I8257=y
>   CONFIG_PARALLEL=y
> -CONFIG_PARALLEL_ISA=y
>   CONFIG_FDC=y
>   CONFIG_PCKBD=y
>   CONFIG_VGA_CIRRUS=y
> 

Re: [Qemu-devel] [Qemu-trivial] [PATCH] configs/alpha: Remove unused CONFIG_PARALLEL_ISA switch
Posted by Laurent Vivier 7 years ago
On 24/10/2018 11:18, Thomas Huth wrote:
> We don't use CONFIG_PARALLEL_ISA in any of our Makefiles, so this
> is just a dead config option which can be removed.
> 
> Fixes: a4cb773928e047b137c6998209cf2eec857fac6b
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
>  default-configs/alpha-softmmu.mak | 1 -
>  1 file changed, 1 deletion(-)

Applied

Thanks,
Laurent