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

RE: [RFCv4,07/35] plat/ofw: Export fdt_find_irq_parent_offset


  • To: Razvan Virtan <virtanrazvan@xxxxxxxxx>
  • From: Justin He <Justin.He@xxxxxxx>
  • Date: Tue, 11 May 2021 15:34:06 +0000
  • Accept-language: en-US, zh-CN
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=arm.com; dmarc=pass action=none header.from=arm.com; dkim=pass header.d=arm.com; arc=none
  • Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=r7u3knyj/4E/FhKVYjT1av13FHICvJ1dVg179keUfHk=; b=a4CabybQd5OXFRAY6jFgFtnFmcwyM5rUlqq4CWY2XDCxOyrGN3YFn880xHf0aQY8IRk+nxbxR85j9c4uGiQ1/amYMfm1eZOUxfsPlBl7CidoDLWusR/3bziOw3+VbZvEiiCdP+Vf+G8+fEghAOi/QcY5nWogjeA4lMldQMDSXC7VE+hpuGTpRjU0t0Bct2i13h7uqJOVmGjEKxCv174zWEvHFmm/mScvf9tyzJv2WrGxXM4mWJa4i5a2DsEn9UPnh6It2rbxI8ET7SzH47ozOtoatFgsvjN2tTb5EbdG8ebFV6oaRSSbCKD1lhZ8UcV24NXsIKr1oFAMUBEQ3aD5cQ==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=ejwpn6/dpyJiOqtQmJiJsqwVz/l67PhN3UNXbWm7i+T0cng1faxElJP3lmlqvSTrAvan6sAVwqcHUV4DcRJBkaclDudn6hYtm/2k9Q/GoF1n7kvW7VZB+3S9/JCv7FsKIyTwpKM5PAMbuU6RyuPvCIx9xVl5hqcDojxVIIFnHOubspO17os9NPapr8bbIXxGBmnYbIoYe+B6caBFeIYA8flW73NEDd2BlQtbKjB3DX2TLMTLIC5aZQREZG0y2kYRcVS6HpQMSDFEC420RAgq8ie/0MdcQxCRdsFioV38lNSB2NUZdfpL+FqhtB6fZ8N17WrgOUjVu1JyiBxD5WyKug==
  • Authentication-results-original: gmail.com; dkim=none (message not signed) header.d=none;gmail.com; dmarc=none action=none header.from=arm.com;
  • Cc: "minios-devel@xxxxxxxxxxxxx" <minios-devel@xxxxxxxxxxxxx>
  • Delivery-date: Tue, 11 May 2021 15:34:24 +0000
  • List-id: Mini-os development list <minios-devel.lists.xenproject.org>
  • Nodisclaimer: true
  • Original-authentication-results: gmail.com; dkim=none (message not signed) header.d=none;gmail.com; dmarc=none action=none header.from=arm.com;
  • Thread-index: AQHXMs2U3jrZqn5pdEKOxMotbMuJx6rekDGg
  • Thread-topic: [RFCv4,07/35] plat/ofw: Export fdt_find_irq_parent_offset

Hi Razvan

> -----Original Message-----
> From: Razvan Virtan <virtanrazvan@xxxxxxxxx>
> Sent: Friday, April 16, 2021 10:34 PM
> To: Justin He <Justin.He@xxxxxxx>
> Cc: minios-devel@xxxxxxxxxxxxx
> Subject: Re: [RFCv4,07/35] plat/ofw: Export fdt_find_irq_parent_offset
>
> Hi, Justin.
> I think the description for fdt_find_irq_parent_offset provided in the
> header file is wrong, since it describes `fdt_node_offset_by_phandle()`
> from `libfdt.h` instead of `fdt_find_irq_parent_offset()`.
>
> /**
>
> The second argument should be something like:
> @offset: offset of the node whose irq parent to find
>
>  * fdt_find_irq_parent_offset - find the irq parent offset
>  * @fdt: pointer to the device tree blob
>  * @phandle: phandle value
>  *
>
> I think the detailed description below should be removed or replaced with
> one
> that describes this function.
Indeed, sorry for the mistake.


--
Cheers,
Justin (Jia He)


>
>  * fdt_node_offset_by_phandle() returns the offset of the node
>  * which has the given phandle value.  If there is more than one node
>  * in the tree with the given phandle (an invalid tree), results are
>  * undefined.
>  *
>  * returns:
>  *    structure block offset of the located node (>= 0), on success
>  *    -FDT_ERR_NOTFOUND, no node with that phandle exists
>  *    -FDT_ERR_BADPHANDLE, given phandle value was invalid (0 or -1)
>  *    -FDT_ERR_BADMAGIC,
>  *    -FDT_ERR_BADVERSION,
>  *    -FDT_ERR_BADSTATE,
>  *    -FDT_ERR_BADSTRUCTURE, standard meanings
>  */
> int fdt_find_irq_parent_offset(const void *fdt, int offset);
>
>
> Reviewed-by: Razvan Virtan <virtanrazvan@xxxxxxxxx>
IMPORTANT NOTICE: The contents of this email and any attachments are 
confidential and may also be privileged. If you are not the intended recipient, 
please notify the sender immediately and do not disclose the contents to any 
other person, use it for any purpose, or store or copy the information in any 
medium. Thank you.



 


Rackspace

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