mail archive of the barebox mailing list
 help / color / mirror / Atom feed
* [PATCH] regulator: discard unused struct regulator_init_data handling
@ 2021-06-09  8:40 Ahmad Fatoum
  2021-06-11 11:25 ` Sascha Hauer
  0 siblings, 1 reply; 2+ messages in thread
From: Ahmad Fatoum @ 2021-06-09  8:40 UTC (permalink / raw)
  To: barebox; +Cc: Enrico Jorns, Ahmad Fatoum

struct regulator_init_data and the contained regulation_constraints
are populated from device tree, but unused anywhere. They are not
required for the stpmic driver to operate, so they can just be dropped.

No functional change.

Tested-by: Enrico Jorns <ejo@pengutronix.de>
Signed-off-by: Ahmad Fatoum <a.fatoum@pengutronix.de>
---
 drivers/regulator/of_regulator.c       | 148 ------------------
 include/linux/regulator/machine.h      | 207 -------------------------
 include/linux/regulator/of_regulator.h |  13 --
 3 files changed, 368 deletions(-)
 delete mode 100644 include/linux/regulator/machine.h

diff --git a/drivers/regulator/of_regulator.c b/drivers/regulator/of_regulator.c
index 3e8caa87104e..c536a82c439a 100644
--- a/drivers/regulator/of_regulator.c
+++ b/drivers/regulator/of_regulator.c
@@ -9,145 +9,6 @@
 #include <common.h>
 #include <of.h>
 #include <linux/regulator/of_regulator.h>
-#include <linux/regulator/machine.h>
-
-static int of_get_regulation_constraints(struct device_d *dev,
-					struct device_node *np,
-					struct regulator_init_data **init_data,
-					const struct regulator_desc *desc)
-{
-	struct regulation_constraints *constraints = &(*init_data)->constraints;
-	int ret;
-	u32 pval;
-
-	constraints->name = of_get_property(np, "regulator-name", NULL);
-
-	if (!of_property_read_u32(np, "regulator-min-microvolt", &pval))
-		constraints->min_uV = pval;
-
-	if (!of_property_read_u32(np, "regulator-max-microvolt", &pval))
-		constraints->max_uV = pval;
-
-	/* Voltage change possible? */
-	if (constraints->min_uV != constraints->max_uV)
-		constraints->valid_ops_mask |= REGULATOR_CHANGE_VOLTAGE;
-
-	/* Do we have a voltage range, if so try to apply it? */
-	if (constraints->min_uV && constraints->max_uV)
-		constraints->apply_uV = true;
-
-	if (!of_property_read_u32(np, "regulator-microvolt-offset", &pval))
-		constraints->uV_offset = pval;
-	if (!of_property_read_u32(np, "regulator-min-microamp", &pval))
-		constraints->min_uA = pval;
-	if (!of_property_read_u32(np, "regulator-max-microamp", &pval))
-		constraints->max_uA = pval;
-
-	if (!of_property_read_u32(np, "regulator-input-current-limit-microamp",
-				  &pval))
-		constraints->ilim_uA = pval;
-
-	/* Current change possible? */
-	if (constraints->min_uA != constraints->max_uA)
-		constraints->valid_ops_mask |= REGULATOR_CHANGE_CURRENT;
-
-	constraints->boot_on = of_property_read_bool(np, "regulator-boot-on");
-	constraints->always_on = of_property_read_bool(np, "regulator-always-on");
-	if (!constraints->always_on) /* status change should be possible. */
-		constraints->valid_ops_mask |= REGULATOR_CHANGE_STATUS;
-
-	constraints->pull_down = of_property_read_bool(np, "regulator-pull-down");
-
-	if (of_property_read_bool(np, "regulator-allow-bypass"))
-		constraints->valid_ops_mask |= REGULATOR_CHANGE_BYPASS;
-
-	if (of_property_read_bool(np, "regulator-allow-set-load"))
-		constraints->valid_ops_mask |= REGULATOR_CHANGE_DRMS;
-
-	ret = of_property_read_u32(np, "regulator-ramp-delay", &pval);
-	if (!ret) {
-		if (pval)
-			constraints->ramp_delay = pval;
-		else
-			constraints->ramp_disable = true;
-	}
-
-	ret = of_property_read_u32(np, "regulator-settling-time-us", &pval);
-	if (!ret)
-		constraints->settling_time = pval;
-
-	ret = of_property_read_u32(np, "regulator-settling-time-up-us", &pval);
-	if (!ret)
-		constraints->settling_time_up = pval;
-	if (constraints->settling_time_up && constraints->settling_time) {
-		pr_warn("%pOFn: ambiguous configuration for settling time, ignoring 'regulator-settling-time-up-us'\n",
-			np);
-		constraints->settling_time_up = 0;
-	}
-
-	ret = of_property_read_u32(np, "regulator-settling-time-down-us",
-				   &pval);
-	if (!ret)
-		constraints->settling_time_down = pval;
-	if (constraints->settling_time_down && constraints->settling_time) {
-		pr_warn("%pOFn: ambiguous configuration for settling time, ignoring 'regulator-settling-time-down-us'\n",
-			np);
-		constraints->settling_time_down = 0;
-	}
-
-	ret = of_property_read_u32(np, "regulator-enable-ramp-delay", &pval);
-	if (!ret)
-		constraints->enable_time = pval;
-
-	constraints->soft_start = of_property_read_bool(np,
-					"regulator-soft-start");
-	ret = of_property_read_u32(np, "regulator-active-discharge", &pval);
-	if (!ret) {
-		constraints->active_discharge =
-				(pval) ? REGULATOR_ACTIVE_DISCHARGE_ENABLE :
-					REGULATOR_ACTIVE_DISCHARGE_DISABLE;
-	}
-
-	if (!of_property_read_u32(np, "regulator-system-load", &pval))
-		constraints->system_load = pval;
-
-	if (!of_property_read_u32(np, "regulator-max-step-microvolt",
-				  &pval))
-		constraints->max_uV_step = pval;
-
-	constraints->over_current_protection = of_property_read_bool(np,
-					"regulator-over-current-protection");
-
-	return 0;
-}
-
-/**
- * of_get_regulator_init_data - extract regulator_init_data structure info
- * @dev: device requesting for regulator_init_data
- * @node: regulator device node
- * @desc: regulator description
- *
- * Populates regulator_init_data structure by extracting data from device
- * tree node, returns a pointer to the populated structure or NULL if memory
- * alloc fails.
- */
-struct regulator_init_data *of_get_regulator_init_data(struct device_d *dev,
-					  struct device_node *node,
-					  const struct regulator_desc *desc)
-{
-	struct regulator_init_data *init_data;
-
-	if (!node)
-		return NULL;
-
-	init_data = xzalloc(sizeof(*init_data));
-
-	if (of_get_regulation_constraints(dev, node, &init_data, desc))
-		return NULL;
-
-	return init_data;
-}
-EXPORT_SYMBOL_GPL(of_get_regulator_init_data);
 
 struct devm_of_regulator_matches {
 	struct of_regulator_match *matches;
@@ -192,7 +53,6 @@ int of_regulator_match(struct device_d *dev, struct device_node *node,
 
 	for (i = 0; i < num_matches; i++) {
 		struct of_regulator_match *match = &matches[i];
-		match->init_data = NULL;
 		match->of_node = NULL;
 	}
 
@@ -210,14 +70,6 @@ int of_regulator_match(struct device_d *dev, struct device_node *node,
 			if (strcmp(match->name, name))
 				continue;
 
-			match->init_data = of_get_regulator_init_data(dev, child,
-								      match->desc);
-			if (!match->init_data) {
-				dev_err(dev,
-					"failed to parse DT for regulator %pOFn\n",
-					child);
-				return -EINVAL;
-			}
 			match->of_node = child;
 			count++;
 			break;
diff --git a/include/linux/regulator/machine.h b/include/linux/regulator/machine.h
deleted file mode 100644
index bb8bc7c4e0b6..000000000000
--- a/include/linux/regulator/machine.h
+++ /dev/null
@@ -1,207 +0,0 @@
-/* SPDX-License-Identifier: GPL-2.0-only */
-/*
- * machine.h -- SoC Regulator support, machine/board driver API.
- *
- * Copyright (C) 2007, 2008 Wolfson Microelectronics PLC.
- *
- * Author: Liam Girdwood <lrg@slimlogic.co.uk>
- *
- * Regulator Machine/Board Interface.
- */
-
-#ifndef __LINUX_REGULATOR_MACHINE_H_
-#define __LINUX_REGULATOR_MACHINE_H_
-
-#include <regulator.h>
-
-struct regulator;
-
-/*
- * Regulator operation constraint flags. These flags are used to enable
- * certain regulator operations and can be OR'ed together.
- *
- * VOLTAGE:  Regulator output voltage can be changed by software on this
- *           board/machine.
- * CURRENT:  Regulator output current can be changed by software on this
- *           board/machine.
- * MODE:     Regulator operating mode can be changed by software on this
- *           board/machine.
- * STATUS:   Regulator can be enabled and disabled.
- * DRMS:     Dynamic Regulator Mode Switching is enabled for this regulator.
- * BYPASS:   Regulator can be put into bypass mode
- */
-
-#define REGULATOR_CHANGE_VOLTAGE	0x1
-#define REGULATOR_CHANGE_CURRENT	0x2
-#define REGULATOR_CHANGE_MODE		0x4
-#define REGULATOR_CHANGE_STATUS		0x8
-#define REGULATOR_CHANGE_DRMS		0x10
-#define REGULATOR_CHANGE_BYPASS		0x20
-
-/* Regulator active discharge flags */
-enum regulator_active_discharge {
-	REGULATOR_ACTIVE_DISCHARGE_DEFAULT,
-	REGULATOR_ACTIVE_DISCHARGE_DISABLE,
-	REGULATOR_ACTIVE_DISCHARGE_ENABLE,
-};
-
-/**
- * struct regulation_constraints - regulator operating constraints.
- *
- * This struct describes regulator and board/machine specific constraints.
- *
- * @name: Descriptive name for the constraints, used for display purposes.
- *
- * @min_uV: Smallest voltage consumers may set.
- * @max_uV: Largest voltage consumers may set.
- * @uV_offset: Offset applied to voltages from consumer to compensate for
- *             voltage drops.
- *
- * @min_uA: Smallest current consumers may set.
- * @max_uA: Largest current consumers may set.
- * @ilim_uA: Maximum input current.
- * @system_load: Load that isn't captured by any consumer requests.
- *
- * @max_spread: Max possible spread between coupled regulators
- * @valid_modes_mask: Mask of modes which may be configured by consumers.
- * @valid_ops_mask: Operations which may be performed by consumers.
- *
- * @always_on: Set if the regulator should never be disabled.
- * @boot_on: Set if the regulator is enabled when the system is initially
- *           started.  If the regulator is not enabled by the hardware or
- *           bootloader then it will be enabled when the constraints are
- *           applied.
- * @apply_uV: Apply the voltage constraint when initialising.
- * @ramp_disable: Disable ramp delay when initialising or when setting voltage.
- * @soft_start: Enable soft start so that voltage ramps slowly.
- * @pull_down: Enable pull down when regulator is disabled.
- * @over_current_protection: Auto disable on over current event.
- *
- * @input_uV: Input voltage for regulator when supplied by another regulator.
- *
- * @initial_mode: Mode to set at startup.
- * @ramp_delay: Time to settle down after voltage change (unit: uV/us)
- * @settling_time: Time to settle down after voltage change when voltage
- *		   change is non-linear (unit: microseconds).
- * @settling_time_up: Time to settle down after voltage increase when voltage
- *		      change is non-linear (unit: microseconds).
- * @settling_time_down : Time to settle down after voltage decrease when
- *			 voltage change is non-linear (unit: microseconds).
- * @active_discharge: Enable/disable active discharge. The enum
- *		      regulator_active_discharge values are used for
- *		      initialisation.
- * @enable_time: Turn-on time of the rails (unit: microseconds)
- */
-struct regulation_constraints {
-
-	const char *name;
-
-	/* voltage output range (inclusive) - for voltage control */
-	int min_uV;
-	int max_uV;
-
-	int uV_offset;
-
-	/* current output range (inclusive) - for current control */
-	int min_uA;
-	int max_uA;
-	int ilim_uA;
-
-	int system_load;
-
-	/* used for coupled regulators */
-	u32 *max_spread;
-
-	/* used for changing voltage in steps */
-	int max_uV_step;
-
-	/* valid regulator operating modes for this machine */
-	unsigned int valid_modes_mask;
-
-	/* valid operations for regulator on this machine */
-	unsigned int valid_ops_mask;
-
-	/* regulator input voltage - only if supply is another regulator */
-	int input_uV;
-
-	/* mode to set on startup */
-	unsigned int initial_mode;
-
-	unsigned int ramp_delay;
-	unsigned int settling_time;
-	unsigned int settling_time_up;
-	unsigned int settling_time_down;
-	unsigned int enable_time;
-
-	unsigned int active_discharge;
-
-	/* constraint flags */
-	unsigned always_on:1;	/* regulator never off when system is on */
-	unsigned boot_on:1;	/* bootloader/firmware enabled regulator */
-	unsigned apply_uV:1;	/* apply uV constraint if min == max */
-	unsigned ramp_disable:1; /* disable ramp delay */
-	unsigned soft_start:1;	/* ramp voltage slowly */
-	unsigned pull_down:1;	/* pull down resistor when regulator off */
-	unsigned over_current_protection:1; /* auto disable on over current */
-};
-
-/**
- * struct regulator_consumer_supply - supply -> device mapping
- *
- * This maps a supply name to a device. Use of dev_name allows support for
- * buses which make struct device available late such as I2C.
- *
- * @dev_name: Result of dev_name() for the consumer.
- * @supply: Name for the supply.
- */
-struct regulator_consumer_supply {
-	const char *dev_name;   /* dev_name() for consumer */
-	const char *supply;	/* consumer supply - e.g. "vcc" */
-};
-
-/* Initialize struct regulator_consumer_supply */
-#define REGULATOR_SUPPLY(_name, _dev_name)			\
-{								\
-	.supply		= _name,				\
-	.dev_name	= _dev_name,				\
-}
-
-/**
- * struct regulator_init_data - regulator platform initialisation data.
- *
- * Initialisation constraints, our supply and consumers supplies.
- *
- * @supply_regulator: Parent regulator.  Specified using the regulator name
- *                    as it appears in the name field in sysfs, which can
- *                    be explicitly set using the constraints field 'name'.
- *
- * @constraints: Constraints.  These must be specified for the regulator to
- *               be usable.
- * @num_consumer_supplies: Number of consumer device supplies.
- * @consumer_supplies: Consumer device supply configuration.
- *
- * @regulator_init: Callback invoked when the regulator has been registered.
- * @driver_data: Data passed to regulator_init.
- */
-struct regulator_init_data {
-	const char *supply_regulator;        /* or NULL for system supply */
-
-	struct regulation_constraints constraints;
-
-	int num_consumer_supplies;
-	struct regulator_consumer_supply *consumer_supplies;
-
-	/* optional regulator machine specific init */
-	int (*regulator_init)(void *driver_data);
-	void *driver_data;	/* core does not touch this */
-};
-
-#ifdef CONFIG_REGULATOR
-void regulator_has_full_constraints(void);
-#else
-static inline void regulator_has_full_constraints(void)
-{
-}
-#endif
-
-#endif
diff --git a/include/linux/regulator/of_regulator.h b/include/linux/regulator/of_regulator.h
index 265b98d1eef3..de6d053e268f 100644
--- a/include/linux/regulator/of_regulator.h
+++ b/include/linux/regulator/of_regulator.h
@@ -14,28 +14,15 @@ struct regulator_desc;
 struct of_regulator_match {
 	const char *name;
 	void *driver_data;
-	struct regulator_init_data *init_data;
 	struct device_node *of_node;
 	const struct regulator_desc *desc;
 };
 
 #if defined(CONFIG_OFDEVICE)
-extern struct regulator_init_data
-	*of_get_regulator_init_data(struct device_d *dev,
-				    struct device_node *node,
-				    const struct regulator_desc *desc);
 extern int of_regulator_match(struct device_d *dev, struct device_node *node,
 			      struct of_regulator_match *matches,
 			      unsigned int num_matches);
 #else
-static inline struct regulator_init_data
-	*of_get_regulator_init_data(struct device_d *dev,
-				    struct device_node *node,
-				    const struct regulator_desc *desc)
-{
-	return NULL;
-}
-
 static inline int of_regulator_match(struct device_d *dev,
 				     struct device_node *node,
 				     struct of_regulator_match *matches,
-- 
2.29.2


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox


^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH] regulator: discard unused struct regulator_init_data handling
  2021-06-09  8:40 [PATCH] regulator: discard unused struct regulator_init_data handling Ahmad Fatoum
@ 2021-06-11 11:25 ` Sascha Hauer
  0 siblings, 0 replies; 2+ messages in thread
From: Sascha Hauer @ 2021-06-11 11:25 UTC (permalink / raw)
  To: Ahmad Fatoum; +Cc: barebox, Enrico Jorns

On Wed, Jun 09, 2021 at 10:40:27AM +0200, Ahmad Fatoum wrote:
> struct regulator_init_data and the contained regulation_constraints
> are populated from device tree, but unused anywhere. They are not
> required for the stpmic driver to operate, so they can just be dropped.
> 
> No functional change.
> 
> Tested-by: Enrico Jorns <ejo@pengutronix.de>
> Signed-off-by: Ahmad Fatoum <a.fatoum@pengutronix.de>
> ---
>  drivers/regulator/of_regulator.c       | 148 ------------------
>  include/linux/regulator/machine.h      | 207 -------------------------
>  include/linux/regulator/of_regulator.h |  13 --
>  3 files changed, 368 deletions(-)
>  delete mode 100644 include/linux/regulator/machine.h

Applied, thanks

Sascha

> 
> diff --git a/drivers/regulator/of_regulator.c b/drivers/regulator/of_regulator.c
> index 3e8caa87104e..c536a82c439a 100644
> --- a/drivers/regulator/of_regulator.c
> +++ b/drivers/regulator/of_regulator.c
> @@ -9,145 +9,6 @@
>  #include <common.h>
>  #include <of.h>
>  #include <linux/regulator/of_regulator.h>
> -#include <linux/regulator/machine.h>
> -
> -static int of_get_regulation_constraints(struct device_d *dev,
> -					struct device_node *np,
> -					struct regulator_init_data **init_data,
> -					const struct regulator_desc *desc)
> -{
> -	struct regulation_constraints *constraints = &(*init_data)->constraints;
> -	int ret;
> -	u32 pval;
> -
> -	constraints->name = of_get_property(np, "regulator-name", NULL);
> -
> -	if (!of_property_read_u32(np, "regulator-min-microvolt", &pval))
> -		constraints->min_uV = pval;
> -
> -	if (!of_property_read_u32(np, "regulator-max-microvolt", &pval))
> -		constraints->max_uV = pval;
> -
> -	/* Voltage change possible? */
> -	if (constraints->min_uV != constraints->max_uV)
> -		constraints->valid_ops_mask |= REGULATOR_CHANGE_VOLTAGE;
> -
> -	/* Do we have a voltage range, if so try to apply it? */
> -	if (constraints->min_uV && constraints->max_uV)
> -		constraints->apply_uV = true;
> -
> -	if (!of_property_read_u32(np, "regulator-microvolt-offset", &pval))
> -		constraints->uV_offset = pval;
> -	if (!of_property_read_u32(np, "regulator-min-microamp", &pval))
> -		constraints->min_uA = pval;
> -	if (!of_property_read_u32(np, "regulator-max-microamp", &pval))
> -		constraints->max_uA = pval;
> -
> -	if (!of_property_read_u32(np, "regulator-input-current-limit-microamp",
> -				  &pval))
> -		constraints->ilim_uA = pval;
> -
> -	/* Current change possible? */
> -	if (constraints->min_uA != constraints->max_uA)
> -		constraints->valid_ops_mask |= REGULATOR_CHANGE_CURRENT;
> -
> -	constraints->boot_on = of_property_read_bool(np, "regulator-boot-on");
> -	constraints->always_on = of_property_read_bool(np, "regulator-always-on");
> -	if (!constraints->always_on) /* status change should be possible. */
> -		constraints->valid_ops_mask |= REGULATOR_CHANGE_STATUS;
> -
> -	constraints->pull_down = of_property_read_bool(np, "regulator-pull-down");
> -
> -	if (of_property_read_bool(np, "regulator-allow-bypass"))
> -		constraints->valid_ops_mask |= REGULATOR_CHANGE_BYPASS;
> -
> -	if (of_property_read_bool(np, "regulator-allow-set-load"))
> -		constraints->valid_ops_mask |= REGULATOR_CHANGE_DRMS;
> -
> -	ret = of_property_read_u32(np, "regulator-ramp-delay", &pval);
> -	if (!ret) {
> -		if (pval)
> -			constraints->ramp_delay = pval;
> -		else
> -			constraints->ramp_disable = true;
> -	}
> -
> -	ret = of_property_read_u32(np, "regulator-settling-time-us", &pval);
> -	if (!ret)
> -		constraints->settling_time = pval;
> -
> -	ret = of_property_read_u32(np, "regulator-settling-time-up-us", &pval);
> -	if (!ret)
> -		constraints->settling_time_up = pval;
> -	if (constraints->settling_time_up && constraints->settling_time) {
> -		pr_warn("%pOFn: ambiguous configuration for settling time, ignoring 'regulator-settling-time-up-us'\n",
> -			np);
> -		constraints->settling_time_up = 0;
> -	}
> -
> -	ret = of_property_read_u32(np, "regulator-settling-time-down-us",
> -				   &pval);
> -	if (!ret)
> -		constraints->settling_time_down = pval;
> -	if (constraints->settling_time_down && constraints->settling_time) {
> -		pr_warn("%pOFn: ambiguous configuration for settling time, ignoring 'regulator-settling-time-down-us'\n",
> -			np);
> -		constraints->settling_time_down = 0;
> -	}
> -
> -	ret = of_property_read_u32(np, "regulator-enable-ramp-delay", &pval);
> -	if (!ret)
> -		constraints->enable_time = pval;
> -
> -	constraints->soft_start = of_property_read_bool(np,
> -					"regulator-soft-start");
> -	ret = of_property_read_u32(np, "regulator-active-discharge", &pval);
> -	if (!ret) {
> -		constraints->active_discharge =
> -				(pval) ? REGULATOR_ACTIVE_DISCHARGE_ENABLE :
> -					REGULATOR_ACTIVE_DISCHARGE_DISABLE;
> -	}
> -
> -	if (!of_property_read_u32(np, "regulator-system-load", &pval))
> -		constraints->system_load = pval;
> -
> -	if (!of_property_read_u32(np, "regulator-max-step-microvolt",
> -				  &pval))
> -		constraints->max_uV_step = pval;
> -
> -	constraints->over_current_protection = of_property_read_bool(np,
> -					"regulator-over-current-protection");
> -
> -	return 0;
> -}
> -
> -/**
> - * of_get_regulator_init_data - extract regulator_init_data structure info
> - * @dev: device requesting for regulator_init_data
> - * @node: regulator device node
> - * @desc: regulator description
> - *
> - * Populates regulator_init_data structure by extracting data from device
> - * tree node, returns a pointer to the populated structure or NULL if memory
> - * alloc fails.
> - */
> -struct regulator_init_data *of_get_regulator_init_data(struct device_d *dev,
> -					  struct device_node *node,
> -					  const struct regulator_desc *desc)
> -{
> -	struct regulator_init_data *init_data;
> -
> -	if (!node)
> -		return NULL;
> -
> -	init_data = xzalloc(sizeof(*init_data));
> -
> -	if (of_get_regulation_constraints(dev, node, &init_data, desc))
> -		return NULL;
> -
> -	return init_data;
> -}
> -EXPORT_SYMBOL_GPL(of_get_regulator_init_data);
>  
>  struct devm_of_regulator_matches {
>  	struct of_regulator_match *matches;
> @@ -192,7 +53,6 @@ int of_regulator_match(struct device_d *dev, struct device_node *node,
>  
>  	for (i = 0; i < num_matches; i++) {
>  		struct of_regulator_match *match = &matches[i];
> -		match->init_data = NULL;
>  		match->of_node = NULL;
>  	}
>  
> @@ -210,14 +70,6 @@ int of_regulator_match(struct device_d *dev, struct device_node *node,
>  			if (strcmp(match->name, name))
>  				continue;
>  
> -			match->init_data = of_get_regulator_init_data(dev, child,
> -								      match->desc);
> -			if (!match->init_data) {
> -				dev_err(dev,
> -					"failed to parse DT for regulator %pOFn\n",
> -					child);
> -				return -EINVAL;
> -			}
>  			match->of_node = child;
>  			count++;
>  			break;
> diff --git a/include/linux/regulator/machine.h b/include/linux/regulator/machine.h
> deleted file mode 100644
> index bb8bc7c4e0b6..000000000000
> --- a/include/linux/regulator/machine.h
> +++ /dev/null
> @@ -1,207 +0,0 @@
> -/* SPDX-License-Identifier: GPL-2.0-only */
> -/*
> - * machine.h -- SoC Regulator support, machine/board driver API.
> - *
> - * Copyright (C) 2007, 2008 Wolfson Microelectronics PLC.
> - *
> - * Author: Liam Girdwood <lrg@slimlogic.co.uk>
> - *
> - * Regulator Machine/Board Interface.
> - */
> -
> -#ifndef __LINUX_REGULATOR_MACHINE_H_
> -#define __LINUX_REGULATOR_MACHINE_H_
> -
> -#include <regulator.h>
> -
> -struct regulator;
> -
> -/*
> - * Regulator operation constraint flags. These flags are used to enable
> - * certain regulator operations and can be OR'ed together.
> - *
> - * VOLTAGE:  Regulator output voltage can be changed by software on this
> - *           board/machine.
> - * CURRENT:  Regulator output current can be changed by software on this
> - *           board/machine.
> - * MODE:     Regulator operating mode can be changed by software on this
> - *           board/machine.
> - * STATUS:   Regulator can be enabled and disabled.
> - * DRMS:     Dynamic Regulator Mode Switching is enabled for this regulator.
> - * BYPASS:   Regulator can be put into bypass mode
> - */
> -
> -#define REGULATOR_CHANGE_VOLTAGE	0x1
> -#define REGULATOR_CHANGE_CURRENT	0x2
> -#define REGULATOR_CHANGE_MODE		0x4
> -#define REGULATOR_CHANGE_STATUS		0x8
> -#define REGULATOR_CHANGE_DRMS		0x10
> -#define REGULATOR_CHANGE_BYPASS		0x20
> -
> -/* Regulator active discharge flags */
> -enum regulator_active_discharge {
> -	REGULATOR_ACTIVE_DISCHARGE_DEFAULT,
> -	REGULATOR_ACTIVE_DISCHARGE_DISABLE,
> -	REGULATOR_ACTIVE_DISCHARGE_ENABLE,
> -};
> -
> -/**
> - * struct regulation_constraints - regulator operating constraints.
> - *
> - * This struct describes regulator and board/machine specific constraints.
> - *
> - * @name: Descriptive name for the constraints, used for display purposes.
> - *
> - * @min_uV: Smallest voltage consumers may set.
> - * @max_uV: Largest voltage consumers may set.
> - * @uV_offset: Offset applied to voltages from consumer to compensate for
> - *             voltage drops.
> - *
> - * @min_uA: Smallest current consumers may set.
> - * @max_uA: Largest current consumers may set.
> - * @ilim_uA: Maximum input current.
> - * @system_load: Load that isn't captured by any consumer requests.
> - *
> - * @max_spread: Max possible spread between coupled regulators
> - * @valid_modes_mask: Mask of modes which may be configured by consumers.
> - * @valid_ops_mask: Operations which may be performed by consumers.
> - *
> - * @always_on: Set if the regulator should never be disabled.
> - * @boot_on: Set if the regulator is enabled when the system is initially
> - *           started.  If the regulator is not enabled by the hardware or
> - *           bootloader then it will be enabled when the constraints are
> - *           applied.
> - * @apply_uV: Apply the voltage constraint when initialising.
> - * @ramp_disable: Disable ramp delay when initialising or when setting voltage.
> - * @soft_start: Enable soft start so that voltage ramps slowly.
> - * @pull_down: Enable pull down when regulator is disabled.
> - * @over_current_protection: Auto disable on over current event.
> - *
> - * @input_uV: Input voltage for regulator when supplied by another regulator.
> - *
> - * @initial_mode: Mode to set at startup.
> - * @ramp_delay: Time to settle down after voltage change (unit: uV/us)
> - * @settling_time: Time to settle down after voltage change when voltage
> - *		   change is non-linear (unit: microseconds).
> - * @settling_time_up: Time to settle down after voltage increase when voltage
> - *		      change is non-linear (unit: microseconds).
> - * @settling_time_down : Time to settle down after voltage decrease when
> - *			 voltage change is non-linear (unit: microseconds).
> - * @active_discharge: Enable/disable active discharge. The enum
> - *		      regulator_active_discharge values are used for
> - *		      initialisation.
> - * @enable_time: Turn-on time of the rails (unit: microseconds)
> - */
> -struct regulation_constraints {
> -
> -	const char *name;
> -
> -	/* voltage output range (inclusive) - for voltage control */
> -	int min_uV;
> -	int max_uV;
> -
> -	int uV_offset;
> -
> -	/* current output range (inclusive) - for current control */
> -	int min_uA;
> -	int max_uA;
> -	int ilim_uA;
> -
> -	int system_load;
> -
> -	/* used for coupled regulators */
> -	u32 *max_spread;
> -
> -	/* used for changing voltage in steps */
> -	int max_uV_step;
> -
> -	/* valid regulator operating modes for this machine */
> -	unsigned int valid_modes_mask;
> -
> -	/* valid operations for regulator on this machine */
> -	unsigned int valid_ops_mask;
> -
> -	/* regulator input voltage - only if supply is another regulator */
> -	int input_uV;
> -
> -	/* mode to set on startup */
> -	unsigned int initial_mode;
> -
> -	unsigned int ramp_delay;
> -	unsigned int settling_time;
> -	unsigned int settling_time_up;
> -	unsigned int settling_time_down;
> -	unsigned int enable_time;
> -
> -	unsigned int active_discharge;
> -
> -	/* constraint flags */
> -	unsigned always_on:1;	/* regulator never off when system is on */
> -	unsigned boot_on:1;	/* bootloader/firmware enabled regulator */
> -	unsigned apply_uV:1;	/* apply uV constraint if min == max */
> -	unsigned ramp_disable:1; /* disable ramp delay */
> -	unsigned soft_start:1;	/* ramp voltage slowly */
> -	unsigned pull_down:1;	/* pull down resistor when regulator off */
> -	unsigned over_current_protection:1; /* auto disable on over current */
> -};
> -
> -/**
> - * struct regulator_consumer_supply - supply -> device mapping
> - *
> - * This maps a supply name to a device. Use of dev_name allows support for
> - * buses which make struct device available late such as I2C.
> - *
> - * @dev_name: Result of dev_name() for the consumer.
> - * @supply: Name for the supply.
> - */
> -struct regulator_consumer_supply {
> -	const char *dev_name;   /* dev_name() for consumer */
> -	const char *supply;	/* consumer supply - e.g. "vcc" */
> -};
> -
> -/* Initialize struct regulator_consumer_supply */
> -#define REGULATOR_SUPPLY(_name, _dev_name)			\
> -{								\
> -	.supply		= _name,				\
> -	.dev_name	= _dev_name,				\
> -}
> -
> -/**
> - * struct regulator_init_data - regulator platform initialisation data.
> - *
> - * Initialisation constraints, our supply and consumers supplies.
> - *
> - * @supply_regulator: Parent regulator.  Specified using the regulator name
> - *                    as it appears in the name field in sysfs, which can
> - *                    be explicitly set using the constraints field 'name'.
> - *
> - * @constraints: Constraints.  These must be specified for the regulator to
> - *               be usable.
> - * @num_consumer_supplies: Number of consumer device supplies.
> - * @consumer_supplies: Consumer device supply configuration.
> - *
> - * @regulator_init: Callback invoked when the regulator has been registered.
> - * @driver_data: Data passed to regulator_init.
> - */
> -struct regulator_init_data {
> -	const char *supply_regulator;        /* or NULL for system supply */
> -
> -	struct regulation_constraints constraints;
> -
> -	int num_consumer_supplies;
> -	struct regulator_consumer_supply *consumer_supplies;
> -
> -	/* optional regulator machine specific init */
> -	int (*regulator_init)(void *driver_data);
> -	void *driver_data;	/* core does not touch this */
> -};
> -
> -#ifdef CONFIG_REGULATOR
> -void regulator_has_full_constraints(void);
> -#else
> -static inline void regulator_has_full_constraints(void)
> -{
> -}
> -#endif
> -
> -#endif
> diff --git a/include/linux/regulator/of_regulator.h b/include/linux/regulator/of_regulator.h
> index 265b98d1eef3..de6d053e268f 100644
> --- a/include/linux/regulator/of_regulator.h
> +++ b/include/linux/regulator/of_regulator.h
> @@ -14,28 +14,15 @@ struct regulator_desc;
>  struct of_regulator_match {
>  	const char *name;
>  	void *driver_data;
> -	struct regulator_init_data *init_data;
>  	struct device_node *of_node;
>  	const struct regulator_desc *desc;
>  };
>  
>  #if defined(CONFIG_OFDEVICE)
> -extern struct regulator_init_data
> -	*of_get_regulator_init_data(struct device_d *dev,
> -				    struct device_node *node,
> -				    const struct regulator_desc *desc);
>  extern int of_regulator_match(struct device_d *dev, struct device_node *node,
>  			      struct of_regulator_match *matches,
>  			      unsigned int num_matches);
>  #else
> -static inline struct regulator_init_data
> -	*of_get_regulator_init_data(struct device_d *dev,
> -				    struct device_node *node,
> -				    const struct regulator_desc *desc)
> -{
> -	return NULL;
> -}
> -
>  static inline int of_regulator_match(struct device_d *dev,
>  				     struct device_node *node,
>  				     struct of_regulator_match *matches,
> -- 
> 2.29.2
> 
> 
> _______________________________________________
> barebox mailing list
> barebox@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/barebox
> 

-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2021-06-11 11:27 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-06-09  8:40 [PATCH] regulator: discard unused struct regulator_init_data handling Ahmad Fatoum
2021-06-11 11:25 ` Sascha Hauer

mail archive of the barebox mailing list

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://lore.barebox.org/barebox/0 barebox/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 barebox barebox/ https://lore.barebox.org/barebox \
		barebox@lists.infradead.org barebox@lists.infradead.org
	public-inbox-index barebox

Example config snippet for mirrors.


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git