Re: [PATCH 2/4] usb: host: ehci.h: cleanup header file

[Sergei Shtylyov]

Hello.

On 12/09/2015 10:57 PM, Geyslan G. Bem wrote:


This patch does align function/macro definitions.

Signed-off-by: Geyslan G. Bem 


USB code just uses different alignment style (2 tabs) than the other 
kernel parts (like networking).


MBR, Sergei

--
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

Re: [PATCH 2/4] usb: host: ehci.h: cleanup header file

[Geyslan G. Bem]

2015-12-09 17:11 GMT-03:00 Sergei Shtylyov :
> Hello.
>
> On 12/09/2015 10:57 PM, Geyslan G. Bem wrote:
>
>> This patch does align function/macro definitions.
>>
>> Signed-off-by: Geyslan G. Bem 
>
>
> USB code just uses different alignment style (2 tabs) than the other
> kernel parts (like networking).
I see. Please, ignore it.

New code has to have 2 tabs too?

>
> MBR, Sergei
>



-- 
Regards,

Geyslan G. Bem
hackingbits.com
--
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

[PATCH 2/4] usb: host: ehci.h: cleanup header file

[Geyslan G. Bem]

This patch does align function/macro definitions.

Signed-off-by: Geyslan G. Bem 
---
 drivers/usb/host/ehci.h | 22 +++---
 1 file changed, 11 insertions(+), 11 deletions(-)

diff --git a/drivers/usb/host/ehci.h b/drivers/usb/host/ehci.h
index a4f5ab5..0daed80 100644
--- a/drivers/usb/host/ehci.h
+++ b/drivers/usb/host/ehci.h
@@ -634,11 +634,11 @@ struct ehci_tt {
 
 /* Prepare the PORTSC wakeup flags during controller suspend/resume */
 
-#define ehci_prepare_ports_for_controller_suspend(ehci, do_wakeup) \
-   ehci_adjust_port_wakeup_flags(ehci, true, do_wakeup)
+#define ehci_prepare_ports_for_controller_suspend(ehci, do_wakeup) \
+   ehci_adjust_port_wakeup_flags(ehci, true, do_wakeup)
 
-#define ehci_prepare_ports_for_controller_resume(ehci) \
-   ehci_adjust_port_wakeup_flags(ehci, false, false)
+#define ehci_prepare_ports_for_controller_resume(ehci) \
+   ehci_adjust_port_wakeup_flags(ehci, false, false)
 
 /*-*/
 
@@ -745,18 +745,18 @@ static inline unsigned int ehci_readl(const struct 
ehci_hcd *ehci,
 
 #ifdef CONFIG_SOC_IMX28
 static inline void imx28_ehci_writel(const unsigned int val,
-   volatile __u32 __iomem *addr)
+volatile __u32 __iomem *addr)
 {
__asm__ ("swp %0, %0, [%1]" : : "r"(val), "r"(addr));
 }
 #else
 static inline void imx28_ehci_writel(const unsigned int val,
-   volatile __u32 __iomem *addr)
+volatile __u32 __iomem *addr)
 {
 }
 #endif
 static inline void ehci_writel(const struct ehci_hcd *ehci,
-   const unsigned int val, __u32 __iomem *regs)
+  const unsigned int val, __u32 __iomem *regs)
 {
 #ifdef CONFIG_USB_EHCI_BIG_ENDIAN_MMIO
ehci_big_endian_mmio(ehci) ?
@@ -874,21 +874,21 @@ struct ehci_driver_overrides {
size_t  extra_priv_size;
int (*reset)(struct usb_hcd *hcd);
int (*port_power)(struct usb_hcd *hcd,
-   int portnum, bool enable);
+ int portnum, bool enable);
 };
 
 extern voidehci_init_driver(struct hc_driver *drv,
-   const struct ehci_driver_overrides *over);
+const struct ehci_driver_overrides *over);
 extern int ehci_setup(struct usb_hcd *hcd);
 extern int ehci_handshake(struct ehci_hcd *ehci, void __iomem *ptr,
-   u32 mask, u32 done, int usec);
+  u32 mask, u32 done, int usec);
 extern int ehci_reset(struct ehci_hcd *ehci);
 
 #ifdef CONFIG_PM
 extern int ehci_suspend(struct usb_hcd *hcd, bool do_wakeup);
 extern int ehci_resume(struct usb_hcd *hcd, bool force_reset);
 extern voidehci_adjust_port_wakeup_flags(struct ehci_hcd *ehci,
-   bool suspending, bool do_wakeup);
+ bool suspending, bool do_wakeup);
 #endif /* CONFIG_PM */
 
 extern int ehci_hub_control(struct usb_hcd *hcd, u16 typeReq, u16 wValue,
-- 
2.6.3

--
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