Unverified Commit 7d34aec5 authored by Christoph Fritz's avatar Christoph Fritz Committed by Mark Brown
Browse files

regulator: da9062: refactor buck modes into header 



This patch refactors buck modes into a header file so that device trees
can make use of these mode constants.

The new header filename uses da9063 because DA9063 was the earlier chip
and its driver code will want updating at some point in a similar manner.

Signed-off-by: default avatarChristoph Fritz <chf.fritz@googlemail.com>
Link: https://lore.kernel.org/r/1573652416-9848-2-git-send-email-chf.fritz@googlemail.com


Signed-off-by: default avatarMark Brown <broonie@kernel.org>
parent 9ebde17c

drivers/regulator/da9062-regulator.c

+10 −18
Original line number Diff line number Diff line
@@ -16,6 +16,7 @@ 
#include <linux/regulator/of_regulator.h>

#include <linux/mfd/da9062/core.h>

#include <linux/mfd/da9062/registers.h>

#include <dt-bindings/regulator/dlg,da9063-regulator.h>



/* Regulator IDs */

enum {
@@ -75,14 +76,6 @@ struct da9062_regulators { 
	struct da9062_regulator			regulator[0];

};



/* BUCK modes */

enum {

	BUCK_MODE_MANUAL,	/* 0 */

	BUCK_MODE_SLEEP,	/* 1 */

	BUCK_MODE_SYNC,		/* 2 */

	BUCK_MODE_AUTO		/* 3 */

};



/* Regulator operations */



/* Current limits array (in uA)
@@ -112,13 +105,13 @@ static int da9062_buck_set_mode(struct regulator_dev *rdev, unsigned mode) 


	switch (mode) {

	case REGULATOR_MODE_FAST:

		val = BUCK_MODE_SYNC;

		val = DA9063_BUCK_MODE_SYNC;

		break;

	case REGULATOR_MODE_NORMAL:

		val = BUCK_MODE_AUTO;

		val = DA9063_BUCK_MODE_AUTO;

		break;

	case REGULATOR_MODE_STANDBY:

		val = BUCK_MODE_SLEEP;

		val = DA9063_BUCK_MODE_SLEEP;

		break;

	default:

		return -EINVAL;
@@ -145,14 +138,13 @@ static unsigned da9062_buck_get_mode(struct regulator_dev *rdev) 


	switch (val) {

	default:

	case BUCK_MODE_MANUAL:

		/* Sleep flag bit decides the mode */

		break;

	case BUCK_MODE_SLEEP:

	case DA9063_BUCK_MODE_SLEEP:

		return REGULATOR_MODE_STANDBY;

	case BUCK_MODE_SYNC:

	case DA9063_BUCK_MODE_SYNC:

		return REGULATOR_MODE_FAST;

	case BUCK_MODE_AUTO:

	case DA9063_BUCK_MODE_AUTO:

		return REGULATOR_MODE_NORMAL;

	}
@@ -279,13 +271,13 @@ static int da9062_buck_set_suspend_mode(struct regulator_dev *rdev, 


	switch (mode) {

	case REGULATOR_MODE_FAST:

		val = BUCK_MODE_SYNC;

		val = DA9063_BUCK_MODE_SYNC;

		break;

	case REGULATOR_MODE_NORMAL:

		val = BUCK_MODE_AUTO;

		val = DA9063_BUCK_MODE_AUTO;

		break;

	case REGULATOR_MODE_STANDBY:

		val = BUCK_MODE_SLEEP;

		val = DA9063_BUCK_MODE_SLEEP;

		break;

	default:

		return -EINVAL;

include/dt-bindings/regulator/dlg,da9063-regulator.h

0 → 100644
+16 −0
Original line number Diff line number Diff line 
/* SPDX-License-Identifier: GPL-2.0 */



#ifndef _DT_BINDINGS_REGULATOR_DLG_DA9063_H

#define _DT_BINDINGS_REGULATOR_DLG_DA9063_H



/*

 * These buck mode constants may be used to specify values in device tree

 * properties (e.g. regulator-initial-mode).

 * A description of the following modes is in the manufacturers datasheet.

 */



#define DA9063_BUCK_MODE_SLEEP		1

#define DA9063_BUCK_MODE_SYNC		2

#define DA9063_BUCK_MODE_AUTO		3



#endif

CRA Git | Maintained and supported by SUSTech CRA and CCSE