diff for duplicates of <20140213113313.GL32508@lee--X1> diff --git a/a/1.txt b/N1/1.txt index 215dbe5..fd33339 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,46 +1,59 @@ -PiA+Pj4gPiAtc3RhdGljIGludCBtYXg2NjUwX3Byb2JlKHN0cnVjdCBpMmNfY2xpZW50ICpjbGll -bnQsCj4gPj4+ID4gLSAgICAgICAgICAgICAgICAgICAgY29uc3Qgc3RydWN0IGkyY19kZXZpY2Vf -aWQgKmlkKTsKPiA+Pj4gPiAtc3RhdGljIGludCBtYXg2NjUwX2luaXRfY2xpZW50KHN0cnVjdCBp -MmNfY2xpZW50ICpjbGllbnQpOwo+ID4+PiA+IC1zdGF0aWMgaW50IG1heDY2NTBfcmVtb3ZlKHN0 -cnVjdCBpMmNfY2xpZW50ICpjbGllbnQpOwo+ID4+PiA+ICtzdGF0aWMgaW50IG1heDY2NTBfcHJv -YmUoc3RydWN0IHBsYXRmb3JtX2RldmljZSAqcGRldik7Cj4gPj4+ID4gK3N0YXRpYyBpbnQgbWF4 -NjY1MF9pbml0X2NsaWVudChzdHJ1Y3QgcGxhdGZvcm1fZGV2aWNlICpwZGV2KTsKPiA+Pj4gPiAr -c3RhdGljIGludCBtYXg2NjUwX3JlbW92ZShzdHJ1Y3QgcGxhdGZvcm1fZGV2aWNlICpwZGV2KTsK -PiA+Pj4gPiAgc3RhdGljIHN0cnVjdCBtYXg2NjUwX2RhdGEgKm1heDY2NTBfdXBkYXRlX2Rldmlj -ZShzdHJ1Y3QgZGV2aWNlICpkZXYpOwo+ID4+Pgo+ID4+PiBJdCB3b3VsZCBiZSBnb29kIHRvIHJl -bW92ZSB0aGVzZSBmb3J3YXJkIGRlY2xhcmF0aW9ucyBpbiB0aGUgZnV0dXJlLgo+ID4+Pgo+ID4+ -PiBJZiBubyBvbmUgdm9sdW50ZWVycyBJJ2xsIGhhcHBpbHkgZG8gaXQuCj4gPj4KPiA+PiBHdWVu -dGVyIGp1c3QgZGlkOgo+ID4+Cj4gPj4gaHR0cDovL2xpc3RzLmxtLXNlbnNvcnMub3JnL3BpcGVy -bWFpbC9sbS1zZW5zb3JzLzIwMTQtRmVicnVhcnkvMDQxMjI0Lmh0bWwKPiA+Pgo+ID4+IEFueSBj -aGFuZ2UgdG8gdGhlIG1heDY2NTAgZHJpdmVyIHNob3VsZCBnbyBvbiB0b3Agb2YgaGlzIHBhdGNo -IHNlcmllcwo+ID4+IHRvIGF2b2lkIGNvbmZsaWN0czoKPiA+Pgo+ID4+IGh0dHA6Ly9saXN0cy5s -bS1zZW5zb3JzLm9yZy9waXBlcm1haWwvbG0tc2Vuc29ycy8yMDE0LUZlYnJ1YXJ5LzA0MTIyMy5o -dG1sCj4gPgo+IEFzIGZhciBhcyBJIGNhbiBzZWUsIHRoYXQgcGF0Y2ggc2V0IHdhcyBub3QgZXZl -biB0ZXN0ZWQsIHNvIGhvdyBjYW4gaXQKPiBnbyBpbj8gSSB3YXMgdG9sZCB0aGF0IGFueSBwYXRj -aCBzaG91bGQgYmUgX3J1bnRpbWVfIHRlc3RlZCwgdG9vLgo+IEZ3aXcsIEkgZG8gbm90IGhhdmUg -dGltZSB0byB0ZXN0IHRob3NlIHBlcnNvbmFsbHksIGhlIHdvdWxkIG5lZWQgdG8KPiBmaW5kIHNv -bWVvbmUgZWxzZSBpZiB0aGF0IHJlcXVpcmVtZW50IHJlYWxseSBob2xkcyB0cnVlLgo+Cj4gSSB3 -b3VsZCBub3QgcmVhbGx5IGxpa2UgdG8gZml4IGJ1Z3MgYXBwZWFyaW5nIGluIHRoYXQgY29kZSB0 -byBnZXQgbXkKPiBmZWF0dXJlcyBpbi4KPiAKPiBBbHNvLCBzaW5jZSBteSBjaGFuZ2UgaGFzIGJl -ZW4gYXJvdW5kIGZvciAyLTMgbW9udGhzIG5vdywgSSB3b3VsZAo+IHJlYWxseSBwcmVmZXIgbm90 -IHRvIGJlIGZvcmNlZCB0byByZXdyaXRlIGl0IGFnYWluIGZyb20gc2NyYXRjaC4KPiBTdXJlbHks -IHlvdSBjYW4gd2FpdCB3aXRoIHRob3NlLCBtb3JlIG9yIGxlc3MsIGNvc21ldGljIG5vbi1ydW50 -aW1lCj4gdGVzdGVkIGNoYW5nZXM/Cj4gCj4gVGhpcyB3b3VsZCBpbXBvc2UgbWUgYSBsb3Qgb2Yg -YWRkaXRpb25hbCB3b3JrIGFnYWluLCBhbmQgSSBwZXJzb25hbGx5Cj4gZG8gbm90IHNlZSB0aGUg -YmVuZWZpdCBvZiBpdC4gSW4gbXkgYm9vayBhdCBsZWFzdCwgZmVhdHVyZSBpcyBvdmVyCj4gaW50 -ZXJuYWwgcG9saXNoaW5nLgoKUmlnaHQsIEkndmUgaGFkIGVub3VnaC4gSSdtIHJlbW92aW5nIHlv -dXIgcGF0Y2ggZnJvbSB0aGUgTUZEIHRyZWUuCgpJJ3ZlIGFza2VkIHRvbyBtYW55IHBlb3BsZSB0 -byBnaXZlIHlvdSBhIHNlY29uZCBjaGFuY2UgYW5kIGFza2VkIHlvdQpwcml2YXRlbHkgdG8gYmVo -YXZlIHlvdXJzZWxmIGFuZCB0cmVhdCBvdGhlcnMgd2l0aCByZXNwZWN0LiBTbyBmYXIgSQpoYXZl -bid0IHNlZW4gYW4gb3VuY2Ugb2Ygc2VsZiBjb250cm9sIG9yIGRlcG9tYWN5IGZyb20geW91LgoK -VGhpcyBpcyBob3cgaXQncyBnb2luZyB0byB3b3JrIGZyb20gbm93IG9uOgoKIC0gWW91IHN1Ym1p -dCBhIHBhdGNoCiAtIEl0IGdldHMgcmV2aWV3ZWQgICAgICAgICAgICAgICAgICAgICAgICAgICAg -PC0tLS1cCiAtIFlvdSBmaXggdXAgdGhlIHJldmlldyBjb21tZW50cyBhcyByZXF1ZXN0ZWQgLS0t -LS0vCiAtIE5vbi1jb21wbGlhbmNlIG9yIGFyZ3VtZW50cyB3aXRoIHRoZSBfZXhwZXJ0c18gcmVz -dWx0cyBpbjoKICAgIGAkSU5URVJFU1QgPiAvZGV2L251bGwgfHwgXAogICAgICBncmVwICJGcm9t -OiBMYXN6aW8gUGFwcCIgfi8ubWFpbCB8IHhhcmdzIHJtIC1yZmAKCi0tIApMZWUgSm9uZXMKTGlu -YXJvIFNUTWljcm9lbGVjdHJvbmljcyBMYW5kaW5nIFRlYW0gTGVhZApMaW5hcm8ub3JnIOKUgiBP -cGVuIHNvdXJjZSBzb2Z0d2FyZSBmb3IgQVJNIFNvQ3MKRm9sbG93IExpbmFybzogRmFjZWJvb2sg -fCBUd2l0dGVyIHwgQmxvZwoKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f -X19fX19fX18KbG0tc2Vuc29ycyBtYWlsaW5nIGxpc3QKbG0tc2Vuc29yc0BsbS1zZW5zb3JzLm9y -ZwpodHRwOi8vbGlzdHMubG0tc2Vuc29ycy5vcmcvbWFpbG1hbi9saXN0aW5mby9sbS1zZW5zb3Jz +> >>> > -static int max6650_probe(struct i2c_client *client, +> >>> > - const struct i2c_device_id *id); +> >>> > -static int max6650_init_client(struct i2c_client *client); +> >>> > -static int max6650_remove(struct i2c_client *client); +> >>> > +static int max6650_probe(struct platform_device *pdev); +> >>> > +static int max6650_init_client(struct platform_device *pdev); +> >>> > +static int max6650_remove(struct platform_device *pdev); +> >>> > static struct max6650_data *max6650_update_device(struct device *dev); +> >>> +> >>> It would be good to remove these forward declarations in the future. +> >>> +> >>> If no one volunteers I'll happily do it. +> >> +> >> Guenter just did: +> >> +> >> http://lists.lm-sensors.org/pipermail/lm-sensors/2014-February/041224.html +> >> +> >> Any change to the max6650 driver should go on top of his patch series +> >> to avoid conflicts: +> >> +> >> http://lists.lm-sensors.org/pipermail/lm-sensors/2014-February/041223.html +> > +> As far as I can see, that patch set was not even tested, so how can it +> go in? I was told that any patch should be _runtime_ tested, too. +> Fwiw, I do not have time to test those personally, he would need to +> find someone else if that requirement really holds true. +> +> I would not really like to fix bugs appearing in that code to get my +> features in. +> +> Also, since my change has been around for 2-3 months now, I would +> really prefer not to be forced to rewrite it again from scratch. +> Surely, you can wait with those, more or less, cosmetic non-runtime +> tested changes? +> +> This would impose me a lot of additional work again, and I personally +> do not see the benefit of it. In my book at least, feature is over +> internal polishing. + +Right, I've had enough. I'm removing your patch from the MFD tree. + +I've asked too many people to give you a second chance and asked you +privately to behave yourself and treat others with respect. So far I +haven't seen an ounce of self control or depomacy from you. + +This is how it's going to work from now on: + + - You submit a patch + - It gets reviewed <----\ + - You fix up the review comments as requested -----/ + - Non-compliance or arguments with the _experts_ results in: + `$INTEREST > /dev/null || \ + grep "From: Laszio Papp" ~/.mail | xargs rm -rf` + +-- +Lee Jones +Linaro STMicroelectronics Landing Team Lead +Linaro.org │ Open source software for ARM SoCs +Follow Linaro: Facebook | Twitter | Blog diff --git a/a/content_digest b/N1/content_digest index ca45f16..efc1216 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -13,51 +13,64 @@ " Guenter Roeck <linux@roeck-us.net>\0" "\00:1\0" "b\0" - "PiA+Pj4gPiAtc3RhdGljIGludCBtYXg2NjUwX3Byb2JlKHN0cnVjdCBpMmNfY2xpZW50ICpjbGll\n" - "bnQsCj4gPj4+ID4gLSAgICAgICAgICAgICAgICAgICAgY29uc3Qgc3RydWN0IGkyY19kZXZpY2Vf\n" - "aWQgKmlkKTsKPiA+Pj4gPiAtc3RhdGljIGludCBtYXg2NjUwX2luaXRfY2xpZW50KHN0cnVjdCBp\n" - "MmNfY2xpZW50ICpjbGllbnQpOwo+ID4+PiA+IC1zdGF0aWMgaW50IG1heDY2NTBfcmVtb3ZlKHN0\n" - "cnVjdCBpMmNfY2xpZW50ICpjbGllbnQpOwo+ID4+PiA+ICtzdGF0aWMgaW50IG1heDY2NTBfcHJv\n" - "YmUoc3RydWN0IHBsYXRmb3JtX2RldmljZSAqcGRldik7Cj4gPj4+ID4gK3N0YXRpYyBpbnQgbWF4\n" - "NjY1MF9pbml0X2NsaWVudChzdHJ1Y3QgcGxhdGZvcm1fZGV2aWNlICpwZGV2KTsKPiA+Pj4gPiAr\n" - "c3RhdGljIGludCBtYXg2NjUwX3JlbW92ZShzdHJ1Y3QgcGxhdGZvcm1fZGV2aWNlICpwZGV2KTsK\n" - "PiA+Pj4gPiAgc3RhdGljIHN0cnVjdCBtYXg2NjUwX2RhdGEgKm1heDY2NTBfdXBkYXRlX2Rldmlj\n" - "ZShzdHJ1Y3QgZGV2aWNlICpkZXYpOwo+ID4+Pgo+ID4+PiBJdCB3b3VsZCBiZSBnb29kIHRvIHJl\n" - "bW92ZSB0aGVzZSBmb3J3YXJkIGRlY2xhcmF0aW9ucyBpbiB0aGUgZnV0dXJlLgo+ID4+Pgo+ID4+\n" - "PiBJZiBubyBvbmUgdm9sdW50ZWVycyBJJ2xsIGhhcHBpbHkgZG8gaXQuCj4gPj4KPiA+PiBHdWVu\n" - "dGVyIGp1c3QgZGlkOgo+ID4+Cj4gPj4gaHR0cDovL2xpc3RzLmxtLXNlbnNvcnMub3JnL3BpcGVy\n" - "bWFpbC9sbS1zZW5zb3JzLzIwMTQtRmVicnVhcnkvMDQxMjI0Lmh0bWwKPiA+Pgo+ID4+IEFueSBj\n" - "aGFuZ2UgdG8gdGhlIG1heDY2NTAgZHJpdmVyIHNob3VsZCBnbyBvbiB0b3Agb2YgaGlzIHBhdGNo\n" - "IHNlcmllcwo+ID4+IHRvIGF2b2lkIGNvbmZsaWN0czoKPiA+Pgo+ID4+IGh0dHA6Ly9saXN0cy5s\n" - "bS1zZW5zb3JzLm9yZy9waXBlcm1haWwvbG0tc2Vuc29ycy8yMDE0LUZlYnJ1YXJ5LzA0MTIyMy5o\n" - "dG1sCj4gPgo+IEFzIGZhciBhcyBJIGNhbiBzZWUsIHRoYXQgcGF0Y2ggc2V0IHdhcyBub3QgZXZl\n" - "biB0ZXN0ZWQsIHNvIGhvdyBjYW4gaXQKPiBnbyBpbj8gSSB3YXMgdG9sZCB0aGF0IGFueSBwYXRj\n" - "aCBzaG91bGQgYmUgX3J1bnRpbWVfIHRlc3RlZCwgdG9vLgo+IEZ3aXcsIEkgZG8gbm90IGhhdmUg\n" - "dGltZSB0byB0ZXN0IHRob3NlIHBlcnNvbmFsbHksIGhlIHdvdWxkIG5lZWQgdG8KPiBmaW5kIHNv\n" - "bWVvbmUgZWxzZSBpZiB0aGF0IHJlcXVpcmVtZW50IHJlYWxseSBob2xkcyB0cnVlLgo+Cj4gSSB3\n" - "b3VsZCBub3QgcmVhbGx5IGxpa2UgdG8gZml4IGJ1Z3MgYXBwZWFyaW5nIGluIHRoYXQgY29kZSB0\n" - "byBnZXQgbXkKPiBmZWF0dXJlcyBpbi4KPiAKPiBBbHNvLCBzaW5jZSBteSBjaGFuZ2UgaGFzIGJl\n" - "ZW4gYXJvdW5kIGZvciAyLTMgbW9udGhzIG5vdywgSSB3b3VsZAo+IHJlYWxseSBwcmVmZXIgbm90\n" - "IHRvIGJlIGZvcmNlZCB0byByZXdyaXRlIGl0IGFnYWluIGZyb20gc2NyYXRjaC4KPiBTdXJlbHks\n" - "IHlvdSBjYW4gd2FpdCB3aXRoIHRob3NlLCBtb3JlIG9yIGxlc3MsIGNvc21ldGljIG5vbi1ydW50\n" - "aW1lCj4gdGVzdGVkIGNoYW5nZXM/Cj4gCj4gVGhpcyB3b3VsZCBpbXBvc2UgbWUgYSBsb3Qgb2Yg\n" - "YWRkaXRpb25hbCB3b3JrIGFnYWluLCBhbmQgSSBwZXJzb25hbGx5Cj4gZG8gbm90IHNlZSB0aGUg\n" - "YmVuZWZpdCBvZiBpdC4gSW4gbXkgYm9vayBhdCBsZWFzdCwgZmVhdHVyZSBpcyBvdmVyCj4gaW50\n" - "ZXJuYWwgcG9saXNoaW5nLgoKUmlnaHQsIEkndmUgaGFkIGVub3VnaC4gSSdtIHJlbW92aW5nIHlv\n" - "dXIgcGF0Y2ggZnJvbSB0aGUgTUZEIHRyZWUuCgpJJ3ZlIGFza2VkIHRvbyBtYW55IHBlb3BsZSB0\n" - "byBnaXZlIHlvdSBhIHNlY29uZCBjaGFuY2UgYW5kIGFza2VkIHlvdQpwcml2YXRlbHkgdG8gYmVo\n" - "YXZlIHlvdXJzZWxmIGFuZCB0cmVhdCBvdGhlcnMgd2l0aCByZXNwZWN0LiBTbyBmYXIgSQpoYXZl\n" - "bid0IHNlZW4gYW4gb3VuY2Ugb2Ygc2VsZiBjb250cm9sIG9yIGRlcG9tYWN5IGZyb20geW91LgoK\n" - "VGhpcyBpcyBob3cgaXQncyBnb2luZyB0byB3b3JrIGZyb20gbm93IG9uOgoKIC0gWW91IHN1Ym1p\n" - "dCBhIHBhdGNoCiAtIEl0IGdldHMgcmV2aWV3ZWQgICAgICAgICAgICAgICAgICAgICAgICAgICAg\n" - "PC0tLS1cCiAtIFlvdSBmaXggdXAgdGhlIHJldmlldyBjb21tZW50cyBhcyByZXF1ZXN0ZWQgLS0t\n" - "LS0vCiAtIE5vbi1jb21wbGlhbmNlIG9yIGFyZ3VtZW50cyB3aXRoIHRoZSBfZXhwZXJ0c18gcmVz\n" - "dWx0cyBpbjoKICAgIGAkSU5URVJFU1QgPiAvZGV2L251bGwgfHwgXAogICAgICBncmVwICJGcm9t\n" - "OiBMYXN6aW8gUGFwcCIgfi8ubWFpbCB8IHhhcmdzIHJtIC1yZmAKCi0tIApMZWUgSm9uZXMKTGlu\n" - "YXJvIFNUTWljcm9lbGVjdHJvbmljcyBMYW5kaW5nIFRlYW0gTGVhZApMaW5hcm8ub3JnIOKUgiBP\n" - "cGVuIHNvdXJjZSBzb2Z0d2FyZSBmb3IgQVJNIFNvQ3MKRm9sbG93IExpbmFybzogRmFjZWJvb2sg\n" - "fCBUd2l0dGVyIHwgQmxvZwoKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f\n" - "X19fX19fX18KbG0tc2Vuc29ycyBtYWlsaW5nIGxpc3QKbG0tc2Vuc29yc0BsbS1zZW5zb3JzLm9y\n" - ZwpodHRwOi8vbGlzdHMubG0tc2Vuc29ycy5vcmcvbWFpbG1hbi9saXN0aW5mby9sbS1zZW5zb3Jz + "> >>> > -static int max6650_probe(struct i2c_client *client,\n" + "> >>> > - const struct i2c_device_id *id);\n" + "> >>> > -static int max6650_init_client(struct i2c_client *client);\n" + "> >>> > -static int max6650_remove(struct i2c_client *client);\n" + "> >>> > +static int max6650_probe(struct platform_device *pdev);\n" + "> >>> > +static int max6650_init_client(struct platform_device *pdev);\n" + "> >>> > +static int max6650_remove(struct platform_device *pdev);\n" + "> >>> > static struct max6650_data *max6650_update_device(struct device *dev);\n" + "> >>>\n" + "> >>> It would be good to remove these forward declarations in the future.\n" + "> >>>\n" + "> >>> If no one volunteers I'll happily do it.\n" + "> >>\n" + "> >> Guenter just did:\n" + "> >>\n" + "> >> http://lists.lm-sensors.org/pipermail/lm-sensors/2014-February/041224.html\n" + "> >>\n" + "> >> Any change to the max6650 driver should go on top of his patch series\n" + "> >> to avoid conflicts:\n" + "> >>\n" + "> >> http://lists.lm-sensors.org/pipermail/lm-sensors/2014-February/041223.html\n" + "> >\n" + "> As far as I can see, that patch set was not even tested, so how can it\n" + "> go in? I was told that any patch should be _runtime_ tested, too.\n" + "> Fwiw, I do not have time to test those personally, he would need to\n" + "> find someone else if that requirement really holds true.\n" + ">\n" + "> I would not really like to fix bugs appearing in that code to get my\n" + "> features in.\n" + "> \n" + "> Also, since my change has been around for 2-3 months now, I would\n" + "> really prefer not to be forced to rewrite it again from scratch.\n" + "> Surely, you can wait with those, more or less, cosmetic non-runtime\n" + "> tested changes?\n" + "> \n" + "> This would impose me a lot of additional work again, and I personally\n" + "> do not see the benefit of it. In my book at least, feature is over\n" + "> internal polishing.\n" + "\n" + "Right, I've had enough. I'm removing your patch from the MFD tree.\n" + "\n" + "I've asked too many people to give you a second chance and asked you\n" + "privately to behave yourself and treat others with respect. So far I\n" + "haven't seen an ounce of self control or depomacy from you.\n" + "\n" + "This is how it's going to work from now on:\n" + "\n" + " - You submit a patch\n" + " - It gets reviewed <----\\\n" + " - You fix up the review comments as requested -----/\n" + " - Non-compliance or arguments with the _experts_ results in:\n" + " `$INTEREST > /dev/null || \\\n" + " grep \"From: Laszio Papp\" ~/.mail | xargs rm -rf`\n" + "\n" + "-- \n" + "Lee Jones\n" + "Linaro STMicroelectronics Landing Team Lead\n" + "Linaro.org \342\224\202 Open source software for ARM SoCs\n" + Follow Linaro: Facebook | Twitter | Blog -5babeb96de59c6ccd3cb058abf893503b8bfd2ecd09db847401ff7ff9ed089f3 +cbdd35871112cbe253fa70a1295c7d898b043c2956030f11ab434d11b7190eac
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.