1. Patchew
  2. linux
  3. [PATCH v3 0/3] MAX77675 regulator driver: Add support for MAX77675 device
  4. View patch

[PATCH v3 1/3] dt-bindings: regulator: Add MAX77675 binding header

Joan-Na-adi posted 3 patches 2 months ago
[PATCH v3 1/3] dt-bindings: regulator: Add MAX77675 binding header
Posted by Joan-Na-adi 2 months ago 
From: Joan Na <joan.na@analog.com>

Add binding header for the MAX77675 PMIC regulator. This header defines
voltage ID and regulator index macros used both in device tree sources
and in the driver implementation.

Fixes:
- Removed unused macros
- Renamed macros for clarity

Signed-off-by: Joan Na <joan.na@analog.com>
---
 .../regulator/maxim,max77675-regulator.h      | 52 +++++++++++++++++++
 1 file changed, 52 insertions(+)
 create mode 100644 include/dt-bindings/regulator/maxim,max77675-regulator.h

diff --git a/include/dt-bindings/regulator/maxim,max77675-regulator.h b/include/dt-bindings/regulator/maxim,max77675-regulator.h
new file mode 100644
index 000000000000..b3b52d1668c2
--- /dev/null
+++ b/include/dt-bindings/regulator/maxim,max77675-regulator.h
@@ -0,0 +1,52 @@
+/* SPDX-License-Identifier: GPL-2.0-only OR BSD 2-Clause */
+/*
+ * This header provides macros for MAXIM MAX77675 device bindings.
+ *
+ * Copyright (c) 2025, Analog Device inc.
+ * Author: Joan Na <joan.na@analog.com>
+ */
+
+#ifndef _DT_BINDINGS_REGULATOR_MAX77675_
+#define _DT_BINDINGS_REGULATOR_MAX77675_
+
+/* FPS source */
+#define MAX77675_FPS_SLOT_0       0x0
+#define MAX77675_FPS_SLOT_1       0x1
+#define MAX77675_FPS_SLOT_2       0x2
+#define MAX77675_FPS_SLOT_3       0x3
+#define MAX77675_FPS_DEF          0x4
+
+/* nEN Manual Reset Time Configuration (MRT) */
+#define MAX77675_MRT_4S           0x0
+#define MAX77675_MRT_8S           0x1
+#define MAX77675_MRT_12S          0x2
+#define MAX77675_MRT_16S          0x3
+
+/* nEN Mode Configuration */
+#define MAX77675_EN_PUSH_BUTTON   0x0
+#define MAX77675_EN_SLIDE_SWITCH  0x1
+#define MAX77675_EN_LOGIC         0x2
+
+/* Debounce Timer Enable (DBEN_nEN) */
+#define MAX77675_DBEN_100US       0x0
+#define MAX77675_DBEN_30000US     0x1
+
+/* Rising slew rate control for SBB0 when ramping up */
+#define MAX77675_SR_2MV_PER_US    0x0  // 2 mV/us
+#define MAX77675_SR_USE_DVS       0x1  // Use DVS slew rate setting (maxim,dvs-slew-rate)
+
+/* Dynamic Voltage Scaling (DVS) Slew Rate */
+#define MAX77675_DVS_SLEW_5MV_PER_US    0x0  // 5 mV/us
+#define MAX77675_DVS_SLEW_10MV_PER_US   0x1  // 10 mV/us
+
+/* Latency Mode */
+#define MAX77675_HIGH_LATENCY_MODE  0x0   // High latency, low quiescent current (~100us)
+#define MAX77675_LOW_LATENCY_MODE   0x1   // Low latency, high quiescent current (~10us)
+
+/* SIMO Buck-Boost Drive Strength (All Channels) */
+#define MAX77675_DRV_SBB_STRENGTH_MAX  0x0  // Maximum drive strength (~0.6 ns transition time)
+#define MAX77675_DRV_SBB_STRENGTH_HIGH 0x1  // High drive strength (~1.2 ns transition time)
+#define MAX77675_DRV_SBB_STRENGTH_LOW  0x2  // Low drive strength (~1.8 ns transition time)
+#define MAX77675_DRV_SBB_STRENGTH_MIN  0x3  // Minimum drive strength (~8 ns transition time)
+
+#endif
--
2.34.1
Re: [PATCH v3 1/3] dt-bindings: regulator: Add MAX77675 binding header
Posted by Krzysztof Kozlowski 1 month, 3 weeks ago 
On Tue, Oct 14, 2025 at 02:31:40PM +0900, Joan-Na-adi wrote:
> diff --git a/include/dt-bindings/regulator/maxim,max77675-regulator.h b/include/dt-bindings/regulator/maxim,max77675-regulator.h
> new file mode 100644
> index 000000000000..b3b52d1668c2
> --- /dev/null
> +++ b/include/dt-bindings/regulator/maxim,max77675-regulator.h
> @@ -0,0 +1,52 @@
> +/* SPDX-License-Identifier: GPL-2.0-only OR BSD 2-Clause */

Wrong text, it looks, please run checkpatch.

Best regards,
Krzysztof
Re: [PATCH v3 1/3] dt-bindings: regulator: Add MAX77675 binding header
Posted by Krzysztof Kozlowski 2 months ago 
On 14/10/2025 07:31, Joan-Na-adi wrote:
> From: Joan Na <joan.na@analog.com>
> 
> Add binding header for the MAX77675 PMIC regulator. This header defines
> voltage ID and regulator index macros used both in device tree sources
> and in the driver implementation.
> 
> Fixes:
> - Removed unused macros
> - Renamed macros for clarity


This makes no sense. Fixes what? There are no macros before.

Please read submitting patches how to write proper changelogs.

> 
> Signed-off-by: Joan Na <joan.na@analog.com>
> ---
>  .../regulator/maxim,max77675-regulator.h      | 52 +++++++++++++++++++
>  1 file changed, 52 insertions(+)
>  create mode 100644 include/dt-bindings/regulator/maxim,max77675-regulator.h
> 
> diff --git a/include/dt-bindings/regulator/maxim,max77675-regulator.h b/include/dt-bindings/regulator/maxim,max77675-regulator.h
> new file mode 100644
> index 000000000000..b3b52d1668c2
> --- /dev/null
> +++ b/include/dt-bindings/regulator/maxim,max77675-regulator.h
> @@ -0,0 +1,52 @@
> +/* SPDX-License-Identifier: GPL-2.0-only OR BSD 2-Clause */
> +/*
> + * This header provides macros for MAXIM MAX77675 device bindings.
> + *
> + * Copyright (c) 2025, Analog Device inc.
> + * Author: Joan Na <joan.na@analog.com>
> + */
> +
> +#ifndef _DT_BINDINGS_REGULATOR_MAX77675_
> +#define _DT_BINDINGS_REGULATOR_MAX77675_
> +
> +/* FPS source */
> +#define MAX77675_FPS_SLOT_0       0x0
> +#define MAX77675_FPS_SLOT_1       0x1
> +#define MAX77675_FPS_SLOT_2       0x2
> +#define MAX77675_FPS_SLOT_3       0x3
> +#define MAX77675_FPS_DEF          0x4
> +
> +/* nEN Manual Reset Time Configuration (MRT) */
> +#define MAX77675_MRT_4S           0x0
> +#define MAX77675_MRT_8S           0x1
> +#define MAX77675_MRT_12S          0x2
> +#define MAX77675_MRT_16S          0x3

None of these are bindings.


> +
> +/* nEN Mode Configuration */
> +#define MAX77675_EN_PUSH_BUTTON   0x0
> +#define MAX77675_EN_SLIDE_SWITCH  0x1
> +#define MAX77675_EN_LOGIC         0x2

Neither these.

> +
> +/* Debounce Timer Enable (DBEN_nEN) */
> +#define MAX77675_DBEN_100US       0x0
> +#define MAX77675_DBEN_30000US     0x1
> +
> +/* Rising slew rate control for SBB0 when ramping up */
> +#define MAX77675_SR_2MV_PER_US    0x0  // 2 mV/us
> +#define MAX77675_SR_USE_DVS       0x1  // Use DVS slew rate setting (maxim,dvs-slew-rate)
> +
> +/* Dynamic Voltage Scaling (DVS) Slew Rate */
> +#define MAX77675_DVS_SLEW_5MV_PER_US    0x0  // 5 mV/us
> +#define MAX77675_DVS_SLEW_10MV_PER_US   0x1  // 10 mV/us
> +
> +/* Latency Mode */
> +#define MAX77675_HIGH_LATENCY_MODE  0x0   // High latency, low quiescent current (~100us)
> +#define MAX77675_LOW_LATENCY_MODE   0x1   // Low latency, high quiescent current (~10us)
> +
> +/* SIMO Buck-Boost Drive Strength (All Channels) */
> +#define MAX77675_DRV_SBB_STRENGTH_MAX  0x0  // Maximum drive strength (~0.6 ns transition time)
> +#define MAX77675_DRV_SBB_STRENGTH_HIGH 0x1  // High drive strength (~1.2 ns transition time)
> +#define MAX77675_DRV_SBB_STRENGTH_LOW  0x2  // Low drive strength (~1.8 ns transition time)
> +#define MAX77675_DRV_SBB_STRENGTH_MIN  0x3  // Minimum drive strength (~8 ns transition time)
> +


Drop entire header. Not a binding. Otherwise explain me which ABI are
you binding?


Best regards,
Krzysztof

Patchew 2.1-devel-b67e4c2

© 2016 - 2025 Red Hat, Inc.