diff for duplicates of <20140213111607.GK32508@lee--X1> diff --git a/a/1.txt b/N1/1.txt index cb33656..72fbad1 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,40 +1,55 @@ -T24gVGh1LCAxMyBGZWIgMjAxNCwgSmVhbiBEZWx2YXJlIHdyb3RlOgoKPiBPbiBUaHUsIDEzIEZl -YiAyMDE0IDA5OjU4OjE3ICswMDAwLCBMZWUgSm9uZXMgd3JvdGU6Cj4gPiA+IFRoZSBNRkQgZHJp -dmVyIGhhcyBub3cgYmVlbiBhZGRlZCwgc28gdGhpcyBkcml2ZXIgaXMgbm93IGJlaW5nIGFkb3B0 -ZWQgdG8gYmUgYQo+ID4gPiBzdWJkZXZpY2UgZHJpdmVyIG9uIHRvcCBvZiBpdC4gVGhpcyBtZWFu -cywgdGhlIGkyYyBkcml2ZXIgdXNhZ2UgaXMgYmVpbmcKPiA+ID4gY29udmVydGVkIHRvIHBsYXRm -b3JtIGRyaXZlciB1c2FnZSBhbGwgYXJvdW5kLgo+ID4gPiAKPiA+ID4gU2lnbmVkLW9mZi1ieTog -TGFzemxvIFBhcHAgPGxwYXBwQGtkZS5vcmc+Cj4gPiA+IC0tLQo+ID4gPiBUaGlzIHBhdGNoIGhh -cyBiZWVuIGNvbXBpbGUgdGVzdGVkIG9ubHkgYW5kIHdpbGwgYmUgdGVzdGVkIHdpdGggcmVhbCBo -YXJkd2FyZSwKPiA+ID4gYnV0IGVhcmx5IHJldmlld3MgdG8gY2F0Y2ggYW55IHRyaXZpYWwgaXNz -dWVzIHdvdWxkIGJlIHdlbGNvbWUuCj4gPiA+ICBkcml2ZXJzL2h3bW9uL0tjb25maWcgICB8ICAg -MiArLQo+ID4gPiAgZHJpdmVycy9od21vbi9tYXg2NjUwLmMgfCAxNTUgKysrKysrKysrKysrKysr -KysrKysrKysrLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tCj4gPiA+ICAyIGZpbGVzIGNoYW5nZWQs -IDc5IGluc2VydGlvbnMoKyksIDc4IGRlbGV0aW9ucygtKQo+ID4gCj4gPiA8c25pcD4KPiA+IAo+ -ID4gPiAgLyoKPiA+ID4gICAqIEluc21vZCBwYXJhbWV0ZXJzCj4gPiA+IEBAIC0xMDUsMjQgKzEw -OCwyMyBAQCBtb2R1bGVfcGFyYW0oY2xvY2ssIGludCwgU19JUlVHTyk7Cj4gPiA+ICAKPiA+ID4g -ICNkZWZpbmUgRElWX0ZST01fUkVHKHJlZykgKDEgPDwgKHJlZyAmIDcpKQo+ID4gPiAgCj4gPiA+ -IC1zdGF0aWMgaW50IG1heDY2NTBfcHJvYmUoc3RydWN0IGkyY19jbGllbnQgKmNsaWVudCwKPiA+ -ID4gLQkJCSBjb25zdCBzdHJ1Y3QgaTJjX2RldmljZV9pZCAqaWQpOwo+ID4gPiAtc3RhdGljIGlu -dCBtYXg2NjUwX2luaXRfY2xpZW50KHN0cnVjdCBpMmNfY2xpZW50ICpjbGllbnQpOwo+ID4gPiAt -c3RhdGljIGludCBtYXg2NjUwX3JlbW92ZShzdHJ1Y3QgaTJjX2NsaWVudCAqY2xpZW50KTsKPiA+ -ID4gK3N0YXRpYyBpbnQgbWF4NjY1MF9wcm9iZShzdHJ1Y3QgcGxhdGZvcm1fZGV2aWNlICpwZGV2 -KTsKPiA+ID4gK3N0YXRpYyBpbnQgbWF4NjY1MF9pbml0X2NsaWVudChzdHJ1Y3QgcGxhdGZvcm1f -ZGV2aWNlICpwZGV2KTsKPiA+ID4gK3N0YXRpYyBpbnQgbWF4NjY1MF9yZW1vdmUoc3RydWN0IHBs -YXRmb3JtX2RldmljZSAqcGRldik7Cj4gPiA+ICBzdGF0aWMgc3RydWN0IG1heDY2NTBfZGF0YSAq -bWF4NjY1MF91cGRhdGVfZGV2aWNlKHN0cnVjdCBkZXZpY2UgKmRldik7Cj4gPiAKPiA+IEl0IHdv -dWxkIGJlIGdvb2QgdG8gcmVtb3ZlIHRoZXNlIGZvcndhcmQgZGVjbGFyYXRpb25zIGluIHRoZSBm -dXR1cmUuCj4gPiAKPiA+IElmIG5vIG9uZSB2b2x1bnRlZXJzIEknbGwgaGFwcGlseSBkbyBpdC4K -PiAKPiBHdWVudGVyIGp1c3QgZGlkOgo+IAo+IGh0dHA6Ly9saXN0cy5sbS1zZW5zb3JzLm9yZy9w -aXBlcm1haWwvbG0tc2Vuc29ycy8yMDE0LUZlYnJ1YXJ5LzA0MTIyNC5odG1sCgpOaWNlLCBGV0lX -OgogIEFja2VkLWJ5OiBMZWUgSm9uZXMgPGxlZS5qb25lc0BsaW5hcm8ub3JnPiAgCgo+IEFueSBj -aGFuZ2UgdG8gdGhlIG1heDY2NTAgZHJpdmVyIHNob3VsZCBnbyBvbiB0b3Agb2YgaGlzIHBhdGNo -IHNlcmllcwo+IHRvIGF2b2lkIGNvbmZsaWN0czoKPiAKPiBodHRwOi8vbGlzdHMubG0tc2Vuc29y -cy5vcmcvcGlwZXJtYWlsL2xtLXNlbnNvcnMvMjAxNC1GZWJydWFyeS8wNDEyMjMuaHRtbAoKRG8g -eW91IGhhdmUgYSB0cmVlIExhc3psbyBjYW4gcmViYXNlIG9uIHRvcCBvZj8KCi0tIApMZWUgSm9u -ZXMKTGluYXJvIFNUTWljcm9lbGVjdHJvbmljcyBMYW5kaW5nIFRlYW0gTGVhZApMaW5hcm8ub3Jn -IOKUgiBPcGVuIHNvdXJjZSBzb2Z0d2FyZSBmb3IgQVJNIFNvQ3MKRm9sbG93IExpbmFybzogRmFj -ZWJvb2sgfCBUd2l0dGVyIHwgQmxvZwoKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f -X19fX19fX19fX19fX18KbG0tc2Vuc29ycyBtYWlsaW5nIGxpc3QKbG0tc2Vuc29yc0BsbS1zZW5z -b3JzLm9yZwpodHRwOi8vbGlzdHMubG0tc2Vuc29ycy5vcmcvbWFpbG1hbi9saXN0aW5mby9sbS1z -ZW5zb3Jz +On Thu, 13 Feb 2014, Jean Delvare wrote: + +> On Thu, 13 Feb 2014 09:58:17 +0000, Lee Jones wrote: +> > > The MFD driver has now been added, so this driver is now being adopted to be a +> > > subdevice driver on top of it. This means, the i2c driver usage is being +> > > converted to platform driver usage all around. +> > > +> > > Signed-off-by: Laszlo Papp <lpapp@kde.org> +> > > --- +> > > This patch has been compile tested only and will be tested with real hardware, +> > > but early reviews to catch any trivial issues would be welcome. +> > > drivers/hwmon/Kconfig | 2 +- +> > > drivers/hwmon/max6650.c | 155 ++++++++++++++++++++++++------------------------ +> > > 2 files changed, 79 insertions(+), 78 deletions(-) +> > +> > <snip> +> > +> > > /* +> > > * Insmod parameters +> > > @@ -105,24 +108,23 @@ module_param(clock, int, S_IRUGO); +> > > +> > > #define DIV_FROM_REG(reg) (1 << (reg & 7)) +> > > +> > > -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 + +Nice, FWIW: + Acked-by: Lee Jones <lee.jones@linaro.org> + +> 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 + +Do you have a tree Laszlo can rebase on top of? + +-- +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 e86cfbe..6f3c108 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -11,45 +11,60 @@ " Guenter Roeck <linux@roeck-us.net>\0" "\00:1\0" "b\0" - "T24gVGh1LCAxMyBGZWIgMjAxNCwgSmVhbiBEZWx2YXJlIHdyb3RlOgoKPiBPbiBUaHUsIDEzIEZl\n" - "YiAyMDE0IDA5OjU4OjE3ICswMDAwLCBMZWUgSm9uZXMgd3JvdGU6Cj4gPiA+IFRoZSBNRkQgZHJp\n" - "dmVyIGhhcyBub3cgYmVlbiBhZGRlZCwgc28gdGhpcyBkcml2ZXIgaXMgbm93IGJlaW5nIGFkb3B0\n" - "ZWQgdG8gYmUgYQo+ID4gPiBzdWJkZXZpY2UgZHJpdmVyIG9uIHRvcCBvZiBpdC4gVGhpcyBtZWFu\n" - "cywgdGhlIGkyYyBkcml2ZXIgdXNhZ2UgaXMgYmVpbmcKPiA+ID4gY29udmVydGVkIHRvIHBsYXRm\n" - "b3JtIGRyaXZlciB1c2FnZSBhbGwgYXJvdW5kLgo+ID4gPiAKPiA+ID4gU2lnbmVkLW9mZi1ieTog\n" - "TGFzemxvIFBhcHAgPGxwYXBwQGtkZS5vcmc+Cj4gPiA+IC0tLQo+ID4gPiBUaGlzIHBhdGNoIGhh\n" - "cyBiZWVuIGNvbXBpbGUgdGVzdGVkIG9ubHkgYW5kIHdpbGwgYmUgdGVzdGVkIHdpdGggcmVhbCBo\n" - "YXJkd2FyZSwKPiA+ID4gYnV0IGVhcmx5IHJldmlld3MgdG8gY2F0Y2ggYW55IHRyaXZpYWwgaXNz\n" - "dWVzIHdvdWxkIGJlIHdlbGNvbWUuCj4gPiA+ICBkcml2ZXJzL2h3bW9uL0tjb25maWcgICB8ICAg\n" - "MiArLQo+ID4gPiAgZHJpdmVycy9od21vbi9tYXg2NjUwLmMgfCAxNTUgKysrKysrKysrKysrKysr\n" - "KysrKysrKysrLS0tLS0tLS0tLS0tLS0tLS0tLS0tLS0tCj4gPiA+ICAyIGZpbGVzIGNoYW5nZWQs\n" - "IDc5IGluc2VydGlvbnMoKyksIDc4IGRlbGV0aW9ucygtKQo+ID4gCj4gPiA8c25pcD4KPiA+IAo+\n" - "ID4gPiAgLyoKPiA+ID4gICAqIEluc21vZCBwYXJhbWV0ZXJzCj4gPiA+IEBAIC0xMDUsMjQgKzEw\n" - "OCwyMyBAQCBtb2R1bGVfcGFyYW0oY2xvY2ssIGludCwgU19JUlVHTyk7Cj4gPiA+ICAKPiA+ID4g\n" - "ICNkZWZpbmUgRElWX0ZST01fUkVHKHJlZykgKDEgPDwgKHJlZyAmIDcpKQo+ID4gPiAgCj4gPiA+\n" - "IC1zdGF0aWMgaW50IG1heDY2NTBfcHJvYmUoc3RydWN0IGkyY19jbGllbnQgKmNsaWVudCwKPiA+\n" - "ID4gLQkJCSBjb25zdCBzdHJ1Y3QgaTJjX2RldmljZV9pZCAqaWQpOwo+ID4gPiAtc3RhdGljIGlu\n" - "dCBtYXg2NjUwX2luaXRfY2xpZW50KHN0cnVjdCBpMmNfY2xpZW50ICpjbGllbnQpOwo+ID4gPiAt\n" - "c3RhdGljIGludCBtYXg2NjUwX3JlbW92ZShzdHJ1Y3QgaTJjX2NsaWVudCAqY2xpZW50KTsKPiA+\n" - "ID4gK3N0YXRpYyBpbnQgbWF4NjY1MF9wcm9iZShzdHJ1Y3QgcGxhdGZvcm1fZGV2aWNlICpwZGV2\n" - "KTsKPiA+ID4gK3N0YXRpYyBpbnQgbWF4NjY1MF9pbml0X2NsaWVudChzdHJ1Y3QgcGxhdGZvcm1f\n" - "ZGV2aWNlICpwZGV2KTsKPiA+ID4gK3N0YXRpYyBpbnQgbWF4NjY1MF9yZW1vdmUoc3RydWN0IHBs\n" - "YXRmb3JtX2RldmljZSAqcGRldik7Cj4gPiA+ICBzdGF0aWMgc3RydWN0IG1heDY2NTBfZGF0YSAq\n" - "bWF4NjY1MF91cGRhdGVfZGV2aWNlKHN0cnVjdCBkZXZpY2UgKmRldik7Cj4gPiAKPiA+IEl0IHdv\n" - "dWxkIGJlIGdvb2QgdG8gcmVtb3ZlIHRoZXNlIGZvcndhcmQgZGVjbGFyYXRpb25zIGluIHRoZSBm\n" - "dXR1cmUuCj4gPiAKPiA+IElmIG5vIG9uZSB2b2x1bnRlZXJzIEknbGwgaGFwcGlseSBkbyBpdC4K\n" - "PiAKPiBHdWVudGVyIGp1c3QgZGlkOgo+IAo+IGh0dHA6Ly9saXN0cy5sbS1zZW5zb3JzLm9yZy9w\n" - "aXBlcm1haWwvbG0tc2Vuc29ycy8yMDE0LUZlYnJ1YXJ5LzA0MTIyNC5odG1sCgpOaWNlLCBGV0lX\n" - "OgogIEFja2VkLWJ5OiBMZWUgSm9uZXMgPGxlZS5qb25lc0BsaW5hcm8ub3JnPiAgCgo+IEFueSBj\n" - "aGFuZ2UgdG8gdGhlIG1heDY2NTAgZHJpdmVyIHNob3VsZCBnbyBvbiB0b3Agb2YgaGlzIHBhdGNo\n" - "IHNlcmllcwo+IHRvIGF2b2lkIGNvbmZsaWN0czoKPiAKPiBodHRwOi8vbGlzdHMubG0tc2Vuc29y\n" - "cy5vcmcvcGlwZXJtYWlsL2xtLXNlbnNvcnMvMjAxNC1GZWJydWFyeS8wNDEyMjMuaHRtbAoKRG8g\n" - "eW91IGhhdmUgYSB0cmVlIExhc3psbyBjYW4gcmViYXNlIG9uIHRvcCBvZj8KCi0tIApMZWUgSm9u\n" - "ZXMKTGluYXJvIFNUTWljcm9lbGVjdHJvbmljcyBMYW5kaW5nIFRlYW0gTGVhZApMaW5hcm8ub3Jn\n" - "IOKUgiBPcGVuIHNvdXJjZSBzb2Z0d2FyZSBmb3IgQVJNIFNvQ3MKRm9sbG93IExpbmFybzogRmFj\n" - "ZWJvb2sgfCBUd2l0dGVyIHwgQmxvZwoKX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f\n" - "X19fX19fX19fX19fX18KbG0tc2Vuc29ycyBtYWlsaW5nIGxpc3QKbG0tc2Vuc29yc0BsbS1zZW5z\n" - "b3JzLm9yZwpodHRwOi8vbGlzdHMubG0tc2Vuc29ycy5vcmcvbWFpbG1hbi9saXN0aW5mby9sbS1z\n" - ZW5zb3Jz + "On Thu, 13 Feb 2014, Jean Delvare wrote:\n" + "\n" + "> On Thu, 13 Feb 2014 09:58:17 +0000, Lee Jones wrote:\n" + "> > > The MFD driver has now been added, so this driver is now being adopted to be a\n" + "> > > subdevice driver on top of it. This means, the i2c driver usage is being\n" + "> > > converted to platform driver usage all around.\n" + "> > > \n" + "> > > Signed-off-by: Laszlo Papp <lpapp@kde.org>\n" + "> > > ---\n" + "> > > This patch has been compile tested only and will be tested with real hardware,\n" + "> > > but early reviews to catch any trivial issues would be welcome.\n" + "> > > drivers/hwmon/Kconfig | 2 +-\n" + "> > > drivers/hwmon/max6650.c | 155 ++++++++++++++++++++++++------------------------\n" + "> > > 2 files changed, 79 insertions(+), 78 deletions(-)\n" + "> > \n" + "> > <snip>\n" + "> > \n" + "> > > /*\n" + "> > > * Insmod parameters\n" + "> > > @@ -105,24 +108,23 @@ module_param(clock, int, S_IRUGO);\n" + "> > > \n" + "> > > #define DIV_FROM_REG(reg) (1 << (reg & 7))\n" + "> > > \n" + "> > > -static int max6650_probe(struct i2c_client *client,\n" + "> > > -\t\t\t 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" + "Nice, FWIW:\n" + " Acked-by: Lee Jones <lee.jones@linaro.org> \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" + "Do you have a tree Laszlo can rebase on top of?\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 -363c87d7c2cb6072f19c5f2e78fa945cc7381d936eb3151f63f0ac5db3536cfb +13692b14bd5a2642be0c93b0c772c0a5cdfcad318791c7bc82be163438ed91bd
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.