From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-ej1-f50.google.com (mail-ej1-f50.google.com [209.85.218.50]) (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 8000C199EAD for ; Sun, 14 Sep 2025 19:26:10 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.218.50 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1757877972; cv=none; b=qfYUJjPdNcAqBOgkFwm/E44woJgAOkDySyB3HXts4EHQI40gkWFdH55IG+Nst4OtHEw34Q+bKzsz0V0efoF3n1xts5waDauhH/YszX21r+c7vWxX8ykXVpNr1EDncunE4i4jd9tLLH9yXAKMO1Q/+tQlKRBC40T3PYJJTPjX0E4= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1757877972; c=relaxed/simple; bh=d7u1JIqjgkuTAjQngOZ7W52Fqia/WzmKATwV61FnzrQ=; h=From:To:Cc:Subject:Date:Message-Id:MIME-Version:Content-Type; b=elvYBaLf65dDe7kBEuSZENaZznF2Xt9V2uaxO0V779lrTykN10791A+hyiXU4eA7Jj1eE7cakkd0/IxX/iQ5tszZmTx1bv+8ReA5SbQGmbeSGOp80GRLcioSZsrLlR+2w4ystTOunGuTm//dSZrnblBXVSP/A2CmeJXVt/fyIgk= 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=EZG4Uf7I; arc=none smtp.client-ip=209.85.218.50 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="EZG4Uf7I" Received: by mail-ej1-f50.google.com with SMTP id a640c23a62f3a-afcb7a16441so523870566b.2 for ; Sun, 14 Sep 2025 12:26:10 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1757877969; x=1758482769; 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=PSIN9RzyqCPmQgqHLeTMza5/h93nH1xHywvp8sYYYA0=; b=EZG4Uf7IDNLDEQrOv+VaRMRDISIqwD7qPwavGS5Sj4d18nUu8i8ZEdZcXt0oV+DqJu LsWol1y1L09td0N+nF3ul5pipB2ELc5qTwDJ5IjIrTtPEn6mkOJ3u7u9/jdf24dajMvJ LPvlzxJsF72di4qFXKqSjgvDpIL1MSZkETDqVfTKCGIcSNqTorF60tPDbljTZYtKQx8C XV5JTxZYVrIo4uU4JutVTNEK2z56Q/HPUYTXffZbaqQAB+h9WziG9VHbXsaX+OlCSNUc zzlVyZcAbjbCtLILxSs2+16oeivKgpvyql+Wj7lMtOIV+60H34YgFYX2TJ2ceHyNYdI2 g3Pw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1757877969; x=1758482769; 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=PSIN9RzyqCPmQgqHLeTMza5/h93nH1xHywvp8sYYYA0=; b=RgTjNDuJeS7SYYHEZNl7WuMs6zSNvRyaRNAtvnCDSZdBFU52F5cclE/2CwIHjjII+G MARV9/f+dQdlbMVN/aviaoECeJUuA3A+M4ZePA8ZT4SAitj+ES9groMtS7WPrrNE/9n2 c8dcXQOwrMVDkI7rJrt3PqQcLjPROFNLO49p1qLsztSIdqJEvUCczffPEkc3yIvUhPGt bKQVto/4OvEkvsxQsNKZmV72eNQovT8YGGT99lEgQrvWSem9l93KQlDxxw4l8Q45+umY 6ICWEzPZzpc50YrFj4/AgNDd1A5e1n5shoFcfjVHgC1Xa5XiXA1kOmLYIEFmeMf+nFFq IwoQ== X-Forwarded-Encrypted: i=1; AJvYcCW6pVCgE1BgXVEdd2YnN1q7f/KEs4K7ydL+MlMka15InICqjTDKAIFyDfdsDQCXY2uWjSnNipG9HCIJXBc=@vger.kernel.org X-Gm-Message-State: AOJu0YwOGyiaKfjvqpqP6JBgYnMm5x+MMQg2+uugiQCnUWs1dWPGsmxS oX7MbVIF6pEH0uEQBp+xfLSPqaqgxsMB/PgkTGJSxkxVzO0VTxRBYJ8= X-Gm-Gg: ASbGncsEcAk7k8m/AjGoWfsyNAvHj3bMwwG9FRBY7onEm0+GlVqo1J2XG9bmWwb3+G6 E2luLa2cIuqSPu7JKXUQ5gyTqcY/NrRCzvexTSUqL+aUNQ+1eDTOdBcf1MWViHtVrOIdKtcy6tb xeJHhZyBBZmCPTORK0LGMfGMN9tXhRj82C18XacL8IbkIbgP4UrRIesmVvNhlHZN/nkwftFxCCi XxfZkxUU8BHGktzC18g1kPpPY/ncnsc3WzaOnPvWA/RPmS21GnouMupZ8fc+LlUKb72+/uSfzgC R9Rmyhjc0o2SQ56f24ZttmfMHd57zG03DkolfpmNPCHzYllmlBqbFp7PuK8+5NAwSyaLPkWB85n F2mIeJ+HI+WetUDWTGyyqWI2AyvYWsN7/ypAcRXnLqsKz4X6QXpIcuObtUw== X-Google-Smtp-Source: AGHT+IEGWv/EAuzgSEUtp2T9EmVVlNbYX1xEJpRNtUQLpEg90WgKSuE7467gfTGWbp2Pkrj8w6UEgw== X-Received: by 2002:a17:907:9443:b0:b09:d915:1416 with SMTP id a640c23a62f3a-b09d91547c7mr511335566b.54.1757877968616; Sun, 14 Sep 2025 12:26:08 -0700 (PDT) Received: from localhost.localdomain ([2a0d:e487:226e:7d72:f695:46fd:3aef:7487]) by smtp.gmail.com with ESMTPSA id a640c23a62f3a-b07baf35488sm707841066b.38.2025.09.14.12.26.07 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 14 Sep 2025 12:26:08 -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 v8 0/3] dt-bindings: mfd: twl: Consolidate and fix TI TWL family bindings Date: Sun, 14 Sep 2025 21:25:13 +0200 Message-Id: <20250914192516.164629-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 Following extensive maintainer feedback, this patch implements the "define then disable" pattern. All possible child nodes are now defined at the top level. Conditional if/then blocks are used to explicitly disallow sub-modules that are not present on specific chip variants. This refactoring also moves pre-existing sub-node definitions (e.g. madc, pwrbutton, gpadc, etc.) out of the conditional blocks to the top level, following the new pattern. 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]. --- Changes in v8: - Restructured binding to define all properties at the top level and use 'property-name: false' to conditionally disable them. This includes refactoring pre-existing sub-node definitions. - Referenced the existing ti,twl6030-charger.yaml and ti,twl6030-gpadc.yaml schemas for the TWL603x charger and gpadc nodes. - No changes for (2/3) & (3/3). 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 | 319 +++++++++++++++--- .../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, 275 insertions(+), 130 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