Re: [PATCH] phy: sunplus: fix error handling in sp_uphy_init()

[email protected] Mon, 03 Aug 2026 15:23:09 +0000
Newsgroups org.infradead.lists.linux-phy
Message-ID <[email protected]>
VGhhbmsgeW91IGZvciB5b3VyIGNvbnRyaWJ1dGlvbiEgU2FzaGlrbyBBSSByZXZpZXcgZm91bmQg
MyBwb3RlbnRpYWwgaXNzdWUocykgdG8gY29uc2lkZXI6CgpQcmUtZXhpc3RpbmcgaXNzdWVzOgot
IFtIaWdoXSBLZXJuZWwgcGFuaWMgZHVlIHRvIGRlcmVmZXJlbmNpbmcgYW4gaW52YWxpZCBlcnJv
ciBwb2ludGVyIGZyb20gbnZtZW1fY2VsbF9nZXQuCi0gW01lZGl1bV0gTWVtb3J5IGxlYWsgb2Yg
dGhlIGJ1ZmZlciByZXR1cm5lZCBieSBudm1lbV9jZWxsX3JlYWQuCi0gW01lZGl1bV0gVW52YWxp
ZGF0ZWQgYXJyYXkgYWNjZXNzIG1heSBsZWFkIHRvIGFuIG91dC1vZi1ib3VuZHMgcmVhZC4KLS0K
Ci0tLSBQYXRjaCBbMV06IFtQQVRDSF0gcGh5OiBzdW5wbHVzOiBmaXggZXJyb3IgaGFuZGxpbmcg
aW4gc3BfdXBoeV9pbml0KCkgLS0tCmNvbW1pdCBiYmRjM2ZiOTIxY2IyNjU3NTk4YjYzNzI0NjYw
MzU1ZmQ5NTZiNDU1CkF1dGhvcjogRmVsaXggR3UgPHVzdGMuZ3VAZ21haWwuY29tPgoKcGh5OiBz
dW5wbHVzOiBmaXggZXJyb3IgaGFuZGxpbmcgaW4gc3BfdXBoeV9pbml0KCkKCkZpeGVzIHRoZSBl
cnJvciBwYXRocyBpbiBzcF91cGh5X2luaXQoKSB0byBjb3JyZWN0bHkgdW5kbyBpbml0aWFsaXph
dGlvbgpzdGVwcy4gSXQgcmV0dXJucyBkaXJlY3RseSBpZiBjbG9jayBlbmFibGUgZmFpbHMsIHJl
bGVhc2VzIHRoZSBjbG9jayBpZgpyZXNldCBkZWFzc2VydCBmYWlscywgYW5kIGFzc2VydHMgcmVz
ZXQgYW5kIGNsb2NrIGlmIGRpc2Nvbm5lY3Qgdm9sdGFnZQp1cGRhdGUgZmFpbHMuCgo+IGRpZmYg
LS1naXQgYS9kcml2ZXJzL3BoeS9zdW5wbHVzL3BoeS1zdW5wbHVzLXVzYjIuYyBiL2RyaXZlcnMv
cGh5L3N1bnBsdXMvcGh5LXN1bnBsdXMtdXNiMi5jCj4gaW5kZXggNjM3YTVmYmFlNmQ5YS4uMGFk
NGM3MTYwZDE3YyAxMDA2NDQKPiAtLS0gYS9kcml2ZXJzL3BoeS9zdW5wbHVzL3BoeS1zdW5wbHVz
LXVzYjIuYwo+ICsrKyBiL2RyaXZlcnMvcGh5L3N1bnBsdXMvcGh5LXN1bnBsdXMtdXNiMi5jCj4g
QEAgLTEyOSw3ICsxMjksNyBAQCBzdGF0aWMgaW50IHNwX3VwaHlfaW5pdChzdHJ1Y3QgcGh5ICpw
aHkpCj4gIAkvKiBkaXNjb25uZWN0IHZvbHRhZ2UgKi8KPiAgCXJldCA9IHVwZGF0ZV9kaXNjX3Zv
bCh1c2JwaHkpOwoKW1NldmVyaXR5OiBIaWdoXQpUaGlzIGlzbid0IGEgYnVnIGludHJvZHVjZWQg
YnkgdGhpcyBwYXRjaCwgYnV0IGxvb2tpbmcgYXQgdXBkYXRlX2Rpc2Nfdm9sKCkKY2FsbGVkIGhl
cmUgaW4gZHJpdmVycy9waHkvc3VucGx1cy9waHktc3VucGx1cy11c2IyLmMsIGlzIHRoZXJlIGEg
cG90ZW50aWFsCmRlcmVmZXJlbmNlIG9mIGFuIGludmFsaWQgcG9pbnRlcj8KCklmIG52bWVtX2Nl
bGxfZ2V0KCkgZmFpbHMgYW5kIHJldHVybnMgYW4gZXJyb3IgcG9pbnRlciBvdGhlciB0aGFuCi1F
UFJPQkVfREVGRVIgKHN1Y2ggYXMgLUVOT0VOVCksIG9yIGlmIGl0IHJldHVybnMgTlVMTCwgdGhl
IGNvZGUgZG9lc24ndApyZXR1cm4gZWFybHkuIEl0IGZhbGxzIHRocm91Z2ggYW5kIHBhc3NlcyB0
aGlzIGludmFsaWQgcG9pbnRlciBkaXJlY3RseSB0bwpudm1lbV9jZWxsX3JlYWQoKToKCgljZWxs
ID0gbnZtZW1fY2VsbF9nZXQodXNicGh5LT5kZXYsIGRpc2NfbmFtZSk7CglpZiAoSVNfRVJSX09S
X05VTEwoY2VsbCkpIHsKCQlpZiAoUFRSX0VSUihjZWxsKSA9PSAtRVBST0JFX0RFRkVSKQoJCQly
ZXR1cm4gLUVQUk9CRV9ERUZFUjsKCX0KCglvdHBfdiA9IG52bWVtX2NlbGxfcmVhZChjZWxsLCAm
b3RwX2wpOwoJbnZtZW1fY2VsbF9wdXQoY2VsbCk7CgpXaWxsIHRoaXMgY2F1c2UgYSBwYW5pYyB3
aGVuIG52bWVtX2NlbGxfcmVhZCgpIGF0dGVtcHRzIHRvIGRlcmVmZXJlbmNlIHRoZQpjZWxsIHBv
aW50ZXI/CgpbU2V2ZXJpdHk6IE1lZGl1bV0KVGhpcyBpcyBhIHByZS1leGlzdGluZyBpc3N1ZSwg
YnV0IGRvZXMgdXBkYXRlX2Rpc2Nfdm9sKCkgbGVhayB0aGUgYnVmZmVyCnJldHVybmVkIGJ5IG52
bWVtX2NlbGxfcmVhZCgpPwoKCW90cF92ID0gbnZtZW1fY2VsbF9yZWFkKGNlbGwsICZvdHBfbCk7
Cgludm1lbV9jZWxsX3B1dChjZWxsKTsKCglpZiAoIUlTX0VSUihvdHBfdikpIHsKCQlzZXQgPSAq
KG90cF92ICsgMSk7CgkJLi4uCgpUaGUgbnZtZW1fY2VsbF9yZWFkKCkgZnVuY3Rpb24gcmV0dXJu
cyBhIGR5bmFtaWNhbGx5IGFsbG9jYXRlZCBidWZmZXIgdGhhdAp0aGUgY2FsbGVyIGlzIHJlc3Bv
bnNpYmxlIGZvciBmcmVlaW5nLiBJdCBhcHBlYXJzIG90cF92IGlzIG5ldmVyIHBhc3NlZCB0bwpr
ZnJlZSgpIGJlZm9yZSB0aGUgZnVuY3Rpb24gcmV0dXJucywgd2hpY2ggd291bGQgcmVzdWx0IGlu
IGEgbWVtb3J5IGxlYWsuCgpbU2V2ZXJpdHk6IE1lZGl1bV0KVGhpcyBpcyBhbm90aGVyIHByZS1l
eGlzdGluZyBpc3N1ZSwgYnV0IGNvdWxkIHRoZSBhcnJheSBhY2Nlc3MgaW4KdXBkYXRlX2Rpc2Nf
dm9sKCkgbGVhZCB0byBhbiBvdXQtb2YtYm91bmRzIHJlYWQ/CgoJb3RwX3YgPSBudm1lbV9jZWxs
X3JlYWQoY2VsbCwgJm90cF9sKTsKCW52bWVtX2NlbGxfcHV0KGNlbGwpOwoKCWlmICghSVNfRVJS
KG90cF92KSkgewoJCXNldCA9ICoob3RwX3YgKyAxKTsKClRoZSBzaXplIG9mIHRoZSByZWFkIGJ1
ZmZlciBpcyBwcm92aWRlZCBpbiBvdHBfbC4gSWYgYSBkZXZpY2UgdHJlZQppbmNvcnJlY3RseSBw
cm92aWRlcyBhIDEtYnl0ZSBudm1lbSBjZWxsLCB1bmNvbmRpdGlvbmFsbHkgYWNjZXNzaW5nIHRo
ZQpzZWNvbmQgYnl0ZSBvZiB0aGUgYnVmZmVyIGF0ICoob3RwX3YgKyAxKSB3aXRob3V0IGNoZWNr
aW5nIGlmIG90cF9sIGlzIGF0CmxlYXN0IDIgbWlnaHQgbGVhZCB0byByZWFkaW5nIG91dC1vZi1i
b3VuZHMgbWVtb3J5LgoKPiAgCWlmIChyZXQgPCAwKQo+IC0JCXJldHVybiByZXQ7Cj4gKwkJZ290
byBlcnJfcmVzZXQ7CgotLSAKU2FzaGlrbyBBSSByZXZpZXcgwrcgaHR0cHM6Ly9zYXNoaWtvLmRl
di8jL3BhdGNoc2V0LzIwMjYwODAzLXN1bnBsdXMtdXNiMy12MS0xLTVhNTYyNTI0Yzg2OUBnbWFp
bC5jb20/cGFydD0xCgotLSAKbGludXgtcGh5IG1haWxpbmcgbGlzdApsaW51eC1waHlAbGlzdHMu
aW5mcmFkZWFkLm9yZwpodHRwczovL2xpc3RzLmluZnJhZGVhZC5vcmcvbWFpbG1hbi9saXN0aW5m
by9saW51eC1waHkK