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

Re: [PATCH v4 06/11] xen/device-tree: Add dt_property_match_string helper


  • To: Rahul Singh <Rahul.Singh@xxxxxxx>
  • From: Bertrand Marquis <Bertrand.Marquis@xxxxxxx>
  • Date: Tue, 12 Jan 2021 11:38:52 +0000
  • Accept-language: en-GB, en-US
  • 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=LoZxgW472f6/pQHqq4yaW+/cizy4YLITAWYPP7qVNa8=; b=VcIE1Ege4LPV4HKgpiIHPb4bBoEk+ZTNvQG0Pl8z5iYmd41OdPbt0EvnPDe9DMySSPgAmKoqfBoJNinGPTPKQJEGz8yaATc0lnKmzyG43kJBAmnNAN7QeZo68gB7hNYfx+uZfhUv844AAiajo9Bq6UakC4VxnseI/sqXwgKDEun4BoxiYp/dtyatMqvaTYABYNpxzFfDg/l7dYb7gsNfzPcpnKxxAqat9X2vxGQN6xkNNjgYgXByluERE6Wvezx7nij4LexKPE9CzQzxFo86ITHRsS5oobcZeziNLsp4CSToxgTEjSAFle6GD5DFIgyog/kFsknB9OG9D/m/iu51GA==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=U/6xx2yCF42ou0Jg2rEUP5egGq2UBujaM9FKvlPWC39vTJdZcOWLgZk5Zqnnjtgf7xgdMNa2IT5WSC2XxYf5q3BHDO5neu7FNqx2f5iY4vvRZAmnp9ATLFl5vthtXXVeREeVKiSx6U7Sa/Hpjl7uu78WXSRlx2abuU/qeEAkZGrO/C/cFulOXL8jgnilw7zKg7GwiXwXBSSP00rM/UOKeheu7+fsEK9/kl2Ei4LasYstB5E67q5oGUy/n/wUtI2oqZilYuD14kZCn+eYLcYTkxW8MOfrk+iydbcNoQFzx8UTRoNIwO5ELQcxrXZRGoftFc6AUeDB/Pp34qQjqgoE0w==
  • Authentication-results-original: arm.com; dkim=none (message not signed) header.d=none;arm.com; dmarc=none action=none header.from=arm.com;
  • Cc: "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Julien Grall <julien@xxxxxxx>
  • Delivery-date: Tue, 12 Jan 2021 11:39:04 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
  • Nodisclaimer: true
  • Original-authentication-results: arm.com; dkim=none (message not signed) header.d=none;arm.com; dmarc=none action=none header.from=arm.com;
  • Thread-index: AQHW5c2nYMNMmoO8sUeHj5NB8HJo8qoj42UA
  • Thread-topic: [PATCH v4 06/11] xen/device-tree: Add dt_property_match_string helper

Hi,

> On 8 Jan 2021, at 14:46, Rahul Singh <Rahul.Singh@xxxxxxx> wrote:
> 
> Import the Linux helper of_property_match_string. This function searches
> a string list property and returns the index of a specific string value.
> 
> Signed-off-by: Rahul Singh <rahul.singh@xxxxxxx>
Reviewed-by: Bertrand Marquis <bertrand.marquis@xxxxxxx>

Cheers
Bertrand

> ---
> Changes in V3:
> - This patch is introduce in this verison.
> Changes in V4: Rebase
> ---
> xen/common/device_tree.c      | 27 +++++++++++++++++++++++++++
> xen/include/xen/device_tree.h | 12 ++++++++++++
> 2 files changed, 39 insertions(+)
> 
> diff --git a/xen/common/device_tree.c b/xen/common/device_tree.c
> index e107c6f89f..18825e333e 100644
> --- a/xen/common/device_tree.c
> +++ b/xen/common/device_tree.c
> @@ -208,6 +208,33 @@ int dt_property_read_string(const struct dt_device_node 
> *np,
>     return 0;
> }
> 
> +int dt_property_match_string(const struct dt_device_node *np,
> +                             const char *propname, const char *string)
> +{
> +    const struct dt_property *dtprop = dt_find_property(np, propname, NULL);
> +    size_t l;
> +    int i;
> +    const char *p, *end;
> +
> +    if ( !dtprop )
> +        return -EINVAL;
> +    if ( !dtprop->value )
> +        return -ENODATA;
> +
> +    p = dtprop->value;
> +    end = p + dtprop->length;
> +
> +    for ( i = 0; p < end; i++, p += l )
> +    {
> +        l = strnlen(p, end - p) + 1;
> +        if ( p + l > end )
> +            return -EILSEQ;
> +        if ( strcmp(string, p) == 0 )
> +            return i; /* Found it; return index */
> +    }
> +    return -ENODATA;
> +}
> +
> bool_t dt_device_is_compatible(const struct dt_device_node *device,
>                                const char *compat)
> {
> diff --git a/xen/include/xen/device_tree.h b/xen/include/xen/device_tree.h
> index f2ad22b79c..b02696be94 100644
> --- a/xen/include/xen/device_tree.h
> +++ b/xen/include/xen/device_tree.h
> @@ -400,6 +400,18 @@ static inline bool_t dt_property_read_bool(const struct 
> dt_device_node *np,
> int dt_property_read_string(const struct dt_device_node *np,
>                             const char *propname, const char **out_string);
> 
> +/**
> + * dt_property_match_string() - Find string in a list and return index
> + * @np: pointer to node containing string list property
> + * @propname: string list property name
> + * @string: pointer to string to search for in string list
> + *
> + * This function searches a string list property and returns the index
> + * of a specific string value.
> + */
> +int dt_property_match_string(const struct dt_device_node *np,
> +                             const char *propname, const char *string);
> +
> /**
>  * Checks if the given "compat" string matches one of the strings in
>  * the device's "compatible" property
> -- 
> 2.17.1
> 




 


Rackspace

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