xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
* [PATCH v1 0/1] xen/arm: Re-add the Xilinx ZynqMP platform
@ 2016-02-26  1:38 Edgar E. Iglesias
  2016-02-26  1:38 ` [PATCH v1 1/1] " Edgar E. Iglesias
  0 siblings, 1 reply; 6+ messages in thread
From: Edgar E. Iglesias @ 2016-02-26  1:38 UTC (permalink / raw)
  To: xen-devel; +Cc: edgar.iglesias, julien.grall, stefano.stabellini, ian.campbell

From: "Edgar E. Iglesias" <edgar.iglesias@xilinx.com>

We need to re-add the Xilinx ZynqMP platform with a blacklisted
zynqmp-pm power management firmware API device. Long term we'll
be looking at finding a way for this to work but this is likely
to take a while. In the meantime, in order to keep recent kernels
from panicing on boots we blacklist it (similar to what vexpress does).

Best regards,
Edgar

Edgar E. Iglesias (1):
  xen/arm: Re-add the Xilinx ZynqMP platform

 xen/arch/arm/platforms/Makefile        |  1 +
 xen/arch/arm/platforms/xilinx-zynqmp.c | 47 ++++++++++++++++++++++++++++++++++
 2 files changed, 48 insertions(+)
 create mode 100644 xen/arch/arm/platforms/xilinx-zynqmp.c

-- 
2.5.0


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

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

* [PATCH v1 1/1] xen/arm: Re-add the Xilinx ZynqMP platform
  2016-02-26  1:38 [PATCH v1 0/1] xen/arm: Re-add the Xilinx ZynqMP platform Edgar E. Iglesias
@ 2016-02-26  1:38 ` Edgar E. Iglesias
  2016-02-26 11:51   ` Stefano Stabellini
  0 siblings, 1 reply; 6+ messages in thread
From: Edgar E. Iglesias @ 2016-02-26  1:38 UTC (permalink / raw)
  To: xen-devel; +Cc: edgar.iglesias, julien.grall, stefano.stabellini, ian.campbell

From: "Edgar E. Iglesias" <edgar.iglesias@xilinx.com>

Re-add the Xilinx ZynqMP platform. This time include a
blacklisted zynqmp-pm (Power Management) device that does
not yet play nicely with Xen.

Signed-off-by: Edgar E. Iglesias <edgar.iglesias@xilinx.com>
---
 xen/arch/arm/platforms/Makefile        |  1 +
 xen/arch/arm/platforms/xilinx-zynqmp.c | 47 ++++++++++++++++++++++++++++++++++
 2 files changed, 48 insertions(+)
 create mode 100644 xen/arch/arm/platforms/xilinx-zynqmp.c

diff --git a/xen/arch/arm/platforms/Makefile b/xen/arch/arm/platforms/Makefile
index e173fec..3689eec 100644
--- a/xen/arch/arm/platforms/Makefile
+++ b/xen/arch/arm/platforms/Makefile
@@ -7,3 +7,4 @@ obj-$(CONFIG_ARM_32) += sunxi.o
 obj-$(CONFIG_ARM_32) += rcar2.o
 obj-$(CONFIG_ARM_64) += seattle.o
 obj-$(CONFIG_ARM_64) += xgene-storm.o
+obj-$(CONFIG_ARM_64) += xilinx-zynqmp.o
diff --git a/xen/arch/arm/platforms/xilinx-zynqmp.c b/xen/arch/arm/platforms/xilinx-zynqmp.c
new file mode 100644
index 0000000..2adee91
--- /dev/null
+++ b/xen/arch/arm/platforms/xilinx-zynqmp.c
@@ -0,0 +1,47 @@
+/*
+ * xen/arch/arm/platforms/xilinx-zynqmp.c
+ *
+ * Xilinx ZynqMP setup
+ *
+ * Copyright (c) 2016 Xilinx Inc.
+ * Written by Edgar E. Iglesias <edgar.iglesias@xilinx.com>
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ */
+
+#include <asm/platform.h>
+
+static const char * const zynqmp_dt_compat[] __initconst =
+{
+    "xlnx,zynqmp",
+    NULL
+};
+
+static const struct dt_device_match zynqmp_blacklist_dev[] __initconst =
+{
+    /* Power management is not yet supported.  */
+    DT_MATCH_COMPATIBLE("xlnx,zynqmp-pm"),
+    { /* sentinel */ },
+};
+
+PLATFORM_START(xgene_storm, "Xilinx ZynqMP")
+    .compatible = zynqmp_dt_compat,
+    .blacklist_dev = zynqmp_blacklist_dev,
+PLATFORM_END
+
+/*
+ * Local variables:
+ * mode: C
+ * c-file-style: "BSD"
+ * c-basic-offset: 4
+ * indent-tabs-mode: nil
+ * End:
+ */
-- 
2.5.0


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

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

* Re: [PATCH v1 1/1] xen/arm: Re-add the Xilinx ZynqMP platform
  2016-02-26  1:38 ` [PATCH v1 1/1] " Edgar E. Iglesias
@ 2016-02-26 11:51   ` Stefano Stabellini
  2016-03-15 17:47     ` Julien Grall
  0 siblings, 1 reply; 6+ messages in thread
From: Stefano Stabellini @ 2016-02-26 11:51 UTC (permalink / raw)
  To: Edgar E. Iglesias
  Cc: edgar.iglesias, julien.grall, stefano.stabellini, ian.campbell,
	xen-devel

On Fri, 26 Feb 2016, Edgar E. Iglesias wrote:
> From: "Edgar E. Iglesias" <edgar.iglesias@xilinx.com>
> 
> Re-add the Xilinx ZynqMP platform. This time include a
> blacklisted zynqmp-pm (Power Management) device that does
> not yet play nicely with Xen.
> 
> Signed-off-by: Edgar E. Iglesias <edgar.iglesias@xilinx.com>

Acked-by: Stefano Stabellini <stefano.stabellini@eu.citrix.com>


>  xen/arch/arm/platforms/Makefile        |  1 +
>  xen/arch/arm/platforms/xilinx-zynqmp.c | 47 ++++++++++++++++++++++++++++++++++
>  2 files changed, 48 insertions(+)
>  create mode 100644 xen/arch/arm/platforms/xilinx-zynqmp.c
> 
> diff --git a/xen/arch/arm/platforms/Makefile b/xen/arch/arm/platforms/Makefile
> index e173fec..3689eec 100644
> --- a/xen/arch/arm/platforms/Makefile
> +++ b/xen/arch/arm/platforms/Makefile
> @@ -7,3 +7,4 @@ obj-$(CONFIG_ARM_32) += sunxi.o
>  obj-$(CONFIG_ARM_32) += rcar2.o
>  obj-$(CONFIG_ARM_64) += seattle.o
>  obj-$(CONFIG_ARM_64) += xgene-storm.o
> +obj-$(CONFIG_ARM_64) += xilinx-zynqmp.o
> diff --git a/xen/arch/arm/platforms/xilinx-zynqmp.c b/xen/arch/arm/platforms/xilinx-zynqmp.c
> new file mode 100644
> index 0000000..2adee91
> --- /dev/null
> +++ b/xen/arch/arm/platforms/xilinx-zynqmp.c
> @@ -0,0 +1,47 @@
> +/*
> + * xen/arch/arm/platforms/xilinx-zynqmp.c
> + *
> + * Xilinx ZynqMP setup
> + *
> + * Copyright (c) 2016 Xilinx Inc.
> + * Written by Edgar E. Iglesias <edgar.iglesias@xilinx.com>
> + *
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License as published by
> + * the Free Software Foundation; either version 2 of the License, or
> + * (at your option) any later version.
> + *
> + * This program is distributed in the hope that it will be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> + * GNU General Public License for more details.
> + */
> +
> +#include <asm/platform.h>
> +
> +static const char * const zynqmp_dt_compat[] __initconst =
> +{
> +    "xlnx,zynqmp",
> +    NULL
> +};
> +
> +static const struct dt_device_match zynqmp_blacklist_dev[] __initconst =
> +{
> +    /* Power management is not yet supported.  */
> +    DT_MATCH_COMPATIBLE("xlnx,zynqmp-pm"),
> +    { /* sentinel */ },
> +};
> +
> +PLATFORM_START(xgene_storm, "Xilinx ZynqMP")
> +    .compatible = zynqmp_dt_compat,
> +    .blacklist_dev = zynqmp_blacklist_dev,
> +PLATFORM_END
> +
> +/*
> + * Local variables:
> + * mode: C
> + * c-file-style: "BSD"
> + * c-basic-offset: 4
> + * indent-tabs-mode: nil
> + * End:
> + */
> -- 
> 2.5.0
> 

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

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

* Re: [PATCH v1 1/1] xen/arm: Re-add the Xilinx ZynqMP platform
  2016-02-26 11:51   ` Stefano Stabellini
@ 2016-03-15 17:47     ` Julien Grall
  2016-03-15 18:18       ` Konrad Rzeszutek Wilk
  2016-03-16 10:19       ` Jan Beulich
  0 siblings, 2 replies; 6+ messages in thread
From: Julien Grall @ 2016-03-15 17:47 UTC (permalink / raw)
  To: Stefano Stabellini, Edgar E. Iglesias
  Cc: edgar.iglesias, julien.grall, Ian Jackson, xen-devel,
	stefano.stabellini, Jan Beulich

Hi Jan and Ian,

This patch has been acked by Stefano and not in the tree.

I'm not sure who is in charge to commit ARM related patches in the tree 
so I've CCed both of you.

Regards,

On 26/02/16 11:51, Stefano Stabellini wrote:
> On Fri, 26 Feb 2016, Edgar E. Iglesias wrote:
>> From: "Edgar E. Iglesias" <edgar.iglesias@xilinx.com>
>>
>> Re-add the Xilinx ZynqMP platform. This time include a
>> blacklisted zynqmp-pm (Power Management) device that does
>> not yet play nicely with Xen.
>>
>> Signed-off-by: Edgar E. Iglesias <edgar.iglesias@xilinx.com>
>
> Acked-by: Stefano Stabellini <stefano.stabellini@eu.citrix.com>
>
>
>>   xen/arch/arm/platforms/Makefile        |  1 +
>>   xen/arch/arm/platforms/xilinx-zynqmp.c | 47 ++++++++++++++++++++++++++++++++++
>>   2 files changed, 48 insertions(+)
>>   create mode 100644 xen/arch/arm/platforms/xilinx-zynqmp.c
>>
>> diff --git a/xen/arch/arm/platforms/Makefile b/xen/arch/arm/platforms/Makefile
>> index e173fec..3689eec 100644
>> --- a/xen/arch/arm/platforms/Makefile
>> +++ b/xen/arch/arm/platforms/Makefile
>> @@ -7,3 +7,4 @@ obj-$(CONFIG_ARM_32) += sunxi.o
>>   obj-$(CONFIG_ARM_32) += rcar2.o
>>   obj-$(CONFIG_ARM_64) += seattle.o
>>   obj-$(CONFIG_ARM_64) += xgene-storm.o
>> +obj-$(CONFIG_ARM_64) += xilinx-zynqmp.o
>> diff --git a/xen/arch/arm/platforms/xilinx-zynqmp.c b/xen/arch/arm/platforms/xilinx-zynqmp.c
>> new file mode 100644
>> index 0000000..2adee91
>> --- /dev/null
>> +++ b/xen/arch/arm/platforms/xilinx-zynqmp.c
>> @@ -0,0 +1,47 @@
>> +/*
>> + * xen/arch/arm/platforms/xilinx-zynqmp.c
>> + *
>> + * Xilinx ZynqMP setup
>> + *
>> + * Copyright (c) 2016 Xilinx Inc.
>> + * Written by Edgar E. Iglesias <edgar.iglesias@xilinx.com>
>> + *
>> + * This program is free software; you can redistribute it and/or modify
>> + * it under the terms of the GNU General Public License as published by
>> + * the Free Software Foundation; either version 2 of the License, or
>> + * (at your option) any later version.
>> + *
>> + * This program is distributed in the hope that it will be useful,
>> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
>> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
>> + * GNU General Public License for more details.
>> + */
>> +
>> +#include <asm/platform.h>
>> +
>> +static const char * const zynqmp_dt_compat[] __initconst =
>> +{
>> +    "xlnx,zynqmp",
>> +    NULL
>> +};
>> +
>> +static const struct dt_device_match zynqmp_blacklist_dev[] __initconst =
>> +{
>> +    /* Power management is not yet supported.  */
>> +    DT_MATCH_COMPATIBLE("xlnx,zynqmp-pm"),
>> +    { /* sentinel */ },
>> +};
>> +
>> +PLATFORM_START(xgene_storm, "Xilinx ZynqMP")
>> +    .compatible = zynqmp_dt_compat,
>> +    .blacklist_dev = zynqmp_blacklist_dev,
>> +PLATFORM_END
>> +
>> +/*
>> + * Local variables:
>> + * mode: C
>> + * c-file-style: "BSD"
>> + * c-basic-offset: 4
>> + * indent-tabs-mode: nil
>> + * End:
>> + */
>> --
>> 2.5.0
>>
>
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xen.org
> http://lists.xen.org/xen-devel
>

-- 
Julien Grall

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

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

* Re: [PATCH v1 1/1] xen/arm: Re-add the Xilinx ZynqMP platform
  2016-03-15 17:47     ` Julien Grall
@ 2016-03-15 18:18       ` Konrad Rzeszutek Wilk
  2016-03-16 10:19       ` Jan Beulich
  1 sibling, 0 replies; 6+ messages in thread
From: Konrad Rzeszutek Wilk @ 2016-03-15 18:18 UTC (permalink / raw)
  To: Julien Grall
  Cc: edgar.iglesias, Stefano Stabellini, julien.grall, Ian Jackson,
	xen-devel, stefano.stabellini, Jan Beulich, Edgar E. Iglesias

On Tue, Mar 15, 2016 at 05:47:16PM +0000, Julien Grall wrote:
> Hi Jan and Ian,
> 
> This patch has been acked by Stefano and not in the tree.
> 
> I'm not sure who is in charge to commit ARM related patches in the tree so
> I've CCed both of you.
> 

Applied.
> Regards,
> 
> On 26/02/16 11:51, Stefano Stabellini wrote:
> >On Fri, 26 Feb 2016, Edgar E. Iglesias wrote:
> >>From: "Edgar E. Iglesias" <edgar.iglesias@xilinx.com>
> >>
> >>Re-add the Xilinx ZynqMP platform. This time include a
> >>blacklisted zynqmp-pm (Power Management) device that does
> >>not yet play nicely with Xen.
> >>
> >>Signed-off-by: Edgar E. Iglesias <edgar.iglesias@xilinx.com>
> >
> >Acked-by: Stefano Stabellini <stefano.stabellini@eu.citrix.com>
> >
> >
> >>  xen/arch/arm/platforms/Makefile        |  1 +
> >>  xen/arch/arm/platforms/xilinx-zynqmp.c | 47 ++++++++++++++++++++++++++++++++++
> >>  2 files changed, 48 insertions(+)
> >>  create mode 100644 xen/arch/arm/platforms/xilinx-zynqmp.c
> >>
> >>diff --git a/xen/arch/arm/platforms/Makefile b/xen/arch/arm/platforms/Makefile
> >>index e173fec..3689eec 100644
> >>--- a/xen/arch/arm/platforms/Makefile
> >>+++ b/xen/arch/arm/platforms/Makefile
> >>@@ -7,3 +7,4 @@ obj-$(CONFIG_ARM_32) += sunxi.o
> >>  obj-$(CONFIG_ARM_32) += rcar2.o
> >>  obj-$(CONFIG_ARM_64) += seattle.o
> >>  obj-$(CONFIG_ARM_64) += xgene-storm.o
> >>+obj-$(CONFIG_ARM_64) += xilinx-zynqmp.o
> >>diff --git a/xen/arch/arm/platforms/xilinx-zynqmp.c b/xen/arch/arm/platforms/xilinx-zynqmp.c
> >>new file mode 100644
> >>index 0000000..2adee91
> >>--- /dev/null
> >>+++ b/xen/arch/arm/platforms/xilinx-zynqmp.c
> >>@@ -0,0 +1,47 @@
> >>+/*
> >>+ * xen/arch/arm/platforms/xilinx-zynqmp.c
> >>+ *
> >>+ * Xilinx ZynqMP setup
> >>+ *
> >>+ * Copyright (c) 2016 Xilinx Inc.
> >>+ * Written by Edgar E. Iglesias <edgar.iglesias@xilinx.com>
> >>+ *
> >>+ * This program is free software; you can redistribute it and/or modify
> >>+ * it under the terms of the GNU General Public License as published by
> >>+ * the Free Software Foundation; either version 2 of the License, or
> >>+ * (at your option) any later version.
> >>+ *
> >>+ * This program is distributed in the hope that it will be useful,
> >>+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
> >>+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> >>+ * GNU General Public License for more details.
> >>+ */
> >>+
> >>+#include <asm/platform.h>
> >>+
> >>+static const char * const zynqmp_dt_compat[] __initconst =
> >>+{
> >>+    "xlnx,zynqmp",
> >>+    NULL
> >>+};
> >>+
> >>+static const struct dt_device_match zynqmp_blacklist_dev[] __initconst =
> >>+{
> >>+    /* Power management is not yet supported.  */
> >>+    DT_MATCH_COMPATIBLE("xlnx,zynqmp-pm"),
> >>+    { /* sentinel */ },
> >>+};
> >>+
> >>+PLATFORM_START(xgene_storm, "Xilinx ZynqMP")
> >>+    .compatible = zynqmp_dt_compat,
> >>+    .blacklist_dev = zynqmp_blacklist_dev,
> >>+PLATFORM_END
> >>+
> >>+/*
> >>+ * Local variables:
> >>+ * mode: C
> >>+ * c-file-style: "BSD"
> >>+ * c-basic-offset: 4
> >>+ * indent-tabs-mode: nil
> >>+ * End:
> >>+ */
> >>--
> >>2.5.0
> >>
> >
> >_______________________________________________
> >Xen-devel mailing list
> >Xen-devel@lists.xen.org
> >http://lists.xen.org/xen-devel
> >
> 
> -- 
> Julien Grall
> 
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xen.org
> http://lists.xen.org/xen-devel

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

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

* Re: [PATCH v1 1/1] xen/arm: Re-add the Xilinx ZynqMP platform
  2016-03-15 17:47     ` Julien Grall
  2016-03-15 18:18       ` Konrad Rzeszutek Wilk
@ 2016-03-16 10:19       ` Jan Beulich
  1 sibling, 0 replies; 6+ messages in thread
From: Jan Beulich @ 2016-03-16 10:19 UTC (permalink / raw)
  To: Julien Grall
  Cc: edgar.iglesias, Stefano Stabellini, julien.grall, Ian Jackson,
	xen-devel, stefano.stabellini, Edgar E. Iglesias

>>> On 15.03.16 at 18:47, <julien.grall@arm.com> wrote:
> Hi Jan and Ian,
> 
> This patch has been acked by Stefano and not in the tree.
> 
> I'm not sure who is in charge to commit ARM related patches in the tree 
> so I've CCed both of you.

Indeed right now the process is to prod the committers if a patch
becomes ready to go in. I'm trying to keep an eye on ARM patches,
but especially when there's a lot of traffic on the list I'm likely to
miss some.

Jan


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

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

end of thread, other threads:[~2016-03-16 10:19 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-02-26  1:38 [PATCH v1 0/1] xen/arm: Re-add the Xilinx ZynqMP platform Edgar E. Iglesias
2016-02-26  1:38 ` [PATCH v1 1/1] " Edgar E. Iglesias
2016-02-26 11:51   ` Stefano Stabellini
2016-03-15 17:47     ` Julien Grall
2016-03-15 18:18       ` Konrad Rzeszutek Wilk
2016-03-16 10:19       ` Jan Beulich

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).