From patchwork Wed Jun 28 13:30:19 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Esteban Blanc X-Patchwork-Id: 697814 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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 6A6C6C001DE for ; Wed, 28 Jun 2023 13:30:44 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231351AbjF1Nam (ORCPT ); Wed, 28 Jun 2023 09:30:42 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:47200 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232398AbjF1Na1 (ORCPT ); Wed, 28 Jun 2023 09:30:27 -0400 Received: from mail-lf1-x136.google.com (mail-lf1-x136.google.com [IPv6:2a00:1450:4864:20::136]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 2A6BC1FCC for ; Wed, 28 Jun 2023 06:30:25 -0700 (PDT) Received: by mail-lf1-x136.google.com with SMTP id 2adb3069b0e04-4fb77f21c63so4871128e87.2 for ; Wed, 28 Jun 2023 06:30:25 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=baylibre-com.20221208.gappssmtp.com; s=20221208; t=1687959023; x=1690551023; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=BXYQ3UvRxgd/GOjXjwkmXpmmOQaoDxWZO5ds+5aOCbQ=; b=VXCNU3Ql55wNm5mKZKjQHVq64XCj73HgsVvJ+ebjBGS3Xk4k8nOVzo6ebkeCJdxwmO a7jOTZ40IGo3LWv1RkZGvxO7GOr1izFvcH9ig7xTn2GCQwyJU3ejuJRoUzjcOPWuTgV3 AY+jtzXOTzXQWvBDqR+BhQQjWpoo43m1WPxhfpNyg9BxtRBovUAEO94OBossbR1uZy3u KJHW1OqxjyDyPiJtJtFGgBRwJ7UBaScQfpZQIpfjXOiETHZ7Jo7fzD59XOSpQIU89ceP 2JJLCNWAoCa2d68DE4w+RVTBrHTw/v3TyHBXRtM1+HjefjPvxWeGUmB4mnpu2sRVV3Ju tinw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1687959023; x=1690551023; 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=BXYQ3UvRxgd/GOjXjwkmXpmmOQaoDxWZO5ds+5aOCbQ=; b=jpjogV0o4b++gbAIXMqV4fDybSQwhZhNr1CLrJuMrmgNc+i+bsTKzhdUIEKrPPMrp3 vjaEFOJ8In+Tblnx0lHsQgh+bZhro8NHo23F5IznEqwPlR8RIqCoafUG8GEhaGNpqP+T CzMlroGNs1dFgGULkVEqGgQna9ihvhJBlFnHiaU76lWHih3DIt34PC7PcZa3f4Tbr2Fy sV5JE18u4I6Z+wokIhwlvuZlEe1WS5/NMCNO5z3EFR+xjz0yU7z6o8fdT+REEh4ebxlG GEetW53GJbb3FR19nf8ej6M6epOIOBtvTf7ced21zA0U6U3N3qNd0LLP4CZZj7khSRc9 PfMA== X-Gm-Message-State: AC+VfDytlKKOED2yLBmPQyteLMV0T8Aazp00ajWLKr6RCiy5MbrLM/Fc xT5QtaD8hx5yleXjpAMb93q1KA== X-Google-Smtp-Source: ACHHUZ6k60XfBnhvzXKMM7ViB35mVDerfENg0F+/+gCPAI697ws7nV9ugBk+f/HD6Z5Xi9JepNsKmw== X-Received: by 2002:a19:5e47:0:b0:4f3:b708:f554 with SMTP id z7-20020a195e47000000b004f3b708f554mr19235829lfi.47.1687959023108; Wed, 28 Jun 2023 06:30:23 -0700 (PDT) Received: from localhost.localdomain ([2a01:e0a:28d:66d0:c6d7:ee89:d0ff:2f52]) by smtp.gmail.com with ESMTPSA id z7-20020a05600c220700b003fa999cefc0sm8067994wml.36.2023.06.28.06.30.21 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 28 Jun 2023 06:30:22 -0700 (PDT) From: Esteban Blanc To: linus.walleij@linaro.org, a.zummo@towertech.it, alexandre.belloni@bootlin.com Cc: linux-kernel@vger.kernel.org, linux-gpio@vger.kernel.org, linux-rtc@vger.kernel.org, jpanis@baylibre.com, jneanne@baylibre.com, aseketeli@baylibre.com, eblanc@baylibre.com, u-kumar1@ti.com Subject: [PATCH v7 0/2] TI TPS6594 PMIC support (RTC, pinctrl, regulators) Date: Wed, 28 Jun 2023 15:30:19 +0200 Message-ID: <20230628133021.500477-1-eblanc@baylibre.com> X-Mailer: git-send-email 2.41.0 MIME-Version: 1.0 Precedence: bulk List-ID: X-Mailing-List: linux-gpio@vger.kernel.org TPS6594 is a Power Management IC which provides regulators and others features like GPIOs, RTC, watchdog, ESMs (Error Signal Monitor), and PFSM (Pre-configurable Finite State Machine). The SoC and the PMIC can communicate through the I2C or SPI interfaces. TPS6594 is the super-set device while TPS6593 and LP8764 are derivatives. This series adds support to TI TPS6594 PMIC and its derivatives. This should be applied on top of other patch series: - https://lore.kernel.org/all/20230511095126.105104-1-jpanis@baylibre.com/ For core MFD driver. The necessary part of this patch series is already applied in linux-next. The features implemented in this series are: - RTC (child device) - Pinmux/GPIO (child device) - Regulator (child device) RTC description: The TPS6594 family has an RTC built-in, except for LP8764. It provides time and an alarm. Pinmux/GPIO: TPS6594 family has 11 GPIOs. Those GPIO can also serve different functions such as I2C or SPI interface, watchdog disable functions. The driver provides both pinmuxing for the functions and GPIO capability. Regulator: TPS6594/TPS6593: 5 BUCKs and 4LDOs LP8764: 4 BUCKs and no LDO Bucks can be used in multipahse mode. Regulators were applied to linux-next by Mark Brown on 06/06/2023 so this patch has been dropped from the patch series. There were some pending comments from Andy Shevchenko so a follow up patch will be sent later. Changes since v1: https://lore.kernel.org/all/20230224133129.887203-1-eblanc@baylibre.com/ Rtc: - Removed struct tps6594_rtc. - Removed some dev_err messages. - Removed some comments. - Remove some whitespaces in comments and error messages. - Check if RTC is running before reading a timestamp in read_rtc. - Stop RTC at the end of probe to wait for a timestamp to be set. - Add default MFD_TPS6594 to Kconfig. Pinctrl: - Removed #define DEBUG. - Add default MFD_TPS6594 to Kconfig. - Fix typo and reword help message of Kconfig. Regulators: Further to Mark Brown review: - File header whole block C++ style. - Configuring modes not supported: omit all mode operations - Log the error before notifying. - Request the interrupts while registering the regulators (then remove the lookup function). Further to Matti review: - Postponed: devm_regulator_irq_helper() and regulator_irq_map_event_simple() can probably be used but code. refactoring is not so trivial. This can be done later as an enhancement after this patch list is merged. Buck Multi phase management: - Multiphase property can take an array when 2 multi phase buck, buck12 and buck34. - Configuration multi phase buck34 without multiphase buck12 is not supported (when only one multiphase, must be buck12). Not clear from the spec but confirmed by TI. - Supported multiphase conficurations: buck12, buck123, buck1234, buck12 + buck34. - All interrupts are attached to the multiphase buck (ie: for regulator buck12, buck1 & buck2 interrupts are registered). Changes since v2: https://lore.kernel.org/all/20230328091448.648452-1-eblanc@baylibre.com/ Rtc: - Add logic to avoid reinitializing a working clock. - Fix some multiline comments format. Regulators: Further to Mark Brown review: - Log the error before notifying. - Request the interrupts while registering the regulators. Further to Krzysztof Kozlowski: https://lore.kernel.org/all/75f0a18d-aed9-8610-2925-4e604b4b0241@baylibre.com/ - Remove ti, multi-phase-id property which is redundant with buck dts naming rules. Changes since v3: https://lore.kernel.org/lkml/20230414101217.1342891-1-eblanc@baylibre.com/ RTC: - Add wakeup source Pinctrl: - Switch to GPIO_REGMAP framework Change since v4: https://lore.kernel.org/lkml/20230512141755.1712358-1-eblanc@baylibre.com/ Update Copyright notice date Reorder includes RTC: - Rework some comments, fixing punctuation and style - Use NANO macro from units.h for PPB_MULT - Rework to use bitwise types - Remove unnecessary casts - Add SAFETY comments - Use `dev_err_probe(...)` instead of print then return Pinctrl: - Reword help message and add module name in Kconfig - Rework code to use struct pinfunction and PINCTRL_PINFUNCTION() macro - Remove unnecessary casts - Use `dev_err_probe(...)` instead of print then return - Replace TPS6594_REG_GPIO1_CONF with a comment for TPS6594_REG_GPIOX_CONF Regulators: - nits: Add missing tabs, standard spaces, group "buck_multi". - Use OF dedicated of_node_cmp API instead of standard strcmp. - Use devm_kmalloc_array(...) API instead of devm_kmalloc(...) wherever possible. - return dev_err_probe(...) wherever possible. Changes since v5: https://lore.kernel.org/lkml/20230522163115.2592883-1-eblanc@baylibre.com/ Pinctrl: - Rework code for clarity - Rework macro to fix checkpatch macro argument reuse - Coding style fixes - Reword some comments Rtc: - Grammar fixes - Removed unused macros - Use type MIN/MAX macro instead of magic numbers - Fix return code in calibration - Use cpu_to_le16 and le16_to_cpu APIs instead of casting. - Reintroduce mdelay before reading BIT_RUN as otherwise both AM62 and J784S4 will report a -ENODEV on a working RTC. Changes since v6: https://lore.kernel.org/all/20230612125248.1235581-1-eblanc@baylibre.com/ Pinctrl: - Remove a comment Esteban Blanc (2): rtc: tps6594: Add driver for TPS6594 RTC pinctrl: tps6594: Add driver for TPS6594 pinctrl and GPIOs drivers/pinctrl/Kconfig | 15 + drivers/pinctrl/Makefile | 1 + drivers/pinctrl/pinctrl-tps6594.c | 368 ++++++++++++++++++++++++ drivers/rtc/Kconfig | 12 + drivers/rtc/Makefile | 1 + drivers/rtc/rtc-tps6594.c | 452 ++++++++++++++++++++++++++++++ 6 files changed, 849 insertions(+) create mode 100644 drivers/pinctrl/pinctrl-tps6594.c create mode 100644 drivers/rtc/rtc-tps6594.c