Re: [PATCH 6/9v2] usb: host: ehci.h: use space after comma
2015-12-10 12:57 GMT-03:00 Alan Stern : > On Thu, 10 Dec 2015, Peter Senna Tschudin wrote: > >> > /* next async queue entry, or pointer to interrupt/periodic QH */ >> > -#define QH_NEXT(ehci,dma) (cpu_to_hc32(ehci, >> > (((u32)dma)&~0x01f)|Q_TYPE_QH)) >> > +#define QH_NEXT(ehci, dma) \ >> > + (cpu_to_hc32(ehci, (((u32) dma) & ~0x01f) | Q_TYPE_QH)) >> >> For the maintainers: Is having two lines here better than having a line with >> 83 chars? > > Two lines is better. This patch is fine and so are the other 7 -- I > never received 9/9v2 (the original 9/9 was okay). Alan, The 9 is v1 only, since it mentions the tool (coccinelle) correctly. Tks. > > Alan Stern > -- 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
Re: [PATCH 6/9v2] usb: host: ehci.h: use space after comma
On Thu, 10 Dec 2015, Peter Senna Tschudin wrote: > > /* next async queue entry, or pointer to interrupt/periodic QH */ > > -#define QH_NEXT(ehci,dma) (cpu_to_hc32(ehci, > > (((u32)dma)&~0x01f)|Q_TYPE_QH)) > > +#define QH_NEXT(ehci, dma) \ > > + (cpu_to_hc32(ehci, (((u32) dma) & ~0x01f) | Q_TYPE_QH)) > > For the maintainers: Is having two lines here better than having a line with > 83 chars? Two lines is better. This patch is fine and so are the other 7 -- I never received 9/9v2 (the original 9/9 was okay). Alan Stern -- 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 6/9v2] usb: host: ehci.h: use space after comma
On Wed, Dec 09, 2015 at 07:32:35PM -0300, Geyslan G. Bem wrote: > Put space after comma. > > This patch also changes QH_NEXT macro for better reading. > > Caught by checkpatch: "ERROR: space required after that ','" > > Signed-off-by: Geyslan G. Bem > --- > drivers/usb/host/ehci.h | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > > diff --git a/drivers/usb/host/ehci.h b/drivers/usb/host/ehci.h > index 7b2b213..ea56147 100644 > --- a/drivers/usb/host/ehci.h > +++ b/drivers/usb/host/ehci.h > @@ -334,7 +334,7 @@ struct ehci_qtd { > /*-*/ > > /* type tag from {qh,itd,sitd,fstn}->hw_next */ > -#define Q_NEXT_TYPE(ehci,dma)((dma) & cpu_to_hc32(ehci, 3 << 1)) > +#define Q_NEXT_TYPE(ehci, dma) ((dma) & cpu_to_hc32(ehci, 3 << 1)) > > /* > * Now the following defines are not converted using the > @@ -350,7 +350,8 @@ struct ehci_qtd { > #define Q_TYPE_FSTN (3 << 1) > > /* next async queue entry, or pointer to interrupt/periodic QH */ > -#define QH_NEXT(ehci,dma)(cpu_to_hc32(ehci, > (((u32)dma)&~0x01f)|Q_TYPE_QH)) > +#define QH_NEXT(ehci, dma) \ > + (cpu_to_hc32(ehci, (((u32) dma) & ~0x01f) | Q_TYPE_QH)) For the maintainers: Is having two lines here better than having a line with 83 chars? > > /* for periodic/async schedules and qtd lists, mark end of list */ > #define EHCI_LIST_END(ehci) cpu_to_hc32(ehci, 1) /* "null pointer" to hw */ > -- > 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
[PATCH 6/9v2] usb: host: ehci.h: use space after comma
Put space after comma. This patch also changes QH_NEXT macro for better reading. Caught by checkpatch: "ERROR: space required after that ','" Signed-off-by: Geyslan G. Bem --- drivers/usb/host/ehci.h | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/usb/host/ehci.h b/drivers/usb/host/ehci.h index 7b2b213..ea56147 100644 --- a/drivers/usb/host/ehci.h +++ b/drivers/usb/host/ehci.h @@ -334,7 +334,7 @@ struct ehci_qtd { /*-*/ /* type tag from {qh,itd,sitd,fstn}->hw_next */ -#define Q_NEXT_TYPE(ehci,dma) ((dma) & cpu_to_hc32(ehci, 3 << 1)) +#define Q_NEXT_TYPE(ehci, dma) ((dma) & cpu_to_hc32(ehci, 3 << 1)) /* * Now the following defines are not converted using the @@ -350,7 +350,8 @@ struct ehci_qtd { #define Q_TYPE_FSTN(3 << 1) /* next async queue entry, or pointer to interrupt/periodic QH */ -#define QH_NEXT(ehci,dma) (cpu_to_hc32(ehci, (((u32)dma)&~0x01f)|Q_TYPE_QH)) +#define QH_NEXT(ehci, dma) \ + (cpu_to_hc32(ehci, (((u32) dma) & ~0x01f) | Q_TYPE_QH)) /* for periodic/async schedules and qtd lists, mark end of list */ #define EHCI_LIST_END(ehci)cpu_to_hc32(ehci, 1) /* "null pointer" to hw */ -- 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