[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH v2] xen/drivers/char: Don't require vpl011 for all non-x86 archs



Hi Alistair,

On 20/05/2019 18:13, Alistair Francis wrote:
Make the asm/vpl011.h dependent on the CONFIG_SBSA_VUART_CONSOLE define.

Your commit message wants a bit more explanation. In this case, you want to say the only user of this include is protected by CONFIG_SBSA_VUART_CONSOLE, hence it makes sense to protect it with the same define.

The title would need to be updated as well.


Signed-off-by: Alistair Francis <alistair.francis@xxxxxxx>
---
  xen/drivers/char/console.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/xen/drivers/char/console.c b/xen/drivers/char/console.c
index 9bbcb0f57a..24287e59cb 100644
--- a/xen/drivers/char/console.c
+++ b/xen/drivers/char/console.c
@@ -36,7 +36,7 @@
  #ifdef CONFIG_X86
  #include <xen/consoled.h>
  #include <asm/guest.h>
-#else
+#elif CONFIG_SBSA_VUART_CONSOLE
  #include <asm/vpl011.h>
  #endif

This is a bit odds to require !CONFIG_X86 && CONFIG_SBSA_VUART_CONSOLE but the code is only protected with the second part.

How about:

#endif
#ifdef CONFIG_SBSA_VUART_CONSOLE
...
#endif

?

--
Julien Grall

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.