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: [v2] usb: chipidea: Hook into mux framework to toggle usb switch From: Yossi Mansharoff Message-Id: Date: Wed, 18 Apr 2018 08:48:10 +0300 To: Peter Rosin Cc: Peter Chen , Greg Kroah-Hartman , "open list:CHIPIDEA USB HIGH SPEED DUAL ROLE CONTROLLER" , open list , robh@kernel.org, swboyd@chromium.org, devicetree@vger.kernel.org, Stephen Boyd List-ID: T24gMjAxOC0wNC0xNyAxNzoxMSwgUGV0ZXIgUm9zaW4gd3JvdGU6Cj4gT24gMjAxOC0wNC0xNyAx NTo1MiwgWW9zc2kgTWFuc2hhcm9mZiB3cm90ZToKPj4gT24gdGhlIGRiNDEwYyA5NmJvYXJkcyBw bGF0Zm9ybSB3ZSBoYXZlIGEgVEM3VVNCNDBNVSBvbiB0aGUgYm9hcmQKPj4gdG8gbXV4IHRoZSBE Ky9ELSBsaW5lcyBjb21pbmcgZnJvbSB0aGUgY29udHJvbGxlciBiZXR3ZWVuIGEgbWljcm8KPj4g dXNiICJkZXZpY2UiIHBvcnQgYW5kIGEgVVNCIGh1YiBmb3IgImhvc3QiIHJvbGVzWzFdLiBEdXJp bmcgYQo+PiByb2xlIHN3aXRjaCwgd2UgbmVlZCB0byB0b2dnbGUgdGhpcyBtdXggdG8gZm9yd2Fy ZCB0aGUgRCsvRC0KPj4gbGluZXMgdG8gZWl0aGVyIHRoZSBwb3J0IG9yIHRoZSBodWIuIEFkZCB0 aGUgbmVjZXNzYXJ5IGNvZGUgdG8gZG8KPj4gdGhlIHJvbGUgc3dpdGNoIGluIGNoaXBpZGVhIGNv cmUgdmlhIHRoZSBnZW5lcmljIG11eCBmcmFtZXdvcmsuCj4+IEJvYXJkIGNvbmZpZ3VyYXRpb25z IGxpa2Ugb24gZGI0MTBjIGFyZSBleHBlY3RlZCB0byBjaGFuZ2Ugcm9sZXMKPj4gdmlhIHRoZSBz eXNmcyBBUEkgZGVzY3JpYmVkIGluCj4+IERvY3VtZW50YXRpb24vQUJJL3Rlc3Rpbmcvc3lzZnMt cGxhdGZvcm0tY2hpcGlkZWEtdXNiMi4KPiAKPiBPaywgc28gdGhpcyBpcyB2Mi4gUGxlYXNlIGRl c2NyaWJlIHdoYXQgaXMgZGlmZmVyZW50IGZyb20gdjEuCj4gSSBoYXZlIHRvbGQgeW91IGJlZm9y ZSB0aGF0IHRoaXMgaW5mb3JtYXRpb24gaGVscHMuCj4gCj4+IFsxXSAKPj4gaHR0cHM6Ly9naXRo dWIuY29tLzk2Ym9hcmRzL2RvY3VtZW50YXRpb24vcmF3L21hc3Rlci9Db25zdW1lckVkaXRpb24v RHJhZ29uQm9hcmQtNDEwYy9IYXJkd2FyZURvY3MvU2NoZW1hdGljc19EcmFnb25Cb2FyZC5wZGYK PiAKPiBUaGlzIGxpbmsgcmV0dXJucyA0MDQgZm9yIG1lLgo+IAo+IENoZWVycywKPiBQZXRlcgoK CkhpLApUaGlzIHBhdGNoIHdhcyBzcGxpdCBhcGFydCBmcm9tIHRoZSBvcmlnaW5hbCBwYXRjaCBp bnRvIHR3byBwYXRjaGVzCm9uZSBmb3IgY2hpcGlkZWEgYW5kIHRoZSBvdGhlciBmb3IgYmluZGlu Z3MuCnRoaXMgcGF0Y2ggaGFzIG5vIG90aGVyIGNoYW5nZXMgdHdvIHRoZSBjb2RlLgoKSSB3aWxs IHVwZGF0ZSB0aGUgbGluay4KCnRoYW5rcwpZb3NzaQotLS0KVG8gdW5zdWJzY3JpYmUgZnJvbSB0 aGlzIGxpc3Q6IHNlbmQgdGhlIGxpbmUgInVuc3Vic2NyaWJlIGxpbnV4LXVzYiIgaW4KdGhlIGJv ZHkgb2YgYSBtZXNzYWdlIHRvIG1ham9yZG9tb0B2Z2VyLmtlcm5lbC5vcmcKTW9yZSBtYWpvcmRv bW8gaW5mbyBhdCAgaHR0cDovL3ZnZXIua2VybmVsLm9yZy9tYWpvcmRvbW8taW5mby5odG1sCg== From mboxrd@z Thu Jan 1 00:00:00 1970 From: yossim@codeaurora.org Subject: Re: [PATCH v2] usb: chipidea: Hook into mux framework to toggle usb switch Date: Wed, 18 Apr 2018 08:48:10 +0300 Message-ID: References: <1523973148-2094-1-git-send-email-yossim@codeaurora.org> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: Peter Rosin Cc: Peter Chen , Greg Kroah-Hartman , "open list:CHIPIDEA USB HIGH SPEED DUAL ROLE CONTROLLER" , open list , robh@kernel.org, swboyd@chromium.org, devicetree@vger.kernel.org, Stephen Boyd List-Id: devicetree@vger.kernel.org On 2018-04-17 17:11, Peter Rosin wrote: > On 2018-04-17 15:52, Yossi Mansharoff wrote: >> On the db410c 96boards platform we have a TC7USB40MU on the board >> to mux the D+/D- lines coming from the controller between a micro >> usb "device" port and a USB hub for "host" roles[1]. During a >> role switch, we need to toggle this mux to forward the D+/D- >> lines to either the port or the hub. Add the necessary code to do >> the role switch in chipidea core via the generic mux framework. >> Board configurations like on db410c are expected to change roles >> via the sysfs API described in >> Documentation/ABI/testing/sysfs-platform-chipidea-usb2. > > Ok, so this is v2. Please describe what is different from v1. > I have told you before that this information helps. > >> [1] >> https://github.com/96boards/documentation/raw/master/ConsumerEdition/DragonBoard-410c/HardwareDocs/Schematics_DragonBoard.pdf > > This link returns 404 for me. > > Cheers, > Peter Hi, This patch was split apart from the original patch into two patches one for chipidea and the other for bindings. this patch has no other changes two the code. I will update the link. thanks Yossi