diff mbox

[v1,1/1] xen/arm: Re-add the Xilinx ZynqMP platform

Message ID 1456450709-14820-2-git-send-email-edgar.iglesias@gmail.com (mailing list archive)
State New, archived
Headers show

Commit Message

Edgar E. Iglesias Feb. 26, 2016, 1:38 a.m. UTC 
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

Comments

Stefano Stabellini Feb. 26, 2016, 11:51 a.m. UTC | #1 
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
>
Julien Grall March 15, 2016, 5:47 p.m. UTC | #2 
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
>
Konrad Rzeszutek Wilk March 15, 2016, 6:18 p.m. UTC | #3 
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
Jan Beulich March 16, 2016, 10:19 a.m. UTC | #4 
>>> 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
diff mbox

Patch

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:
+ */