From: Rob Herring <robh@kernel.org>
To: Shannon Zhao <zhaoshenglong@huawei.com>
Cc: "linux-arm-kernel@lists.infradead.org"
<linux-arm-kernel@lists.infradead.org>,
Ard Biesheuvel <ard.biesheuvel@linaro.org>,
Stefano Stabellini <stefano.stabellini@citrix.com>,
david.vrabel@citrix.com,
Catalin Marinas <catalin.marinas@arm.com>,
Will Deacon <will.deacon@arm.com>,
Mark Rutland <mark.rutland@arm.com>,
julien.grall@citrix.com,
"xen-devel@lists.xen.org" <xen-devel@lists.xen.org>,
"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
"linux-efi@vger.kernel.org" <linux-efi@vger.kernel.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
Ian Campbell <ian.campbell@citrix.com>,
Shannon Zhao <shannon.zhao@linaro.org>,
peter.huangpeng@huawei.com
Subject: Re: [PATCH v4 16/17] FDT: Add a helper to get specified name subnode
Date: Wed, 24 Feb 2016 08:53:46 -0600 [thread overview]
Message-ID: <CAL_JsqJJx_agcQRa=vU25Db33t8LtaoBw+dZB5jS2N7LA2KX2g@mail.gmail.com> (raw)
In-Reply-To: <56CC3ED9.5000101@huawei.com>
On Tue, Feb 23, 2016 at 5:13 AM, Shannon Zhao <zhaoshenglong@huawei.com> wrote:
>
>
> On 2016/2/9 13:04, Rob Herring wrote:
>> On Thu, Feb 4, 2016 at 9:05 PM, Shannon Zhao <zhaoshenglong@huawei.com> wrote:
>>> From: Shannon Zhao <shannon.zhao@linaro.org>
>>>
>>> Sometimes it needs to check if there is a node in FDT by full path.
>>
>> I'm confused. Are you searching by full path or...
>>
>>> Introduce this helper to get the specified name subnode if it exists.
>>
>> name of sub node?
>>
> get the offset of sub node by specified name.
>
>> Either way, fdt_subnode_offset or fdt_path_offset doesn't work?
>>
> Ah，right，both of them should work. Does it need to add a wrapper in
> drivers/of/fdt.c for them or directly use them with the parameter
> initial_boot_params?
Add a wrapper in fdt.c.
Rob
WARNING: multiple messages have this Message-ID (diff)
From: Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
To: Shannon Zhao <zhaoshenglong-hv44wF8Li93QT0dZR+AlfA@public.gmane.org>
Cc: "linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org"
<linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org>,
Ard Biesheuvel
<ard.biesheuvel-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>,
Stefano Stabellini
<stefano.stabellini-Sxgqhf6Nn4DQT0dZR+AlfA@public.gmane.org>,
david.vrabel-Sxgqhf6Nn4DQT0dZR+AlfA@public.gmane.org,
Catalin Marinas <catalin.marinas-5wv7dgnIgG8@public.gmane.org>,
Will Deacon <will.deacon-5wv7dgnIgG8@public.gmane.org>,
Mark Rutland <mark.rutland-5wv7dgnIgG8@public.gmane.org>,
julien.grall-Sxgqhf6Nn4DQT0dZR+AlfA@public.gmane.org,
"xen-devel-GuqFBffKawuEi8DpZVb4nw@public.gmane.org"
<xen-devel-GuqFBffKawuEi8DpZVb4nw@public.gmane.org>,
"devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
<devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
"linux-efi-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
<linux-efi-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
"linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
<linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
Ian Campbell
<ian.campbell-Sxgqhf6Nn4DQT0dZR+AlfA@public.gmane.org>,
Shannon Zhao
<shannon.zhao-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>,
peter.huangpeng-hv44wF8Li93QT0dZR+AlfA@public.gmane.org
Subject: Re: [PATCH v4 16/17] FDT: Add a helper to get specified name subnode
Date: Wed, 24 Feb 2016 08:53:46 -0600 [thread overview]
Message-ID: <CAL_JsqJJx_agcQRa=vU25Db33t8LtaoBw+dZB5jS2N7LA2KX2g@mail.gmail.com> (raw)
In-Reply-To: <56CC3ED9.5000101-hv44wF8Li93QT0dZR+AlfA@public.gmane.org>
On Tue, Feb 23, 2016 at 5:13 AM, Shannon Zhao <zhaoshenglong-hv44wF8Li93QT0dZR+AlfA@public.gmane.org> wrote:
>
>
> On 2016/2/9 13:04, Rob Herring wrote:
>> On Thu, Feb 4, 2016 at 9:05 PM, Shannon Zhao <zhaoshenglong@huawei.com> 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.
>>
>> I'm confused. Are you searching by full path or...
>>
>>> Introduce this helper to get the specified name subnode if it exists.
>>
>> name of sub node?
>>
> get the offset of sub node by specified name.
>
>> Either way, fdt_subnode_offset or fdt_path_offset doesn't work?
>>
> Ah，right，both of them should work. Does it need to add a wrapper in
> drivers/of/fdt.c for them or directly use them with the parameter
> initial_boot_params?
Add a wrapper in fdt.c.
Rob
WARNING: multiple messages have this Message-ID (diff)
From: robh@kernel.org (Rob Herring)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v4 16/17] FDT: Add a helper to get specified name subnode
Date: Wed, 24 Feb 2016 08:53:46 -0600 [thread overview]
Message-ID: <CAL_JsqJJx_agcQRa=vU25Db33t8LtaoBw+dZB5jS2N7LA2KX2g@mail.gmail.com> (raw)
In-Reply-To: <56CC3ED9.5000101@huawei.com>
On Tue, Feb 23, 2016 at 5:13 AM, Shannon Zhao <zhaoshenglong@huawei.com> wrote:
>
>
> On 2016/2/9 13:04, Rob Herring wrote:
>> On Thu, Feb 4, 2016 at 9:05 PM, Shannon Zhao <zhaoshenglong@huawei.com> wrote:
>>> From: Shannon Zhao <shannon.zhao@linaro.org>
>>>
>>> Sometimes it needs to check if there is a node in FDT by full path.
>>
>> I'm confused. Are you searching by full path or...
>>
>>> Introduce this helper to get the specified name subnode if it exists.
>>
>> name of sub node?
>>
> get the offset of sub node by specified name.
>
>> Either way, fdt_subnode_offset or fdt_path_offset doesn't work?
>>
> Ah?right?both of them should work. Does it need to add a wrapper in
> drivers/of/fdt.c for them or directly use them with the parameter
> initial_boot_params?
Add a wrapper in fdt.c.
Rob
next prev parent reply other threads:[~2016-02-24 14:54 UTC|newest]
Thread overview: 127+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-02-05 3:05 [PATCH v4 00/17] Add ACPI support for Xen Dom0 on ARM64 Shannon Zhao
2016-02-05 3:05 ` Shannon Zhao
2016-02-05 3:05 ` Shannon Zhao
2016-02-05 3:05 ` [PATCH v4 01/17] Xen: ACPI: Hide UART used by Xen Shannon Zhao
2016-02-05 3:05 ` Shannon Zhao
2016-02-05 3:05 ` Shannon Zhao
2016-02-05 3:05 ` Shannon Zhao
2016-02-06 10:23 ` Rafael J. Wysocki
2016-02-06 10:23 ` Rafael J. Wysocki
2016-02-06 10:23 ` Rafael J. Wysocki
2016-02-06 10:23 ` Rafael J. Wysocki
2016-02-08 10:57 ` Stefano Stabellini
[not found] ` <CAJZ5v0id-035AtyQ_SccPuCxD7+pa6GCgj3WBAojM+rFqssGQQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2016-02-08 10:57 ` Stefano Stabellini
2016-02-08 10:57 ` Stefano Stabellini
2016-02-08 10:57 ` Stefano Stabellini
[not found] ` <alpine.DEB.2.02.1602081054350.27008-7Z66fg9igcxYtxbxJUhB2Dgeux46jI+i@public.gmane.org>
2016-02-08 22:25 ` Rafael J. Wysocki
2016-02-08 22:25 ` Rafael J. Wysocki
2016-02-08 22:25 ` Rafael J. Wysocki
2016-02-09 11:19 ` Stefano Stabellini
2016-02-09 11:19 ` Stefano Stabellini
2016-02-09 11:19 ` Stefano Stabellini
2016-02-09 11:19 ` Stefano Stabellini
2016-02-10 22:29 ` Rafael J. Wysocki
2016-02-10 22:29 ` Rafael J. Wysocki
2016-02-10 22:29 ` Rafael J. Wysocki
2016-02-10 22:29 ` Rafael J. Wysocki
[not found] ` <44113385.2QnPtvxQx0-sKB8Sp2ER+y1GS7QM15AGw@public.gmane.org>
2016-02-11 16:04 ` Stefano Stabellini
2016-02-11 16:04 ` Stefano Stabellini
2016-02-11 16:04 ` Stefano Stabellini
[not found] ` <alpine.DEB.2.02.1602111557430.27008-7Z66fg9igcxYtxbxJUhB2Dgeux46jI+i@public.gmane.org>
2016-02-11 22:22 ` Rafael J. Wysocki
2016-02-11 22:22 ` Rafael J. Wysocki
2016-02-11 22:22 ` Rafael J. Wysocki
2016-02-12 11:50 ` Stefano Stabellini
2016-02-29 2:02 ` Shannon Zhao
[not found] ` <1667812.hY9vsC2epc-sKB8Sp2ER+y1GS7QM15AGw@public.gmane.org>
2016-02-12 11:50 ` Stefano Stabellini
2016-02-12 11:50 ` Stefano Stabellini
2016-02-12 11:50 ` Stefano Stabellini
2016-02-12 12:57 ` Rafael J. Wysocki
[not found] ` <alpine.DEB.2.02.1602121138090.27008-7Z66fg9igcxYtxbxJUhB2Dgeux46jI+i@public.gmane.org>
2016-02-12 12:57 ` Rafael J. Wysocki
2016-02-12 12:57 ` Rafael J. Wysocki
2016-02-12 12:57 ` Rafael J. Wysocki
2016-02-29 2:02 ` Shannon Zhao
2016-02-29 2:02 ` Shannon Zhao
2016-02-29 2:02 ` Shannon Zhao
2016-02-11 22:22 ` Rafael J. Wysocki
2016-02-11 16:04 ` Stefano Stabellini
2016-02-08 22:25 ` Rafael J. Wysocki
2016-02-05 3:05 ` [PATCH v4 02/17] xen/grant-table: Move xlated_setup_gnttab_pages to common place Shannon Zhao
2016-02-05 3:05 ` Shannon Zhao
2016-02-05 3:05 ` Shannon Zhao
2016-02-05 3:05 ` Shannon Zhao
2016-02-05 3:05 ` [PATCH v4 03/17] Xen: xlate: Use page_to_xen_pfn instead of page_to_pfn Shannon Zhao
2016-02-05 3:05 ` Shannon Zhao
2016-02-05 3:05 ` Shannon Zhao
2016-02-05 3:05 ` Shannon Zhao
2016-02-05 3:05 ` [PATCH v4 04/17] arm/xen: Use xen_xlate_map_ballooned_pages to setup grant table Shannon Zhao
2016-02-05 3:05 ` Shannon Zhao
2016-02-05 3:05 ` Shannon Zhao
2016-02-05 3:05 ` [PATCH v4 05/17] xen: memory : Add new XENMAPSPACE type XENMAPSPACE_dev_mmio Shannon Zhao
2016-02-05 3:05 ` Shannon Zhao
2016-02-05 3:05 ` Shannon Zhao
2016-02-08 11:43 ` Stefano Stabellini
2016-02-08 11:43 ` Stefano Stabellini
2016-02-08 11:43 ` Stefano Stabellini
2016-02-08 11:43 ` Stefano Stabellini
2016-02-05 3:05 ` [PATCH v4 06/17] Xen: ARM: Add support for mapping platform device mmio Shannon Zhao
2016-02-05 3:05 ` Shannon Zhao
2016-02-05 3:05 ` Shannon Zhao
2016-02-05 3:05 ` [PATCH v4 07/17] Xen: ARM: Add support for mapping AMBA " Shannon Zhao
2016-02-05 3:05 ` Shannon Zhao
2016-02-05 3:05 ` Shannon Zhao
2016-02-05 3:05 ` [PATCH v4 08/17] Xen: public/hvm: sync changes of HVM_PARAM_CALLBACK_VIA ABI from Xen Shannon Zhao
2016-02-05 3:05 ` Shannon Zhao
2016-02-05 3:05 ` Shannon Zhao
2016-02-05 3:05 ` Shannon Zhao
2016-02-05 3:05 ` [PATCH v4 09/17] xen/hvm/params: Add a new delivery type for event-channel in HVM_PARAM_CALLBACK_IRQ Shannon Zhao
2016-02-05 3:05 ` Shannon Zhao
2016-02-05 3:05 ` Shannon Zhao
2016-02-05 3:05 ` [PATCH v4 10/17] arm/xen: Get event-channel irq through HVM_PARAM when booting with ACPI Shannon Zhao
2016-02-05 3:05 ` Shannon Zhao
2016-02-05 3:05 ` Shannon Zhao
2016-02-05 3:05 ` [PATCH v4 11/17] ARM: XEN: Move xen_early_init() before efi_init() Shannon Zhao
2016-02-05 3:05 ` Shannon Zhao
2016-02-05 3:05 ` Shannon Zhao
2016-02-05 3:05 ` [PATCH v4 12/17] ARM64: ACPI: Check if it runs on Xen to enable or disable ACPI Shannon Zhao
2016-02-05 3:05 ` Shannon Zhao
2016-02-05 3:05 ` Shannon Zhao
2016-02-05 3:05 ` Shannon Zhao
2016-02-05 3:05 ` [PATCH v4 13/17] ARM: Xen: Document UEFI support on Xen ARM virtual platforms Shannon Zhao
2016-02-05 3:05 ` Shannon Zhao
2016-02-05 3:05 ` Shannon Zhao
2016-02-08 11:31 ` Stefano Stabellini
2016-02-08 11:31 ` Stefano Stabellini
2016-02-08 11:31 ` Stefano Stabellini
2016-02-08 11:31 ` Stefano Stabellini
2016-02-05 3:05 ` [PATCH v4 14/17] XEN: EFI: Move x86 specific codes to architecture directory Shannon Zhao
2016-02-05 3:05 ` Shannon Zhao
2016-02-05 3:05 ` Shannon Zhao
2016-02-05 3:05 ` Shannon Zhao
2016-02-05 3:05 ` [PATCH v4 15/17] ARM64: XEN: Add a function to initialize Xen specific UEFI runtime services Shannon Zhao
2016-02-05 3:05 ` Shannon Zhao
2016-02-05 3:05 ` Shannon Zhao
2016-02-05 3:05 ` Shannon Zhao
2016-02-05 3:05 ` [PATCH v4 16/17] FDT: Add a helper to get specified name subnode Shannon Zhao
2016-02-05 3:05 ` Shannon Zhao
2016-02-05 3:05 ` Shannon Zhao
2016-02-05 14:15 ` Stefano Stabellini
2016-02-05 14:15 ` Stefano Stabellini
2016-02-05 14:15 ` Stefano Stabellini
2016-02-05 14:15 ` Stefano Stabellini
2016-02-09 5:04 ` Rob Herring
2016-02-09 5:04 ` Rob Herring
2016-02-09 5:04 ` Rob Herring
2016-02-23 11:13 ` Shannon Zhao
2016-02-23 11:13 ` Shannon Zhao
2016-02-23 11:13 ` Shannon Zhao
2016-02-23 11:13 ` Shannon Zhao
2016-02-24 14:53 ` Rob Herring [this message]
2016-02-24 14:53 ` Rob Herring
2016-02-24 14:53 ` Rob Herring
2016-02-24 14:53 ` Rob Herring
2016-02-09 5:04 ` Rob Herring
2016-02-05 3:05 ` Shannon Zhao
2016-02-05 3:05 ` [PATCH v4 17/17] Xen: EFI: Parse DT parameters for Xen specific UEFI Shannon Zhao
2016-02-05 3:05 ` Shannon Zhao
2016-02-05 3:05 ` Shannon Zhao
2016-02-05 3:05 ` Shannon Zhao
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='CAL_JsqJJx_agcQRa=vU25Db33t8LtaoBw+dZB5jS2N7LA2KX2g@mail.gmail.com' \
--to=robh@kernel.org \
--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=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.