Re: [PATCH v2 06/22] usb: common: add API to get usb otg features from device tree

[Roger Quadros <rogerq-l0cyMroinI0@public.gmane.org>]

Hi,

On Mon, 8 Jun 2015 23:02:01 +0800
Li Jun <jun.li-KZfg59tc24xl57MIdRCFDg@public.gmane.org> wrote:

> Check property of usb hardware to get otg version and if SRP, HNP and ADP
> are supported.
> 
> Signed-off-by: Li Jun <jun.li-KZfg59tc24xl57MIdRCFDg@public.gmane.org>
> ---
>  drivers/usb/common/common.c | 64 +++++++++++++++++++++++++++++++++++++++++++++
>  include/linux/usb/of.h      | 20 ++++++++++++++
>  2 files changed, 84 insertions(+)
> 
> diff --git a/drivers/usb/common/common.c b/drivers/usb/common/common.c
> index b530fd4..7b0594b 100644
> --- a/drivers/usb/common/common.c
> +++ b/drivers/usb/common/common.c
> @@ -154,6 +154,70 @@ bool of_usb_host_tpl_support(struct device_node *np)
>  	return false;
>  }
>  EXPORT_SYMBOL_GPL(of_usb_host_tpl_support);
> +
> +/**
> + * of_usb_otg_srp_support -  to get if otg srp is supported by the otg controller

This is not really about otg controller supporting SRP but whether
the DT user wants it enabled or not.

IMO this function should be named of_usb_get_otg_srp_enable()

> + * @np: Pointer to the given device_node
> + *
> + * The function gets if the otg port supports SRP
> + */
> +bool of_usb_otg_srp_support(struct device_node *np)
> +{
> +	if (of_find_property(np, "srp-support", NULL))
> +		return true;
> +
> +	return false;
> +}
> +EXPORT_SYMBOL_GPL(of_usb_otg_srp_support);
> +
> +/**
> + * of_usb_otg_hnp_support -  to get if otg hnp is supported by the otg controller
> + * @np: Pointer to the given device_node
> + *
> + * The function gets if the otg port supports HNP
> + */
> +bool of_usb_otg_hnp_support(struct device_node *np)
> +{
> +	if (of_find_property(np, "hnp-support", NULL))
> +		return true;
> +
> +	return false;
> +}
> +EXPORT_SYMBOL_GPL(of_usb_otg_hnp_support);
> +
> +/**
> + * of_usb_otg_adp_support -  to get if otg adp is supported by the otg controller
> + * @np: Pointer to the given device_node
> + *
> + * The function gets if the otg port supports ADP
> + */
> +bool of_usb_otg_adp_support(struct device_node *np)
> +{
> +	if (of_find_property(np, "adp-support", NULL))
> +		return true;
> +
> +	return false;
> +}
> +EXPORT_SYMBOL_GPL(of_usb_otg_adp_support);
> +
> +/**
> + * of_usb_otg_rev_number -  to get the otg release number the device is
> + * compliant with.
> + * @np: Pointer to the given device_node
> + *
> + * The function gets OTG and EH supplement release number in binary-coded
> + * decimal (i.e. 2.0 is 0200H).
> + */
> +int of_usb_otg_rev_number(struct device_node *np)

how about of_usb_get_otg_rev()?

> +{
> +	u32 otg_rev;
> +
> +	if (!of_property_read_u32(np, "otg-rev", &otg_rev))
> +		return otg_rev;
> +
> +	return 0;
> +}
> +EXPORT_SYMBOL_GPL(of_usb_otg_rev_number);
>  #endif
>  

cheers,
-roger
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html