qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [Qemu-devel] [PATCH] configs/alpha: Remove unused CONFIG_PARALLEL_ISA switch
@ 2018-10-24 10:18 Thomas Huth
  2018-10-24 11:18 ` Richard Henderson
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Thomas Huth @ 2018-10-24 10:18 UTC (permalink / raw)
  To: Richard Henderson, qemu-devel; +Cc: Philippe Mathieu-Daudé, qemu-trivial

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

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [Qemu-devel] [PATCH] configs/alpha: Remove unused CONFIG_PARALLEL_ISA switch
  2018-10-24 10:18 [Qemu-devel] [PATCH] configs/alpha: Remove unused CONFIG_PARALLEL_ISA switch Thomas Huth
@ 2018-10-24 11:18 ` Richard Henderson
  2018-10-24 23:18 ` Philippe Mathieu-Daudé
  2018-10-25 13:49 ` [Qemu-devel] [Qemu-trivial] " Laurent Vivier
  2 siblings, 0 replies; 4+ messages in thread
From: Richard Henderson @ 2018-10-24 11:18 UTC (permalink / raw)
  To: Thomas Huth, qemu-devel; +Cc: Philippe Mathieu-Daudé, qemu-trivial

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~

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [Qemu-devel] [PATCH] configs/alpha: Remove unused CONFIG_PARALLEL_ISA switch
  2018-10-24 10:18 [Qemu-devel] [PATCH] configs/alpha: Remove unused CONFIG_PARALLEL_ISA switch Thomas Huth
  2018-10-24 11:18 ` Richard Henderson
@ 2018-10-24 23:18 ` Philippe Mathieu-Daudé
  2018-10-25 13:49 ` [Qemu-devel] [Qemu-trivial] " Laurent Vivier
  2 siblings, 0 replies; 4+ messages in thread
From: Philippe Mathieu-Daudé @ 2018-10-24 23:18 UTC (permalink / raw)
  To: Thomas Huth, Richard Henderson, qemu-devel
  Cc: qemu-trivial, Philippe Mathieu-Daudé

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
> 

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [Qemu-devel] [Qemu-trivial] [PATCH] configs/alpha: Remove unused CONFIG_PARALLEL_ISA switch
  2018-10-24 10:18 [Qemu-devel] [PATCH] configs/alpha: Remove unused CONFIG_PARALLEL_ISA switch Thomas Huth
  2018-10-24 11:18 ` Richard Henderson
  2018-10-24 23:18 ` Philippe Mathieu-Daudé
@ 2018-10-25 13:49 ` Laurent Vivier
  2 siblings, 0 replies; 4+ messages in thread
From: Laurent Vivier @ 2018-10-25 13:49 UTC (permalink / raw)
  To: Thomas Huth, Richard Henderson, qemu-devel
  Cc: qemu-trivial, Philippe Mathieu-Daudé

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

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2018-10-25 13:49 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-10-24 10:18 [Qemu-devel] [PATCH] configs/alpha: Remove unused CONFIG_PARALLEL_ISA switch Thomas Huth
2018-10-24 11:18 ` Richard Henderson
2018-10-24 23:18 ` Philippe Mathieu-Daudé
2018-10-25 13:49 ` [Qemu-devel] [Qemu-trivial] " Laurent Vivier

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).