From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tony Lindgren Subject: Re: [PATCHv4] phy: mapphone-mdm6600: Add USB PHY driver for MDM6600 on Droid 4 Date: Sun, 25 Mar 2018 08:45:12 -0700 Message-ID: <20180325154512.GC5700@atomide.com> References: <20180322135553.GA27648@amd> <20180322164633.GF5799@atomide.com> <20180322192843.GA17189@amd> <20180323105455.GB21644@amd> <20180323113521.45va72bhzis6ym6m@earth.universe> <20180323201343.GC23658@amd> <20180324142517.GA94954@atomide.com> <20180324200224.GA17857@amd> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20180324200224.GA17857@amd> Sender: linux-kernel-owner@vger.kernel.org To: Pavel Machek Cc: Dan Williams , Sebastian Reichel , Kishon Vijay Abraham I , linux-kernel@vger.kernel.org, linux-usb@vger.kernel.org, linux-omap@vger.kernel.org, devicetree@vger.kernel.org, Mark Rutland , Marcel Partap , Michael Scott , Rob Herring List-Id: linux-omap@vger.kernel.org * Pavel Machek [180324 20:03]: > On Sat 2018-03-24 07:25:17, Tony Lindgren wrote: > > * Dan Williams [180324 14:00]: > > > On Fri, 2018-03-23 at 21:13 +0100, Pavel Machek wrote: > > > > Does ofonod work for you? I could not get that one to work... > > > > > > Because it's looking for a Gobi modem but the MDM6600 isn't one and > > > doesn't expose that layout (and doesn't really need to anyway). I > > > don't think ofono has a generic QMI driver, so you'd either need to for > > > ce it to use the telitqmi or quectelqmi drivers, or write your own > > > generic QMI one. > > > > We also get five USB uarts if we add the device id with something > > like the patch below. I don't quite get why we get five UARTS? > > Not sure, either. Often more than one uart is useful, you get AT > commands on one, while GPRS data flow on some other. > > > Also not sure if we should be using drivers/usb/serial/qcaux.c > > instead of qcserial.c? > > > > And from what I recall trying it out, adding the USB UARTs > > somehow confused ModemManager I think, that needs to be retested > > though :) > > > > And the USB UARTs added do not offer the same set of AT commands > > as the n_gsm serial mux. > > Hmm. Interesting. Anyway, for me ttyUSB4 is interesting, as it seems > to react to AT commands, and in particular reacts to ADT123; (; is > important). Is that to dial a voice call? > AT+CMGF=1 > AT+CMGS="123" > foo^Z > > Works for SMS sending. Good. So what do you use for reading notifying and reading sms? > Now, if someone knows what needs to be done to get GSM audio working, > let me know. That's something I'd really like. I think it's the cpcap based config to route voice call audio to SoC, Sebastian knows the details :) The way to figure that one out is to dump the cpcap registers before and during voice call on android with cpcaprw, then diff the output for the audio registers. Probably some SoC registers need to be diffed too with rwmem or similar tool for the mcbsp instance(s) used. Regards, Tony From mboxrd@z Thu Jan 1 00:00:00 1970 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Subject: [PATCHv4] phy: mapphone-mdm6600: Add USB PHY driver for MDM6600 on Droid 4 From: Tony Lindgren Message-Id: <20180325154512.GC5700@atomide.com> Date: Sun, 25 Mar 2018 08:45:12 -0700 To: Pavel Machek Cc: Dan Williams , Sebastian Reichel , Kishon Vijay Abraham I , linux-kernel@vger.kernel.org, linux-usb@vger.kernel.org, linux-omap@vger.kernel.org, devicetree@vger.kernel.org, Mark Rutland , Marcel Partap , Michael Scott , Rob Herring List-ID: KiBQYXZlbCBNYWNoZWsgPHBhdmVsQHVjdy5jej4gWzE4MDMyNCAyMDowM106Cj4gT24gU2F0IDIw MTgtMDMtMjQgMDc6MjU6MTcsIFRvbnkgTGluZGdyZW4gd3JvdGU6Cj4gPiAqIERhbiBXaWxsaWFt cyA8ZGNid0ByZWRoYXQuY29tPiBbMTgwMzI0IDE0OjAwXToKPiA+ID4gT24gRnJpLCAyMDE4LTAz LTIzIGF0IDIxOjEzICswMTAwLCBQYXZlbCBNYWNoZWsgd3JvdGU6Cj4gPiA+ID4gRG9lcyBvZm9u b2Qgd29yayBmb3IgeW91PyBJIGNvdWxkIG5vdCBnZXQgdGhhdCBvbmUgdG8gd29yay4uLgo+ID4g PiAKPiA+ID4gQmVjYXVzZSBpdCdzIGxvb2tpbmcgZm9yIGEgR29iaSBtb2RlbSBidXQgdGhlIE1E TTY2MDAgaXNuJ3Qgb25lIGFuZAo+ID4gPiBkb2Vzbid0IGV4cG9zZSB0aGF0IGxheW91dCAoYW5k IGRvZXNuJ3QgcmVhbGx5IG5lZWQgdG8gYW55d2F5KS4gIEkKPiA+ID4gZG9uJ3QgdGhpbmsgb2Zv bm8gaGFzIGEgZ2VuZXJpYyBRTUkgZHJpdmVyLCBzbyB5b3UnZCBlaXRoZXIgbmVlZCB0byBmb3IK PiA+ID4gY2UgaXQgdG8gdXNlIHRoZSB0ZWxpdHFtaSBvciBxdWVjdGVscW1pIGRyaXZlcnMsIG9y IHdyaXRlIHlvdXIgb3duCj4gPiA+IGdlbmVyaWMgUU1JIG9uZS4KPiA+IAo+ID4gV2UgYWxzbyBn ZXQgZml2ZSBVU0IgdWFydHMgaWYgd2UgYWRkIHRoZSBkZXZpY2UgaWQgd2l0aCBzb21ldGhpbmcK PiA+IGxpa2UgdGhlIHBhdGNoIGJlbG93LiBJIGRvbid0IHF1aXRlIGdldCB3aHkgd2UgZ2V0IGZp dmUgVUFSVFM/Cj4gCj4gTm90IHN1cmUsIGVpdGhlci4gT2Z0ZW4gbW9yZSB0aGFuIG9uZSB1YXJ0 IGlzIHVzZWZ1bCwgeW91IGdldCBBVAo+IGNvbW1hbmRzIG9uIG9uZSwgd2hpbGUgR1BSUyBkYXRh IGZsb3cgb24gc29tZSBvdGhlci4gCj4gCj4gPiBBbHNvIG5vdCBzdXJlIGlmIHdlIHNob3VsZCBi ZSB1c2luZyBkcml2ZXJzL3VzYi9zZXJpYWwvcWNhdXguYwo+ID4gaW5zdGVhZCBvZiBxY3Nlcmlh bC5jPwo+ID4gCj4gPiBBbmQgZnJvbSB3aGF0IEkgcmVjYWxsIHRyeWluZyBpdCBvdXQsIGFkZGlu ZyB0aGUgVVNCIFVBUlRzCj4gPiBzb21laG93IGNvbmZ1c2VkIE1vZGVtTWFuYWdlciBJIHRoaW5r LCB0aGF0IG5lZWRzIHRvIGJlIHJldGVzdGVkCj4gPiB0aG91Z2ggOikKPiA+IAo+ID4gQW5kIHRo ZSBVU0IgVUFSVHMgYWRkZWQgZG8gbm90IG9mZmVyIHRoZSBzYW1lIHNldCBvZiBBVCBjb21tYW5k cwo+ID4gYXMgdGhlIG5fZ3NtIHNlcmlhbCBtdXguCj4gCj4gSG1tLiBJbnRlcmVzdGluZy4gQW55 d2F5LCBmb3IgbWUgdHR5VVNCNCBpcyBpbnRlcmVzdGluZywgYXMgaXQgc2VlbXMKPiB0byByZWFj dCB0byBBVCBjb21tYW5kcywgYW5kIGluIHBhcnRpY3VsYXIgcmVhY3RzIHRvIEFEVDEyMzsgKDsg aXMKPiBpbXBvcnRhbnQpLgoKSXMgdGhhdCB0byBkaWFsIGEgdm9pY2UgY2FsbD8KCj4gQVQrQ01H Rj0xCj4gQVQrQ01HUz0iMTIzIgo+IGZvb15aCj4gCj4gV29ya3MgZm9yIFNNUyBzZW5kaW5nLiBH b29kLgoKU28gd2hhdCBkbyB5b3UgdXNlIGZvciByZWFkaW5nIG5vdGlmeWluZyBhbmQgcmVhZGlu ZyBzbXM/Cgo+IE5vdywgaWYgc29tZW9uZSBrbm93cyB3aGF0IG5lZWRzIHRvIGJlIGRvbmUgdG8g Z2V0IEdTTSBhdWRpbyB3b3JraW5nLAo+IGxldCBtZSBrbm93LiBUaGF0J3Mgc29tZXRoaW5nIEkn ZCByZWFsbHkgbGlrZS4KCkkgdGhpbmsgaXQncyB0aGUgY3BjYXAgYmFzZWQgY29uZmlnIHRvIHJv dXRlIHZvaWNlIGNhbGwgYXVkaW8KdG8gU29DLCBTZWJhc3RpYW4ga25vd3MgdGhlIGRldGFpbHMg OikKClRoZSB3YXkgdG8gZmlndXJlIHRoYXQgb25lIG91dCBpcyB0byBkdW1wIHRoZSBjcGNhcCBy ZWdpc3RlcnMKYmVmb3JlIGFuZCBkdXJpbmcgdm9pY2UgY2FsbCBvbiBhbmRyb2lkIHdpdGggY3Bj YXBydywgdGhlbgpkaWZmIHRoZSBvdXRwdXQgZm9yIHRoZSBhdWRpbyByZWdpc3RlcnMuIFByb2Jh Ymx5IHNvbWUgU29DCnJlZ2lzdGVycyBuZWVkIHRvIGJlIGRpZmZlZCB0b28gd2l0aCByd21lbSBv ciBzaW1pbGFyIHRvb2wKZm9yIHRoZSBtY2JzcCBpbnN0YW5jZShzKSB1c2VkLgoKUmVnYXJkcywK ClRvbnkKLS0tClRvIHVuc3Vic2NyaWJlIGZyb20gdGhpcyBsaXN0OiBzZW5kIHRoZSBsaW5lICJ1 bnN1YnNjcmliZSBsaW51eC11c2IiIGluCnRoZSBib2R5IG9mIGEgbWVzc2FnZSB0byBtYWpvcmRv bW9Admdlci5rZXJuZWwub3JnCk1vcmUgbWFqb3Jkb21vIGluZm8gYXQgIGh0dHA6Ly92Z2VyLmtl cm5lbC5vcmcvbWFqb3Jkb21vLWluZm8uaHRtbAo=