diff for duplicates of <1452515294.26146.10.camel@linux.intel.com> diff --git a/a/1.txt b/N1/1.txt index 52b5664..2a12de3 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -2,8 +2,8 @@ On Fri, 2016-01-08 at 18:43 +0100, Anton Wuerfel wrote: > This patch fixes whitespace errors reported by checkpatch to increase > readability. Main focus is on missing spaces after commas in > function headers and macros (like foo,bar edited to foo, bar). ->=20 -> Signed-off-by: Anton W=C3=BCrfel <anton.wuerfel@fau.de> +> +> Signed-off-by: Anton Würfel <anton.wuerfel@fau.de> > Signed-off-by: Phillip Raffeck <phillip.raffeck@fau.de> > CC: linux-kernel@i4.cs.fau.de @@ -12,23 +12,22 @@ Cc: looks more standard. > --- a/drivers/tty/serial/8250/8250_accent.c > +++ b/drivers/tty/serial/8250/8250_accent.c > @@ -10,7 +10,7 @@ -> =C2=A0#include <linux/init.h> -> =C2=A0#include <linux/serial_8250.h> -> =C2=A0 +> #include <linux/init.h> +> #include <linux/serial_8250.h> +> > -#define PORT(_base,_irq) \ > +#define PORT(_base, _irq) \ -> =C2=A0 { \ -> =C2=A0 .iobase =3D _base, \ -> =C2=A0 .irq =3D _irq, \ ->=20 +> { \ +> .iobase = _base, \ +> .irq = _irq, \ +> While those are okay, I would suggest to make a macro in -drivers/=E2=80=A6/8250.h one for all, something like this: +drivers/…/8250.h one for all, something like this: #define 8250_PORT_FLAGS(_base, _irq, _flags) -=E2=80=A6 - .flags=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=3D = -UPF_BOOT_AUTOCONF | (_flags) \ +… + .flags = UPF_BOOT_AUTOCONF | (_flags) \ #define 8250_PORT(_base, _irq) 8250_PORT_FLAGS(_base, _irq, 0) @@ -37,54 +36,48 @@ And use it. > --- a/drivers/tty/serial/8250/8250_boca.c > +++ b/drivers/tty/serial/8250/8250_boca.c > @@ -10,7 +10,7 @@ -> =C2=A0#include <linux/init.h> -> =C2=A0#include <linux/serial_8250.h> -> =C2=A0 +> #include <linux/init.h> +> #include <linux/serial_8250.h> +> > -#define PORT(_base,_irq) \ > +#define PORT(_base, _irq) \ -> =C2=A0 { \ -> =C2=A0 .iobase =3D _base, \ -> =C2=A0 .irq =3D _irq, \ ->=20 +> { \ +> .iobase = _base, \ +> .irq = _irq, \ +> > --- a/drivers/tty/serial/8250/8250_exar_st16c554.c > +++ b/drivers/tty/serial/8250/8250_exar_st16c554.c > @@ -13,7 +13,7 @@ -> =C2=A0#include <linux/init.h> -> =C2=A0#include <linux/serial_8250.h> -> =C2=A0 +> #include <linux/init.h> +> #include <linux/serial_8250.h> +> > -#define PORT(_base,_irq) \ > +#define PORT(_base, _irq) \ -> =C2=A0 { \ -> =C2=A0 .iobase =3D _base, \ -> =C2=A0 .irq =3D _irq, \ ->=20 +> { \ +> .iobase = _base, \ +> .irq = _irq, \ +> > --- a/drivers/tty/serial/8250/8250_fourport.c > +++ b/drivers/tty/serial/8250/8250_fourport.c > @@ -10,7 +10,7 @@ -> =C2=A0#include <linux/init.h> -> =C2=A0#include <linux/serial_8250.h> -> =C2=A0 -> -#define PORT(_base,_irq) =09 +> #include <linux/init.h> +> #include <linux/serial_8250.h> +> +> -#define PORT(_base,_irq) > \ -> +#define PORT(_base, _irq) =09 +> +#define PORT(_base, _irq) > \ -> =C2=A0 { =09 +> { > \ -> =C2=A0 .iobase =3D _base, =09 +> .iobase = _base, > \ -> =C2=A0 .irq =3D _irq, =09 +> .irq = _irq, > \ ---=20 +-- Andy Shevchenko <andriy.shevchenko@linux.intel.com> Intel Finland Oy - --- -To unsubscribe from this list: send the line "unsubscribe linux-parisc"= - in -the body of a message to majordomo@vger.kernel.org -More majordomo info at http://vger.kernel.org/majordomo-info.html diff --git a/a/content_digest b/N1/content_digest index 4553de8..17734ec 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -32,8 +32,8 @@ "> This patch fixes whitespace errors reported by checkpatch to increase\n" "> readability. Main focus is on missing spaces after commas in\n" "> function headers and macros (like foo,bar edited to foo, bar).\n" - ">=20\n" - "> Signed-off-by: Anton W=C3=BCrfel <anton.wuerfel@fau.de>\n" + "> \n" + "> Signed-off-by: Anton W\303\274rfel <anton.wuerfel@fau.de>\n" "> Signed-off-by: Phillip Raffeck <phillip.raffeck@fau.de>\n" "> CC: linux-kernel@i4.cs.fau.de\n" "\n" @@ -42,23 +42,22 @@ "> --- a/drivers/tty/serial/8250/8250_accent.c\n" "> +++ b/drivers/tty/serial/8250/8250_accent.c\n" "> @@ -10,7 +10,7 @@\n" - "> =C2=A0#include <linux/init.h>\n" - "> =C2=A0#include <linux/serial_8250.h>\n" - "> =C2=A0\n" + "> \302\240#include <linux/init.h>\n" + "> \302\240#include <linux/serial_8250.h>\n" + "> \302\240\n" "> -#define PORT(_base,_irq)\t\t\t\t\\\n" "> +#define PORT(_base, _irq)\t\t\t\t\\\n" - "> =C2=A0\t{\t\t\t\t\t\t\\\n" - "> =C2=A0\t\t.iobase\t\t=3D _base,\t\t\\\n" - "> =C2=A0\t\t.irq\t\t=3D _irq,\t\t\t\\\n" - ">=20\n" + "> \302\240\t{\t\t\t\t\t\t\\\n" + "> \302\240\t\t.iobase\t\t= _base,\t\t\\\n" + "> \302\240\t\t.irq\t\t= _irq,\t\t\t\\\n" + "> \n" "\n" "While those are okay, I would suggest to make a macro in\n" - "drivers/=E2=80=A6/8250.h one for all, something like this:\n" + "drivers/\342\200\246/8250.h one for all, something like this:\n" "\n" "#define 8250_PORT_FLAGS(_base, _irq, _flags)\n" - "=E2=80=A6\n" - "\t.flags=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=3D =\n" - "UPF_BOOT_AUTOCONF | (_flags) \\\n" + "\342\200\246\n" + "\t.flags\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240= UPF_BOOT_AUTOCONF | (_flags) \\\n" "\n" "#define 8250_PORT(_base, _irq)\t8250_PORT_FLAGS(_base, _irq, 0)\n" "\n" @@ -67,56 +66,50 @@ "> --- a/drivers/tty/serial/8250/8250_boca.c\n" "> +++ b/drivers/tty/serial/8250/8250_boca.c\n" "> @@ -10,7 +10,7 @@\n" - "> =C2=A0#include <linux/init.h>\n" - "> =C2=A0#include <linux/serial_8250.h>\n" - "> =C2=A0\n" + "> \302\240#include <linux/init.h>\n" + "> \302\240#include <linux/serial_8250.h>\n" + "> \302\240\n" "> -#define PORT(_base,_irq)\t\t\t\t\\\n" "> +#define PORT(_base, _irq)\t\t\t\t\\\n" - "> =C2=A0\t{\t\t\t\t\t\t\\\n" - "> =C2=A0\t\t.iobase\t\t=3D _base,\t\t\\\n" - "> =C2=A0\t\t.irq\t\t=3D _irq,\t\t\t\\\n" - ">=20\n" + "> \302\240\t{\t\t\t\t\t\t\\\n" + "> \302\240\t\t.iobase\t\t= _base,\t\t\\\n" + "> \302\240\t\t.irq\t\t= _irq,\t\t\t\\\n" + "> \n" "\n" "> --- a/drivers/tty/serial/8250/8250_exar_st16c554.c\n" "> +++ b/drivers/tty/serial/8250/8250_exar_st16c554.c\n" "> @@ -13,7 +13,7 @@\n" - "> =C2=A0#include <linux/init.h>\n" - "> =C2=A0#include <linux/serial_8250.h>\n" - "> =C2=A0\n" + "> \302\240#include <linux/init.h>\n" + "> \302\240#include <linux/serial_8250.h>\n" + "> \302\240\n" "> -#define PORT(_base,_irq)\t\t\t\t\\\n" "> +#define PORT(_base, _irq)\t\t\t\t\\\n" - "> =C2=A0\t{\t\t\t\t\t\t\\\n" - "> =C2=A0\t\t.iobase\t\t=3D _base,\t\t\\\n" - "> =C2=A0\t\t.irq\t\t=3D _irq,\t\t\t\\\n" - ">=20\n" + "> \302\240\t{\t\t\t\t\t\t\\\n" + "> \302\240\t\t.iobase\t\t= _base,\t\t\\\n" + "> \302\240\t\t.irq\t\t= _irq,\t\t\t\\\n" + "> \n" "\n" "> --- a/drivers/tty/serial/8250/8250_fourport.c\n" "> +++ b/drivers/tty/serial/8250/8250_fourport.c\n" "> @@ -10,7 +10,7 @@\n" - "> =C2=A0#include <linux/init.h>\n" - "> =C2=A0#include <linux/serial_8250.h>\n" - "> =C2=A0\n" - "> -#define PORT(_base,_irq)\t\t\t\t\t=09\n" + "> \302\240#include <linux/init.h>\n" + "> \302\240#include <linux/serial_8250.h>\n" + "> \302\240\n" + "> -#define PORT(_base,_irq)\t\t\t\t\t\t\n" "> \\\n" - "> +#define PORT(_base, _irq)\t\t\t\t\t=09\n" + "> +#define PORT(_base, _irq)\t\t\t\t\t\t\n" "> \\\n" - "> =C2=A0\t{\t\t\t\t\t\t\t=09\n" + "> \302\240\t{\t\t\t\t\t\t\t\t\n" "> \\\n" - "> =C2=A0\t\t.iobase\t\t=3D _base,\t\t=09\n" + "> \302\240\t\t.iobase\t\t= _base,\t\t\t\n" "> \t\\\n" - "> =C2=A0\t\t.irq\t\t=3D _irq,\t\t\t=09\n" + "> \302\240\t\t.irq\t\t= _irq,\t\t\t\t\n" "> \t\\\n" "\n" "\n" "\n" - "--=20\n" + "-- \n" "Andy Shevchenko <andriy.shevchenko@linux.intel.com>\n" - "Intel Finland Oy\n" - "\n" - "--\n" - "To unsubscribe from this list: send the line \"unsubscribe linux-parisc\"=\n" - " in\n" - "the body of a message to majordomo@vger.kernel.org\n" - More majordomo info at http://vger.kernel.org/majordomo-info.html + Intel Finland Oy -471e9695ba5352cae57724f3214fb60cb032533de5254be8504686149ec9647e +905315424153119ed0a86442239b51c41502610a12e746f94c5638887d686378
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.