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 6A244C43602 for ; Wed, 8 Jul 2026 10:49:03 +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=oKYd04ZoJsGJsJJ2yFXHkqXywpMnyfuB7+OWK3A/ioM=; b=eGkfbcQQE4I6vu i1pGxsFk3tCh9CcedbmuQ70d2DxN37wyVKAFo51V7oSy1gWrN9TkxIsPcnoCHjjQWYgMCbZg9UI+7 UViBXGtsaw318CsyLGTc0MbcX1diesoQJnaY22BlJmv8eTsboE2bqo3UGzPIvq519AtpKpRTSznSB 381yzXdSYF1qOStbyNvM4xmmxGEbj+eECUotEeq7Buana7qCwC/59MgO2M0stzu2PjxEEtdsqJtHD iEuB4+kta9Gje3+Dqh5p1RMdT42441I8tp26iogXfrZ2OrB4gWqp6wzGnRjYZe/PPji8USmPNSETr 1v9mfiDw+uStnfW/2tDQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.99.1 #2 (Red Hat Linux)) id 1whPpv-0000000Gwbg-0hBS; Wed, 08 Jul 2026 10:49:03 +0000 Received: from tor.source.kernel.org ([172.105.4.254]) by bombadil.infradead.org with esmtps (Exim 4.99.1 #2 (Red Hat Linux)) id 1whPpu-0000000GwbY-16Ln for linux-phy@lists.infradead.org; Wed, 08 Jul 2026 10:49:02 +0000 Received: from smtp.kernel.org (quasi.space.kernel.org [100.103.45.18]) by tor.source.kernel.org (Postfix) with ESMTP id B0968601D9; Wed, 8 Jul 2026 10:49:01 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id D616E1F000E9; Wed, 8 Jul 2026 10:49:00 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1783507741; bh=NjmxgfT50Pozt5M6c45G6mi5aTkKtqZO4ykzegNi2+A=; h=From:Subject:Reply-To:To:Cc:In-Reply-To:References:Date; b=DrKY8UAxwTpzhIk3f558wYx1DM2BWiMkyACdArzrh6nGPuezDj7FlM/O1Y+/CFVKy 8LqkZJ+kTQQhCeNzNwbx0StMQCllHmGgSauclAniz+jfWt7avENlbCUmfavjzVaBPz ygNJ5fxXQdyu7P0UFOxDFnLDUVJEGT03sjOSWIuemtR3Biai8LD/1AdGRvk2Pz6otc rfasjEpKU6hrbnlx3Pn+U0ktUspeFU+7yhVqf7lAsvxQlsVKbcWaSs1v/G+n/lVjGw zer9dZtx1ErUZjg2iMcyBW3Bg4L1/TCaIX9FUp2eSOiCXELOJMSVo2R/py3dVMtE0G CfvKzI90E08AA== From: sashiko-bot@kernel.org Subject: Re: [PATCH v3 1/3] dt-bindings: phy: nuvoton,ma35d1-usb2-phy: extend for dual-port and OTG To: "Joey Lu" Cc: neil.armstrong@linaro.org, devicetree@vger.kernel.org, olteanv@gmail.com, linux-phy@lists.infradead.org, vkoul@kernel.org, robh@kernel.org, conor+dt@kernel.org In-Reply-To: <20260708103606.1462960-2-a0987203069@gmail.com> References: <20260708103606.1462960-1-a0987203069@gmail.com> <20260708103606.1462960-2-a0987203069@gmail.com> Date: Wed, 08 Jul 2026 10:49:00 +0000 Message-Id: <20260708104900.D616E1F000E9@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 MiBwb3RlbnRpYWwgaXNzdWUocykgdG8gY29uc2lkZXI6Ci0gW0hpZ2hdIFRoZSBEVCBiaW5kaW5n IGludHJvZHVjZXMgbXVsdGktcG9ydCBzdXBwb3J0LCBidXQgdGhlIGNvcnJlc3BvbmRpbmcgZHJp dmVyIGxhY2tzIGltcGxlbWVudGF0aW9uLCBsZWFkaW5nIHRvIHNpbGVudCBoYXJkd2FyZSBtYWxm dW5jdGlvbi4KLSBbTWVkaXVtXSBUaGUgYCNwaHktY2VsbHNgIHByb3BlcnR5IHVzZXMgYW4gZW51 bSBjb25zdHJhaW50IGluc3RlYWQgb2YgYSBzdHJpY3QgY29uc3QgY29uc3RyYWludCwgdmlvbGF0 aW5nIHN1YnN5c3RlbSBndWlkZWxpbmVzLgotLQoKLS0tIFBhdGNoIFsxXTogW1BBVENIIHYzIDEv M10gZHQtYmluZGluZ3M6IHBoeTogbnV2b3RvbixtYTM1ZDEtdXNiMi1waHk6IGV4dGVuZCBmb3Ig ZHVhbC1wb3J0IGFuZCBPVEcgLS0tCmNvbW1pdCA3ZjcyYThjYzRiNjRlMWRmY2Y0ZWZiYTFjZDY3 YmY5ZWFmZDMxZDcyCkF1dGhvcjogSm9leSBMdSA8YTA5ODcyMDMwNjlAZ21haWwuY29tPgoKZHQt YmluZGluZ3M6IHBoeTogbnV2b3RvbixtYTM1ZDEtdXNiMi1waHk6IGV4dGVuZCBmb3IgZHVhbC1w b3J0IGFuZCBPVEcKClRoaXMgY29tbWl0IHVwZGF0ZXMgdGhlIG51dm90b24sbWEzNWQxLXVzYjIt cGh5IGRldmljZSB0cmVlIGJpbmRpbmcgdG8Kc3VwcG9ydCBkdWFsIFVTQiBQSFkgcG9ydHMgKFBI WTAgYW5kIFBIWTEpLiBJdCByZWxheGVzICNwaHktY2VsbHMgZnJvbSBjb25zdDogMAp0byBlbnVt OiBbMCwgMV0gdG8gYWxsb3cgcG9ydCBzZWxlY3Rpb24gd2hpbGUgcHJlc2VydmluZyBiYWNrd2Fy ZApjb21wYXRpYmlsaXR5LiBJdCBhbHNvIGFkZHMgbmV3IG9wdGlvbmFsIHByb3BlcnRpZXMgZm9y IHJlc2lzdG9yIGNhbGlicmF0aW9uCmFuZCBvdmVyLWN1cnJlbnQgcG9sYXJpdHkuCgo+IGRpZmYg LS1naXQgYS9Eb2N1bWVudGF0aW9uL2RldmljZXRyZWUvYmluZGluZ3MvcGh5L251dm90b24sbWEz NWQxLXVzYjItcGh5LnlhbWwgYi9Eb2N1bWVudGF0aW9uL2RldmljZXRyZWUvYmluZGluZ3MvcGh5 L251dm90b24sbWEzNWQxLXVzYjItcGh5LnlhbWwKPiBpbmRleCBmZmY4NThjOTA5YTA2Li4xOTFk MWIyMjcyZWM1IDEwMDY0NAo+IC0tLSBhL0RvY3VtZW50YXRpb24vZGV2aWNldHJlZS9iaW5kaW5n cy9waHkvbnV2b3RvbixtYTM1ZDEtdXNiMi1waHkueWFtbAo+ICsrKyBiL0RvY3VtZW50YXRpb24v ZGV2aWNldHJlZS9iaW5kaW5ncy9waHkvbnV2b3RvbixtYTM1ZDEtdXNiMi1waHkueWFtbApbIC4u LiBdCj4gQEAgLTE1LDcgKzI3LDEyIEBAIHByb3BlcnRpZXM6Cj4gICAgICAgIC0gbnV2b3Rvbixt YTM1ZDEtdXNiMi1waHkKPiAgCj4gICAgIiNwaHktY2VsbHMiOgo+IC0gICAgY29uc3Q6IDAKPiAr ICAgIGVudW06IFswLCAxXQoKW1NldmVyaXR5OiBIaWdoXQpEb2VzIHRoZSBkcml2ZXIgY3VycmVu dGx5IHN1cHBvcnQgdGhpcyBuZXcgI3BoeS1jZWxscyA9IDwxPiBjb25maWd1cmF0aW9uPwoKTG9v a2luZyBhdCBtYTM1X3VzYl9waHlfcHJvYmUoKSBpbiBkcml2ZXJzL3BoeS9udXZvdG9uL3BoeS1t YTM1ZDEtdXNiMi5jLCB0aGUgCnByb3ZpZGVyIHJlZ2lzdGVycyB1c2luZyBvZl9waHlfc2ltcGxl X3hsYXRlOgoKICAgIHByb3ZpZGVyID0gZGV2bV9vZl9waHlfcHJvdmlkZXJfcmVnaXN0ZXIoJnBk ZXYtPmRldiwgb2ZfcGh5X3NpbXBsZV94bGF0ZSk7CgpCZWNhdXNlIG9mX3BoeV9zaW1wbGVfeGxh dGUgaWdub3JlcyBjZWxsIGFyZ3VtZW50cywgYW55IHZhbGlkIERUIHJlcXVlc3QgZm9yClBIWTEg d2lsbCBzaWxlbnRseSBtYXAgdG8gdGhlIHNpbmdsZSBpbml0aWFsaXplZCBQSFkuCgpGdXJ0aGVy bW9yZSwgbWEzNV91c2JfcGh5X3Bvd2VyX29uKCkgc3RyaWN0bHkgaGFyZGNvZGVzIHJlZ2lzdGVy cyBmb3IgUEhZMDoKCiAgICByZWdtYXBfdXBkYXRlX2JpdHMocF9waHktPnN5c3JlZywgTUEzNV9T WVNfUkVHX1VTQlBNSVNDUiwgMHg3LCAoUEhZMFBPUiB8IFBIWTBTVVNQRU5EKSk7CgpXaWxsIHRo aXMgY2F1c2UgYSBjb25zdW1lciBhdHRlbXB0aW5nIHRvIGNvbnRyb2wgUEhZMSB0byBpbmFkdmVy dGVudGx5IHJlc2V0IGFuZApwb3dlciBvbiBQSFkwIGluc3RlYWQ/CgpbU2V2ZXJpdHk6IE1lZGl1 bV0KSXMgaXQgYWNjZXB0YWJsZSB0byB1c2UgYW4gZW51bSBjb25zdHJhaW50IGZvciBhIGNlbGwt Y291bnQgcHJvcGVydHk/CgpBY2NvcmRpbmcgdG8gdGhlIHN1YnN5c3RlbS9kdC1iaW5kaW5ncy5t ZCBnbG9iYWwgcmV2aWV3IGd1aWRlbGluZXMsIGVhY2gKY2VsbC1jb3VudCBwcm9wZXJ0eSBtdXN0 IGhhdmUgYSBzdHJpY3QgY29uc3QgY29uc3RyYWludCBtYXRjaGluZyB0aGUgaGFyZHdhcmUuClNp bmNlIHRoZSBoYXJkd2FyZSBwaHlzaWNhbGx5IGhhcyB0d28gcG9ydHMsIHNob3VsZG4ndCAjcGh5 LWNlbGxzIGJlIGRlZmluZWQKYXMgY29uc3Q6IDEgdG8gYWNjdXJhdGVseSBtb2RlbCB0aGUgaGFy ZHdhcmUncyBjYXBhYmlsaXRpZXM/Cgo+ICsgICAgZGVzY3JpcHRpb246Cj4gKyAgICAgIFdoZW4g MCwgdGhlIG5vZGUgZXhwb3NlcyBQSFkwIG9ubHkgYW5kIGNvbnN1bWVycyByZWZlcmVuY2UgaXQK PiArICAgICAgd2l0aCBubyBjZWxsIGFyZ3VtZW50LiBXaGVuIDEsIHRoZSBzaW5nbGUgY2VsbCBz ZWxlY3RzIHRoZSBwb3J0LAo+ICsgICAgICB3aXRoIDAgZm9yIHRoZSBPVEcgcG9ydCAoVVNCMCwg c2hhcmVkIHdpdGggRFdDMiBnYWRnZXQgY29udHJvbGxlcikKPiArICAgICAgYW5kIDEgZm9yIHRo ZSBob3N0LW9ubHkgcG9ydCAoVVNCMSkuClsgLi4uIF0KPiBAQCAtMzksNyArNzgsNyBAQCBleGFt cGxlczoKPiAgCj4gICAgICB1c2JfcGh5OiB1c2ItcGh5IHsKPiAgICAgICAgICBjb21wYXRpYmxl ID0gIm51dm90b24sbWEzNWQxLXVzYjItcGh5IjsKPiAtICAgICAgICBjbG9ja3MgPSA8JmNsayBV U0JEX0dBVEU+Owo+ICsgICAgICAgIGNsb2NrcyA9IDwmY2xrIEhVU0JIMF9HQVRFPjsKPiAgICAg ICAgICBudXZvdG9uLHN5cyA9IDwmc3lzPjsKPiAgICAgICAgICAjcGh5LWNlbGxzID0gPDA+OwoK W1NldmVyaXR5OiBNZWRpdW1dCldpdGggdGhlIGhhcmR3YXJlIHN1cHBvcnRpbmcgbXVsdGlwbGUg cG9ydHMsIHNob3VsZCB0aGUgZXhhbXBsZSBiZSB1cGRhdGVkIHRvCmRlbW9uc3RyYXRlIHRoZSBj b3JyZWN0IG11bHRpLXBvcnQgY29uZmlndXJhdGlvbiAoI3BoeS1jZWxscyA9IDwxPjspIGluc3Rl YWQKb2YgcHJvcGFnYXRpbmcgdGhlIGxlZ2FjeSBjb25maWd1cmF0aW9uPwoKPiAgICAgIH07Cgot LSAKU2FzaGlrbyBBSSByZXZpZXcgwrcgaHR0cHM6Ly9zYXNoaWtvLmRldi8jL3BhdGNoc2V0LzIw MjYwNzA4MTAzNjA2LjE0NjI5NjAtMS1hMDk4NzIwMzA2OUBnbWFpbC5jb20/cGFydD0xCgotLSAK bGludXgtcGh5IG1haWxpbmcgbGlzdApsaW51eC1waHlAbGlzdHMuaW5mcmFkZWFkLm9yZwpodHRw czovL2xpc3RzLmluZnJhZGVhZC5vcmcvbWFpbG1hbi9saXN0aW5mby9saW51eC1waHkK 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 E0AD23F8EAF for ; Wed, 8 Jul 2026 10:49:04 +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=1783507753; cv=none; b=MR4q8+ehMkr9WmkxrVEmEoU7m+Wt1WRtyPakMD7AHX5F66mz7Y2TS1Hq0++l3t6sWG3/niL5omggDr6aM2IfClYZ1LsEuFGJCzcsx4i4nvY6AWYPvMDWsJKbOGyNXdql80pVPDRWwuuQBtemwtOfLm7N+v/HvNDWa+6jqjVjCmU= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1783507753; c=relaxed/simple; bh=0NpZcEcGAGgvy2ljcRmhmoQE7fhi+nqi0fcU8Mpshh0=; h=From:Subject:To:Cc:In-Reply-To:References:Content-Type:Date: Message-Id; b=NJU2usbdbSR8sN6uxT1krTXz7cyfqcVZZWS1yELjVP49wtoSES/g2BDFcnQNFcvzKwOrXRhihZBaj+mAQh2XekMOLYeZ+FweebAxnpSTYBB+6qdx1WnKiVo0IgHCd2C3zW1SahfK+LhM1ywkyij6lFehDg8WecGwHHxILlJfttE= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=DrKY8UAx; 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="DrKY8UAx" Received: by smtp.kernel.org (Postfix) with ESMTPSA id D616E1F000E9; Wed, 8 Jul 2026 10:49:00 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1783507741; bh=NjmxgfT50Pozt5M6c45G6mi5aTkKtqZO4ykzegNi2+A=; h=From:Subject:Reply-To:To:Cc:In-Reply-To:References:Date; b=DrKY8UAxwTpzhIk3f558wYx1DM2BWiMkyACdArzrh6nGPuezDj7FlM/O1Y+/CFVKy 8LqkZJ+kTQQhCeNzNwbx0StMQCllHmGgSauclAniz+jfWt7avENlbCUmfavjzVaBPz ygNJ5fxXQdyu7P0UFOxDFnLDUVJEGT03sjOSWIuemtR3Biai8LD/1AdGRvk2Pz6otc rfasjEpKU6hrbnlx3Pn+U0ktUspeFU+7yhVqf7lAsvxQlsVKbcWaSs1v/G+n/lVjGw zer9dZtx1ErUZjg2iMcyBW3Bg4L1/TCaIX9FUp2eSOiCXELOJMSVo2R/py3dVMtE0G CfvKzI90E08AA== From: sashiko-bot@kernel.org Subject: Re: [PATCH v3 1/3] dt-bindings: phy: nuvoton,ma35d1-usb2-phy: extend for dual-port and OTG Reply-To: sashiko-reviews@lists.linux.dev To: "Joey Lu" Cc: neil.armstrong@linaro.org, devicetree@vger.kernel.org, olteanv@gmail.com, linux-phy@lists.infradead.org, vkoul@kernel.org, robh@kernel.org, conor+dt@kernel.org In-Reply-To: <20260708103606.1462960-2-a0987203069@gmail.com> References: <20260708103606.1462960-1-a0987203069@gmail.com> <20260708103606.1462960-2-a0987203069@gmail.com> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Date: Wed, 08 Jul 2026 10:49:00 +0000 Message-Id: <20260708104900.D616E1F000E9@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 2 potential issue(= s) to consider: - [High] The DT binding introduces multi-port support, but the correspondin= g driver lacks implementation, leading to silent hardware malfunction. - [Medium] The `#phy-cells` property uses an enum constraint instead of a s= trict const constraint, violating subsystem guidelines. -- --- Patch [1]: [PATCH v3 1/3] dt-bindings: phy: nuvoton,ma35d1-usb2-phy: ex= tend for dual-port and OTG --- commit 7f72a8cc4b64e1dfcf4efba1cd67bf9eafd31d72 Author: Joey Lu dt-bindings: phy: nuvoton,ma35d1-usb2-phy: extend for dual-port and OTG This commit updates the nuvoton,ma35d1-usb2-phy device tree binding to support dual USB PHY ports (PHY0 and PHY1). It relaxes #phy-cells from cons= t: 0 to enum: [0, 1] to allow port selection while preserving backward compatibility. It also adds new optional properties for resistor calibration and over-current polarity. > diff --git a/Documentation/devicetree/bindings/phy/nuvoton,ma35d1-usb2-ph= y.yaml b/Documentation/devicetree/bindings/phy/nuvoton,ma35d1-usb2-phy.yaml > index fff858c909a06..191d1b2272ec5 100644 > --- a/Documentation/devicetree/bindings/phy/nuvoton,ma35d1-usb2-phy.yaml > +++ b/Documentation/devicetree/bindings/phy/nuvoton,ma35d1-usb2-phy.yaml [ ... ] > @@ -15,7 +27,12 @@ properties: > - nuvoton,ma35d1-usb2-phy > =20 > "#phy-cells": > - const: 0 > + enum: [0, 1] [Severity: High] Does the driver currently support this new #phy-cells =3D <1> configuration? Looking at ma35_usb_phy_probe() in drivers/phy/nuvoton/phy-ma35d1-usb2.c, t= he=20 provider registers using of_phy_simple_xlate: provider =3D devm_of_phy_provider_register(&pdev->dev, of_phy_simple_xl= ate); Because of_phy_simple_xlate ignores cell arguments, any valid DT request for PHY1 will silently map to the single initialized PHY. Furthermore, ma35_usb_phy_power_on() strictly hardcodes registers for PHY0: regmap_update_bits(p_phy->sysreg, MA35_SYS_REG_USBPMISCR, 0x7, (PHY0POR= | PHY0SUSPEND)); Will this cause a consumer attempting to control PHY1 to inadvertently rese= t and power on PHY0 instead? [Severity: Medium] Is it acceptable to use an enum constraint for a cell-count property? According to the subsystem/dt-bindings.md global review guidelines, each cell-count property must have a strict const constraint matching the hardwa= re. Since the hardware physically has two ports, shouldn't #phy-cells be defined as const: 1 to accurately model the hardware's capabilities? > + description: > + When 0, the node exposes PHY0 only and consumers reference it > + with no cell argument. When 1, the single cell selects the port, > + with 0 for the OTG port (USB0, shared with DWC2 gadget controller) > + and 1 for the host-only port (USB1). [ ... ] > @@ -39,7 +78,7 @@ examples: > =20 > usb_phy: usb-phy { > compatible =3D "nuvoton,ma35d1-usb2-phy"; > - clocks =3D <&clk USBD_GATE>; > + clocks =3D <&clk HUSBH0_GATE>; > nuvoton,sys =3D <&sys>; > #phy-cells =3D <0>; [Severity: Medium] With the hardware supporting multiple ports, should the example be updated = to demonstrate the correct multi-port configuration (#phy-cells =3D <1>;) inst= ead of propagating the legacy configuration? > }; --=20 Sashiko AI review =C2=B7 https://sashiko.dev/#/patchset/20260708103606.1462= 960-1-a0987203069@gmail.com?part=3D1