All of lore.kernel.org
 help / color / mirror / Atom feed
From: Stefano Stabellini <stefano.stabellini@eu.citrix.com>
To: Shannon Zhao <zhaoshenglong@huawei.com>
Cc: <linux-arm-kernel@lists.infradead.org>,
	<ard.biesheuvel@linaro.org>, <mark.rutland@arm.com>,
	<stefano.stabellini@citrix.com>, <david.vrabel@citrix.com>,
	<catalin.marinas@arm.com>, <will.deacon@arm.com>,
	<julien.grall@citrix.com>, <xen-devel@lists.xen.org>,
	<devicetree@vger.kernel.org>, <linux-efi@vger.kernel.org>,
	<linux-kernel@vger.kernel.org>, <ian.campbell@citrix.com>,
	<shannon.zhao@linaro.org>, <peter.huangpeng@huawei.com>,
	Rob Herring <robh@kernel.org>
Subject: Re: [PATCH v3 16/17] FDT: Add a helper to get specified name subnode
Date: Tue, 26 Jan 2016 12:11:24 +0000	[thread overview]
Message-ID: <alpine.DEB.2.02.1601261156340.3619@kaball.uk.xensource.com> (raw)
In-Reply-To: <1453519184-11908-17-git-send-email-zhaoshenglong@huawei.com>

On Sat, 23 Jan 2016, Shannon Zhao wrote:
> From: Shannon Zhao <shannon.zhao@linaro.org>
> 
> Sometimes it needs to check if there is a node in FDT by full path.
> Introduce this helper to get the specified name subnode if it exists.
> 
> Signed-off-by: Shannon Zhao <shannon.zhao@linaro.org>
> ---
> CC: Rob Herring <robh@kernel.org>
> ---
>  drivers/of/fdt.c       | 35 +++++++++++++++++++++++++++++++++++
>  include/linux/of_fdt.h |  2 ++
>  2 files changed, 37 insertions(+)
> 
> diff --git a/drivers/of/fdt.c b/drivers/of/fdt.c
> index 655f79d..112ec16 100644
> --- a/drivers/of/fdt.c
> +++ b/drivers/of/fdt.c
> @@ -645,6 +645,41 @@ int __init of_scan_flat_dt(int (*it)(unsigned long node,
>  }
>  
>  /**
> + * of_get_flat_dt_subnode_by_name - get subnode of specified node by name
> + *
> + * @node: the parent node
> + * @uname: the name of subnode
> + * @return offset of the subnode, or -FDT_ERR_NOTFOUND if there is none
> + */
> +
> +int of_get_flat_dt_subnode_by_name(unsigned long node, const char *uname)
> +{
> +	const void *blob = initial_boot_params;
> +	int offset;
> +	const char *pathp;
> +
> +	/* Find first subnode if it exists */
> +	offset = fdt_first_subnode(blob, node);
> +	if (offset < 0)
> +		return -FDT_ERR_NOTFOUND;
> +	pathp = fdt_get_name(blob, offset, NULL);
> +	if (strncmp(pathp, uname, strlen(uname)) == 0)
> +		return offset;

Wouldn't this check succeed even if uname is "uefi" and the node
name is actually "uefiiiii"?  You might have to use strcmp.


> +	/* Find other subnodes */
> +	do {
> +		offset = fdt_next_subnode(blob, offset);
> +		if (offset < 0)
> +			return -FDT_ERR_NOTFOUND;
> +		pathp = fdt_get_name(blob, offset, NULL);
> +		if (strncmp(pathp, uname, strlen(uname)) == 0)
> +			return offset;
> +	} while (offset >= 0);

Rather than writing the name check twice, I think it would be best to
code this loop as:

    for (offset = fdt_first_subnode(blob, offset);
         offset >= 0;
         offset = fdt_next_subnode(blob, offset)) {

         /* do name check */



> +	return -FDT_ERR_NOTFOUND;
> +}
> +
> +/**
>   * of_get_flat_dt_root - find the root node in the flat blob
>   */
>  unsigned long __init of_get_flat_dt_root(void)
> diff --git a/include/linux/of_fdt.h b/include/linux/of_fdt.h
> index df9ef38..fc28162 100644
> --- a/include/linux/of_fdt.h
> +++ b/include/linux/of_fdt.h
> @@ -52,6 +52,8 @@ extern char __dtb_end[];
>  extern int of_scan_flat_dt(int (*it)(unsigned long node, const char *uname,
>  				     int depth, void *data),
>  			   void *data);
> +extern int of_get_flat_dt_subnode_by_name(unsigned long node,
> +					  const char *uname);
>  extern const void *of_get_flat_dt_prop(unsigned long node, const char *name,
>  				       int *size);
>  extern int of_flat_dt_is_compatible(unsigned long node, const char *name);
> -- 
> 2.0.4
> 
> 

WARNING: multiple messages have this Message-ID (diff)
From: Stefano Stabellini <stefano.stabellini-mvvWK6WmYclDPfheJLI6IQ@public.gmane.org>
To: Shannon Zhao <zhaoshenglong-hv44wF8Li93QT0dZR+AlfA@public.gmane.org>
Cc: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
	ard.biesheuvel-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org,
	mark.rutland-5wv7dgnIgG8@public.gmane.org,
	stefano.stabellini-Sxgqhf6Nn4DQT0dZR+AlfA@public.gmane.org,
	david.vrabel-Sxgqhf6Nn4DQT0dZR+AlfA@public.gmane.org,
	catalin.marinas-5wv7dgnIgG8@public.gmane.org,
	will.deacon-5wv7dgnIgG8@public.gmane.org,
	julien.grall-Sxgqhf6Nn4DQT0dZR+AlfA@public.gmane.org,
	xen-devel-GuqFBffKawuEi8DpZVb4nw@public.gmane.org,
	devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linux-efi-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	ian.campbell-Sxgqhf6Nn4DQT0dZR+AlfA@public.gmane.org,
	shannon.zhao-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org,
	peter.huangpeng-hv44wF8Li93QT0dZR+AlfA@public.gmane.org,
	Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
Subject: Re: [PATCH v3 16/17] FDT: Add a helper to get specified name subnode
Date: Tue, 26 Jan 2016 12:11:24 +0000	[thread overview]
Message-ID: <alpine.DEB.2.02.1601261156340.3619@kaball.uk.xensource.com> (raw)
In-Reply-To: <1453519184-11908-17-git-send-email-zhaoshenglong-hv44wF8Li93QT0dZR+AlfA@public.gmane.org>

On Sat, 23 Jan 2016, Shannon Zhao wrote:
> From: Shannon Zhao <shannon.zhao-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
> 
> Sometimes it needs to check if there is a node in FDT by full path.
> Introduce this helper to get the specified name subnode if it exists.
> 
> Signed-off-by: Shannon Zhao <shannon.zhao-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
> ---
> CC: Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
> ---
>  drivers/of/fdt.c       | 35 +++++++++++++++++++++++++++++++++++
>  include/linux/of_fdt.h |  2 ++
>  2 files changed, 37 insertions(+)
> 
> diff --git a/drivers/of/fdt.c b/drivers/of/fdt.c
> index 655f79d..112ec16 100644
> --- a/drivers/of/fdt.c
> +++ b/drivers/of/fdt.c
> @@ -645,6 +645,41 @@ int __init of_scan_flat_dt(int (*it)(unsigned long node,
>  }
>  
>  /**
> + * of_get_flat_dt_subnode_by_name - get subnode of specified node by name
> + *
> + * @node: the parent node
> + * @uname: the name of subnode
> + * @return offset of the subnode, or -FDT_ERR_NOTFOUND if there is none
> + */
> +
> +int of_get_flat_dt_subnode_by_name(unsigned long node, const char *uname)
> +{
> +	const void *blob = initial_boot_params;
> +	int offset;
> +	const char *pathp;
> +
> +	/* Find first subnode if it exists */
> +	offset = fdt_first_subnode(blob, node);
> +	if (offset < 0)
> +		return -FDT_ERR_NOTFOUND;
> +	pathp = fdt_get_name(blob, offset, NULL);
> +	if (strncmp(pathp, uname, strlen(uname)) == 0)
> +		return offset;

Wouldn't this check succeed even if uname is "uefi" and the node
name is actually "uefiiiii"?  You might have to use strcmp.


> +	/* Find other subnodes */
> +	do {
> +		offset = fdt_next_subnode(blob, offset);
> +		if (offset < 0)
> +			return -FDT_ERR_NOTFOUND;
> +		pathp = fdt_get_name(blob, offset, NULL);
> +		if (strncmp(pathp, uname, strlen(uname)) == 0)
> +			return offset;
> +	} while (offset >= 0);

Rather than writing the name check twice, I think it would be best to
code this loop as:

    for (offset = fdt_first_subnode(blob, offset);
         offset >= 0;
         offset = fdt_next_subnode(blob, offset)) {

         /* do name check */



> +	return -FDT_ERR_NOTFOUND;
> +}
> +
> +/**
>   * of_get_flat_dt_root - find the root node in the flat blob
>   */
>  unsigned long __init of_get_flat_dt_root(void)
> diff --git a/include/linux/of_fdt.h b/include/linux/of_fdt.h
> index df9ef38..fc28162 100644
> --- a/include/linux/of_fdt.h
> +++ b/include/linux/of_fdt.h
> @@ -52,6 +52,8 @@ extern char __dtb_end[];
>  extern int of_scan_flat_dt(int (*it)(unsigned long node, const char *uname,
>  				     int depth, void *data),
>  			   void *data);
> +extern int of_get_flat_dt_subnode_by_name(unsigned long node,
> +					  const char *uname);
>  extern const void *of_get_flat_dt_prop(unsigned long node, const char *name,
>  				       int *size);
>  extern int of_flat_dt_is_compatible(unsigned long node, const char *name);
> -- 
> 2.0.4
> 
> 
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

WARNING: multiple messages have this Message-ID (diff)
From: stefano.stabellini@eu.citrix.com (Stefano Stabellini)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v3 16/17] FDT: Add a helper to get specified name subnode
Date: Tue, 26 Jan 2016 12:11:24 +0000	[thread overview]
Message-ID: <alpine.DEB.2.02.1601261156340.3619@kaball.uk.xensource.com> (raw)
In-Reply-To: <1453519184-11908-17-git-send-email-zhaoshenglong@huawei.com>

On Sat, 23 Jan 2016, Shannon Zhao wrote:
> From: Shannon Zhao <shannon.zhao@linaro.org>
> 
> Sometimes it needs to check if there is a node in FDT by full path.
> Introduce this helper to get the specified name subnode if it exists.
> 
> Signed-off-by: Shannon Zhao <shannon.zhao@linaro.org>
> ---
> CC: Rob Herring <robh@kernel.org>
> ---
>  drivers/of/fdt.c       | 35 +++++++++++++++++++++++++++++++++++
>  include/linux/of_fdt.h |  2 ++
>  2 files changed, 37 insertions(+)
> 
> diff --git a/drivers/of/fdt.c b/drivers/of/fdt.c
> index 655f79d..112ec16 100644
> --- a/drivers/of/fdt.c
> +++ b/drivers/of/fdt.c
> @@ -645,6 +645,41 @@ int __init of_scan_flat_dt(int (*it)(unsigned long node,
>  }
>  
>  /**
> + * of_get_flat_dt_subnode_by_name - get subnode of specified node by name
> + *
> + * @node: the parent node
> + * @uname: the name of subnode
> + * @return offset of the subnode, or -FDT_ERR_NOTFOUND if there is none
> + */
> +
> +int of_get_flat_dt_subnode_by_name(unsigned long node, const char *uname)
> +{
> +	const void *blob = initial_boot_params;
> +	int offset;
> +	const char *pathp;
> +
> +	/* Find first subnode if it exists */
> +	offset = fdt_first_subnode(blob, node);
> +	if (offset < 0)
> +		return -FDT_ERR_NOTFOUND;
> +	pathp = fdt_get_name(blob, offset, NULL);
> +	if (strncmp(pathp, uname, strlen(uname)) == 0)
> +		return offset;

Wouldn't this check succeed even if uname is "uefi" and the node
name is actually "uefiiiii"?  You might have to use strcmp.


> +	/* Find other subnodes */
> +	do {
> +		offset = fdt_next_subnode(blob, offset);
> +		if (offset < 0)
> +			return -FDT_ERR_NOTFOUND;
> +		pathp = fdt_get_name(blob, offset, NULL);
> +		if (strncmp(pathp, uname, strlen(uname)) == 0)
> +			return offset;
> +	} while (offset >= 0);

Rather than writing the name check twice, I think it would be best to
code this loop as:

    for (offset = fdt_first_subnode(blob, offset);
         offset >= 0;
         offset = fdt_next_subnode(blob, offset)) {

         /* do name check */



> +	return -FDT_ERR_NOTFOUND;
> +}
> +
> +/**
>   * of_get_flat_dt_root - find the root node in the flat blob
>   */
>  unsigned long __init of_get_flat_dt_root(void)
> diff --git a/include/linux/of_fdt.h b/include/linux/of_fdt.h
> index df9ef38..fc28162 100644
> --- a/include/linux/of_fdt.h
> +++ b/include/linux/of_fdt.h
> @@ -52,6 +52,8 @@ extern char __dtb_end[];
>  extern int of_scan_flat_dt(int (*it)(unsigned long node, const char *uname,
>  				     int depth, void *data),
>  			   void *data);
> +extern int of_get_flat_dt_subnode_by_name(unsigned long node,
> +					  const char *uname);
>  extern const void *of_get_flat_dt_prop(unsigned long node, const char *name,
>  				       int *size);
>  extern int of_flat_dt_is_compatible(unsigned long node, const char *name);
> -- 
> 2.0.4
> 
> 

  reply	other threads:[~2016-01-26 12:12 UTC|newest]

Thread overview: 144+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-01-23  3:19 [PATCH v3 00/17] Add ACPI support for Xen Dom0 on ARM64 Shannon Zhao
2016-01-23  3:19 ` Shannon Zhao
2016-01-23  3:19 ` Shannon Zhao
2016-01-23  3:19 ` [PATCH v3 01/17] Xen: ACPI: Hide UART used by Xen Shannon Zhao
2016-01-23  3:19 ` Shannon Zhao
2016-01-23  3:19   ` Shannon Zhao
2016-01-23  3:19   ` Shannon Zhao
2016-01-25 12:20   ` Stefano Stabellini
2016-01-25 12:20     ` Stefano Stabellini
2016-01-25 12:20     ` Stefano Stabellini
2016-01-25 12:20   ` Stefano Stabellini
2016-01-23  3:19 ` [PATCH v3 02/17] xen/grant-table: Move xlated_setup_gnttab_pages to common place Shannon Zhao
2016-01-23  3:19   ` Shannon Zhao
2016-01-23  3:19   ` Shannon Zhao
2016-01-23  3:19 ` [PATCH v3 03/17] Xen: xlate: Use page_to_xen_pfn instead of page_to_pfn Shannon Zhao
2016-01-23  3:19 ` Shannon Zhao
2016-01-23  3:19   ` Shannon Zhao
2016-01-23  3:19   ` Shannon Zhao
2016-01-25 13:55   ` Stefano Stabellini
2016-01-25 13:55   ` Stefano Stabellini
2016-01-25 13:55     ` Stefano Stabellini
2016-01-25 13:55     ` Stefano Stabellini
2016-01-23  3:19 ` [PATCH v3 04/17] arm/xen: Use xen_xlate_map_ballooned_pages to setup grant table Shannon Zhao
2016-01-23  3:19 ` Shannon Zhao
2016-01-23  3:19   ` Shannon Zhao
2016-01-23  3:19   ` Shannon Zhao
2016-01-23  3:19 ` [PATCH v3 05/17] xen: memory : Add new XENMAPSPACE type XENMAPSPACE_dev_mmio Shannon Zhao
2016-01-23  3:19 ` Shannon Zhao
2016-01-23  3:19   ` Shannon Zhao
2016-01-23  3:19   ` Shannon Zhao
2016-01-23  3:19 ` [PATCH v3 06/17] Xen: ARM: Add support for mapping platform device mmio Shannon Zhao
2016-01-23  3:19   ` Shannon Zhao
2016-01-23  3:19   ` Shannon Zhao
2016-01-23  3:19 ` [PATCH v3 07/17] Xen: ARM: Add support for mapping AMBA " Shannon Zhao
2016-01-23  3:19   ` Shannon Zhao
2016-01-23  3:19   ` Shannon Zhao
2016-01-23  3:19 ` [PATCH v3 08/17] Xen: public/hvm: sync changes of HVM_PARAM_CALLBACK_VIA ABI from Xen Shannon Zhao
2016-01-23  3:19   ` Shannon Zhao
2016-01-23  3:19   ` Shannon Zhao
2016-01-25 15:06   ` Stefano Stabellini
2016-01-25 15:06     ` Stefano Stabellini
2016-01-25 15:06     ` Stefano Stabellini
2016-01-25 15:06   ` Stefano Stabellini
2016-01-23  3:19 ` [PATCH v3 09/17] xen/hvm/params: Add a new delivery type for event-channel in HVM_PARAM_CALLBACK_IRQ Shannon Zhao
2016-01-23  3:19   ` Shannon Zhao
2016-01-23  3:19   ` Shannon Zhao
2016-01-25 15:08   ` Stefano Stabellini
2016-01-25 15:08   ` Stefano Stabellini
2016-01-25 15:08     ` Stefano Stabellini
2016-01-25 15:08     ` Stefano Stabellini
2016-01-23  3:19 ` Shannon Zhao
2016-01-23  3:19 ` [PATCH v3 10/17] arm/xen: Get event-channel irq through HVM_PARAM when booting with ACPI Shannon Zhao
2016-01-23  3:19   ` Shannon Zhao
2016-01-23  3:19   ` Shannon Zhao
2016-01-25 15:19   ` Stefano Stabellini
2016-01-25 15:19   ` Stefano Stabellini
2016-01-25 15:19     ` Stefano Stabellini
2016-01-25 15:19     ` Stefano Stabellini
2016-01-23  3:19 ` [PATCH v3 11/17] ARM: XEN: Move xen_early_init() before efi_init() Shannon Zhao
2016-01-23  3:19 ` Shannon Zhao
2016-01-23  3:19   ` Shannon Zhao
2016-01-23  3:19   ` Shannon Zhao
2016-01-25 15:25   ` Stefano Stabellini
2016-01-25 15:25     ` Stefano Stabellini
2016-01-25 15:25     ` Stefano Stabellini
2016-01-25 15:25   ` Stefano Stabellini
2016-01-23  3:19 ` [PATCH v3 12/17] ARM64: ACPI: Check if it runs on Xen to enable or disable ACPI Shannon Zhao
2016-01-23  3:19 ` Shannon Zhao
2016-01-23  3:19   ` Shannon Zhao
2016-01-23  3:19   ` Shannon Zhao
2016-01-25 15:32   ` Stefano Stabellini
2016-01-25 15:32     ` Stefano Stabellini
2016-01-25 15:32     ` Stefano Stabellini
2016-01-25 15:32   ` Stefano Stabellini
2016-01-27  8:27   ` Hanjun Guo
2016-01-27  8:27     ` Hanjun Guo
2016-01-27  8:57     ` Shannon Zhao
2016-01-27  8:57       ` Shannon Zhao
2016-01-27  8:57       ` Shannon Zhao
2016-01-27  8:57     ` Shannon Zhao
2016-01-27  8:27   ` Hanjun Guo
2016-01-23  3:19 ` [PATCH v3 13/17] ARM: Xen: Document UEFI support on Xen ARM virtual platforms Shannon Zhao
2016-01-23  3:19 ` Shannon Zhao
2016-01-23  3:19   ` Shannon Zhao
2016-01-23  3:19   ` Shannon Zhao
2016-01-25 15:50   ` Stefano Stabellini
2016-01-25 15:50     ` Stefano Stabellini
2016-01-25 15:50     ` Stefano Stabellini
2016-01-25 15:59     ` Mark Rutland
2016-01-25 15:59       ` Mark Rutland
2016-01-25 15:59       ` Mark Rutland
2016-01-25 16:06       ` Stefano Stabellini
2016-01-25 16:06       ` Stefano Stabellini
2016-01-25 16:06         ` Stefano Stabellini
2016-01-25 16:06         ` Stefano Stabellini
2016-01-25 15:50   ` Stefano Stabellini
2016-01-23  3:19 ` [PATCH v3 14/17] XEN: EFI: Move x86 specific codes to architecture directory Shannon Zhao
2016-01-23  3:19   ` Shannon Zhao
2016-01-23  3:19   ` Shannon Zhao
2016-01-25 16:44   ` Stefano Stabellini
2016-01-25 16:44     ` Stefano Stabellini
2016-01-25 16:44     ` Stefano Stabellini
2016-01-26  8:16     ` Shannon Zhao
2016-01-26  8:16     ` Shannon Zhao
2016-01-26  8:16       ` Shannon Zhao
2016-01-26  8:16       ` Shannon Zhao
2016-01-26 11:31       ` Stefano Stabellini
2016-01-26 11:31         ` Stefano Stabellini
2016-01-26 11:31         ` Stefano Stabellini
2016-01-26 12:26         ` Shannon Zhao
2016-01-26 12:26         ` Shannon Zhao
2016-01-26 12:26           ` Shannon Zhao
2016-01-26 12:26           ` Shannon Zhao
2016-01-26 11:31       ` Stefano Stabellini
2016-01-25 16:44   ` Stefano Stabellini
2016-01-23  3:19 ` [PATCH v3 15/17] ARM64: XEN: Add a function to initialize Xen specific UEFI runtime services Shannon Zhao
2016-01-23  3:19   ` Shannon Zhao
2016-01-23  3:19   ` Shannon Zhao
2016-01-25 16:47   ` Stefano Stabellini
2016-01-25 16:47   ` Stefano Stabellini
2016-01-25 16:47     ` Stefano Stabellini
2016-01-25 16:47     ` Stefano Stabellini
2016-01-23  3:19 ` [PATCH v3 16/17] FDT: Add a helper to get specified name subnode Shannon Zhao
2016-01-23  3:19   ` Shannon Zhao
2016-01-23  3:19   ` Shannon Zhao
2016-01-26 12:11   ` Stefano Stabellini [this message]
2016-01-26 12:11     ` Stefano Stabellini
2016-01-26 12:11     ` Stefano Stabellini
2016-01-26 12:27     ` Shannon Zhao
2016-01-26 12:27     ` Shannon Zhao
2016-01-26 12:27       ` Shannon Zhao
2016-01-26 12:27       ` Shannon Zhao
2016-01-26 12:11   ` Stefano Stabellini
2016-01-23  3:19 ` [PATCH v3 17/17] Xen: EFI: Parse DT parameters for Xen specific UEFI Shannon Zhao
2016-01-23  3:19 ` Shannon Zhao
2016-01-23  3:19   ` Shannon Zhao
2016-01-23  3:19   ` Shannon Zhao
2016-01-26 11:41   ` Stefano Stabellini
2016-01-26 11:41     ` Stefano Stabellini
2016-01-26 11:41     ` Stefano Stabellini
2016-01-26 11:41   ` Stefano Stabellini
2016-01-26 12:05   ` Matt Fleming
2016-01-26 12:05     ` Matt Fleming
2016-01-26 12:05   ` Matt Fleming

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=alpine.DEB.2.02.1601261156340.3619@kaball.uk.xensource.com \
    --to=stefano.stabellini@eu.citrix.com \
    --cc=ard.biesheuvel@linaro.org \
    --cc=catalin.marinas@arm.com \
    --cc=david.vrabel@citrix.com \
    --cc=devicetree@vger.kernel.org \
    --cc=ian.campbell@citrix.com \
    --cc=julien.grall@citrix.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-efi@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mark.rutland@arm.com \
    --cc=peter.huangpeng@huawei.com \
    --cc=robh@kernel.org \
    --cc=shannon.zhao@linaro.org \
    --cc=stefano.stabellini@citrix.com \
    --cc=will.deacon@arm.com \
    --cc=xen-devel@lists.xen.org \
    --cc=zhaoshenglong@huawei.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.