From nobody Fri Sep 25 17:42:53 2026 Received: from mail-ot1-f51.google.com (mail-ot1-f51.google.com [209.85.210.51]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 7D8DA2DFA4A for ; Thu, 10 Sep 2026 00:26:55 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.210.51 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1789000017; cv=none; b=LILmcoEpDOLO61Cn5QmQoU0V4SZaTW2dC8ykKtMHxo/xo3UkzPcUBxNMO8GAINz+NeWjr0zfJhBLhMB1pDFaHALjqwRSPZDxt/b5KjL1KoseOJUnWNnMgYVqtjjP5ZWIDtrHcCAHaXIDwA93u7Tt3vhHSAORtWWOVyEK1I5+MAA= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1789000017; c=relaxed/simple; bh=LUDT1kHiXHBCnePbIaYBUJQHAxuAaQl4Sg3kYNxuzRM=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=MUy/gE6lIy66lnRQKIuoe0pXcDn4PklJQd7U6Fh/bsm9wAczvh8R12DvlznUpTC/hPy+nKOi405hA6+X34id7KOgvwzDx3z19TZO4bP8iRS47FFzv3EOjRcAUPHrbmekzYZztoEx1x/2++8n8m/7jOzBcJxUszYPd2gfL0b6dDo= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=YPub2A6S; arc=none smtp.client-ip=209.85.210.51 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="YPub2A6S" Received: by mail-ot1-f51.google.com with SMTP id 46e09a7af769-7f4e729368fso5834785a34.0 for ; Wed, 09 Sep 2026 17:26:55 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1789000014; x=1789604814; darn=vger.kernel.org; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :content-type:mime-version:subject:date:from:from:to:cc:subject:date :message-id:reply-to:content-type; bh=eTa0efMcyPx+a3sRs/5/R1BWxQ19AFGAD7UOiV7rwak=; b=YPub2A6S/UP3hFAXhnVcdhWvveAR+mT6APt4gGg0VtItSMNF8Q6blGLnReXI4078iF YIrRumPrF2tcdHdlKm+EaynuG5oaQ9Qg3BAxGDr7cvtiBBxhan8gg5j6W9+tzjgSEFmn aeW2FOYS35GfeVUJASdeJpWNnUwX2x7TGC4XnztkMvmuAYQnCP+5P3me0ficBNZDehI0 ndnMwqdW4GbqvVdOiRJOGJXp4z8eAE39kUw+Zipgt8F74vmfu9m5vvbRl0HJqC7ypFu7 hjizTmIEZdRr/QZ796fhZigKr3bxKxo0xwmSjlRqPq9kiUA4sfSsz/G3UavNmuf9iO+k Y5iQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1789000014; x=1789604814; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :content-type:mime-version:subject:date:from:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to :content-type; bh=eTa0efMcyPx+a3sRs/5/R1BWxQ19AFGAD7UOiV7rwak=; b=lnQHeli1ffgogd+wQPzfMRRksbiN9EhQ3oA91sggWk2eWUSqdTdBZqtNA95FiJMsQI 8XpMOrlv/bEofrvxy/qL/68hQN9TljxbnE3Q7m9apIHzwJ4FyrI9JUf76YJR6ewdJM6O bHv/+NNQyQpOjXURiSuaXJgTeG5FzLHoztsgqL85IaNrp+fa2OZ09o1wYR9Iy2AbvwFY +rIv2GfsHUlxt2NeDRv8Uc20XmvAguqALE5aWCO1h1tCc8++O0fZTI+S/Ud0PYupMDCw 7DTEPosshGzyzNo4nFUEw2VnGnb/4PmOCGjQ6cOGKZnosXTwz7gMum37i/8qR5OGZtIK AC/g== X-Forwarded-Encrypted: i=1; AKwUvBwjOtBWYTjhflgyVErdCFoxQpmCScokTV8/s1KJhjfA7EG3Jfv//dh6cVK1FQ/Hu6uwmgu26XdjXzmBRhw=@vger.kernel.org X-Gm-Message-State: AFuF++kthoH0uWS5R6MHFLfN/IrXQBWhPBUjokF+flCEEuOGd/B/bfXb cdxr2MM2mTMCaCvRgnQ9gJdift+ZSFkXbFcHdzdbsyF01Z4HdpREj9aF X-Gm-Gg: AYBFou0B59Mm9Gvv3sIKywiB2yMRyG8g6nft+MiikynilSK5tJdaT8HvEjOQBTDF0Gf 5ZecrlFT/qTwvXgNluGofXcpx4oOEoBxe9JC0w8ju2mz/l/zgpLeMnyrvTRKEEpNCyjIL4l/nmD NdAAQPOkzbHmLmxpH4AUFs/Gn41lJCE7hu7KlESPiprCymYbbQrDidBwxulCbLh8dMbnwow3zJH qWQk559706OFjGr22pqZ5WBpQVimVvTLfpp3g8F2qpnzwVeYff3NOUKvXI1SZTdUPDB/E5H25F/ zeSI/SzwfX2VE9l+GWZ2zgf/1D2sPLzYkRtN6IeUp2G8KH4bE+Kkh0Dr6fWz8ZpmWpen2ZzSQiJ W0oWaqSuMkHVNpVMKf44V3EpPntwBue3TOwj+HLk4fXqbn+WaftNgztKMAnXvAcG0VNchsbMIP+ t6JRBROozNrXHVitxUItYAyc3nCgY1YGARZit7LDPtzfcv7QAm8BR3BElUmpaT12JB12p+e2MDB 8fNqSjhTuA35lOJdpIoJWEe6Fgs2BjkMSwTPSXFBD2KdUwXVI9Xzbb0SkeZ9hCw1L+xLNC7zUvA HsAHE9cj8fixLbH5z635NURXer54KXiamMyJPghu99sBRm+HFAPMi/ULDZkCkRtC9UlKI3f0rj1 l2dRRBQW0Q/R+dCLtiw5c1QlyYgHHP4NgbWDuQ7Bj X-Received: by 2002:a05:6830:3509:b0:7dc:c749:ade3 with SMTP id 46e09a7af769-7fa1deab695mr24907263a34.3.1789000014269; Wed, 09 Sep 2026 17:26:54 -0700 (PDT) Received: from [127.0.1.1] (184-96-151-165.hlrn.qwest.net. [184.96.151.165]) by smtp.gmail.com with ESMTPSA id 46e09a7af769-7fe2fbf5db8sm10182789a34.25.2026.09.09.17.26.53 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 09 Sep 2026 17:26:53 -0700 (PDT) From: James Hilliard Date: Wed, 09 Sep 2026 18:26:50 -0600 Subject: [PATCH v12 1/2] dt-bindings: mfd: x-powers: Add AC200 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20260909-submit-ac200-mfd-v12-1-a44d6bc30a4f@gmail.com> References: <20260909-submit-ac200-mfd-v12-0-a44d6bc30a4f@gmail.com> In-Reply-To: <20260909-submit-ac200-mfd-v12-0-a44d6bc30a4f@gmail.com> To: Jernej Skrabec , James Hilliard , Lee Jones , Arnd Bergmann , Rob Herring , Krzysztof Kozlowski , Conor Dooley Cc: Andrew Lunn , "Jagielski, Jedrzej" , Andre Przywara , Chen-Yu Tsai , linux-sunxi@lists.linux.dev, mfd@lists.linux.dev, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Conor Dooley X-Mailer: b4 0.15.2 The AC200 is an I2C-controlled mixed-signal companion IC containing audio, video, RTC and Fast Ethernet PHY functions. Describe the shared input clock, audio DAI, TV encoder display graph and optional bandgap calibration in one parent node. The codec and TV encoder do not need separate firmware nodes. AC_LDOIN and TV_VCC are distinct 3.3 V supply inputs. Describe them separately without requiring different external regulators. Require the audio supply when the DAI is described and the video supply when the display graph is present, while allowing PHY-only configurations. Describe the optional shared INTB connection without an internal interrupt domain. Cable detection status remains readable over I2C when INTB is not connected. Include wired and unwired examples. The Ethernet PHY is represented by a PHY package on its MDIO bus, with a phandle back to this device for access to the package control registers. It therefore does not require an MFD child node. The input clock must run at 24 or 27 MHz when the Ethernet PHY is used, matching the rates encoded by its documented clock selector. Co-developed-by: Jernej Skrabec Signed-off-by: Jernej Skrabec Signed-off-by: James Hilliard Reviewed-by: Conor Dooley --- .../devicetree/bindings/mfd/x-powers,ac200.yaml | 168 +++++++++++++++++= ++++ MAINTAINERS | 7 + 2 files changed, 175 insertions(+) diff --git a/Documentation/devicetree/bindings/mfd/x-powers,ac200.yaml b/Do= cumentation/devicetree/bindings/mfd/x-powers,ac200.yaml new file mode 100644 index 000000000000..75dafa7c1071 --- /dev/null +++ b/Documentation/devicetree/bindings/mfd/x-powers,ac200.yaml @@ -0,0 +1,168 @@ +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/mfd/x-powers,ac200.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: X-Powers AC200 mixed-signal IC + +maintainers: + - James Hilliard + - Jernej Skrabec + +description: + The AC200 is a mixed-signal companion IC containing audio, video, RTC and + Fast Ethernet PHY functions. Its control registers are accessed over I2C. + The audio codec and TV encoder are described by properties of this node. + The Ethernet PHY is described separately on its MDIO bus. + +allOf: + - $ref: /schemas/sound/dai-common.yaml# + +properties: + compatible: + const: x-powers,ac200 + + reg: + maxItems: 1 + + clocks: + maxItems: 1 + description: + AC200 input clock. When using the Ethernet PHY, its configured rate = must + be 24 or 27 MHz, matching the rates encoded by the documented EPHY c= lock + selector. + + interrupts: + maxItems: 1 + description: + The shared, level-triggered, open-drain INTB output for the TV encod= er, + Ethernet PHY and RTC interrupts. Omit when INTB is not connected. TV + encoder cable detection status can also be read over I2C without an + interrupt. + + '#sound-dai-cells': + const: 0 + + ac-ldoin-supply: + description: + The 3.3 V supply connected to the AC_LDOIN pin for the audio codec L= DO + input. This may be the same regulator as tv-vcc-supply. + + tv-vcc-supply: + description: + The 3.3 V supply connected to the TV_VCC pin for the composite-video + DAC. This may be the same regulator as ac-ldoin-supply. + + nvmem-cells: + maxItems: 1 + description: TV encoder bandgap calibration. + + nvmem-cell-names: + items: + - const: bandgap + + ports: + $ref: /schemas/graph.yaml#/properties/ports + + properties: + port@0: + $ref: /schemas/graph.yaml#/properties/port + description: Input to the TV encoder, carrying CCIR656. + + port@1: + $ref: /schemas/graph.yaml#/properties/port + description: Output to the composite-video connector. + + required: + - port@0 + - port@1 + +required: + - compatible + - reg + - clocks + +dependencies: + '#sound-dai-cells': [ ac-ldoin-supply ] + ports: [ tv-vcc-supply ] + nvmem-cells: [ nvmem-cell-names ] + nvmem-cell-names: [ nvmem-cells ] + +unevaluatedProperties: false + +examples: + - | + #include + + i2c { + #address-cells =3D <1>; + #size-cells =3D <0>; + + mixed-signal@10 { + compatible =3D "x-powers,ac200"; + reg =3D <0x10>; + clocks =3D <&pwm 5>; + interrupt-parent =3D <&pio>; + interrupts =3D <1 20 IRQ_TYPE_LEVEL_LOW>; + #sound-dai-cells =3D <0>; + ac-ldoin-supply =3D <®_aldo2>; + tv-vcc-supply =3D <®_aldo2>; + + ports { + #address-cells =3D <1>; + #size-cells =3D <0>; + + port@0 { + reg =3D <0>; + + tve_in: endpoint { + remote-endpoint =3D <&tcon_out_tve>; + }; + }; + + port@1 { + reg =3D <1>; + + tve_out: endpoint { + remote-endpoint =3D <&composite_in>; + }; + }; + }; + }; + }; + - | + i2c { + #address-cells =3D <1>; + #size-cells =3D <0>; + + mixed-signal@10 { + compatible =3D "x-powers,ac200"; + reg =3D <0x10>; + clocks =3D <&pwm 5>; + /* INTB is not connected. */ + tv-vcc-supply =3D <®_aldo2>; + + ports { + #address-cells =3D <1>; + #size-cells =3D <0>; + + port@0 { + reg =3D <0>; + + endpoint { + remote-endpoint =3D <&tcon_out_tve>; + }; + }; + + port@1 { + reg =3D <1>; + + endpoint { + remote-endpoint =3D <&composite_in>; + }; + }; + }; + }; + }; +... diff --git a/MAINTAINERS b/MAINTAINERS index 3a19da74d00c..419340093c9b 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -29504,6 +29504,13 @@ F: drivers/net/wwan/ F: include/linux/wwan.h F: include/uapi/linux/wwan.h =20 +X-POWERS AC200 DRIVERS +M: Jernej Skrabec +M: James Hilliard +L: linux-sunxi@lists.linux.dev +S: Maintained +F: Documentation/devicetree/bindings/mfd/x-powers,ac200.yaml + X-POWERS AXP288 PMIC DRIVERS M: Hans de Goede S: Maintained --=20 2.53.0 From nobody Fri Sep 25 17:42:53 2026 Received: from mail-ot1-f52.google.com (mail-ot1-f52.google.com [209.85.210.52]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 71A5D2EBBA1 for ; Thu, 10 Sep 2026 00:26:56 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.210.52 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1789000018; cv=none; b=FbFkxVPKk583JNOattMR5O5C7jWQbH+u4BZ/EQ3f0L/aH6h9xYQrrQyrD4aHsBSSLGRyYzBTP4maXpdOzRyPvagf1W48eXYFO3B3ptT9hjhC1pa9WPuQAiWXDrmUlxiMGHVbdy/D2EGmTT0UC3qQNOdckjJfddDZtYeWwg/J350= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1789000018; c=relaxed/simple; bh=FW5SV01EPHNg3l3FrgB1ILkB0Ify9YcERXPtAQCkkMw=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:References: In-Reply-To:To:Cc; b=i+5/mYpci4EKHKsc7z/oRM2Ok6oWZjM30c57SqyY7/MfwuK2WtAIlz46KHhdS1oFrZPfMrxO6SlVkdrysyaiSNcwI+Fi96qaY7KQuhGGOytAUXMIf0DVen6nrFLm8vNpak7xdSY1Kz51voUy/fESAvIVpcLdRe3AYY2WbdkuXEk= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=oxNXLEwn; arc=none smtp.client-ip=209.85.210.52 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="oxNXLEwn" Received: by mail-ot1-f52.google.com with SMTP id 46e09a7af769-7f84a55cc06so5074031a34.2 for ; Wed, 09 Sep 2026 17:26:56 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1789000015; x=1789604815; darn=vger.kernel.org; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :content-type:mime-version:subject:date:from:from:to:cc:subject:date :message-id:reply-to:content-type; bh=Q+4sP2C3v2ZdGguoW3qixGSWTMWMqlk3YkqM2/qh2KM=; b=oxNXLEwnp9d7gRpRqa//2zKgvY8CQDJdT45rQmnzub/bXIlJEPPMdSAIXSF19hvPXc VyCLdsIENfiaQYJdVHrU4dQLvWRJyiFlPTVJKh5/Eegpa/NBf5X0knWtdNMNozDeaXjG dFsVB/Y/exJui768tNRa9WJwQZD1nnVI4MEtAi0WLTSXSQZl8S6JGeNNASoYxaHk6m7J JAfcdUidMH0T1cb5dW/79J9driNysAhe8yYeBXPmMSiUPIgTCFBe4Wd0xfKfNamM4qlG K42Jk2UOud1Ht5sGQvT+VDghmf4+e4PCJGULDgvNeyAq4gXqG+5ypElwSQKnHAe5SHoy cVWg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1789000015; x=1789604815; h=cc:to:in-reply-to:references:message-id:content-transfer-encoding :content-type:mime-version:subject:date:from:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to :content-type; bh=Q+4sP2C3v2ZdGguoW3qixGSWTMWMqlk3YkqM2/qh2KM=; b=hDYdPdZZCivff+IEImCNHdJDVrTzf3IbNx6yeedYtfwPS2r9dq1mUyK50oY0cP6rE7 zETmSghRXRj1qxkUB8azRaxyNmBTOdA7e6f2utmhBAO13qwFW3Iy4gLrc9yri3KKN3oH VW4GJ+ZyWG6Jwa7ANUJOLURKyFSpv7OJ0+Ar+CSZhgMiojH0/Jj1+bS8s3Vr1GX92Jr7 Nv341JD+aZKQQ+fhWr2qz/lHo3LpWywc7slKWWrC5Cr98l1231s2CE3swy84sDJIRuIo kGqL9WqRQqdBQ38vPx4YfgDutdijzhCkPz7EW1WBLnWIKZ56mwmxsLFYRmrHxmk/wlrK rLSA== X-Forwarded-Encrypted: i=1; AKwUvBw85t0tLuKpVI3R91wwNflQ3jOfghBh+9ySPIv6Wn+4vG2ygtR7q51WI5NSaKx14korODu5ifkMaOx38ew=@vger.kernel.org X-Gm-Message-State: AFuF++l4smIVeh5BNcjC2xiBwS2IkMowAVYovpT+uTc4xDYrjr5w0EBR JPRDbm+zdutT9zgCxUnEuDLheWqXUxXGYVwgbdafoQvJg1C8aJg8z23N X-Gm-Gg: AYBFou15S2tfmXXny+MTUrfWw5QWKCtgzTz4xhmZrsnmzbktvHfprw06iTkwB8sNmkM IlXlEPu5fLsK+pZ6uOvGrPZR4NMXA43Li5x1RzcP+WHCm/I3dclxNcCL7e+/Uqt1cWefYbz+t5R 5QRePSDM0LZfy3q7X/knPASIuDyWrrhG1im0kXv/buzvHoWN1RrUb7daBUsu2U0i5umZsjl4+M2 9fB3wMXfwHuiDvj3xcxsgNPSBJWrcZsJQkO5a1lUjCKncBc/KLI8hZSEZlE+vi6rRalZmtwiF7l qqFtdIQckeYQpJOanwPDx+XabXIuoxUZJ8m/c8zdOmuMd9udQpHZuthIMYhW0njtcBlK2mYc+j2 pbJeTMnCIHRV53VJgeBYaWlY5gju3ZDwal/o2FT7sy6CcyqSC5JpB5Vbj705NCJFIm7Ikzs+Zfk jxc3jX5nJg43P/D+EDd+c/iewCpwfVqt7ZkorGvFm3VPPkcXpGFYXJoZcjNbu8elu3gHzxtodOA bCnTTOIaFbOT10OOEE7D8pluA2ym62qqSTepFsEvpoHarHoGqAq7mAf+tKBWSCllRc8qnIG066U jiun2MP07f6MGG/hC6JKQJdIZqacpyvop6pdEzFXVbglPd0u01BcR6Br/n7bdck09n4F3yTwAC1 gp5OCBiAamymf4AU3b5qlAENuRZO8Rg== X-Received: by 2002:a05:6830:6415:b0:7fa:5c49:5251 with SMTP id 46e09a7af769-7fa5c4954demr23897006a34.24.1789000015237; Wed, 09 Sep 2026 17:26:55 -0700 (PDT) Received: from [127.0.1.1] (184-96-151-165.hlrn.qwest.net. [184.96.151.165]) by smtp.gmail.com with ESMTPSA id 46e09a7af769-7fe2fbf5db8sm10182789a34.25.2026.09.09.17.26.54 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 09 Sep 2026 17:26:54 -0700 (PDT) From: James Hilliard Date: Wed, 09 Sep 2026 18:26:51 -0600 Subject: [PATCH v12 2/2] mfd: ac200: Add X-Powers AC200 support Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20260909-submit-ac200-mfd-v12-2-a44d6bc30a4f@gmail.com> References: <20260909-submit-ac200-mfd-v12-0-a44d6bc30a4f@gmail.com> In-Reply-To: <20260909-submit-ac200-mfd-v12-0-a44d6bc30a4f@gmail.com> To: Jernej Skrabec , James Hilliard , Lee Jones , Arnd Bergmann , Rob Herring , Krzysztof Kozlowski , Conor Dooley Cc: Andrew Lunn , "Jagielski, Jedrzej" , Andre Przywara , Chen-Yu Tsai , linux-sunxi@lists.linux.dev, mfd@lists.linux.dev, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org X-Mailer: b4 0.15.2 The X-Powers AC200 is a mixed-signal companion IC with a paged register map accessed over I2C. Enable and rate-lock the shared input clock, retain the vendor resume path's 40 ms wait before creating the regmap and accessing registers, and deassert the common reset. No minimum delay is documented. Set only the deassert bit instead of forcing a reset cycle, avoiding a chip-wide reset of unrelated function registers. Leave the common reset deasserted during driver removal and system shutdown; function drivers own their block resets. Supplier unbind still tears down linked consumers and releases the provider's clock references. Cache only the common page selector. Individual functions can reset independently and invalidate their other registers, so leave all functional registers volatile. Register the audio codec and TV encoder using static MFD cells and automatically assigned platform device IDs. Their firmware properties belong to the parent node. Map their supply lookups to the parent with MFD supply aliases, without assigning separate OF nodes to the cells. When INTB is connected, pass its physical IRQ to the TV encoder cell. Select a cell array without IRQ resources when INTB is absent. No regmap IRQ controller or private IRQ domain is needed. Configure the level-triggered INTB output, defaulting to level-low when the upstream IRQ has no trigger type. Mask all sources before enabling INTB. Function drivers request shared threaded IRQs before enabling their own source, check and clear their own status, and mask their source before freeing the IRQ. Disable INTB after removing the children on failure or removal, while the regmap and clock are still available. The Ethernet PHY is enumerated on its MDIO bus rather than as an MFD child. It follows the x-powers,ac200 phandle and uses this regmap for ancillary package-control access. Co-developed-by: Jernej Skrabec Signed-off-by: Jernej Skrabec Signed-off-by: Andre Przywara Signed-off-by: James Hilliard --- MAINTAINERS | 1 + drivers/mfd/Kconfig | 14 ++++ drivers/mfd/Makefile | 1 + drivers/mfd/ac200.c | 231 +++++++++++++++++++++++++++++++++++++++++++++++= ++++ 4 files changed, 247 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 419340093c9b..1d03b0060bda 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -29510,6 +29510,7 @@ M: James Hilliard L: linux-sunxi@lists.linux.dev S: Maintained F: Documentation/devicetree/bindings/mfd/x-powers,ac200.yaml +F: drivers/mfd/ac200.c =20 X-POWERS AXP288 PMIC DRIVERS M: Hans de Goede diff --git a/drivers/mfd/Kconfig b/drivers/mfd/Kconfig index 857ca3bb0d5b..071cba7c775b 100644 --- a/drivers/mfd/Kconfig +++ b/drivers/mfd/Kconfig @@ -205,6 +205,20 @@ config MFD_AC100 This driver include only the core APIs. You have to select individual components like codecs or RTC under the corresponding menus. =20 +config MFD_AC200 + tristate "X-Powers AC200" + depends on COMMON_CLK + depends on I2C + depends on OF + select MFD_CORE + select REGMAP_I2C + help + Support for the X-Powers AC200 mixed-signal companion IC. The AC200 + contains audio, video, RTC and Fast Ethernet PHY functions and is + co-packaged with some Allwinner H6 and H616 SoCs. This driver provides + the shared register access and instantiates the individual function + devices. + config MFD_AXP20X tristate select MFD_CORE diff --git a/drivers/mfd/Makefile b/drivers/mfd/Makefile index 72d3944b0ad8..f8101d2a9ce9 100644 --- a/drivers/mfd/Makefile +++ b/drivers/mfd/Makefile @@ -150,6 +150,7 @@ obj-$(CONFIG_MFD_DA9052_SPI) +=3D da9052-spi.o obj-$(CONFIG_MFD_DA9052_I2C) +=3D da9052-i2c.o =20 obj-$(CONFIG_MFD_AC100) +=3D ac100.o +obj-$(CONFIG_MFD_AC200) +=3D ac200.o obj-$(CONFIG_MFD_AXP20X) +=3D axp20x.o obj-$(CONFIG_MFD_AXP20X_I2C) +=3D axp20x-i2c.o obj-$(CONFIG_MFD_AXP20X_RSB) +=3D axp20x-rsb.o diff --git a/drivers/mfd/ac200.c b/drivers/mfd/ac200.c new file mode 100644 index 000000000000..af0d27ff5f45 --- /dev/null +++ b/drivers/mfd/ac200.c @@ -0,0 +1,231 @@ +// SPDX-License-Identifier: GPL-2.0-only +/* + * MFD core driver for the X-Powers AC200 + * + * Copyright (C) 2019 Jernej Skrabec + * Copyright (C) 2026 James Hilliard + */ + +#include +#include +#include +#include +#include +#include +#include +#include + +#define AC200_SYS_CONTROL_REG 0x0002 +#define AC200_SYS_CONTROL_CHIP_RESET_DEASSERT BIT(0) +#define AC200_SYS_IRQ_ENABLE_REG 0x0004 +#define AC200_SYS_IRQ_INTB_ENABLE BIT(15) +#define AC200_SYS_IRQ_INTB_ACTIVE_HIGH BIT(14) +#define AC200_SYS_IRQ_RTC BIT(12) +#define AC200_SYS_IRQ_EPHY BIT(8) +#define AC200_SYS_IRQ_TVE BIT(4) + +/* Interface register accessible from every register page. */ +#define AC200_TWI_REG_ADDR_H 0x00fe +#define AC200_MAX_REG 0xa1f2 + +static const struct regmap_range_cfg ac200_range_cfg[] =3D { + { + .range_max =3D AC200_MAX_REG, + .selector_reg =3D AC200_TWI_REG_ADDR_H, + .selector_mask =3D 0xff, + .window_len =3D 256, + }, +}; + +/* + * Each AC200 sub-block can reset independently, invalidating its register + * contents without regmap's knowledge. Cache only the common page selecto= r; + * this avoids a selector read-modify-write for every access on the same p= age + * without ever returning stale functional-register values. + */ +static bool ac200_volatile_reg(struct device *dev, unsigned int reg) +{ + return reg !=3D AC200_TWI_REG_ADDR_H; +} + +static const struct regmap_config ac200_regmap_config =3D { + .name =3D "ac200", + .reg_bits =3D 8, + .reg_stride =3D 2, + .val_bits =3D 16, + .ranges =3D ac200_range_cfg, + .num_ranges =3D ARRAY_SIZE(ac200_range_cfg), + .max_register =3D AC200_MAX_REG, + .volatile_reg =3D ac200_volatile_reg, + .cache_type =3D REGCACHE_MAPLE, +}; + +static const char * const ac200_codec_supplies[] =3D { + "ac-ldoin", +}; + +static const char * const ac200_tve_supplies[] =3D { + "tv-vcc", +}; + +static const struct resource ac200_tve_resources[] =3D { + DEFINE_RES_IRQ_NAMED(0, "intb"), +}; + +#define AC200_CELL(_name, _supplies, _resources) \ + { \ + .name =3D (_name), \ + .parent_supplies =3D (_supplies), \ + .num_parent_supplies =3D ARRAY_SIZE(_supplies), \ + .resources =3D (_resources), \ + .num_resources =3D MFD_RES_SIZE(_resources), \ + } + +static const struct mfd_cell ac200_cells[] =3D { + AC200_CELL("ac200-codec", ac200_codec_supplies, NULL), + AC200_CELL("ac200-tve", ac200_tve_supplies, ac200_tve_resources), +}; + +static const struct mfd_cell ac200_noirq_cells[] =3D { + AC200_CELL("ac200-codec", ac200_codec_supplies, NULL), + AC200_CELL("ac200-tve", ac200_tve_supplies, NULL), +}; + +static void ac200_disable_intb(void *data) +{ + struct regmap *regmap =3D data; + int ret; + + ret =3D regmap_clear_bits(regmap, AC200_SYS_IRQ_ENABLE_REG, + AC200_SYS_IRQ_INTB_ENABLE); + if (ret) + dev_err(regmap_get_device(regmap), "failed to disable INTB: %d\n", + ret); +} + +static int ac200_init_irq(struct device *dev, struct regmap *regmap, int i= rq) +{ + unsigned int trigger; + u16 value =3D 0; + int ret; + + trigger =3D irq_get_trigger_type(irq); + switch (trigger) { + case IRQ_TYPE_LEVEL_HIGH: + value |=3D AC200_SYS_IRQ_INTB_ACTIVE_HIGH; + break; + case IRQ_TYPE_NONE: + case IRQ_TYPE_LEVEL_LOW: + break; + default: + return dev_err_probe(dev, -EINVAL, + "INTB is level triggered, not type %u\n", + trigger); + } + + /* Mask every source before enabling the shared output. */ + ret =3D regmap_update_bits(regmap, AC200_SYS_IRQ_ENABLE_REG, + AC200_SYS_IRQ_INTB_ENABLE | + AC200_SYS_IRQ_INTB_ACTIVE_HIGH | + AC200_SYS_IRQ_RTC | AC200_SYS_IRQ_EPHY | + AC200_SYS_IRQ_TVE, value); + if (ret) + return ret; + + /* Children are removed before INTB is disabled and the regmap released. = */ + ret =3D devm_add_action_or_reset(dev, ac200_disable_intb, regmap); + if (ret) + return ret; + + if (trigger =3D=3D IRQ_TYPE_NONE) { + ret =3D irq_set_irq_type(irq, IRQ_TYPE_LEVEL_LOW); + if (ret) + return dev_err_probe(dev, ret, "failed to set INTB trigger\n"); + } + + /* + * Function drivers request INTB with IRQF_SHARED | IRQF_ONESHOT before + * enabling their own source. They check and clear their own status and + * mask their source before freeing the IRQ. No IRQ domain is needed. + */ + return regmap_set_bits(regmap, AC200_SYS_IRQ_ENABLE_REG, + AC200_SYS_IRQ_INTB_ENABLE); +} + +static int ac200_probe(struct i2c_client *client) +{ + const struct mfd_cell *cells =3D ac200_noirq_cells; + unsigned int num_cells =3D ARRAY_SIZE(ac200_noirq_cells); + struct device *dev =3D &client->dev; + struct regmap *regmap; + struct clk *clk; + int ret; + + clk =3D devm_clk_get_enabled(dev, NULL); + if (IS_ERR(clk)) + return dev_err_probe(dev, PTR_ERR(clk), + "failed to enable input clock\n"); + + ret =3D devm_clk_rate_exclusive_get(dev, clk); + if (ret) + return dev_err_probe(dev, ret, "failed to lock clock rate\n"); + + /* + * No minimum delay is documented. Retain the vendor resume path's 40 ms + * wait after enabling the input clock and before register access. + */ + msleep(40); + + regmap =3D devm_regmap_init_i2c(client, &ac200_regmap_config); + if (IS_ERR(regmap)) + return dev_err_probe(dev, PTR_ERR(regmap), + "failed to initialize regmap\n"); + + ret =3D regmap_set_bits(regmap, AC200_SYS_CONTROL_REG, + AC200_SYS_CONTROL_CHIP_RESET_DEASSERT); + if (ret) + return ret; + + if (client->irq > 0) { + ret =3D ac200_init_irq(dev, regmap, client->irq); + if (ret) + return ret; + cells =3D ac200_cells; + num_cells =3D ARRAY_SIZE(ac200_cells); + } + + /* Function drivers read their firmware properties from the parent. */ + ret =3D devm_mfd_add_devices(dev, PLATFORM_DEVID_AUTO, cells, num_cells, + NULL, client->irq, NULL); + if (ret) + return dev_err_probe(dev, ret, "failed to add function devices\n"); + + return 0; +} + +static const struct of_device_id ac200_of_match[] =3D { + { .compatible =3D "x-powers,ac200" }, + { } +}; +MODULE_DEVICE_TABLE(of, ac200_of_match); + +static const struct i2c_device_id ac200_i2c_ids[] =3D { + { .name =3D "ac200" }, + { } +}; +MODULE_DEVICE_TABLE(i2c, ac200_i2c_ids); + +static struct i2c_driver ac200_driver =3D { + .driver =3D { + .name =3D "ac200", + .of_match_table =3D ac200_of_match, + }, + .probe =3D ac200_probe, + .id_table =3D ac200_i2c_ids, +}; +module_i2c_driver(ac200_driver); + +MODULE_AUTHOR("Jernej Skrabec "); +MODULE_AUTHOR("James Hilliard "); +MODULE_DESCRIPTION("X-Powers AC200 MFD core driver"); +MODULE_LICENSE("GPL"); --=20 2.53.0