From mboxrd@z Thu Jan 1 00:00:00 1970 From: Pavel Machek Subject: Re: [PATCHv4] phy: mapphone-mdm6600: Add USB PHY driver for MDM6600 on Droid 4 Date: Fri, 23 Mar 2018 21:13:43 +0100 Message-ID: <20180323201343.GC23658@amd> References: <20180309023750.108942-1-tony@atomide.com> <20180322135553.GA27648@amd> <20180322164633.GF5799@atomide.com> <20180322192843.GA17189@amd> <20180323105455.GB21644@amd> <20180323113521.45va72bhzis6ym6m@earth.universe> Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="1SQmhf2mF2YjsYvc" Return-path: Content-Disposition: inline In-Reply-To: <20180323113521.45va72bhzis6ym6m@earth.universe> Sender: linux-kernel-owner@vger.kernel.org To: Sebastian Reichel Cc: Dan Williams , Tony Lindgren , 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 --1SQmhf2mF2YjsYvc Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Fri 2018-03-23 12:35:21, Sebastian Reichel wrote: > Hi, >=20 > On Fri, Mar 23, 2018 at 11:54:55AM +0100, Pavel Machek wrote: > > Hi! > >=20 > > > > > Do you have the related dts patches picked from next? > > > > >=20 > > > > > fdd192037fce ("ARM: dts: omap4-droid4: Fix USB PHY port naming") > > > > > e5b9fd7bdeb5 ("ARM: dts: omap4-droid4: Configure MDM6600 USB PHY") > > > > >=20 > > > > > But yeah all you need to do is have phy-mapphone-mdm6600 and > > > > > ohci-platform loaded and then ifconfig should show four wwan > > > > > interfaces being added. > > > >=20 > > > > ifconfig? I thought I should get /dev/ttyUSB0..3? > > >=20 > > > I believe they are QMI via qmi_wwan, not TTYs. > >=20 > > Well, qmicli expects device path... and I see nothing on ifconfig. Any > > idea how the device would be named? > >=20 > > But I believe it just does not work, see the qcserial experiment with > > new_id below. >=20 > Dan is right, you need qmi_wwan driver. qmicli expects > /dev/cdc-wdm device. I use this on Droid 4: >=20 > qmicli -d /dev/cdc-wdm0 --some-other-option Aha, there's ./drivers/usb/serial/usb_wwan.c and there's =2E/drivers/net/usb/qmi_wwan.o . That confused me. qmicli now works for me, thanks! Tested-by: Pavel Machek Does ofonod work for you? I could not get that one to work... ofonod[4083]: plugins/udevng.c:create_modem() /sys/devices/platform/44000000.ocp/4a064000.usbhshost/4a064800.ohci/usb2/2-1 ofonod[4083]: plugins/udevng.c:create_modem() driver=3Dgobi ofonod[4083]: src/modem.c:ofono_modem_create() name: (null), type: gobi ofonod[4083]: plugins/udevng.c:setup_gobi() /sys/devices/platform/44000000.ocp/4a064000.usbhshost/4a064800.ohci/usb2/2-1 ofonod[4083]: plugins/udevng.c:setup_gobi() /dev/cdc-wdm0 255/251/255 05 (null) (null) usbmisc ofonod[4083]: plugins/udevng.c:setup_gobi() wwan0 255/251/255 05 (null) (null) net ofonod[4083]: plugins/udevng.c:setup_gobi() /dev/cdc-wdm1 255/251/255 06 (null) (null) usbmisc ofonod[4083]: plugins/udevng.c:setup_gobi() wwan1 255/251/255 06 (null) (null) net ofonod[4083]: plugins/udevng.c:setup_gobi() /dev/cdc-wdm2 255/251/255 07 (null) (null) usbmisc ofonod[4083]: plugins/udevng.c:setup_gobi() wwan2 255/251/255 07 (null) (null) net ofonod[4083]: plugins/udevng.c:setup_gobi() /dev/cdc-wdm3 255/251/255 08 (null) (null) usbmisc ofonod[4083]: plugins/udevng.c:setup_gobi() wwan3 255/251/255 08 (null) (null) net ofonod[4083]: plugins/udevng.c:destroy_modem() /sys/devices/platform/44000000.ocp/4a064000.usbhshost/4a064800.ohci/usb2/2-1 ofonod[4083]: src/modem.c:ofono_modem_remove() 0x5eb380 ofonod[4083]: plugins/udevng.c:destroy_modem() /dev/cdc-wdm0 ofonod[4083]: plugins/udevng.c:destroy_modem() wwan0 ofonod[4083]: plugins/udevng.c:destroy_modem() /dev/cdc-wdm1 ofonod[4083]: plugins/udevng.c:destroy_modem() wwan1 ofonod[4083]: plugins/udevng.c:destroy_modem() /dev/cdc-wdm2 ofonod[4083]: plugins/udevng.c:destroy_modem() wwan2 ofonod[4083]: plugins/udevng.c:destroy_modem() /dev/cdc-wdm3 ofonod[4083]: plugins/udevng.c:destroy_modem() wwan3 ofonod[4083]: plugins/upower.c:upower_connect() upower connect ofonod[4083]: plugins/hfp_hf_bluez5.c:connect_handler() Registering External Profile handler ... Pavel --=20 (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blo= g.html --1SQmhf2mF2YjsYvc Content-Type: application/pgp-signature; name="signature.asc" Content-Description: Digital signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 iEYEARECAAYFAlq1X/cACgkQMOfwapXb+vIbcwCgivIAngwiUQOVnKt6eFB5BSuk IzEAnA+JUc0uloOKnfZijU76/HFBzaj5 =3ybB -----END PGP SIGNATURE----- --1SQmhf2mF2YjsYvc-- 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: Pavel Machek Message-Id: <20180323201343.GC23658@amd> Date: Fri, 23 Mar 2018 21:13:43 +0100 To: Sebastian Reichel Cc: Dan Williams , Tony Lindgren , 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: T24gRnJpIDIwMTgtMDMtMjMgMTI6MzU6MjEsIFNlYmFzdGlhbiBSZWljaGVsIHdyb3RlOgo+IEhp LAo+IAo+IE9uIEZyaSwgTWFyIDIzLCAyMDE4IGF0IDExOjU0OjU1QU0gKzAxMDAsIFBhdmVsIE1h Y2hlayB3cm90ZToKPiA+IEhpIQo+ID4gCj4gPiA+ID4gPiBEbyB5b3UgaGF2ZSB0aGUgcmVsYXRl ZCBkdHMgcGF0Y2hlcyBwaWNrZWQgZnJvbSBuZXh0Pwo+ID4gPiA+ID4gCj4gPiA+ID4gPiBmZGQx OTIwMzdmY2UgKCJBUk06IGR0czogb21hcDQtZHJvaWQ0OiBGaXggVVNCIFBIWSBwb3J0IG5hbWlu ZyIpCj4gPiA+ID4gPiBlNWI5ZmQ3YmRlYjUgKCJBUk06IGR0czogb21hcDQtZHJvaWQ0OiBDb25m aWd1cmUgTURNNjYwMCBVU0IgUEhZIikKPiA+ID4gPiA+IAo+ID4gPiA+ID4gQnV0IHllYWggYWxs IHlvdSBuZWVkIHRvIGRvIGlzIGhhdmUgcGh5LW1hcHBob25lLW1kbTY2MDAgYW5kCj4gPiA+ID4g PiBvaGNpLXBsYXRmb3JtIGxvYWRlZCBhbmQgdGhlbiBpZmNvbmZpZyBzaG91bGQgc2hvdyBmb3Vy IHd3YW4KPiA+ID4gPiA+IGludGVyZmFjZXMgYmVpbmcgYWRkZWQuCj4gPiA+ID4gCj4gPiA+ID4g aWZjb25maWc/IEkgdGhvdWdodCBJIHNob3VsZCBnZXQgL2Rldi90dHlVU0IwLi4zPwo+ID4gPiAK PiA+ID4gSSBiZWxpZXZlIHRoZXkgYXJlIFFNSSB2aWEgcW1pX3d3YW4sIG5vdCBUVFlzLgo+ID4g Cj4gPiBXZWxsLCBxbWljbGkgZXhwZWN0cyBkZXZpY2UgcGF0aC4uLiBhbmQgSSBzZWUgbm90aGlu ZyBvbiBpZmNvbmZpZy4gQW55Cj4gPiBpZGVhIGhvdyB0aGUgZGV2aWNlIHdvdWxkIGJlIG5hbWVk Pwo+ID4gCj4gPiBCdXQgSSBiZWxpZXZlIGl0IGp1c3QgZG9lcyBub3Qgd29yaywgc2VlIHRoZSBx Y3NlcmlhbCBleHBlcmltZW50IHdpdGgKPiA+IG5ld19pZCBiZWxvdy4KPiAKPiBEYW4gaXMgcmln aHQsIHlvdSBuZWVkIHFtaV93d2FuIGRyaXZlci4gcW1pY2xpIGV4cGVjdHMKPiAvZGV2L2NkYy13 ZG08bnVtPiBkZXZpY2UuIEkgdXNlIHRoaXMgb24gRHJvaWQgNDoKPiAKPiBxbWljbGkgLWQgL2Rl di9jZGMtd2RtMCAtLXNvbWUtb3RoZXItb3B0aW9uCgpBaGEsIHRoZXJlJ3MgLi9kcml2ZXJzL3Vz Yi9zZXJpYWwvdXNiX3d3YW4uYyBhbmQgdGhlcmUncwouL2RyaXZlcnMvbmV0L3VzYi9xbWlfd3dh bi5vIC4gVGhhdCBjb25mdXNlZCBtZS4KCnFtaWNsaSBub3cgd29ya3MgZm9yIG1lLCB0aGFua3Mh CgpUZXN0ZWQtYnk6IFBhdmVsIE1hY2hlayA8cGF2ZWxAdWN3LmN6PgoKRG9lcyBvZm9ub2Qgd29y ayBmb3IgeW91PyBJIGNvdWxkIG5vdCBnZXQgdGhhdCBvbmUgdG8gd29yay4uLgoKb2Zvbm9kWzQw ODNdOiBwbHVnaW5zL3VkZXZuZy5jOmNyZWF0ZV9tb2RlbSgpCi9zeXMvZGV2aWNlcy9wbGF0Zm9y bS80NDAwMDAwMC5vY3AvNGEwNjQwMDAudXNiaHNob3N0LzRhMDY0ODAwLm9oY2kvdXNiMi8yLTEK b2Zvbm9kWzQwODNdOiBwbHVnaW5zL3VkZXZuZy5jOmNyZWF0ZV9tb2RlbSgpIGRyaXZlcj1nb2Jp Cm9mb25vZFs0MDgzXTogc3JjL21vZGVtLmM6b2Zvbm9fbW9kZW1fY3JlYXRlKCkgbmFtZTogKG51 bGwpLCB0eXBlOgpnb2JpCm9mb25vZFs0MDgzXTogcGx1Z2lucy91ZGV2bmcuYzpzZXR1cF9nb2Jp KCkKL3N5cy9kZXZpY2VzL3BsYXRmb3JtLzQ0MDAwMDAwLm9jcC80YTA2NDAwMC51c2Joc2hvc3Qv NGEwNjQ4MDAub2hjaS91c2IyLzItMQpvZm9ub2RbNDA4M106IHBsdWdpbnMvdWRldm5nLmM6c2V0 dXBfZ29iaSgpIC9kZXYvY2RjLXdkbTAgMjU1LzI1MS8yNTUKMDUgKG51bGwpIChudWxsKSB1c2Jt aXNjCm9mb25vZFs0MDgzXTogcGx1Z2lucy91ZGV2bmcuYzpzZXR1cF9nb2JpKCkgd3dhbjAgMjU1 LzI1MS8yNTUgMDUKKG51bGwpIChudWxsKSBuZXQKb2Zvbm9kWzQwODNdOiBwbHVnaW5zL3VkZXZu Zy5jOnNldHVwX2dvYmkoKSAvZGV2L2NkYy13ZG0xIDI1NS8yNTEvMjU1CjA2IChudWxsKSAobnVs bCkgdXNibWlzYwpvZm9ub2RbNDA4M106IHBsdWdpbnMvdWRldm5nLmM6c2V0dXBfZ29iaSgpIHd3 YW4xIDI1NS8yNTEvMjU1IDA2CihudWxsKSAobnVsbCkgbmV0Cm9mb25vZFs0MDgzXTogcGx1Z2lu cy91ZGV2bmcuYzpzZXR1cF9nb2JpKCkgL2Rldi9jZGMtd2RtMiAyNTUvMjUxLzI1NQowNyAobnVs bCkgKG51bGwpIHVzYm1pc2MKb2Zvbm9kWzQwODNdOiBwbHVnaW5zL3VkZXZuZy5jOnNldHVwX2dv YmkoKSB3d2FuMiAyNTUvMjUxLzI1NSAwNwoobnVsbCkgKG51bGwpIG5ldApvZm9ub2RbNDA4M106 IHBsdWdpbnMvdWRldm5nLmM6c2V0dXBfZ29iaSgpIC9kZXYvY2RjLXdkbTMgMjU1LzI1MS8yNTUK MDggKG51bGwpIChudWxsKSB1c2JtaXNjCm9mb25vZFs0MDgzXTogcGx1Z2lucy91ZGV2bmcuYzpz ZXR1cF9nb2JpKCkgd3dhbjMgMjU1LzI1MS8yNTUgMDgKKG51bGwpIChudWxsKSBuZXQKb2Zvbm9k WzQwODNdOiBwbHVnaW5zL3VkZXZuZy5jOmRlc3Ryb3lfbW9kZW0oKQovc3lzL2RldmljZXMvcGxh dGZvcm0vNDQwMDAwMDAub2NwLzRhMDY0MDAwLnVzYmhzaG9zdC80YTA2NDgwMC5vaGNpL3VzYjIv Mi0xCm9mb25vZFs0MDgzXTogc3JjL21vZGVtLmM6b2Zvbm9fbW9kZW1fcmVtb3ZlKCkgMHg1ZWIz ODAKb2Zvbm9kWzQwODNdOiBwbHVnaW5zL3VkZXZuZy5jOmRlc3Ryb3lfbW9kZW0oKSAvZGV2L2Nk Yy13ZG0wCm9mb25vZFs0MDgzXTogcGx1Z2lucy91ZGV2bmcuYzpkZXN0cm95X21vZGVtKCkgd3dh bjAKb2Zvbm9kWzQwODNdOiBwbHVnaW5zL3VkZXZuZy5jOmRlc3Ryb3lfbW9kZW0oKSAvZGV2L2Nk Yy13ZG0xCm9mb25vZFs0MDgzXTogcGx1Z2lucy91ZGV2bmcuYzpkZXN0cm95X21vZGVtKCkgd3dh bjEKb2Zvbm9kWzQwODNdOiBwbHVnaW5zL3VkZXZuZy5jOmRlc3Ryb3lfbW9kZW0oKSAvZGV2L2Nk Yy13ZG0yCm9mb25vZFs0MDgzXTogcGx1Z2lucy91ZGV2bmcuYzpkZXN0cm95X21vZGVtKCkgd3dh bjIKb2Zvbm9kWzQwODNdOiBwbHVnaW5zL3VkZXZuZy5jOmRlc3Ryb3lfbW9kZW0oKSAvZGV2L2Nk Yy13ZG0zCm9mb25vZFs0MDgzXTogcGx1Z2lucy91ZGV2bmcuYzpkZXN0cm95X21vZGVtKCkgd3dh bjMKb2Zvbm9kWzQwODNdOiBwbHVnaW5zL3Vwb3dlci5jOnVwb3dlcl9jb25uZWN0KCkgdXBvd2Vy IGNvbm5lY3QKb2Zvbm9kWzQwODNdOiBwbHVnaW5zL2hmcF9oZl9ibHVlejUuYzpjb25uZWN0X2hh bmRsZXIoKSBSZWdpc3RlcmluZwpFeHRlcm5hbCBQcm9maWxlIGhhbmRsZXIgLi4uCgoJCQkJCQkJ CQlQYXZlbAo=