From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-ej1-f46.google.com (mail-ej1-f46.google.com [209.85.218.46]) (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 1F7EA277CAB for ; Wed, 10 Sep 2025 16:07:22 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.218.46 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1757520445; cv=none; b=ndT4cVBixcklyHU0Ke1Z5lqeKfvI08Bass+8qt1eSrSrQxVf86PeEvbZn5Ob5m/DhZ3HSYZUg5oFgcsUY9D/MzYSRS0sAbUJUGAxsw78X+EvYwBXiiyNUjCI1+vk8Cle5pRJXrJdG7gsOZt5tpKGuisQ+x+monbL9ZUiwFV4RKU= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1757520445; c=relaxed/simple; bh=9JM8zXnpGCXjAshCsShUwOSUA0HzjprAbIs3oGMpa2s=; h=From:To:Cc:Subject:Date:Message-Id:MIME-Version:Content-Type; b=XUOgt8vjjFTHrdbVO+TqO5qcOMI8PQJ1uWVC3vu7rs/fCqlNIMzQBEd98Nc979EFzhlKUNSUVF5WOtHplLZ45FnDmsfZdPF8kOcal1xq48DlvQwWvqbkDt7Ib0GGyOSidkcrp0zkYMGErc8+nDmqt8qXmHrQCoGU6wEZLO+WFD0= 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=lo94Pf0i; arc=none smtp.client-ip=209.85.218.46 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="lo94Pf0i" Received: by mail-ej1-f46.google.com with SMTP id a640c23a62f3a-b04770a25f2so978942866b.2 for ; Wed, 10 Sep 2025 09:07:22 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1757520441; x=1758125241; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=mIxXSxIMQ3326DXwyl7d19buZ73lRdljmi6FSs0+JJ0=; b=lo94Pf0ivpehjcbLmUd/EHc8VdIoX/egmiqUSTPqyn2St82q2ydV44OvJQ0KYDxMWx 7H196cT8i8v+aL8Mrpz7F3yImVeKZ45rOH5WMTV+7F0yeZ4Lax0MCEVVTtCTd7mmQv75 LP3xYedFas+tozpkVVuZ3YQOPm5g2vjT9b9Dh8dMohFrBXiV1MQPmAKAKpNe7Vc9S8j8 rfwk31mnF1lkW+RXPGw23kk9NXe1Gyrm9G3AwAikwhEZizkitTBlnw3Qy/Wcq7sO9oev xEXDAr+3bAHJnGalsN10Yta9/UED/eBSk5Pw2EuwGje93eO9Oj0S3FPjz3TnudUhB7lF JT+Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1757520441; x=1758125241; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=mIxXSxIMQ3326DXwyl7d19buZ73lRdljmi6FSs0+JJ0=; b=at3dGT2U0O66qG+I65GRPx5qINwt+OnRsjuSCncAxXhSFHOnjHSVY3ycjabr5RgX/i vX1rqd6npz2hqL2Xirw9K8NW/VKz2T0LQIWDt4smrAHMkc2qMMlvY+JomPCgwBSPA23U jHvS9jHY5jB/VSOJMRjXVGMCQ4yfy58VZlFjmh7bn0dtj2cRw9aTI7693Sr6Tz1BaubV uU43vFODlBSIOS2BRC5r1bP9A07ipl7RE1hwcn99m17jYvkNLpeoqCU7cpkkKjRT/DhS hO/yI4U3gNjSZncc9z5LgDKGa1vBdVO1gynZPbZjPfECB4qYSYDDE0xo4ozO9fvVCiu2 fFgg== X-Forwarded-Encrypted: i=1; AJvYcCVqqyugbeFCj1ZTEZaMEDiSuLiVub9dN9z6zp5SOgGkE0/aBttnlQ0KGTH87GyaNbGLMvzXLfsuMN/iWAM=@vger.kernel.org X-Gm-Message-State: AOJu0YxY64DIbYn6V5VR+nAY7t5a74W1pt5oXyUs1hLgDChH7ac9jfE1 3InACmjaTot2f6qvlyWRxSWrmJzQZe6mLDB9AAqyup9mYM8ycRtO/Bg= X-Gm-Gg: ASbGncvWvqSHrJHZ5yDMqrJOUqMyjKCcnhPn4obgy41xcorWplcZdm4OoL+z5i6I90r feh7z01ox/a2LSbZmmKk/52TovXmRm90EVJFTCi61p2ELDmUBshRC9R5SMH7H1WDxZ1LZon1KqK iRqzKbj++sZPI2xTX0LpZKOxDyZLgh7A7fV5wLnWkcuQV138kXGCd5NELReUjx0/O2OR3kJvbpq Gz0A2mrKFBnXX4sqsl41oNmKXmeaDmHGz0yuf59WAGn/LBhZ1r3i+6q1h5kZjIdBhQKNdojALFY RsiGXJ/tNkkzM69UWDyN/3dO2vlpLVTgFsqgnme+IjCsx9CqQ/s7BJ4nXeUAYyRQuA84mv1dAdg /SDMOmE9ZNGM6XzRLFSZu/1l5DIF1OTlSBsBSdR1QjOMfhl/7FZE5mjTyydQBh1LZdvw= X-Google-Smtp-Source: AGHT+IHUnlYKS+LJGenEhgjXAoecAwaDL0J5OwJwpWMxJCRcFIpXNlW/vy+X963BIZzpFhsZr8LRZg== X-Received: by 2002:a17:907:2d8e:b0:b04:563f:e120 with SMTP id a640c23a62f3a-b04b16d641emr1475440566b.53.1757520441048; Wed, 10 Sep 2025 09:07:21 -0700 (PDT) Received: from localhost.localdomain ([2a0d:e487:229f:d114:32a:5671:8dbf:7357]) by smtp.gmail.com with ESMTPSA id a640c23a62f3a-b0783047b9bsm189424766b.13.2025.09.10.09.07.19 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 10 Sep 2025 09:07:20 -0700 (PDT) From: Jihed Chaibi To: andreas@kemnade.info Cc: lee@kernel.org, krzk+dt@kernel.org, tony@atomide.com, robh@kernel.org, conor+dt@kernel.org, ukleinek@kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-pwm@vger.kernel.org, linux-omap@vger.kernel.org, jihed.chaibi.dev@gmail.com Subject: [PATCH v7 0/3] dt-bindings: mfd: twl: Consolidate and fix TI TWL family bindings Date: Wed, 10 Sep 2025 18:07:01 +0200 Message-Id: <20250910160704.115565-1-jihed.chaibi.dev@gmail.com> X-Mailer: git-send-email 2.39.5 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: 8bit This version addresses a final piece of feedback from Andreas to make the twl4030/twl6030-specific child nodes (audio, usb, keypad etc.) conditional by moving them out of the common block, which now only contains common properties (rtc, charger, pwm, pwmled..) ensuring the schema is fully accurate. The complete dtbs_check for this binding is clean except for two warnings originating from pre-existing bugs in the OMAP DTS files, for which fixes have already been submitted separately [1][2]. Reviewed-by: Rob Herring Acked-by: Uwe Kleine-König --- Changes in v7: - (1/3): Moved twl4030/twl6030-specific child node definitions (audio, usb etc.) into the conditional 'if/then' block to improve schema accuracy. - (2/3 & 3/3): No changes. Changes in v6: - Refactored the ti,twl4030-power compatible schema to be much stricter, removing obsolete board-specific compatibles (-n900, -beagleboard-xm), that were added in v5. The schema now only permits specific, valid fallback combinations. - This series presents two new patches (2/3) & (3/3), which update the affected DTS files by removing obsolete entries. - Enforced the presence of the compatible property on all relevant sub-nodes by adding 'required: - compatible', closing a key validation loophole. - Applied various formatting cleanups for readability and correctness. Changes in v5: - Restructured the entire binding to define properties at the top level instead of if/then blocks, per maintainer feedback. - Added specific compatible enums for new child nodes instead of a generic 'compatible: true'. - Set 'unevaluatedProperties: false' for 'pwm' and 'pwmled' nodes to enforce strict validation. - Expanded 'power' node compatible enum to include all board-specific compatible strings (used in existing device trees, e.g. OMAP3-based boards) for more complete coverage. - Corrected the schema for the 'power' node compatible to properly handle single and fallback entries. Changes in v4: - Reworked binding to be independent and bisectable per maintainer feedback by using 'additionalProperties: true' for child nodes. - Added board-specific compatibles to the 'power' node enum. - Added definitions for 'clocks' and 'clock-names' properties. - Renamed 'twl6030-usb' child node to 'usb-comparator' to match existing Device Tree usage (twl6030.dtsi). - Fixed some spelling/grammar erros in the description. Changes in v3: - New patch to consolidate simple bindings (power, pwm) and add definitions for all child nodes to fix dtbs_check validation errors found in v2. Changes in v2: - This patch is split from larger series [3] per maintainer feedback. - Added missing sub-node definitions, resolving dtbs_check errors. [1] https://lore.kernel.org/all/20250822222530.113520-1-jihed.chaibi.dev@gmail.com/ [2] https://lore.kernel.org/all/20250822225052.136919-1-jihed.chaibi.dev@gmail.com/ [3] https://lore.kernel.org/all/20250816021523.167049-1-jihed.chaibi.dev@gmail.com/ Jihed Chaibi (3): dt-bindings: mfd: twl: Add missing sub-nodes for TWL4030 & TWL603x ARM: dts: omap3: beagle-xm: Correct obsolete TWL4030 power compatible ARM: dts: omap3: n900: Correct obsolete TWL4030 power compatible .../devicetree/bindings/mfd/ti,twl.yaml | 232 +++++++++++++++++- .../devicetree/bindings/mfd/twl4030-power.txt | 48 ---- .../devicetree/bindings/pwm/ti,twl-pwm.txt | 17 -- .../devicetree/bindings/pwm/ti,twl-pwmled.txt | 17 -- arch/arm/boot/dts/ti/omap/omap3-beagle-xm.dts | 2 +- arch/arm/boot/dts/ti/omap/omap3-n900.dts | 2 +- 6 files changed, 223 insertions(+), 95 deletions(-) delete mode 100644 Documentation/devicetree/bindings/mfd/twl4030-power.txt delete mode 100644 Documentation/devicetree/bindings/pwm/ti,twl-pwm.txt delete mode 100644 Documentation/devicetree/bindings/pwm/ti,twl-pwmled.txt -- 2.39.5