On Wed, May 27, 2015 at 07:50:07PM +0800, Li Jun wrote:
> Check property of usb hardware to get if SRP, HNP and ADP is supported.
> 
> Signed-off-by: Li Jun <jun...@freescale.com>
> ---
>  drivers/usb/common/common.c | 45 
> +++++++++++++++++++++++++++++++++++++++++++++
>  include/linux/usb/of.h      | 15 +++++++++++++++
>  2 files changed, 60 insertions(+)
> 
> diff --git a/drivers/usb/common/common.c b/drivers/usb/common/common.c
> index b530fd4..9944bee 100644
> --- a/drivers/usb/common/common.c
> +++ b/drivers/usb/common/common.c
> @@ -154,6 +154,51 @@ 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
> + * @np: Pointer to the given device_node
> + *
> + * The function gets if the otg port support SRP
> + */

%s/support/supports

> +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 support HNP

%s/support/supports

> + */
> +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 support ADP

%s/support/supports

Except for above minor changes, others are ok for me.

Acked-by: Peter Chen <peter.c...@freescale.com>
> + */
> +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);
>  #endif
>  
>  MODULE_LICENSE("GPL");
> diff --git a/include/linux/usb/of.h b/include/linux/usb/of.h
> index cfe0528..22f1ce2 100644
> --- a/include/linux/usb/of.h
> +++ b/include/linux/usb/of.h
> @@ -15,6 +15,9 @@
>  enum usb_dr_mode of_usb_get_dr_mode(struct device_node *np);
>  enum usb_device_speed of_usb_get_maximum_speed(struct device_node *np);
>  bool of_usb_host_tpl_support(struct device_node *np);
> +bool of_usb_otg_srp_support(struct device_node *np);
> +bool of_usb_otg_hnp_support(struct device_node *np);
> +bool of_usb_otg_adp_support(struct device_node *np);
>  #else
>  static inline enum usb_dr_mode of_usb_get_dr_mode(struct device_node *np)
>  {
> @@ -30,6 +33,18 @@ static inline bool of_usb_host_tpl_support(struct 
> device_node *np)
>  {
>       return false;
>  }
> +static inline bool of_usb_otg_srp_support(struct device_node *np)
> +{
> +     return false;
> +}
> +static inline bool of_usb_otg_hnp_support(struct device_node *np)
> +{
> +     return false;
> +}
> +static inline bool of_usb_otg_adp_support(struct device_node *np)
> +{
> +     return false;
> +}
>  #endif
>  
>  #if IS_ENABLED(CONFIG_OF) && IS_ENABLED(CONFIG_USB_SUPPORT)
> -- 
> 1.9.1
> 

-- 

Best Regards,
Peter Chen
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to