From mboxrd@z Thu Jan 1 00:00:00 1970 From: balbi@kernel.org (Felipe Balbi) Date: Mon, 29 Jan 2018 10:18:28 +0200 Subject: [PATCH 2/3] usb: dwc3: of-simple: add support for shared and pulsed reset lines In-Reply-To: <20180128200333.20093-3-martin.blumenstingl@googlemail.com> References: <20180128200333.20093-1-martin.blumenstingl@googlemail.com> <20180128200333.20093-3-martin.blumenstingl@googlemail.com> Message-ID: <87fu6pjcx7.fsf@linux.intel.com> To: linus-amlogic@lists.infradead.org List-Id: linus-amlogic.lists.infradead.org Hi, Martin Blumenstingl writes: > Some SoCs (such as Amlogic Meson GXL for example) share the reset line > with other components (in case of the Meson GXL example there's a shared > reset line between the USB2 PHYs, USB3 PHYs and the dwc3 controller). > Additionally SoC implementations may prefer a reset pulse over level > resets. > > Add an internal per-of_device_id struct which can be used to configure > whether the reset lines are shared and whether they use level or pulse > resets. > > For now this falls back to the old defaults, which are: > - reset lines are exclusive > - level resets are being used > > Signed-off-by: Martin Blumenstingl > --- > drivers/usb/dwc3/dwc3-of-simple.c | 65 ++++++++++++++++++++++++++++++++------- > 1 file changed, 54 insertions(+), 11 deletions(-) > > diff --git a/drivers/usb/dwc3/dwc3-of-simple.c b/drivers/usb/dwc3/dwc3-of-simple.c > index 7ae0eefc7cc7..ceb9f0cd822a 100644 > --- a/drivers/usb/dwc3/dwc3-of-simple.c > +++ b/drivers/usb/dwc3/dwc3-of-simple.c > @@ -22,11 +22,22 @@ > #include > #include > > +/** > + * struct dwc3_of_simple_params - hardware specific parameters > + * @shared_resets: indicates that the resets are shared or exclusive > + * @pulse_resets: use a reset pulse instead of level based resets > + */ > +struct dwc3_of_simple_params { > + bool shared_resets; > + bool pulse_resets; > +}; > + > struct dwc3_of_simple { > struct device *dev; > struct clk **clks; > int num_clocks; > struct reset_control *resets; > + const struct dwc3_of_simple_params *params; instead, you can add these two fields here: bool shared_resets; bool pulse_resets; and ... > @@ -90,17 +101,26 @@ static int dwc3_of_simple_probe(struct platform_device *pdev) > > platform_set_drvdata(pdev, simple); > simple->dev = dev; > + simple->params = of_device_get_match_data(dev); > > - simple->resets = of_reset_control_array_get_optional_exclusive(np); > + simple->resets = of_reset_control_array_get(np, > + simple->params->shared_resets, > + true); wrap this with a of_device_is_compatible() check: if (of_device_is_compatible(dev->of_node, "foobar")) { simple->shared_resets = true; simple->pulse_resets = true; } or something like that. Then we don't need to add a new dwc3_of_simple_params for everybody. Also, the why isn't the reset type (pulse vs level) handled by reset framework itself? Why does dwc3-of-simple need to know about it? -- balbi -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 832 bytes Desc: not available URL: 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: [2/3] usb: dwc3: of-simple: add support for shared and pulsed reset lines From: Felipe Balbi Message-Id: <87fu6pjcx7.fsf@linux.intel.com> Date: Mon, 29 Jan 2018 10:18:28 +0200 To: Martin Blumenstingl , gregkh@linuxfoundation.org, robh+dt@kernel.org, mark.rutland@arm.com, linux-usb@vger.kernel.org, devicetree@vger.kernel.org Cc: linux-amlogic@lists.infradead.org List-ID: SGksCgpNYXJ0aW4gQmx1bWVuc3RpbmdsIDxtYXJ0aW4uYmx1bWVuc3RpbmdsQGdvb2dsZW1haWwu Y29tPiB3cml0ZXM6Cj4gU29tZSBTb0NzIChzdWNoIGFzIEFtbG9naWMgTWVzb24gR1hMIGZvciBl eGFtcGxlKSBzaGFyZSB0aGUgcmVzZXQgbGluZQo+IHdpdGggb3RoZXIgY29tcG9uZW50cyAoaW4g Y2FzZSBvZiB0aGUgTWVzb24gR1hMIGV4YW1wbGUgdGhlcmUncyBhIHNoYXJlZAo+IHJlc2V0IGxp bmUgYmV0d2VlbiB0aGUgVVNCMiBQSFlzLCBVU0IzIFBIWXMgYW5kIHRoZSBkd2MzIGNvbnRyb2xs ZXIpLgo+IEFkZGl0aW9uYWxseSBTb0MgaW1wbGVtZW50YXRpb25zIG1heSBwcmVmZXIgYSByZXNl dCBwdWxzZSBvdmVyIGxldmVsCj4gcmVzZXRzLgo+Cj4gQWRkIGFuIGludGVybmFsIHBlci1vZl9k ZXZpY2VfaWQgc3RydWN0IHdoaWNoIGNhbiBiZSB1c2VkIHRvIGNvbmZpZ3VyZQo+IHdoZXRoZXIg dGhlIHJlc2V0IGxpbmVzIGFyZSBzaGFyZWQgYW5kIHdoZXRoZXIgdGhleSB1c2UgbGV2ZWwgb3Ig cHVsc2UKPiByZXNldHMuCj4KPiBGb3Igbm93IHRoaXMgZmFsbHMgYmFjayB0byB0aGUgb2xkIGRl ZmF1bHRzLCB3aGljaCBhcmU6Cj4gLSByZXNldCBsaW5lcyBhcmUgZXhjbHVzaXZlCj4gLSBsZXZl bCByZXNldHMgYXJlIGJlaW5nIHVzZWQKPgo+IFNpZ25lZC1vZmYtYnk6IE1hcnRpbiBCbHVtZW5z dGluZ2wgPG1hcnRpbi5ibHVtZW5zdGluZ2xAZ29vZ2xlbWFpbC5jb20+Cj4gLS0tCj4gIGRyaXZl cnMvdXNiL2R3YzMvZHdjMy1vZi1zaW1wbGUuYyB8IDY1ICsrKysrKysrKysrKysrKysrKysrKysr KysrKysrKysrLS0tLS0tLQo+ICAxIGZpbGUgY2hhbmdlZCwgNTQgaW5zZXJ0aW9ucygrKSwgMTEg ZGVsZXRpb25zKC0pCj4KPiBkaWZmIC0tZ2l0IGEvZHJpdmVycy91c2IvZHdjMy9kd2MzLW9mLXNp bXBsZS5jIGIvZHJpdmVycy91c2IvZHdjMy9kd2MzLW9mLXNpbXBsZS5jCj4gaW5kZXggN2FlMGVl ZmM3Y2M3Li5jZWI5ZjBjZDgyMmEgMTAwNjQ0Cj4gLS0tIGEvZHJpdmVycy91c2IvZHdjMy9kd2Mz LW9mLXNpbXBsZS5jCj4gKysrIGIvZHJpdmVycy91c2IvZHdjMy9kd2MzLW9mLXNpbXBsZS5jCj4g QEAgLTIyLDExICsyMiwyMiBAQAo+ICAjaW5jbHVkZSA8bGludXgvcG1fcnVudGltZS5oPgo+ICAj aW5jbHVkZSA8bGludXgvcmVzZXQuaD4KPiAgCj4gKy8qKgo+ICsgKiBzdHJ1Y3QgZHdjM19vZl9z aW1wbGVfcGFyYW1zIC0gaGFyZHdhcmUgc3BlY2lmaWMgcGFyYW1ldGVycwo+ICsgKiBAc2hhcmVk X3Jlc2V0czogaW5kaWNhdGVzIHRoYXQgdGhlIHJlc2V0cyBhcmUgc2hhcmVkIG9yIGV4Y2x1c2l2 ZQo+ICsgKiBAcHVsc2VfcmVzZXRzOiB1c2UgYSByZXNldCBwdWxzZSBpbnN0ZWFkIG9mIGxldmVs IGJhc2VkIHJlc2V0cwo+ICsgKi8KPiArc3RydWN0IGR3YzNfb2Zfc2ltcGxlX3BhcmFtcyB7Cj4g Kwlib29sCQkJc2hhcmVkX3Jlc2V0czsKPiArCWJvb2wJCQlwdWxzZV9yZXNldHM7Cj4gK307Cj4g Kwo+ICBzdHJ1Y3QgZHdjM19vZl9zaW1wbGUgewo+ICAJc3RydWN0IGRldmljZQkJKmRldjsKPiAg CXN0cnVjdCBjbGsJCSoqY2xrczsKPiAgCWludAkJCW51bV9jbG9ja3M7Cj4gIAlzdHJ1Y3QgcmVz ZXRfY29udHJvbAkqcmVzZXRzOwo+ICsJY29uc3Qgc3RydWN0IGR3YzNfb2Zfc2ltcGxlX3BhcmFt cwkqcGFyYW1zOwoKaW5zdGVhZCwgeW91IGNhbiBhZGQgdGhlc2UgdHdvIGZpZWxkcyBoZXJlOgoK CWJvb2wgc2hhcmVkX3Jlc2V0czsKICAgICAgICBib29sIHB1bHNlX3Jlc2V0czsKCmFuZCAuLi4K Cj4gQEAgLTkwLDE3ICsxMDEsMjYgQEAgc3RhdGljIGludCBkd2MzX29mX3NpbXBsZV9wcm9iZShz dHJ1Y3QgcGxhdGZvcm1fZGV2aWNlICpwZGV2KQo+ICAKPiAgCXBsYXRmb3JtX3NldF9kcnZkYXRh KHBkZXYsIHNpbXBsZSk7Cj4gIAlzaW1wbGUtPmRldiA9IGRldjsKPiArCXNpbXBsZS0+cGFyYW1z ID0gb2ZfZGV2aWNlX2dldF9tYXRjaF9kYXRhKGRldik7Cj4gIAo+IC0Jc2ltcGxlLT5yZXNldHMg PSBvZl9yZXNldF9jb250cm9sX2FycmF5X2dldF9vcHRpb25hbF9leGNsdXNpdmUobnApOwo+ICsJ c2ltcGxlLT5yZXNldHMgPSBvZl9yZXNldF9jb250cm9sX2FycmF5X2dldChucCwKPiArCQkJCQkJ c2ltcGxlLT5wYXJhbXMtPnNoYXJlZF9yZXNldHMsCj4gKwkJCQkJCXRydWUpOwoKd3JhcCB0aGlz IHdpdGggYSBvZl9kZXZpY2VfaXNfY29tcGF0aWJsZSgpIGNoZWNrOgoKCWlmIChvZl9kZXZpY2Vf aXNfY29tcGF0aWJsZShkZXYtPm9mX25vZGUsICJmb29iYXIiKSkgewogICAgICAgIAlzaW1wbGUt PnNoYXJlZF9yZXNldHMgPSB0cnVlOwoJCXNpbXBsZS0+cHVsc2VfcmVzZXRzID0gdHJ1ZTsKCX0K Cm9yIHNvbWV0aGluZyBsaWtlIHRoYXQuIFRoZW4gd2UgZG9uJ3QgbmVlZCB0byBhZGQgYSBuZXcK ZHdjM19vZl9zaW1wbGVfcGFyYW1zIGZvciBldmVyeWJvZHkuCgpBbHNvLCB0aGUgd2h5IGlzbid0 IHRoZSByZXNldCB0eXBlIChwdWxzZSB2cyBsZXZlbCkgaGFuZGxlZCBieSByZXNldApmcmFtZXdv cmsgaXRzZWxmPyBXaHkgZG9lcyBkd2MzLW9mLXNpbXBsZSBuZWVkIHRvIGtub3cgYWJvdXQgaXQ/ Cg== From mboxrd@z Thu Jan 1 00:00:00 1970 From: Felipe Balbi Subject: Re: [PATCH 2/3] usb: dwc3: of-simple: add support for shared and pulsed reset lines Date: Mon, 29 Jan 2018 10:18:28 +0200 Message-ID: <87fu6pjcx7.fsf@linux.intel.com> References: <20180128200333.20093-1-martin.blumenstingl@googlemail.com> <20180128200333.20093-3-martin.blumenstingl@googlemail.com> Mime-Version: 1.0 Content-Type: multipart/signed; boundary="=-=-="; micalg=pgp-sha512; protocol="application/pgp-signature" Return-path: In-Reply-To: <20180128200333.20093-3-martin.blumenstingl-gM/Ye1E23mwN+BqQ9rBEUg@public.gmane.org> Sender: linux-usb-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: gregkh-hQyY1W1yCW8ekmWlsbkhG0B+6BGkLq7r@public.gmane.org, robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, mark.rutland-5wv7dgnIgG8@public.gmane.org, linux-usb-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org Cc: linux-amlogic-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, Martin Blumenstingl List-Id: devicetree@vger.kernel.org --=-=-= Content-Type: text/plain Content-Transfer-Encoding: quoted-printable Hi, Martin Blumenstingl writes: > Some SoCs (such as Amlogic Meson GXL for example) share the reset line > with other components (in case of the Meson GXL example there's a shared > reset line between the USB2 PHYs, USB3 PHYs and the dwc3 controller). > Additionally SoC implementations may prefer a reset pulse over level > resets. > > Add an internal per-of_device_id struct which can be used to configure > whether the reset lines are shared and whether they use level or pulse > resets. > > For now this falls back to the old defaults, which are: > - reset lines are exclusive > - level resets are being used > > Signed-off-by: Martin Blumenstingl > --- > drivers/usb/dwc3/dwc3-of-simple.c | 65 ++++++++++++++++++++++++++++++++-= ------ > 1 file changed, 54 insertions(+), 11 deletions(-) > > diff --git a/drivers/usb/dwc3/dwc3-of-simple.c b/drivers/usb/dwc3/dwc3-of= -simple.c > index 7ae0eefc7cc7..ceb9f0cd822a 100644 > --- a/drivers/usb/dwc3/dwc3-of-simple.c > +++ b/drivers/usb/dwc3/dwc3-of-simple.c > @@ -22,11 +22,22 @@ > #include > #include >=20=20 > +/** > + * struct dwc3_of_simple_params - hardware specific parameters > + * @shared_resets: indicates that the resets are shared or exclusive > + * @pulse_resets: use a reset pulse instead of level based resets > + */ > +struct dwc3_of_simple_params { > + bool shared_resets; > + bool pulse_resets; > +}; > + > struct dwc3_of_simple { > struct device *dev; > struct clk **clks; > int num_clocks; > struct reset_control *resets; > + const struct dwc3_of_simple_params *params; instead, you can add these two fields here: bool shared_resets; bool pulse_resets; and ... > @@ -90,17 +101,26 @@ static int dwc3_of_simple_probe(struct platform_devi= ce *pdev) >=20=20 > platform_set_drvdata(pdev, simple); > simple->dev =3D dev; > + simple->params =3D of_device_get_match_data(dev); >=20=20 > - simple->resets =3D of_reset_control_array_get_optional_exclusive(np); > + simple->resets =3D of_reset_control_array_get(np, > + simple->params->shared_resets, > + true); wrap this with a of_device_is_compatible() check: if (of_device_is_compatible(dev->of_node, "foobar")) { simple->shared_resets =3D true; simple->pulse_resets =3D true; } or something like that. Then we don't need to add a new dwc3_of_simple_params for everybody. Also, the why isn't the reset type (pulse vs level) handled by reset framework itself? Why does dwc3-of-simple need to know about it? =2D-=20 balbi --=-=-= Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAEBCgAdFiEElLzh7wn96CXwjh2IzL64meEamQYFAlpu2NYACgkQzL64meEa mQYGgRAA3ep5SbIUVP+PfD0lzEaNyeumwtBOFAbWHo/BaUyE+HdQYjOcCmZjcC17 kPfO3c4i1e+GK/H2GY/uhlnKfDt3SbKxB6FZH2Fn0mQUOjlY9CKiXwf5hDtD8bwU 2889EyTO0ms8OZGXGNruuAS/pKbEYqOx1FDd0Ibl9JibLNO35Ofz/jVo9mpl2+GR SIQU/qtKLd6DUnvX54ymWvsoTvy4UEqu9M0lhB9YE+NJgV2iLaoWQ6V+n8jQsBOb rxFmzAbt+4/SPPp7DNDoeRN//3GmG/W7geKiYvLFbM+0I5sseNHDXKlI9ofDML7L xxGO5Hn5s34NUMCw0KZkBZsW2ZceDRCW83czbaZCzr5vEHWmcawP+iX/6evbasZ1 49gKUGc/QxlJgn6TVgx8V5F8GfePq0J3sNqzfbXAGMuczgYFy//mU0wnSemGDnRu 3rA8AQwAtZ17OMk5BnRAQrxxXFdZE3SS6wb/bnoLJqRATj47W4iR2E0ufYy7L6HA h331rxZgW2vVTkvS30gO1dFdBchD02dG3fccvyGXsw3Ah55MBawtpVjFA52vMgvy xiyBs1IXt0OKI6uiWEJ5f7nDG4T57Nvh3lxilNZK6U+zpWKvg+fb7rtIkQF1kwOv YBx1OaQ4mfUIHr/42KBIvsXrCKwi0RJTkoRzhvSx5wnIeOoSioI= =yeQ5 -----END PGP SIGNATURE----- --=-=-=-- -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html