diff for duplicates of <4518414.hTLWuvFlBl@wuerfel> diff --git a/a/1.txt b/N1/1.txt index cc3e26b..91b51b9 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,29 +1,35 @@ -T24gV2VkbmVzZGF5LCBTZXB0ZW1iZXIgMjEsIDIwMTYgNjoxMjoyOCBQTSBDRVNUIHpoaWNoYW5n -IHdyb3RlOgo+IE9uIDIwMTblubQwOeaciDE15pelIDA1OjMzLCBBcm5kIEJlcmdtYW5uIHdyb3Rl -Ogo+ID4gT24gV2VkbmVzZGF5LCBTZXB0ZW1iZXIgMTQsIDIwMTYgMTE6MDQ6MzMgUE0gQ0VTVCB6 -aGljaGFuZy55dWFuIHdyb3RlOgo+ID4+IFRoZSA4MjUwX2hpc2lfbHBjLmMgc3VwcG9ydCBib3Ro -IEFDUEkgYW5kIGR0cyBzaW1pbGFyIHRvIDgyNTBfZHcgOgo+ID4+Cj4gPj4gK3N0YXRpYyBzdHJ1 -Y3QgcGxhdGZvcm1fZHJpdmVyIGhzX2xwYzgyNTBfZHJpdmVyID0gewo+ID4+ICsgICAgICAgLmRy -aXZlciA9IHsKPiA+PiArICAgICAgICAgICAgICAgLm5hbWUgICAgICAgICAgID0gImhpc2ktbHBj -LXVhcnQiLAo+ID4+ICsgICAgICAgICAgICAgICAub2ZfbWF0Y2hfdGFibGUgPSBoczgyNTBfb2Zf -bWF0Y2gsCj4gPj4gKyAgICAgICAgICAgICAgIC5hY3BpX21hdGNoX3RhYmxlID0gQUNQSV9QVFIo -aHM4MjUwX2FjcGlfbWF0Y2gpLAo+ID4+Cj4gPj4gU28sIEkgYW0gYSBsaXR0bGUgY29uZnVzZWQg -d2h5IHdlIG5lZWQgdG8gc3VwcG9ydCBkdHMgaW4gODI1MF9vZi5jIGFuZCBzdXBwb3J0IEFDUEkg -aW4gYW5vdGhlcgo+ID4+IGRyaXZlciBmaWxlLgo+ID4+Cj4gPiAKPiA+IEFmdGVyIGxvb2tpbmcg -YWdhaW4sIEknbSBwcmV0dHkgc3VyZSB0aGF0IGRyaXZlcnMvdHR5L3NlcmlhbC84MjUwLzgyNTBf -cG5wLmMKPiA+IHdpbGwgaGFuZGxlIHRoZSBBQ1BJIGNhc2Ugd2l0aG91dCBtb2RpZmljYXRpb25z -LCB5b3UganVzdCBuZWVkIHRvIGFkYXB0Cj4gPiB0aGUgODI1MF9vZiBkcml2ZXIgdG8gaGFuZGxl -IElPUkVTT1VSQ0VfSU8gcG9ydHMuIFRoaXMgd2lsbCBiZSByZXF1aXJlZAo+ID4gYW55d2F5IHdp -dGggdGhlIG5leHQgdmVyc2lvbiBvbmNlIHRoZSBMUEMgYnJpZGdlIGlzIG9uIHRoZSBQQ0kgYnVz -Lgo+ID4gCj4gCj4gWWVzLiBZb3UgYXJlIHJpZ2h0Lgo+IAo+IFdoZW4gdGhlIGluZGlyZWN0SU8g -cGF0Y2ggaXMgYWNjZXB0ZWQsIG5vIG11Y2ggY2hhbmdlcyBuZWVkZWQgaW4gdGhlc2UgdHdvIGZp -bGUgdG8gc3VwcG9ydCBMUEMgVUFSVC4KPiAKPiBCVFcsIHdoYXQgaXMgeW91ciB0YXJnZXQgdG8g -Y2hhbmdlIHRoZXNlIHR3byBmaWxlcyB0byBzdXBwb3J0IExQQyB1YXJ0Pwo+IERvIHlvdSB0aGlu -ayB3ZSBkb24ndCBuZWVkIHRvIGFkZCBtYW55IG5ldyBjIGZpbGVzPz8KCkkgdGhpbmsgYnkgbW9k -aWZ5aW5nIDgyNTBfb2YuYywgeW91IHNob3VsZCBiZSBhYmxlIHRvIG1ha2UgYW55IFVBUlQgb24K -c29tZSBMUEMgYnVzIHdvcmssIHJlZ2FyZGxlc3Mgb2Ygd2hhdCBjb250cm9sbGVyIGlzIHVzZWQu -CgpUaGUgQUNQSSB2ZXJzaW9uICg4MjUwX3BucC5jKSBzaG91bGRuJ3QgbmVlZCBhbnkgY2hhbmdl -cyBhdCBhbGwgaWYgSQpyZWFkIHRoYXQgY29ycmVjdGx5LgoKCUFybmQKCl9fX19fX19fX19fX19f -X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCmxpbnV4LWFybS1rZXJuZWwgbWFpbGlu -ZyBsaXN0CmxpbnV4LWFybS1rZXJuZWxAbGlzdHMuaW5mcmFkZWFkLm9yZwpodHRwOi8vbGlzdHMu -aW5mcmFkZWFkLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2xpbnV4LWFybS1rZXJuZWwK +On Wednesday, September 21, 2016 6:12:28 PM CEST zhichang wrote: +> On 2016年09月15日 05:33, Arnd Bergmann wrote: +> > On Wednesday, September 14, 2016 11:04:33 PM CEST zhichang.yuan wrote: +> >> The 8250_hisi_lpc.c support both ACPI and dts similar to 8250_dw : +> >> +> >> +static struct platform_driver hs_lpc8250_driver = { +> >> + .driver = { +> >> + .name = "hisi-lpc-uart", +> >> + .of_match_table = hs8250_of_match, +> >> + .acpi_match_table = ACPI_PTR(hs8250_acpi_match), +> >> +> >> So, I am a little confused why we need to support dts in 8250_of.c and support ACPI in another +> >> driver file. +> >> +> > +> > After looking again, I'm pretty sure that drivers/tty/serial/8250/8250_pnp.c +> > will handle the ACPI case without modifications, you just need to adapt +> > the 8250_of driver to handle IORESOURCE_IO ports. This will be required +> > anyway with the next version once the LPC bridge is on the PCI bus. +> > +> +> Yes. You are right. +> +> When the indirectIO patch is accepted, no much changes needed in these two file to support LPC UART. +> +> BTW, what is your target to change these two files to support LPC uart? +> Do you think we don't need to add many new c files?? + +I think by modifying 8250_of.c, you should be able to make any UART on +some LPC bus work, regardless of what controller is used. + +The ACPI version (8250_pnp.c) shouldn't need any changes at all if I +read that correctly. + + Arnd diff --git a/a/content_digest b/N1/content_digest index e99cbc0..a942c37 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -5,54 +5,60 @@ "Subject\0Re: [PATCH V3 3/4] ARM64 LPC: support serial based on low-pin-count\0" "Date\0Wed, 21 Sep 2016 21:29:22 +0200\0" "To\0zhichang <zhichang.yuan02@gmail.com>\0" - "Cc\0devicetree@vger.kernel.org" + "Cc\0linux-arm-kernel@lists.infradead.org" + zhichang.yuan <yuanzhichang@hisilicon.com> + devicetree@vger.kernel.org lorenzo.pieralisi@arm.com gabriele.paoloni@huawei.com minyard@acm.org - linux-pci@vger.kernel.org gregkh@linuxfoundation.org + benh@kernel.crashing.org john.garry@huawei.com will.deacon@arm.com linux-kernel@vger.kernel.org - zhichang.yuan <yuanzhichang@hisilicon.com> - linuxarm@huawei.com xuwei5@hisilicon.com + linuxarm@huawei.com linux-serial@vger.kernel.org - benh@kernel.crashing.org + linux-pci@vger.kernel.org zourongrong@gmail.com liviu.dudau@arm.com - kantyzc@163.com - " linux-arm-kernel@lists.infradead.org\0" + " kantyzc@163.com\0" "\00:1\0" "b\0" - "T24gV2VkbmVzZGF5LCBTZXB0ZW1iZXIgMjEsIDIwMTYgNjoxMjoyOCBQTSBDRVNUIHpoaWNoYW5n\n" - "IHdyb3RlOgo+IE9uIDIwMTblubQwOeaciDE15pelIDA1OjMzLCBBcm5kIEJlcmdtYW5uIHdyb3Rl\n" - "Ogo+ID4gT24gV2VkbmVzZGF5LCBTZXB0ZW1iZXIgMTQsIDIwMTYgMTE6MDQ6MzMgUE0gQ0VTVCB6\n" - "aGljaGFuZy55dWFuIHdyb3RlOgo+ID4+IFRoZSA4MjUwX2hpc2lfbHBjLmMgc3VwcG9ydCBib3Ro\n" - "IEFDUEkgYW5kIGR0cyBzaW1pbGFyIHRvIDgyNTBfZHcgOgo+ID4+Cj4gPj4gK3N0YXRpYyBzdHJ1\n" - "Y3QgcGxhdGZvcm1fZHJpdmVyIGhzX2xwYzgyNTBfZHJpdmVyID0gewo+ID4+ICsgICAgICAgLmRy\n" - "aXZlciA9IHsKPiA+PiArICAgICAgICAgICAgICAgLm5hbWUgICAgICAgICAgID0gImhpc2ktbHBj\n" - "LXVhcnQiLAo+ID4+ICsgICAgICAgICAgICAgICAub2ZfbWF0Y2hfdGFibGUgPSBoczgyNTBfb2Zf\n" - "bWF0Y2gsCj4gPj4gKyAgICAgICAgICAgICAgIC5hY3BpX21hdGNoX3RhYmxlID0gQUNQSV9QVFIo\n" - "aHM4MjUwX2FjcGlfbWF0Y2gpLAo+ID4+Cj4gPj4gU28sIEkgYW0gYSBsaXR0bGUgY29uZnVzZWQg\n" - "d2h5IHdlIG5lZWQgdG8gc3VwcG9ydCBkdHMgaW4gODI1MF9vZi5jIGFuZCBzdXBwb3J0IEFDUEkg\n" - "aW4gYW5vdGhlcgo+ID4+IGRyaXZlciBmaWxlLgo+ID4+Cj4gPiAKPiA+IEFmdGVyIGxvb2tpbmcg\n" - "YWdhaW4sIEknbSBwcmV0dHkgc3VyZSB0aGF0IGRyaXZlcnMvdHR5L3NlcmlhbC84MjUwLzgyNTBf\n" - "cG5wLmMKPiA+IHdpbGwgaGFuZGxlIHRoZSBBQ1BJIGNhc2Ugd2l0aG91dCBtb2RpZmljYXRpb25z\n" - "LCB5b3UganVzdCBuZWVkIHRvIGFkYXB0Cj4gPiB0aGUgODI1MF9vZiBkcml2ZXIgdG8gaGFuZGxl\n" - "IElPUkVTT1VSQ0VfSU8gcG9ydHMuIFRoaXMgd2lsbCBiZSByZXF1aXJlZAo+ID4gYW55d2F5IHdp\n" - "dGggdGhlIG5leHQgdmVyc2lvbiBvbmNlIHRoZSBMUEMgYnJpZGdlIGlzIG9uIHRoZSBQQ0kgYnVz\n" - "Lgo+ID4gCj4gCj4gWWVzLiBZb3UgYXJlIHJpZ2h0Lgo+IAo+IFdoZW4gdGhlIGluZGlyZWN0SU8g\n" - "cGF0Y2ggaXMgYWNjZXB0ZWQsIG5vIG11Y2ggY2hhbmdlcyBuZWVkZWQgaW4gdGhlc2UgdHdvIGZp\n" - "bGUgdG8gc3VwcG9ydCBMUEMgVUFSVC4KPiAKPiBCVFcsIHdoYXQgaXMgeW91ciB0YXJnZXQgdG8g\n" - "Y2hhbmdlIHRoZXNlIHR3byBmaWxlcyB0byBzdXBwb3J0IExQQyB1YXJ0Pwo+IERvIHlvdSB0aGlu\n" - "ayB3ZSBkb24ndCBuZWVkIHRvIGFkZCBtYW55IG5ldyBjIGZpbGVzPz8KCkkgdGhpbmsgYnkgbW9k\n" - "aWZ5aW5nIDgyNTBfb2YuYywgeW91IHNob3VsZCBiZSBhYmxlIHRvIG1ha2UgYW55IFVBUlQgb24K\n" - "c29tZSBMUEMgYnVzIHdvcmssIHJlZ2FyZGxlc3Mgb2Ygd2hhdCBjb250cm9sbGVyIGlzIHVzZWQu\n" - "CgpUaGUgQUNQSSB2ZXJzaW9uICg4MjUwX3BucC5jKSBzaG91bGRuJ3QgbmVlZCBhbnkgY2hhbmdl\n" - "cyBhdCBhbGwgaWYgSQpyZWFkIHRoYXQgY29ycmVjdGx5LgoKCUFybmQKCl9fX19fX19fX19fX19f\n" - "X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCmxpbnV4LWFybS1rZXJuZWwgbWFpbGlu\n" - "ZyBsaXN0CmxpbnV4LWFybS1rZXJuZWxAbGlzdHMuaW5mcmFkZWFkLm9yZwpodHRwOi8vbGlzdHMu\n" - aW5mcmFkZWFkLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2xpbnV4LWFybS1rZXJuZWwK + "On Wednesday, September 21, 2016 6:12:28 PM CEST zhichang wrote:\n" + "> On 2016\345\271\26409\346\234\21015\346\227\245 05:33, Arnd Bergmann wrote:\n" + "> > On Wednesday, September 14, 2016 11:04:33 PM CEST zhichang.yuan wrote:\n" + "> >> The 8250_hisi_lpc.c support both ACPI and dts similar to 8250_dw :\n" + "> >>\n" + "> >> +static struct platform_driver hs_lpc8250_driver = {\n" + "> >> + .driver = {\n" + "> >> + .name = \"hisi-lpc-uart\",\n" + "> >> + .of_match_table = hs8250_of_match,\n" + "> >> + .acpi_match_table = ACPI_PTR(hs8250_acpi_match),\n" + "> >>\n" + "> >> So, I am a little confused why we need to support dts in 8250_of.c and support ACPI in another\n" + "> >> driver file.\n" + "> >>\n" + "> > \n" + "> > After looking again, I'm pretty sure that drivers/tty/serial/8250/8250_pnp.c\n" + "> > will handle the ACPI case without modifications, you just need to adapt\n" + "> > the 8250_of driver to handle IORESOURCE_IO ports. This will be required\n" + "> > anyway with the next version once the LPC bridge is on the PCI bus.\n" + "> > \n" + "> \n" + "> Yes. You are right.\n" + "> \n" + "> When the indirectIO patch is accepted, no much changes needed in these two file to support LPC UART.\n" + "> \n" + "> BTW, what is your target to change these two files to support LPC uart?\n" + "> Do you think we don't need to add many new c files??\n" + "\n" + "I think by modifying 8250_of.c, you should be able to make any UART on\n" + "some LPC bus work, regardless of what controller is used.\n" + "\n" + "The ACPI version (8250_pnp.c) shouldn't need any changes at all if I\n" + "read that correctly.\n" + "\n" + "\tArnd" -24522c2fdd2a6da88db2aeabcc55175caac9ea42e28d091796e5bcd9cfd4ba0c +f3fa3a41a314ed9684ff579d172f943e4f512aaf8a76dc84fa11c41a36de8e21
diff --git a/a/1.txt b/N2/1.txt index cc3e26b..eb12040 100644 --- a/a/1.txt +++ b/N2/1.txt @@ -1,29 +1,35 @@ -T24gV2VkbmVzZGF5LCBTZXB0ZW1iZXIgMjEsIDIwMTYgNjoxMjoyOCBQTSBDRVNUIHpoaWNoYW5n -IHdyb3RlOgo+IE9uIDIwMTblubQwOeaciDE15pelIDA1OjMzLCBBcm5kIEJlcmdtYW5uIHdyb3Rl -Ogo+ID4gT24gV2VkbmVzZGF5LCBTZXB0ZW1iZXIgMTQsIDIwMTYgMTE6MDQ6MzMgUE0gQ0VTVCB6 -aGljaGFuZy55dWFuIHdyb3RlOgo+ID4+IFRoZSA4MjUwX2hpc2lfbHBjLmMgc3VwcG9ydCBib3Ro -IEFDUEkgYW5kIGR0cyBzaW1pbGFyIHRvIDgyNTBfZHcgOgo+ID4+Cj4gPj4gK3N0YXRpYyBzdHJ1 -Y3QgcGxhdGZvcm1fZHJpdmVyIGhzX2xwYzgyNTBfZHJpdmVyID0gewo+ID4+ICsgICAgICAgLmRy -aXZlciA9IHsKPiA+PiArICAgICAgICAgICAgICAgLm5hbWUgICAgICAgICAgID0gImhpc2ktbHBj -LXVhcnQiLAo+ID4+ICsgICAgICAgICAgICAgICAub2ZfbWF0Y2hfdGFibGUgPSBoczgyNTBfb2Zf -bWF0Y2gsCj4gPj4gKyAgICAgICAgICAgICAgIC5hY3BpX21hdGNoX3RhYmxlID0gQUNQSV9QVFIo -aHM4MjUwX2FjcGlfbWF0Y2gpLAo+ID4+Cj4gPj4gU28sIEkgYW0gYSBsaXR0bGUgY29uZnVzZWQg -d2h5IHdlIG5lZWQgdG8gc3VwcG9ydCBkdHMgaW4gODI1MF9vZi5jIGFuZCBzdXBwb3J0IEFDUEkg -aW4gYW5vdGhlcgo+ID4+IGRyaXZlciBmaWxlLgo+ID4+Cj4gPiAKPiA+IEFmdGVyIGxvb2tpbmcg -YWdhaW4sIEknbSBwcmV0dHkgc3VyZSB0aGF0IGRyaXZlcnMvdHR5L3NlcmlhbC84MjUwLzgyNTBf -cG5wLmMKPiA+IHdpbGwgaGFuZGxlIHRoZSBBQ1BJIGNhc2Ugd2l0aG91dCBtb2RpZmljYXRpb25z -LCB5b3UganVzdCBuZWVkIHRvIGFkYXB0Cj4gPiB0aGUgODI1MF9vZiBkcml2ZXIgdG8gaGFuZGxl -IElPUkVTT1VSQ0VfSU8gcG9ydHMuIFRoaXMgd2lsbCBiZSByZXF1aXJlZAo+ID4gYW55d2F5IHdp -dGggdGhlIG5leHQgdmVyc2lvbiBvbmNlIHRoZSBMUEMgYnJpZGdlIGlzIG9uIHRoZSBQQ0kgYnVz -Lgo+ID4gCj4gCj4gWWVzLiBZb3UgYXJlIHJpZ2h0Lgo+IAo+IFdoZW4gdGhlIGluZGlyZWN0SU8g -cGF0Y2ggaXMgYWNjZXB0ZWQsIG5vIG11Y2ggY2hhbmdlcyBuZWVkZWQgaW4gdGhlc2UgdHdvIGZp -bGUgdG8gc3VwcG9ydCBMUEMgVUFSVC4KPiAKPiBCVFcsIHdoYXQgaXMgeW91ciB0YXJnZXQgdG8g -Y2hhbmdlIHRoZXNlIHR3byBmaWxlcyB0byBzdXBwb3J0IExQQyB1YXJ0Pwo+IERvIHlvdSB0aGlu -ayB3ZSBkb24ndCBuZWVkIHRvIGFkZCBtYW55IG5ldyBjIGZpbGVzPz8KCkkgdGhpbmsgYnkgbW9k -aWZ5aW5nIDgyNTBfb2YuYywgeW91IHNob3VsZCBiZSBhYmxlIHRvIG1ha2UgYW55IFVBUlQgb24K -c29tZSBMUEMgYnVzIHdvcmssIHJlZ2FyZGxlc3Mgb2Ygd2hhdCBjb250cm9sbGVyIGlzIHVzZWQu -CgpUaGUgQUNQSSB2ZXJzaW9uICg4MjUwX3BucC5jKSBzaG91bGRuJ3QgbmVlZCBhbnkgY2hhbmdl -cyBhdCBhbGwgaWYgSQpyZWFkIHRoYXQgY29ycmVjdGx5LgoKCUFybmQKCl9fX19fX19fX19fX19f -X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCmxpbnV4LWFybS1rZXJuZWwgbWFpbGlu -ZyBsaXN0CmxpbnV4LWFybS1rZXJuZWxAbGlzdHMuaW5mcmFkZWFkLm9yZwpodHRwOi8vbGlzdHMu -aW5mcmFkZWFkLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2xpbnV4LWFybS1rZXJuZWwK +On Wednesday, September 21, 2016 6:12:28 PM CEST zhichang wrote: +> On 2016?09?15? 05:33, Arnd Bergmann wrote: +> > On Wednesday, September 14, 2016 11:04:33 PM CEST zhichang.yuan wrote: +> >> The 8250_hisi_lpc.c support both ACPI and dts similar to 8250_dw : +> >> +> >> +static struct platform_driver hs_lpc8250_driver = { +> >> + .driver = { +> >> + .name = "hisi-lpc-uart", +> >> + .of_match_table = hs8250_of_match, +> >> + .acpi_match_table = ACPI_PTR(hs8250_acpi_match), +> >> +> >> So, I am a little confused why we need to support dts in 8250_of.c and support ACPI in another +> >> driver file. +> >> +> > +> > After looking again, I'm pretty sure that drivers/tty/serial/8250/8250_pnp.c +> > will handle the ACPI case without modifications, you just need to adapt +> > the 8250_of driver to handle IORESOURCE_IO ports. This will be required +> > anyway with the next version once the LPC bridge is on the PCI bus. +> > +> +> Yes. You are right. +> +> When the indirectIO patch is accepted, no much changes needed in these two file to support LPC UART. +> +> BTW, what is your target to change these two files to support LPC uart? +> Do you think we don't need to add many new c files?? + +I think by modifying 8250_of.c, you should be able to make any UART on +some LPC bus work, regardless of what controller is used. + +The ACPI version (8250_pnp.c) shouldn't need any changes at all if I +read that correctly. + + Arnd diff --git a/a/content_digest b/N2/content_digest index e99cbc0..a1db8c1 100644 --- a/a/content_digest +++ b/N2/content_digest @@ -1,58 +1,46 @@ "ref\01473855354-150093-1-git-send-email-yuanzhichang@hisilicon.com\0" "ref\05525856.MfDSSsiP5d@wuerfel\0" "ref\0815bebc1-96c9-2131-930d-bccdd4bf1c55@gmail.com\0" - "From\0Arnd Bergmann <arnd@arndb.de>\0" - "Subject\0Re: [PATCH V3 3/4] ARM64 LPC: support serial based on low-pin-count\0" + "From\0arnd@arndb.de (Arnd Bergmann)\0" + "Subject\0[PATCH V3 3/4] ARM64 LPC: support serial based on low-pin-count\0" "Date\0Wed, 21 Sep 2016 21:29:22 +0200\0" - "To\0zhichang <zhichang.yuan02@gmail.com>\0" - "Cc\0devicetree@vger.kernel.org" - lorenzo.pieralisi@arm.com - gabriele.paoloni@huawei.com - minyard@acm.org - linux-pci@vger.kernel.org - gregkh@linuxfoundation.org - john.garry@huawei.com - will.deacon@arm.com - linux-kernel@vger.kernel.org - zhichang.yuan <yuanzhichang@hisilicon.com> - linuxarm@huawei.com - xuwei5@hisilicon.com - linux-serial@vger.kernel.org - benh@kernel.crashing.org - zourongrong@gmail.com - liviu.dudau@arm.com - kantyzc@163.com - " linux-arm-kernel@lists.infradead.org\0" + "To\0linux-arm-kernel@lists.infradead.org\0" "\00:1\0" "b\0" - "T24gV2VkbmVzZGF5LCBTZXB0ZW1iZXIgMjEsIDIwMTYgNjoxMjoyOCBQTSBDRVNUIHpoaWNoYW5n\n" - "IHdyb3RlOgo+IE9uIDIwMTblubQwOeaciDE15pelIDA1OjMzLCBBcm5kIEJlcmdtYW5uIHdyb3Rl\n" - "Ogo+ID4gT24gV2VkbmVzZGF5LCBTZXB0ZW1iZXIgMTQsIDIwMTYgMTE6MDQ6MzMgUE0gQ0VTVCB6\n" - "aGljaGFuZy55dWFuIHdyb3RlOgo+ID4+IFRoZSA4MjUwX2hpc2lfbHBjLmMgc3VwcG9ydCBib3Ro\n" - "IEFDUEkgYW5kIGR0cyBzaW1pbGFyIHRvIDgyNTBfZHcgOgo+ID4+Cj4gPj4gK3N0YXRpYyBzdHJ1\n" - "Y3QgcGxhdGZvcm1fZHJpdmVyIGhzX2xwYzgyNTBfZHJpdmVyID0gewo+ID4+ICsgICAgICAgLmRy\n" - "aXZlciA9IHsKPiA+PiArICAgICAgICAgICAgICAgLm5hbWUgICAgICAgICAgID0gImhpc2ktbHBj\n" - "LXVhcnQiLAo+ID4+ICsgICAgICAgICAgICAgICAub2ZfbWF0Y2hfdGFibGUgPSBoczgyNTBfb2Zf\n" - "bWF0Y2gsCj4gPj4gKyAgICAgICAgICAgICAgIC5hY3BpX21hdGNoX3RhYmxlID0gQUNQSV9QVFIo\n" - "aHM4MjUwX2FjcGlfbWF0Y2gpLAo+ID4+Cj4gPj4gU28sIEkgYW0gYSBsaXR0bGUgY29uZnVzZWQg\n" - "d2h5IHdlIG5lZWQgdG8gc3VwcG9ydCBkdHMgaW4gODI1MF9vZi5jIGFuZCBzdXBwb3J0IEFDUEkg\n" - "aW4gYW5vdGhlcgo+ID4+IGRyaXZlciBmaWxlLgo+ID4+Cj4gPiAKPiA+IEFmdGVyIGxvb2tpbmcg\n" - "YWdhaW4sIEknbSBwcmV0dHkgc3VyZSB0aGF0IGRyaXZlcnMvdHR5L3NlcmlhbC84MjUwLzgyNTBf\n" - "cG5wLmMKPiA+IHdpbGwgaGFuZGxlIHRoZSBBQ1BJIGNhc2Ugd2l0aG91dCBtb2RpZmljYXRpb25z\n" - "LCB5b3UganVzdCBuZWVkIHRvIGFkYXB0Cj4gPiB0aGUgODI1MF9vZiBkcml2ZXIgdG8gaGFuZGxl\n" - "IElPUkVTT1VSQ0VfSU8gcG9ydHMuIFRoaXMgd2lsbCBiZSByZXF1aXJlZAo+ID4gYW55d2F5IHdp\n" - "dGggdGhlIG5leHQgdmVyc2lvbiBvbmNlIHRoZSBMUEMgYnJpZGdlIGlzIG9uIHRoZSBQQ0kgYnVz\n" - "Lgo+ID4gCj4gCj4gWWVzLiBZb3UgYXJlIHJpZ2h0Lgo+IAo+IFdoZW4gdGhlIGluZGlyZWN0SU8g\n" - "cGF0Y2ggaXMgYWNjZXB0ZWQsIG5vIG11Y2ggY2hhbmdlcyBuZWVkZWQgaW4gdGhlc2UgdHdvIGZp\n" - "bGUgdG8gc3VwcG9ydCBMUEMgVUFSVC4KPiAKPiBCVFcsIHdoYXQgaXMgeW91ciB0YXJnZXQgdG8g\n" - "Y2hhbmdlIHRoZXNlIHR3byBmaWxlcyB0byBzdXBwb3J0IExQQyB1YXJ0Pwo+IERvIHlvdSB0aGlu\n" - "ayB3ZSBkb24ndCBuZWVkIHRvIGFkZCBtYW55IG5ldyBjIGZpbGVzPz8KCkkgdGhpbmsgYnkgbW9k\n" - "aWZ5aW5nIDgyNTBfb2YuYywgeW91IHNob3VsZCBiZSBhYmxlIHRvIG1ha2UgYW55IFVBUlQgb24K\n" - "c29tZSBMUEMgYnVzIHdvcmssIHJlZ2FyZGxlc3Mgb2Ygd2hhdCBjb250cm9sbGVyIGlzIHVzZWQu\n" - "CgpUaGUgQUNQSSB2ZXJzaW9uICg4MjUwX3BucC5jKSBzaG91bGRuJ3QgbmVlZCBhbnkgY2hhbmdl\n" - "cyBhdCBhbGwgaWYgSQpyZWFkIHRoYXQgY29ycmVjdGx5LgoKCUFybmQKCl9fX19fX19fX19fX19f\n" - "X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCmxpbnV4LWFybS1rZXJuZWwgbWFpbGlu\n" - "ZyBsaXN0CmxpbnV4LWFybS1rZXJuZWxAbGlzdHMuaW5mcmFkZWFkLm9yZwpodHRwOi8vbGlzdHMu\n" - aW5mcmFkZWFkLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2xpbnV4LWFybS1rZXJuZWwK + "On Wednesday, September 21, 2016 6:12:28 PM CEST zhichang wrote:\n" + "> On 2016?09?15? 05:33, Arnd Bergmann wrote:\n" + "> > On Wednesday, September 14, 2016 11:04:33 PM CEST zhichang.yuan wrote:\n" + "> >> The 8250_hisi_lpc.c support both ACPI and dts similar to 8250_dw :\n" + "> >>\n" + "> >> +static struct platform_driver hs_lpc8250_driver = {\n" + "> >> + .driver = {\n" + "> >> + .name = \"hisi-lpc-uart\",\n" + "> >> + .of_match_table = hs8250_of_match,\n" + "> >> + .acpi_match_table = ACPI_PTR(hs8250_acpi_match),\n" + "> >>\n" + "> >> So, I am a little confused why we need to support dts in 8250_of.c and support ACPI in another\n" + "> >> driver file.\n" + "> >>\n" + "> > \n" + "> > After looking again, I'm pretty sure that drivers/tty/serial/8250/8250_pnp.c\n" + "> > will handle the ACPI case without modifications, you just need to adapt\n" + "> > the 8250_of driver to handle IORESOURCE_IO ports. This will be required\n" + "> > anyway with the next version once the LPC bridge is on the PCI bus.\n" + "> > \n" + "> \n" + "> Yes. You are right.\n" + "> \n" + "> When the indirectIO patch is accepted, no much changes needed in these two file to support LPC UART.\n" + "> \n" + "> BTW, what is your target to change these two files to support LPC uart?\n" + "> Do you think we don't need to add many new c files??\n" + "\n" + "I think by modifying 8250_of.c, you should be able to make any UART on\n" + "some LPC bus work, regardless of what controller is used.\n" + "\n" + "The ACPI version (8250_pnp.c) shouldn't need any changes at all if I\n" + "read that correctly.\n" + "\n" + "\tArnd" -24522c2fdd2a6da88db2aeabcc55175caac9ea42e28d091796e5bcd9cfd4ba0c +891b09bc755e98823f5914a232f37418fe37c373ca9e7222b8261fe8e5111a0d
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.