diff for duplicates of <1452516646.26146.22.camel@intel.com> diff --git a/a/1.txt b/N1/1.txt index 658b5e4..4935c47 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,40 +1,77 @@ -T24gTW9uLCAyMDE2LTAxLTExIGF0IDE0OjI4ICswMjAwLCBBbmR5IFNoZXZjaGVua28gd3JvdGU6 -DQo+IE9uIEZyaSwgMjAxNi0wMS0wOCBhdCAxODo0MyArMDEwMCwgQW50b24gV3VlcmZlbCB3cm90 -ZToNCj4gPiBUaGlzIHBhdGNoIGZpeGVzIHdoaXRlc3BhY2UgZXJyb3JzIHJlcG9ydGVkIGJ5IGNo -ZWNrcGF0Y2ggdG8NCj4gPiBpbmNyZWFzZQ0KPiA+IHJlYWRhYmlsaXR5LiBNYWluIGZvY3VzIGlz -IG9uIG1pc3Npbmcgc3BhY2VzIGFmdGVyIGNvbW1hcyBpbg0KPiA+IGZ1bmN0aW9uIGhlYWRlcnMg -YW5kIG1hY3JvcyAobGlrZSBmb28sYmFyIGVkaXRlZCB0byBmb28sIGJhcikuDQo+ID4gDQo+ID4g -U2lnbmVkLW9mZi1ieTogQW50b24gV8O8cmZlbCA8YW50b24ud3VlcmZlbEBmYXUuZGU+DQo+ID4g -U2lnbmVkLW9mZi1ieTogUGhpbGxpcCBSYWZmZWNrIDxwaGlsbGlwLnJhZmZlY2tAZmF1LmRlPg0K -PiA+IENDOiBsaW51eC1rZXJuZWxAaTQuY3MuZmF1LmRlDQo+IA0KPiBDYzogbG9va3MgbW9yZSBz -dGFuZGFyZC4NCj4gDQo+ID4gLS0tIGEvZHJpdmVycy90dHkvc2VyaWFsLzgyNTAvODI1MF9hY2Nl -bnQuYw0KPiA+ICsrKyBiL2RyaXZlcnMvdHR5L3NlcmlhbC84MjUwLzgyNTBfYWNjZW50LmMNCj4g -PiBAQCAtMTAsNyArMTAsNyBAQA0KPiA+IMKgI2luY2x1ZGUgPGxpbnV4L2luaXQuaD4NCj4gPiDC -oCNpbmNsdWRlIDxsaW51eC9zZXJpYWxfODI1MC5oPg0KPiA+IMKgDQo+ID4gLSNkZWZpbmUgUE9S -VChfYmFzZSxfaXJxKQkJCQlcDQo+ID4gKyNkZWZpbmUgUE9SVChfYmFzZSwgX2lycSkJCQkJXA0K -PiA+IMKgCXsJCQkJCQlcDQo+ID4gwqAJCS5pb2Jhc2UJCT0gX2Jhc2UsCQlcDQo+ID4gwqAJCS5p -cnEJCT0gX2lycSwJCQkNCj4gPiBcDQoNCjgyNTBfUE9SVCgpDQoNCj4gPiANCj4gDQo+IFdoaWxl -IHRob3NlIGFyZSBva2F5LCBJIHdvdWxkIHN1Z2dlc3QgdG8gbWFrZSBhIG1hY3JvIGluDQo+IGRy -aXZlcnMv4oCmLzgyNTAuaCBvbmUgZm9yIGFsbCwgc29tZXRoaW5nIGxpa2UgdGhpczoNCj4gDQo+ -ICNkZWZpbmUgODI1MF9QT1JUX0ZMQUdTKF9iYXNlLCBfaXJxLCBfZmxhZ3MpDQo+IOKApg0KPiAJ -LmZsYWdzwqDCoMKgwqDCoMKgwqDCoMKgwqA9IFVQRl9CT09UX0FVVE9DT05GIHwgKF9mbGFncykg -XA0KPiANCj4gI2RlZmluZSA4MjUwX1BPUlQoX2Jhc2UsIF9pcnEpCTgyNTBfUE9SVF9GTEFHUyhf -YmFzZSwgX2lycSwgMCkNCj4gDQo+IEFuZCB1c2UgaXQuDQoNCiNkZWZpbmUgODI1MF9GT1VSUE9S -VChfYmFzZSwgX2lycSkJODI1MF9QT1JUX0ZMQUdTKF9iYXNlLCBfaXJxLA0KVVBGX0ZPVVJQT1JU -KQ0KDQo+ID4gLS0tIGEvZHJpdmVycy90dHkvc2VyaWFsLzgyNTAvODI1MF9mb3VycG9ydC5jDQo+ -ID4gKysrIGIvZHJpdmVycy90dHkvc2VyaWFsLzgyNTAvODI1MF9mb3VycG9ydC5jDQo+ID4gQEAg -LTEwLDcgKzEwLDcgQEANCj4gPiDCoCNpbmNsdWRlIDxsaW51eC9pbml0Lmg+DQo+ID4gwqAjaW5j -bHVkZSA8bGludXgvc2VyaWFsXzgyNTAuaD4NCj4gPiDCoA0KPiA+IC0jZGVmaW5lIFBPUlQoX2Jh -c2UsX2lycSkJCQkJCQkNCj4gPiBcDQo+ID4gKyNkZWZpbmUgUE9SVChfYmFzZSwgX2lycSkJCQkJ -CQkNCj4gPiBcDQo+ID4gwqAJewkJCQkJCQkJDQo+ID4gXA0KPiA+IMKgCQkuaW9iYXNlCQk9IF9i -YXNlLAkJCQ0KPiA+IAlcDQo+ID4gwqAJCS5pcnEJCT0gX2lycSwJCQkNCj4gPiAJDQo+ID4gCVwN -Cg0KODI1MF9GT1VSUE9SVCgpDQoNCi0tIA0KQW5keSBTaGV2Y2hlbmtvIDxhbmRyaXkuc2hldmNo -ZW5rb0BpbnRlbC5jb20+DQpJbnRlbCBGaW5sYW5kIE95DQotLS0tLS0tLS0tLS0tLS0tLS0tLS0t -LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0KSW50ZWwgRmlu -bGFuZCBPeQpSZWdpc3RlcmVkIEFkZHJlc3M6IFBMIDI4MSwgMDAxODEgSGVsc2lua2kgCkJ1c2lu -ZXNzIElkZW50aXR5IENvZGU6IDAzNTc2MDYgLSA0IApEb21pY2lsZWQgaW4gSGVsc2lua2kgCgpU -aGlzIGUtbWFpbCBhbmQgYW55IGF0dGFjaG1lbnRzIG1heSBjb250YWluIGNvbmZpZGVudGlhbCBt -YXRlcmlhbCBmb3IKdGhlIHNvbGUgdXNlIG9mIHRoZSBpbnRlbmRlZCByZWNpcGllbnQocykuIEFu -eSByZXZpZXcgb3IgZGlzdHJpYnV0aW9uCmJ5IG90aGVycyBpcyBzdHJpY3RseSBwcm9oaWJpdGVk -LiBJZiB5b3UgYXJlIG5vdCB0aGUgaW50ZW5kZWQKcmVjaXBpZW50LCBwbGVhc2UgY29udGFjdCB0 -aGUgc2VuZGVyIGFuZCBkZWxldGUgYWxsIGNvcGllcy4K +On Mon, 2016-01-11 at 14:28 +0200, Andy Shevchenko wrote: +> 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). +> > +> > 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 +> +> Cc: looks more standard. +> +> > --- a/drivers/tty/serial/8250/8250_accent.c +> > +++ b/drivers/tty/serial/8250/8250_accent.c +> > @@ -10,7 +10,7 @@ +> > #include <linux/init.h> +> > #include <linux/serial_8250.h> +> > +> > -#define PORT(_base,_irq) \ +> > +#define PORT(_base, _irq) \ +> > { \ +> > .iobase = _base, \ +> > .irq = _irq, +> > \ + +8250_PORT() + +> > +> +> While those are okay, I would suggest to make a macro in +> drivers/…/8250.h one for all, something like this: +> +> #define 8250_PORT_FLAGS(_base, _irq, _flags) +> … +> .flags = UPF_BOOT_AUTOCONF | (_flags) \ +> +> #define 8250_PORT(_base, _irq) 8250_PORT_FLAGS(_base, _irq, 0) +> +> And use it. + +#define 8250_FOURPORT(_base, _irq) 8250_PORT_FLAGS(_base, _irq, +UPF_FOURPORT) + +> > --- a/drivers/tty/serial/8250/8250_fourport.c +> > +++ b/drivers/tty/serial/8250/8250_fourport.c +> > @@ -10,7 +10,7 @@ +> > #include <linux/init.h> +> > #include <linux/serial_8250.h> +> > +> > -#define PORT(_base,_irq) +> > \ +> > +#define PORT(_base, _irq) +> > \ +> > { +> > \ +> > .iobase = _base, +> > \ +> > .irq = _irq, +> > +> > \ + +8250_FOURPORT() + +-- +Andy Shevchenko <andriy.shevchenko@intel.com> +Intel Finland Oy +--------------------------------------------------------------------- +Intel Finland Oy +Registered Address: PL 281, 00181 Helsinki +Business Identity Code: 0357606 - 4 +Domiciled in Helsinki + +This e-mail and any attachments may contain confidential material for +the sole use of the intended recipient(s). Any review or distribution +by others is strictly prohibited. If you are not the intended +recipient, please contact the sender and delete all copies. diff --git a/a/content_digest b/N1/content_digest index 844fb06..e7be93a 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -24,48 +24,88 @@ mans@mansr.com <mans@mansr.com> adam.lee@canonical.com <adam.lee@canonical.com> jejb@parisc-linux.org <jejb@parisc-linux.org> - " linux-parisc@vger.kernel.org <linux-paris\0" + linux-parisc@vger.kernel.org <linux-parisc@vger.kernel.org> + peter@hurleysoftware.com <peter@hurleysoftware.com> + matt.redfearn@imgtec.com <matt.redfearn@imgtec.com> + " k.kozlowski@samsung.com <k.kozlowski@samsung.com>\0" "\00:1\0" "b\0" - "T24gTW9uLCAyMDE2LTAxLTExIGF0IDE0OjI4ICswMjAwLCBBbmR5IFNoZXZjaGVua28gd3JvdGU6\n" - "DQo+IE9uIEZyaSwgMjAxNi0wMS0wOCBhdCAxODo0MyArMDEwMCwgQW50b24gV3VlcmZlbCB3cm90\n" - "ZToNCj4gPiBUaGlzIHBhdGNoIGZpeGVzIHdoaXRlc3BhY2UgZXJyb3JzIHJlcG9ydGVkIGJ5IGNo\n" - "ZWNrcGF0Y2ggdG8NCj4gPiBpbmNyZWFzZQ0KPiA+IHJlYWRhYmlsaXR5LiBNYWluIGZvY3VzIGlz\n" - "IG9uIG1pc3Npbmcgc3BhY2VzIGFmdGVyIGNvbW1hcyBpbg0KPiA+IGZ1bmN0aW9uIGhlYWRlcnMg\n" - "YW5kIG1hY3JvcyAobGlrZSBmb28sYmFyIGVkaXRlZCB0byBmb28sIGJhcikuDQo+ID4gDQo+ID4g\n" - "U2lnbmVkLW9mZi1ieTogQW50b24gV8O8cmZlbCA8YW50b24ud3VlcmZlbEBmYXUuZGU+DQo+ID4g\n" - "U2lnbmVkLW9mZi1ieTogUGhpbGxpcCBSYWZmZWNrIDxwaGlsbGlwLnJhZmZlY2tAZmF1LmRlPg0K\n" - "PiA+IENDOiBsaW51eC1rZXJuZWxAaTQuY3MuZmF1LmRlDQo+IA0KPiBDYzogbG9va3MgbW9yZSBz\n" - "dGFuZGFyZC4NCj4gDQo+ID4gLS0tIGEvZHJpdmVycy90dHkvc2VyaWFsLzgyNTAvODI1MF9hY2Nl\n" - "bnQuYw0KPiA+ICsrKyBiL2RyaXZlcnMvdHR5L3NlcmlhbC84MjUwLzgyNTBfYWNjZW50LmMNCj4g\n" - "PiBAQCAtMTAsNyArMTAsNyBAQA0KPiA+IMKgI2luY2x1ZGUgPGxpbnV4L2luaXQuaD4NCj4gPiDC\n" - "oCNpbmNsdWRlIDxsaW51eC9zZXJpYWxfODI1MC5oPg0KPiA+IMKgDQo+ID4gLSNkZWZpbmUgUE9S\n" - "VChfYmFzZSxfaXJxKQkJCQlcDQo+ID4gKyNkZWZpbmUgUE9SVChfYmFzZSwgX2lycSkJCQkJXA0K\n" - "PiA+IMKgCXsJCQkJCQlcDQo+ID4gwqAJCS5pb2Jhc2UJCT0gX2Jhc2UsCQlcDQo+ID4gwqAJCS5p\n" - "cnEJCT0gX2lycSwJCQkNCj4gPiBcDQoNCjgyNTBfUE9SVCgpDQoNCj4gPiANCj4gDQo+IFdoaWxl\n" - "IHRob3NlIGFyZSBva2F5LCBJIHdvdWxkIHN1Z2dlc3QgdG8gbWFrZSBhIG1hY3JvIGluDQo+IGRy\n" - "aXZlcnMv4oCmLzgyNTAuaCBvbmUgZm9yIGFsbCwgc29tZXRoaW5nIGxpa2UgdGhpczoNCj4gDQo+\n" - "ICNkZWZpbmUgODI1MF9QT1JUX0ZMQUdTKF9iYXNlLCBfaXJxLCBfZmxhZ3MpDQo+IOKApg0KPiAJ\n" - "LmZsYWdzwqDCoMKgwqDCoMKgwqDCoMKgwqA9IFVQRl9CT09UX0FVVE9DT05GIHwgKF9mbGFncykg\n" - "XA0KPiANCj4gI2RlZmluZSA4MjUwX1BPUlQoX2Jhc2UsIF9pcnEpCTgyNTBfUE9SVF9GTEFHUyhf\n" - "YmFzZSwgX2lycSwgMCkNCj4gDQo+IEFuZCB1c2UgaXQuDQoNCiNkZWZpbmUgODI1MF9GT1VSUE9S\n" - "VChfYmFzZSwgX2lycSkJODI1MF9QT1JUX0ZMQUdTKF9iYXNlLCBfaXJxLA0KVVBGX0ZPVVJQT1JU\n" - "KQ0KDQo+ID4gLS0tIGEvZHJpdmVycy90dHkvc2VyaWFsLzgyNTAvODI1MF9mb3VycG9ydC5jDQo+\n" - "ID4gKysrIGIvZHJpdmVycy90dHkvc2VyaWFsLzgyNTAvODI1MF9mb3VycG9ydC5jDQo+ID4gQEAg\n" - "LTEwLDcgKzEwLDcgQEANCj4gPiDCoCNpbmNsdWRlIDxsaW51eC9pbml0Lmg+DQo+ID4gwqAjaW5j\n" - "bHVkZSA8bGludXgvc2VyaWFsXzgyNTAuaD4NCj4gPiDCoA0KPiA+IC0jZGVmaW5lIFBPUlQoX2Jh\n" - "c2UsX2lycSkJCQkJCQkNCj4gPiBcDQo+ID4gKyNkZWZpbmUgUE9SVChfYmFzZSwgX2lycSkJCQkJ\n" - "CQkNCj4gPiBcDQo+ID4gwqAJewkJCQkJCQkJDQo+ID4gXA0KPiA+IMKgCQkuaW9iYXNlCQk9IF9i\n" - "YXNlLAkJCQ0KPiA+IAlcDQo+ID4gwqAJCS5pcnEJCT0gX2lycSwJCQkNCj4gPiAJDQo+ID4gCVwN\n" - "Cg0KODI1MF9GT1VSUE9SVCgpDQoNCi0tIA0KQW5keSBTaGV2Y2hlbmtvIDxhbmRyaXkuc2hldmNo\n" - "ZW5rb0BpbnRlbC5jb20+DQpJbnRlbCBGaW5sYW5kIE95DQotLS0tLS0tLS0tLS0tLS0tLS0tLS0t\n" - "LS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0KSW50ZWwgRmlu\n" - "bGFuZCBPeQpSZWdpc3RlcmVkIEFkZHJlc3M6IFBMIDI4MSwgMDAxODEgSGVsc2lua2kgCkJ1c2lu\n" - "ZXNzIElkZW50aXR5IENvZGU6IDAzNTc2MDYgLSA0IApEb21pY2lsZWQgaW4gSGVsc2lua2kgCgpU\n" - "aGlzIGUtbWFpbCBhbmQgYW55IGF0dGFjaG1lbnRzIG1heSBjb250YWluIGNvbmZpZGVudGlhbCBt\n" - "YXRlcmlhbCBmb3IKdGhlIHNvbGUgdXNlIG9mIHRoZSBpbnRlbmRlZCByZWNpcGllbnQocykuIEFu\n" - "eSByZXZpZXcgb3IgZGlzdHJpYnV0aW9uCmJ5IG90aGVycyBpcyBzdHJpY3RseSBwcm9oaWJpdGVk\n" - "LiBJZiB5b3UgYXJlIG5vdCB0aGUgaW50ZW5kZWQKcmVjaXBpZW50LCBwbGVhc2UgY29udGFjdCB0\n" - aGUgc2VuZGVyIGFuZCBkZWxldGUgYWxsIGNvcGllcy4K + "On Mon, 2016-01-11 at 14:28 +0200, Andy Shevchenko wrote:\n" + "> On Fri, 2016-01-08 at 18:43 +0100, Anton Wuerfel wrote:\n" + "> > This patch fixes whitespace errors reported by checkpatch to\n" + "> > 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" + "> > \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" + "> Cc: looks more standard.\n" + "> \n" + "> > --- a/drivers/tty/serial/8250/8250_accent.c\n" + "> > +++ b/drivers/tty/serial/8250/8250_accent.c\n" + "> > @@ -10,7 +10,7 @@\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" + "> > \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" + "8250_PORT()\n" + "\n" + "> > \n" + "> \n" + "> While those are okay, I would suggest to make a macro in\n" + "> drivers/\342\200\246/8250.h one for all, something like this:\n" + "> \n" + "> #define 8250_PORT_FLAGS(_base, _irq, _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" + "> And use it.\n" + "\n" + "#define 8250_FOURPORT(_base, _irq)\t8250_PORT_FLAGS(_base, _irq,\n" + "UPF_FOURPORT)\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" + "> > \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\t\n" + "> > \\\n" + "> > \302\240\t{\t\t\t\t\t\t\t\t\n" + "> > \\\n" + "> > \302\240\t\t.iobase\t\t= _base,\t\t\t\n" + "> > \t\\\n" + "> > \302\240\t\t.irq\t\t= _irq,\t\t\t\n" + "> > \t\n" + "> > \t\\\n" + "\n" + "8250_FOURPORT()\n" + "\n" + "-- \n" + "Andy Shevchenko <andriy.shevchenko@intel.com>\n" + "Intel Finland Oy\n" + "---------------------------------------------------------------------\n" + "Intel Finland Oy\n" + "Registered Address: PL 281, 00181 Helsinki \n" + "Business Identity Code: 0357606 - 4 \n" + "Domiciled in Helsinki \n" + "\n" + "This e-mail and any attachments may contain confidential material for\n" + "the sole use of the intended recipient(s). Any review or distribution\n" + "by others is strictly prohibited. If you are not the intended\n" + recipient, please contact the sender and delete all copies. -447cef4326f73b3319b618a75b7eb1e4481a76217c4e1cacc786161c3be72923 +e322a5bf081e44a41bf8991e7430d55ef81fcd34cf2485cac81bbfc6b068ab16
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.