All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] usb: cdns3: Removes xhci_cdns3_suspend_quirk from host-export.h
@ 2020-12-11  5:09 Pawel Laszczak
  2020-12-11  8:51 ` Roger Quadros
  0 siblings, 1 reply; 2+ messages in thread
From: Pawel Laszczak @ 2020-12-11  5:09 UTC (permalink / raw)
  To: peter.chen
  Cc: linux-usb, gregkh, kurahul, rogerq, a-govindraju, Pawel Laszczak

Function xhci_cdns3_suspend_quirk is used only in host.c file.
We can make it as static and removes it from host-export.h.

Signed-off-by: Pawel Laszczak <pawell@cadence.com>
---
 drivers/usb/cdns3/host-export.h | 7 -------
 drivers/usb/cdns3/host.c        | 4 +++-
 2 files changed, 3 insertions(+), 8 deletions(-)

diff --git a/drivers/usb/cdns3/host-export.h b/drivers/usb/cdns3/host-export.h
index fb8541b8adbc..cf92173ecf00 100644
--- a/drivers/usb/cdns3/host-export.h
+++ b/drivers/usb/cdns3/host-export.h
@@ -9,12 +9,9 @@
 #ifndef __LINUX_CDNS3_HOST_EXPORT
 #define __LINUX_CDNS3_HOST_EXPORT
 
-struct usb_hcd;
-
 #if IS_ENABLED(CONFIG_USB_CDNS_HOST)
 
 int cdns_host_init(struct cdns *cdns);
-int xhci_cdns3_suspend_quirk(struct usb_hcd *hcd);
 
 #else
 
@@ -24,10 +21,6 @@ static inline int cdns_host_init(struct cdns *cdns)
 }
 
 static inline void cdns_host_exit(struct cdns *cdns) { }
-static inline int xhci_cdns3_suspend_quirk(struct usb_hcd *hcd)
-{
-	return 0;
-}
 
 #endif /* USB_CDNS_HOST */
 
diff --git a/drivers/usb/cdns3/host.c b/drivers/usb/cdns3/host.c
index 3ff19182b0c0..84dadfa726aa 100644
--- a/drivers/usb/cdns3/host.c
+++ b/drivers/usb/cdns3/host.c
@@ -23,6 +23,8 @@
 #define CFG_RXDET_P3_EN		BIT(15)
 #define LPM_2_STB_SWITCH_EN	BIT(25)
 
+static int xhci_cdns3_suspend_quirk(struct usb_hcd *hcd);
+
 static const struct xhci_plat_priv xhci_plat_cdns3_xhci = {
 	.quirks = XHCI_SKIP_PHY_INIT | XHCI_AVOID_BEI,
 	.suspend_quirk = xhci_cdns3_suspend_quirk,
@@ -87,7 +89,7 @@ static int __cdns_host_init(struct cdns *cdns)
 	return ret;
 }
 
-int xhci_cdns3_suspend_quirk(struct usb_hcd *hcd)
+static int xhci_cdns3_suspend_quirk(struct usb_hcd *hcd)
 {
 	struct xhci_hcd	*xhci = hcd_to_xhci(hcd);
 	u32 value;
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] usb: cdns3: Removes xhci_cdns3_suspend_quirk from host-export.h
  2020-12-11  5:09 [PATCH] usb: cdns3: Removes xhci_cdns3_suspend_quirk from host-export.h Pawel Laszczak
@ 2020-12-11  8:51 ` Roger Quadros
  0 siblings, 0 replies; 2+ messages in thread
From: Roger Quadros @ 2020-12-11  8:51 UTC (permalink / raw)
  To: Pawel Laszczak, peter.chen; +Cc: linux-usb, gregkh, kurahul, a-govindraju



On 11/12/2020 07:09, Pawel Laszczak wrote:
> Function xhci_cdns3_suspend_quirk is used only in host.c file.
> We can make it as static and removes it from host-export.h.
> 
> Signed-off-by: Pawel Laszczak <pawell@cadence.com>

Acked-by: Roger Quadros <rogerq@ti.com>

cheers,
-roger

> ---
>   drivers/usb/cdns3/host-export.h | 7 -------
>   drivers/usb/cdns3/host.c        | 4 +++-
>   2 files changed, 3 insertions(+), 8 deletions(-)
> 
> diff --git a/drivers/usb/cdns3/host-export.h b/drivers/usb/cdns3/host-export.h
> index fb8541b8adbc..cf92173ecf00 100644
> --- a/drivers/usb/cdns3/host-export.h
> +++ b/drivers/usb/cdns3/host-export.h
> @@ -9,12 +9,9 @@
>   #ifndef __LINUX_CDNS3_HOST_EXPORT
>   #define __LINUX_CDNS3_HOST_EXPORT
>   
> -struct usb_hcd;
> -
>   #if IS_ENABLED(CONFIG_USB_CDNS_HOST)
>   
>   int cdns_host_init(struct cdns *cdns);
> -int xhci_cdns3_suspend_quirk(struct usb_hcd *hcd);
>   
>   #else
>   
> @@ -24,10 +21,6 @@ static inline int cdns_host_init(struct cdns *cdns)
>   }
>   
>   static inline void cdns_host_exit(struct cdns *cdns) { }
> -static inline int xhci_cdns3_suspend_quirk(struct usb_hcd *hcd)
> -{
> -	return 0;
> -}
>   
>   #endif /* USB_CDNS_HOST */
>   
> diff --git a/drivers/usb/cdns3/host.c b/drivers/usb/cdns3/host.c
> index 3ff19182b0c0..84dadfa726aa 100644
> --- a/drivers/usb/cdns3/host.c
> +++ b/drivers/usb/cdns3/host.c
> @@ -23,6 +23,8 @@
>   #define CFG_RXDET_P3_EN		BIT(15)
>   #define LPM_2_STB_SWITCH_EN	BIT(25)
>   
> +static int xhci_cdns3_suspend_quirk(struct usb_hcd *hcd);
> +
>   static const struct xhci_plat_priv xhci_plat_cdns3_xhci = {
>   	.quirks = XHCI_SKIP_PHY_INIT | XHCI_AVOID_BEI,
>   	.suspend_quirk = xhci_cdns3_suspend_quirk,
> @@ -87,7 +89,7 @@ static int __cdns_host_init(struct cdns *cdns)
>   	return ret;
>   }
>   
> -int xhci_cdns3_suspend_quirk(struct usb_hcd *hcd)
> +static int xhci_cdns3_suspend_quirk(struct usb_hcd *hcd)
>   {
>   	struct xhci_hcd	*xhci = hcd_to_xhci(hcd);
>   	u32 value;
> 

-- 
Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki.
Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2020-12-11  8:53 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-12-11  5:09 [PATCH] usb: cdns3: Removes xhci_cdns3_suspend_quirk from host-export.h Pawel Laszczak
2020-12-11  8:51 ` Roger Quadros

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.