From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id D0C21C5CFDB for ; Sat, 15 Aug 2026 09:29:57 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:MIME-Version:Reply-To:List-Subscribe: List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id:Message-Id:Date: References:In-Reply-To:Cc:To:Subject:From:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=W6RdR7++FQxC8Phj5XOOselDxEBjmIshX4ihMdRqVTE=; b=kZbzVQc1W2I79Q xvguWLvNTfRy1VumrOHMq9TWw2dWuI7/JK7Lss1r//7AS9j7kPZrNa/YEqE32VFjU7BI+r6X+GiXf ydzzad4leRjQfttVX+uL9oJ2bus7+6jVHknO9zKOtw1QwRzlwKXKdrz3CpeL/wjK9/bL6Jv/Mqwwv 4CykBzvlr5V6zXnkttvsxYAGqdO8jq7b33XlCTjLp4mnGM6xK6AWgJ5LxHYXw8adH0ShgiIagUYdV rv0rlyHjTrqz3+v8LVp203b/HVg1KxMtnMMEdf4awIzxQmgnC23iTN7BleqbFpHuEFRjQWl6XQO2x q/FuOCfhfqxS0v37IoPQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.99.1 #2 (Red Hat Linux)) id 1wvAiD-00000003i3q-2B0P; Sat, 15 Aug 2026 09:29:57 +0000 Received: from tor.source.kernel.org ([2600:3c04:e001:324:0:1991:8:25]) by bombadil.infradead.org with esmtps (Exim 4.99.1 #2 (Red Hat Linux)) id 1wvAiB-00000003i3k-1v8p for linux-phy@lists.infradead.org; Sat, 15 Aug 2026 09:29:55 +0000 Received: from smtp.kernel.org (quasi.space.kernel.org [100.103.45.18]) by tor.source.kernel.org (Postfix) with ESMTP id C1D7F60008; Sat, 15 Aug 2026 09:29:54 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id E61ED1F000E9; Sat, 15 Aug 2026 09:29:53 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1786786194; bh=yxcmxBndzvKDGTFViWT6R+/5sUpaWul9J56V9sHIJck=; h=From:Subject:Reply-To:To:Cc:In-Reply-To:References:Date; b=kDbSp7dtycgOgBWpuG9GwZ0ACsGk0MfRmok2AYXCdC+vcrN28C3TgY0Q2nNvDKSJK bifxKP6l1+PmOTAPJt6mImEyfBfvH6xv0Ra6ZqwWadjH/cLpFnOGIw3INfE68K/kIA AYtBb7lnDKFlUGkpliS4A/+8lCdgGk15bsBa+Na1r+y/WRodUGuUXQQIK0fJyu6nqk sLLcoZi4MZ9uPAL3MUG6V4f2zZZGn57rPcY+UkebmZlEtEhbpa6vF5xA7kD6csdjiW qXGshdNjVYVR3oPkD+wzvIbfWtMi3ts4kKZYlPskK5tiKodaOsT0KuO0tZleReNcAo EV4pO6EChB+ow== From: sashiko-bot@kernel.org Subject: Re: [PATCH v7 5/6] phy: realtek: usb2: add support for RTL9607C USB2 PHY To: "Rustam Adilov" Cc: neil.armstrong@linaro.org, linux-phy@lists.infradead.org, olteanv@gmail.com, robh@kernel.org, conor+dt@kernel.org, devicetree@vger.kernel.org, vkoul@kernel.org In-Reply-To: <20260815091340.26910-6-adilov@disroot.org> References: <20260815091340.26910-1-adilov@disroot.org> <20260815091340.26910-6-adilov@disroot.org> Date: Sat, 15 Aug 2026 09:29:53 +0000 Message-Id: <20260815092953.E61ED1F000E9@smtp.kernel.org> X-BeenThere: linux-phy@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Linux Phy Mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: sashiko-reviews@lists.linux.dev MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Sender: "linux-phy" Errors-To: linux-phy-bounces+linux-phy=archiver.kernel.org@lists.infradead.org VGhhbmsgeW91IGZvciB5b3VyIGNvbnRyaWJ1dGlvbiEgU2FzaGlrbyBBSSByZXZpZXcgZm91bmQg NiBwb3RlbnRpYWwgaXNzdWUocykgdG8gY29uc2lkZXI6CgpOZXcgaXNzdWVzOgotIFtNZWRpdW1d IFRoZSBydGw5NjA3X3BoeV9jZmcgc3RydWN0dXJlIG9taXRzIHRoZSAuY2hlY2tfZWZ1c2UgZmxh ZywgY2F1c2luZyBlRnVzZSBwYXJhbWV0ZXJzIHRvIGJlIGlnbm9yZWQuCi0gW01lZGl1bV0gVGhl IHBhZ2UwIHJlZ2lzdGVyIGNvbmZpZ3VyYXRpb24gc3RvcmVzIGFkZHJlc3MgMHhmMyBhdCB0aGUg d3JvbmcgaW5kZXgsIGNvcnJ1cHRpbmcgZGVidWdmcyBvdXRwdXQuCgpQcmUtZXhpc3RpbmcgaXNz dWVzOgotIFtIaWdoXSBVbmNoZWNrZWQgcmV0dXJuIHZhbHVlIGZyb20gb2ZfaW9tYXAoKSBsZWFk cyB0byBhIE5VTEwgcG9pbnRlciBkZXJlZmVyZW5jZS4KLSBbSGlnaF0gUG9pbnRlciBhcml0aG1l dGljIG9uIHZvaWQgX19pb21lbSAqIGNhdXNlcyB1bmFsaWduZWQgZGV2aWNlIHJlZ2lzdGVyIGFj Y2Vzc2VzLgotIFtIaWdoXSBTaWduZWQgY2hhciBpbnRlZ2VyIHByb21vdGlvbiBjb3JydXB0cyBy ZWdpc3RlciB3cml0ZXMgb24gYXJjaGl0ZWN0dXJlcyB3aXRoIHNpZ25lZCBjaGFycyAoZS5nLiwg TUlQUykuCi0gW0hpZ2hdIFNpbGVudCBmYWlsdXJlIHdoZW4gZG9fcnRrX3BoeV9pbml0KCkgZmFp bHMgZm9yIG11bHRpLXBvcnQgU29Dcy4KLS0KCi0tLSBQYXRjaCBbNV06IFtQQVRDSCB2NyA1LzZd IHBoeTogcmVhbHRlazogdXNiMjogYWRkIHN1cHBvcnQgZm9yIFJUTDk2MDdDIFVTQjIgUEhZIC0t LQpjb21taXQgNmE1YjRlMjIzNTMxNGQ0NzMwY2NkMTQ3YmUxMjE0OGIxOWFmYzk4NApBdXRob3I6 IFJ1c3RhbSBBZGlsb3YgPGFkaWxvdkBkaXNyb290Lm9yZz4KCiAgICBwaHk6IHJlYWx0ZWs6IHVz YjI6IGFkZCBzdXBwb3J0IGZvciBSVEw5NjA3QyBVU0IyIFBIWQogICAgCiAgICBBZGQgc3VwcG9y dCBmb3IgdGhlIHVzYjIgcGh5IG9mIFJUTDk2MDdDIHNlcmllcyBiYXNlZCBTb0NzLgogICAgQWRk IHRoZSBtYWNyb3MgYW5kIHBoeSBjb25maWcgc3RydWN0IGZvciBydGw5NjA3LgoKPiBkaWZmIC0t Z2l0IGEvZHJpdmVycy9waHkvcmVhbHRlay9waHktcnRrLXVzYjIuYyBiL2RyaXZlcnMvcGh5L3Jl YWx0ZWsvcGh5LXJ0ay11c2IyLmMKPiBpbmRleCAyZGRhZDdiZTczNTM0Li45OTBjZWNhMzdlZjhh IDEwMDY0NAo+IC0tLSBhL2RyaXZlcnMvcGh5L3JlYWx0ZWsvcGh5LXJ0ay11c2IyLmMKPiArKysg Yi9kcml2ZXJzL3BoeS9yZWFsdGVrL3BoeS1ydGstdXNiMi5jCgpbIC4uLiBdCgo+IEBAIC0xMzQy LDYgKzEzODQsMjkgQEAgc3RhdGljIGNvbnN0IHN0cnVjdCBwaHlfY2ZnIHJ0ZDEzMTVlX3BoeV9j ZmcgPSB7Cj4gIAkucmVnX2Rlc2MgPSAmcnRkX3BoeV9yZWdfZGVzYywKPiAgfTsKPiAgCj4gK3N0 YXRpYyBjb25zdCBzdHJ1Y3QgcGh5X2NmZyBydGw5NjA3X3BoeV9jZmcgPSB7Cj4gKwkucGFnZTBf c2l6ZSA9IE1BWF9VU0JfUEhZX1BBR0UwX0RBVEFfU0laRSwKPiArCS5wYWdlMCA9IHsgWzBdID0g ezB4ZTAsIDB4OTV9LAoKW1NldmVyaXR5OiBIaWdoXQpUaGlzIGlzIGEgcHJlLWV4aXN0aW5nIGlz c3VlLCBidXQgZG9lcyB3cml0aW5nIDB4OTUgaGVyZSBjb3JydXB0IHRoZSByZWdpc3RlcgpvbiBh cmNoaXRlY3R1cmVzIGxpa2UgTUlQUz8KCnJ0a19waHlfd3JpdGUoKSB0YWtlcyBkYXRhIGFzIGEg c2lnbmVkIGNoYXI6Cgpkcml2ZXJzL3BoeS9yZWFsdGVrL3BoeS1ydGstdXNiMi5jOnJ0a19waHlf d3JpdGUoKSB7Ci4uLgoJcmVnX2Rlc2MtPndyaXRlKCh1MzIpZGF0YSA8PCBzaGlmdF9iaXRzLAou Li4KfQoKT24gYXJjaGl0ZWN0dXJlcyB3aGVyZSBjaGFyIGlzIHNpZ25lZCBieSBkZWZhdWx0LCBh IHZhbHVlIG9mIDB4OTUgaXMKaW50ZXJwcmV0ZWQgYXMgYSBuZWdhdGl2ZSBudW1iZXIgKC0xMDcp LiBXaGVuIGNhc3QgdG8gdTMyLCBpdCB1bmRlcmdvZXMgc2lnbgpleHRlbnNpb24gdG8gMHhGRkZG RkY5NSwgYW5kIGFmdGVyIHNoaWZ0aW5nLCBpdCBjYW4gY2xvYmJlciB0aGUgdXBwZXIgYml0cwpv ZiB0aGUgc2hhcmVkIHJlZ193cmFwX3ZzdGF0dXMgcmVnaXN0ZXIuCgo+ICsJCSAgIFs0XSA9IHsw eGU0LCAweDZhfSwKPiArCQkgIFsxMl0gPSB7MHhmMywgMHgzMX0sIH0sCgpbU2V2ZXJpdHk6IE1l ZGl1bV0KV2lsbCB0aGlzIG1hcCB0byB0aGUgY29ycmVjdCBkZWJ1Z2ZzIG91dHB1dD8KCkFkZHJl c3MgMHhmMyBtYXRoZW1hdGljYWxseSBtYXBzIHRvIGFycmF5IGluZGV4IDExIHZpYSB0aGUgc3Ry aWN0IG1hcHBpbmcKZm9ybXVsYSBpbiBwYWdlX2FkZHJfdG9fYXJyYXlfaW5kZXgoKS4gQmVjYXVz ZSBpdCBpcyBwbGFjZWQgYXQgaW5kZXggMTIKaGVyZSwgd2hlbiBydGtfdXNiMl9wYXJhbWV0ZXJf c2hvdygpIGNvbXB1dGVzIHRoZSB0YXJnZXQgcmVhZCBhZGRyZXNzCmR5bmFtaWNhbGx5LCBpdCB3 aWxsIHJlYWQgZnJvbSAweGY0IGFuZCBkaXNwbGF5IGl0IGFsb25nc2lkZSB0aGUgZXhwZWN0ZWQK MHhmMyBkYXRhLgoKPiArCS5wYWdlMV9zaXplID0gTUFYX1VTQl9QSFlfUEFHRTFfREFUQV9TSVpF LAo+ICsJLnBhZ2UxID0geyBbMF0gPSB7MHhlMCwgMHgyNn0sIH0sCj4gKwkucGFnZTJfc2l6ZSA9 IE1BWF9VU0JfUEhZX1BBR0UyX0RBVEFfU0laRSwKPiArCS5wYWdlMiA9IHsgWzddID0gezB4ZTcs IDB4MzN9LCB9LAo+ICsJLm51bV9waHkgPSAxLAo+ICsJLmNoZWNrX2VmdXNlX3ZlcnNpb24gPSBD SEVDS19FRlVTRV9WMiwKCltTZXZlcml0eTogTWVkaXVtXQpJcyB0aGUgLmNoZWNrX2VmdXNlIGZs YWcgbWlzc2luZyBoZXJlPwoKV2l0aG91dCAuY2hlY2tfZWZ1c2UgPSB0cnVlLCB0aGUgZUZ1c2Ug cGFyYW1ldGVycyBsaWtlIGNoZWNrX2VmdXNlX3ZlcnNpb24KYW5kIGVmdXNlX2RjX2RyaXZpbmdf cmF0ZSBzcGVjaWZpZWQgaW4gdGhpcyBzdHJ1Y3R1cmUgYXBwZWFyIHRvIGJlIGlnbm9yZWQKYnkg Z2V0X3BoeV9kYXRhX2J5X2VmdXNlKCksIGxlYXZpbmcgdGhlIGhhcmR3YXJlIHVuY2FsaWJyYXRl ZC4KCj4gKwkuZWZ1c2VfZGNfZHJpdmluZ19yYXRlID0gRUZVU19VU0JfRENfQ0FMX1JBVEUsCj4g KwkuZGNfZHJpdmluZ19tYXNrID0gMHgxZiwKPiArCS5lZnVzZV9kY19kaXNjb25uZWN0X3JhdGUg PSBFRlVTX1VTQl9EQ19ESVNfUkFURSwKPiArCS5kY19kaXNjb25uZWN0X21hc2sgPSAweGYsCj4g KwkudXNiX2RjX2Rpc2Nvbm5lY3RfYXRfcGFnZTAgPSB0cnVlLAo+ICsJLmRvX3RvZ2dsZSA9IHRy dWUsCj4gKwkuZHJpdmluZ191cGRhdGVkX2Zvcl9kZXZfZGlzID0gMHg4LAo+ICsJLmlzX2RvdWJs ZV9zZW5zaXRpdml0eV9tb2RlID0gdHJ1ZSwKPiArCS5yZWdfZGVzYyA9ICZydGw5NjA3X3BoeV9y ZWdfZGVzYywKPiArCS5mb3JjZV9ob3N0X2Rpc2Nvbm5lY3QgPSB0cnVlLAo+ICt9Owo+ICsKClsg Li4uIF0KCltTZXZlcml0eTogSGlnaF0KVGhpcyBpc24ndCBhIGJ1ZyBpbnRyb2R1Y2VkIGJ5IHRo aXMgcGF0Y2gsIGJ1dCBsb29raW5nIGF0IHRoZSBleGlzdGluZwpwYXJzZV9waHlfZGF0YSgpIGZ1 bmN0aW9uOgoKZHJpdmVycy9waHkvcmVhbHRlay9waHktcnRrLXVzYjIuYzpwYXJzZV9waHlfZGF0 YSgpIHsKLi4uCglwaHlfcGFyYW1ldGVyLT5waHlfcmVnLnJlZ193cmFwX3ZzdGF0dXMgPSBvZl9p b21hcChucCwgMCk7Ci4uLgp9CgpEb2VzIHRoaXMgY29kZSBkZXJlZmVyZW5jZSBhIE5VTEwgcG9p bnRlciBkdXJpbmcgZGV2aWNlIHByb2JlIGlmIHRoZSBkZXZpY2UKdHJlZSBsYWNrcyB0aGUgcmVx dWlyZWQgcmVnIHByb3BlcnRpZXM/CgpUaGUgcmV0dXJuIHZhbHVlcyBmcm9tIG9mX2lvbWFwKCkg YXJlIGFzc2lnbmVkIGFuZCBzdWJzZXF1ZW50bHkgZGVyZWZlcmVuY2VkCndpdGhvdXQgYW55IHZh bGlkYXRpb24uCgpbU2V2ZXJpdHk6IEhpZ2hdClRoaXMgaXMgYSBwcmUtZXhpc3RpbmcgaXNzdWUs IGJ1dCBkb2VzIGFkZGluZyBpbmRleCB0byB0aGUgb2ZfaW9tYXAoKSByZXN1bHQKY2F1c2UgdW5h bGlnbmVkIGRldmljZSByZWdpc3RlciBhY2Nlc3Nlcz8KCkxvb2tpbmcgYXQgdGhlIGV4aXN0aW5n IHBhcnNlX3BoeV9kYXRhKCkgZnVuY3Rpb246Cgpkcml2ZXJzL3BoeS9yZWFsdGVrL3BoeS1ydGst dXNiMi5jOnBhcnNlX3BoeV9kYXRhKCkgewouLi4KCXBoeV9wYXJhbWV0ZXItPnBoeV9yZWcucmVn X2d1c2IycGh5YWNjMCA9IG9mX2lvbWFwKG5wLCAxKSArIGluZGV4OwouLi4KfQoKU2luY2Ugb2Zf aW9tYXAoKSByZXR1cm5zIHZvaWQgX19pb21lbSAqLCBHQ0MgdHJlYXRzIHRoZSBwb2ludGVyIGFy aXRobWV0aWMKYXMgMS1ieXRlIGVsZW1lbnRzLiBGb3IgbXVsdGktcG9ydCBQSFlzIHdoZXJlIGlu ZGV4ID4gMCwgdGhpcyBhZGRzIGV4YWN0bHkKMSBieXRlIHRvIHRoZSBiYXNlIGFkZHJlc3MsIHdo aWNoIHdpbGwgdHJpZ2dlciBhbGlnbm1lbnQgZXhjZXB0aW9ucyBkdXJpbmcKc3Vic2VxdWVudCAz Mi1iaXQgd3JpdGVsKCkgY2FsbHMgb24gYXJjaGl0ZWN0dXJlcyBsaWtlIEFSTTY0LgoKW1NldmVy aXR5OiBIaWdoXQpUaGlzIGlzIGEgcHJlLWV4aXN0aW5nIGlzc3VlLCBidXQgZG9lcyB0aGUgaW5p dGlhbGl6YXRpb24gbG9vcCBtYXNrIGVycm9ycwpmb3IgbXVsdGktcG9ydCBTb0NzPwoKTG9va2lu ZyBhdCB0aGUgZXhpc3RpbmcgcnRrX3BoeV9pbml0KCkgZnVuY3Rpb246Cgpkcml2ZXJzL3BoeS9y ZWFsdGVrL3BoeS1ydGstdXNiMi5jOnJ0a19waHlfaW5pdCgpIHsKLi4uCglmb3IgKGkgPSAwOyBp IDwgcnRrX3BoeS0+bnVtX3BoeTsgaSsrKQoJCXJldCA9IGRvX3J0a19waHlfaW5pdChydGtfcGh5 LCBpKTsKLi4uCn0KCklmIGRvX3J0a19waHlfaW5pdCgpIGZhaWxzIGZvciBhbnkgcG9ydCBleGNl cHQgdGhlIGxhc3Qgb25lLCByZXQgaXMKb3ZlcndyaXR0ZW4gYnkgdGhlIHJlc3VsdCBvZiB0aGUg c3Vic2VxdWVudCBQSFkncyBpbml0aWFsaXphdGlvbiwgc2lsZW50bHkKaWdub3JpbmcgZWFybGll ciBmYWlsdXJlcyB3aGlsZSBpbmNvcnJlY3RseSByZXBvcnRpbmcgcHJvYmUgc3VjY2Vzcy4KCi0t IApTYXNoaWtvIEFJIHJldmlldyDCtyBodHRwczovL3Nhc2hpa28uZGV2LyMvcGF0Y2hzZXQvMjAy NjA4MTUwOTEzNDAuMjY5MTAtMS1hZGlsb3ZAZGlzcm9vdC5vcmc/cGFydD01CgotLSAKbGludXgt cGh5IG1haWxpbmcgbGlzdApsaW51eC1waHlAbGlzdHMuaW5mcmFkZWFkLm9yZwpodHRwczovL2xp c3RzLmluZnJhZGVhZC5vcmcvbWFpbG1hbi9saXN0aW5mby9saW51eC1waHkK From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-alma10-1.taild15c8.ts.net [100.103.45.18]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 8C19B375F8E for ; Sat, 15 Aug 2026 09:29:54 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=100.103.45.18 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786786195; cv=none; b=nWOmfR+WXmEDZFbvrCx2wERNaIid9AtPPH4WtgymSMin/nOfEVdhadmTrGNzY9lCALNVsu6XzUPV7iPjjcFB5y0dfdcHKIKyo4ld/+TOYs6G1bferjE7Ff8jBlEUcxrYpP3Bf957hQxsIjufDs0Yp7sbTjM6eFDdaV3H6R8exHc= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786786195; c=relaxed/simple; bh=Gf4+h5Z8PMXznnzIcNpdG3O88hTKGadpxDTpT6jO4/0=; h=From:Subject:To:Cc:In-Reply-To:References:Content-Type:Date: Message-Id; b=HeyuqsSaA7PQIUcse79lh69wS4gAen/5oxs6EjK3g/vDjofn3LB8GTmkrqKY2mcYNUFeDaFtUIXiA66Ipas9r2cFNRVXkbQ8zKEsREKfVdvYsKUkOpHNhurAjM3U24rp+fBUWGnsKjoI8IfyM73oWscu9ltxVzFrVaq5/vEFXSs= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=kDbSp7dt; arc=none smtp.client-ip=100.103.45.18 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="kDbSp7dt" Received: by smtp.kernel.org (Postfix) with ESMTPSA id E61ED1F000E9; Sat, 15 Aug 2026 09:29:53 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1786786194; bh=yxcmxBndzvKDGTFViWT6R+/5sUpaWul9J56V9sHIJck=; h=From:Subject:Reply-To:To:Cc:In-Reply-To:References:Date; b=kDbSp7dtycgOgBWpuG9GwZ0ACsGk0MfRmok2AYXCdC+vcrN28C3TgY0Q2nNvDKSJK bifxKP6l1+PmOTAPJt6mImEyfBfvH6xv0Ra6ZqwWadjH/cLpFnOGIw3INfE68K/kIA AYtBb7lnDKFlUGkpliS4A/+8lCdgGk15bsBa+Na1r+y/WRodUGuUXQQIK0fJyu6nqk sLLcoZi4MZ9uPAL3MUG6V4f2zZZGn57rPcY+UkebmZlEtEhbpa6vF5xA7kD6csdjiW qXGshdNjVYVR3oPkD+wzvIbfWtMi3ts4kKZYlPskK5tiKodaOsT0KuO0tZleReNcAo EV4pO6EChB+ow== From: sashiko-bot@kernel.org Subject: Re: [PATCH v7 5/6] phy: realtek: usb2: add support for RTL9607C USB2 PHY Reply-To: sashiko-reviews@lists.linux.dev To: "Rustam Adilov" Cc: neil.armstrong@linaro.org, linux-phy@lists.infradead.org, olteanv@gmail.com, robh@kernel.org, conor+dt@kernel.org, devicetree@vger.kernel.org, vkoul@kernel.org In-Reply-To: <20260815091340.26910-6-adilov@disroot.org> References: <20260815091340.26910-1-adilov@disroot.org> <20260815091340.26910-6-adilov@disroot.org> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Date: Sat, 15 Aug 2026 09:29:53 +0000 Message-Id: <20260815092953.E61ED1F000E9@smtp.kernel.org> Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Thank you for your contribution! Sashiko AI review found 6 potential issue(= s) to consider: New issues: - [Medium] The rtl9607_phy_cfg structure omits the .check_efuse flag, causi= ng eFuse parameters to be ignored. - [Medium] The page0 register configuration stores address 0xf3 at the wron= g index, corrupting debugfs output. Pre-existing issues: - [High] Unchecked return value from of_iomap() leads to a NULL pointer der= eference. - [High] Pointer arithmetic on void __iomem * causes unaligned device regis= ter accesses. - [High] Signed char integer promotion corrupts register writes on architec= tures with signed chars (e.g., MIPS). - [High] Silent failure when do_rtk_phy_init() fails for multi-port SoCs. -- --- Patch [5]: [PATCH v7 5/6] phy: realtek: usb2: add support for RTL9607C = USB2 PHY --- commit 6a5b4e2235314d4730ccd147be12148b19afc984 Author: Rustam Adilov phy: realtek: usb2: add support for RTL9607C USB2 PHY =20 Add support for the usb2 phy of RTL9607C series based SoCs. Add the macros and phy config struct for rtl9607. > diff --git a/drivers/phy/realtek/phy-rtk-usb2.c b/drivers/phy/realtek/phy= -rtk-usb2.c > index 2ddad7be73534..990ceca37ef8a 100644 > --- a/drivers/phy/realtek/phy-rtk-usb2.c > +++ b/drivers/phy/realtek/phy-rtk-usb2.c [ ... ] > @@ -1342,6 +1384,29 @@ static const struct phy_cfg rtd1315e_phy_cfg =3D { > .reg_desc =3D &rtd_phy_reg_desc, > }; > =20 > +static const struct phy_cfg rtl9607_phy_cfg =3D { > + .page0_size =3D MAX_USB_PHY_PAGE0_DATA_SIZE, > + .page0 =3D { [0] =3D {0xe0, 0x95}, [Severity: High] This is a pre-existing issue, but does writing 0x95 here corrupt the regist= er on architectures like MIPS? rtk_phy_write() takes data as a signed char: drivers/phy/realtek/phy-rtk-usb2.c:rtk_phy_write() { ... reg_desc->write((u32)data << shift_bits, ... } On architectures where char is signed by default, a value of 0x95 is interpreted as a negative number (-107). When cast to u32, it undergoes sign extension to 0xFFFFFF95, and after shifting, it can clobber the upper bits of the shared reg_wrap_vstatus register. > + [4] =3D {0xe4, 0x6a}, > + [12] =3D {0xf3, 0x31}, }, [Severity: Medium] Will this map to the correct debugfs output? Address 0xf3 mathematically maps to array index 11 via the strict mapping formula in page_addr_to_array_index(). Because it is placed at index 12 here, when rtk_usb2_parameter_show() computes the target read address dynamically, it will read from 0xf4 and display it alongside the expected 0xf3 data. > + .page1_size =3D MAX_USB_PHY_PAGE1_DATA_SIZE, > + .page1 =3D { [0] =3D {0xe0, 0x26}, }, > + .page2_size =3D MAX_USB_PHY_PAGE2_DATA_SIZE, > + .page2 =3D { [7] =3D {0xe7, 0x33}, }, > + .num_phy =3D 1, > + .check_efuse_version =3D CHECK_EFUSE_V2, [Severity: Medium] Is the .check_efuse flag missing here? Without .check_efuse =3D true, the eFuse parameters like check_efuse_version and efuse_dc_driving_rate specified in this structure appear to be ignored by get_phy_data_by_efuse(), leaving the hardware uncalibrated. > + .efuse_dc_driving_rate =3D EFUS_USB_DC_CAL_RATE, > + .dc_driving_mask =3D 0x1f, > + .efuse_dc_disconnect_rate =3D EFUS_USB_DC_DIS_RATE, > + .dc_disconnect_mask =3D 0xf, > + .usb_dc_disconnect_at_page0 =3D true, > + .do_toggle =3D true, > + .driving_updated_for_dev_dis =3D 0x8, > + .is_double_sensitivity_mode =3D true, > + .reg_desc =3D &rtl9607_phy_reg_desc, > + .force_host_disconnect =3D true, > +}; > + [ ... ] [Severity: High] This isn't a bug introduced by this patch, but looking at the existing parse_phy_data() function: drivers/phy/realtek/phy-rtk-usb2.c:parse_phy_data() { ... phy_parameter->phy_reg.reg_wrap_vstatus =3D of_iomap(np, 0); ... } Does this code dereference a NULL pointer during device probe if the device tree lacks the required reg properties? The return values from of_iomap() are assigned and subsequently dereferenced without any validation. [Severity: High] This is a pre-existing issue, but does adding index to the of_iomap() result cause unaligned device register accesses? Looking at the existing parse_phy_data() function: drivers/phy/realtek/phy-rtk-usb2.c:parse_phy_data() { ... phy_parameter->phy_reg.reg_gusb2phyacc0 =3D of_iomap(np, 1) + index; ... } Since of_iomap() returns void __iomem *, GCC treats the pointer arithmetic as 1-byte elements. For multi-port PHYs where index > 0, this adds exactly 1 byte to the base address, which will trigger alignment exceptions during subsequent 32-bit writel() calls on architectures like ARM64. [Severity: High] This is a pre-existing issue, but does the initialization loop mask errors for multi-port SoCs? Looking at the existing rtk_phy_init() function: drivers/phy/realtek/phy-rtk-usb2.c:rtk_phy_init() { ... for (i =3D 0; i < rtk_phy->num_phy; i++) ret =3D do_rtk_phy_init(rtk_phy, i); ... } If do_rtk_phy_init() fails for any port except the last one, ret is overwritten by the result of the subsequent PHY's initialization, silently ignoring earlier failures while incorrectly reporting probe success. --=20 Sashiko AI review =C2=B7 https://sashiko.dev/#/patchset/20260815091340.2691= 0-1-adilov@disroot.org?part=3D5