Commit 52b461b8 authored by Mark Brown's avatar Mark Brown Committed by Samuel Ortiz

mfd: Add regmap cache support for wm8350

Use the most simple possible transformation on the existing code so keep
the table sitting around, further patches in this series will delete the
existing cache code - the main purpose of this patch is to ensure that
we always have a cache for bisection.
Signed-off-by: default avatarMark Brown <broonie@opensource.wolfsonmicro.com>
Signed-off-by: default avatarSamuel Ortiz <sameo@linux.intel.com>
parent b0ab907d
...@@ -32,9 +32,6 @@ ...@@ -32,9 +32,6 @@
#include <linux/mfd/wm8350/supply.h> #include <linux/mfd/wm8350/supply.h>
#include <linux/mfd/wm8350/wdt.h> #include <linux/mfd/wm8350/wdt.h>
#define WM8350_UNLOCK_KEY 0x0013
#define WM8350_LOCK_KEY 0x0000
#define WM8350_CLOCK_CONTROL_1 0x28 #define WM8350_CLOCK_CONTROL_1 0x28
#define WM8350_AIF_TEST 0x74 #define WM8350_AIF_TEST 0x74
...@@ -295,15 +292,20 @@ EXPORT_SYMBOL_GPL(wm8350_block_write); ...@@ -295,15 +292,20 @@ EXPORT_SYMBOL_GPL(wm8350_block_write);
*/ */
int wm8350_reg_lock(struct wm8350 *wm8350) int wm8350_reg_lock(struct wm8350 *wm8350)
{ {
u16 key = WM8350_LOCK_KEY;
int ret; int ret;
mutex_lock(&reg_lock_mutex);
ldbg(__func__); ldbg(__func__);
mutex_lock(&io_mutex);
ret = wm8350_write(wm8350, WM8350_SECURITY, 1, &key); ret = wm8350_reg_write(wm8350, WM8350_SECURITY, WM8350_LOCK_KEY);
if (ret) if (ret)
dev_err(wm8350->dev, "lock failed\n"); dev_err(wm8350->dev, "lock failed\n");
mutex_unlock(&io_mutex);
wm8350->unlocked = false;
mutex_unlock(&reg_lock_mutex);
return ret; return ret;
} }
EXPORT_SYMBOL_GPL(wm8350_reg_lock); EXPORT_SYMBOL_GPL(wm8350_reg_lock);
...@@ -319,15 +321,20 @@ EXPORT_SYMBOL_GPL(wm8350_reg_lock); ...@@ -319,15 +321,20 @@ EXPORT_SYMBOL_GPL(wm8350_reg_lock);
*/ */
int wm8350_reg_unlock(struct wm8350 *wm8350) int wm8350_reg_unlock(struct wm8350 *wm8350)
{ {
u16 key = WM8350_UNLOCK_KEY;
int ret; int ret;
mutex_lock(&reg_lock_mutex);
ldbg(__func__); ldbg(__func__);
mutex_lock(&io_mutex);
ret = wm8350_write(wm8350, WM8350_SECURITY, 1, &key); ret = wm8350_reg_write(wm8350, WM8350_SECURITY, WM8350_UNLOCK_KEY);
if (ret) if (ret)
dev_err(wm8350->dev, "unlock failed\n"); dev_err(wm8350->dev, "unlock failed\n");
mutex_unlock(&io_mutex);
wm8350->unlocked = true;
mutex_unlock(&reg_lock_mutex);
return ret; return ret;
} }
EXPORT_SYMBOL_GPL(wm8350_reg_unlock); EXPORT_SYMBOL_GPL(wm8350_reg_unlock);
......
...@@ -23,11 +23,6 @@ ...@@ -23,11 +23,6 @@
#include <linux/regmap.h> #include <linux/regmap.h>
#include <linux/slab.h> #include <linux/slab.h>
static const struct regmap_config wm8350_regmap = {
.reg_bits = 8,
.val_bits = 16,
};
static int wm8350_i2c_probe(struct i2c_client *i2c, static int wm8350_i2c_probe(struct i2c_client *i2c,
const struct i2c_device_id *id) const struct i2c_device_id *id)
{ {
......
...@@ -3433,3 +3433,59 @@ const struct wm8350_reg_access wm8350_reg_io_map[] = { ...@@ -3433,3 +3433,59 @@ const struct wm8350_reg_access wm8350_reg_io_map[] = {
{ 0x0000, 0x0000, 0x0000 }, /* R254 */ { 0x0000, 0x0000, 0x0000 }, /* R254 */
{ 0x0000, 0x0000, 0x0000 }, /* R255 */ { 0x0000, 0x0000, 0x0000 }, /* R255 */
}; };
static bool wm8350_readable(struct device *dev, unsigned int reg)
{
return wm8350_reg_io_map[reg].readable;
}
static bool wm8350_writeable(struct device *dev, unsigned int reg)
{
struct wm8350 *wm8350 = dev_get_drvdata(dev);
if (!wm8350->unlocked) {
if ((reg >= WM8350_GPIO_FUNCTION_SELECT_1 &&
reg <= WM8350_GPIO_FUNCTION_SELECT_4) ||
(reg >= WM8350_BATTERY_CHARGER_CONTROL_1 &&
reg <= WM8350_BATTERY_CHARGER_CONTROL_3))
return false;
}
return wm8350_reg_io_map[reg].writable;
}
static bool wm8350_volatile(struct device *dev, unsigned int reg)
{
return wm8350_reg_io_map[reg].vol;
}
static bool wm8350_precious(struct device *dev, unsigned int reg)
{
switch (reg) {
case WM8350_SYSTEM_INTERRUPTS:
case WM8350_INT_STATUS_1:
case WM8350_INT_STATUS_2:
case WM8350_POWER_UP_INT_STATUS:
case WM8350_UNDER_VOLTAGE_INT_STATUS:
case WM8350_OVER_CURRENT_INT_STATUS:
case WM8350_GPIO_INT_STATUS:
case WM8350_COMPARATOR_INT_STATUS:
return true;
default:
return false;
}
}
const struct regmap_config wm8350_regmap = {
.reg_bits = 8,
.val_bits = 16,
.cache_type = REGCACHE_RBTREE,
.max_register = WM8350_MAX_REGISTER,
.readable_reg = wm8350_readable,
.writeable_reg = wm8350_writeable,
.volatile_reg = wm8350_volatile,
.precious_reg = wm8350_precious,
};
...@@ -17,6 +17,7 @@ ...@@ -17,6 +17,7 @@
#include <linux/mutex.h> #include <linux/mutex.h>
#include <linux/interrupt.h> #include <linux/interrupt.h>
#include <linux/completion.h> #include <linux/completion.h>
#include <linux/regmap.h>
#include <linux/mfd/wm8350/audio.h> #include <linux/mfd/wm8350/audio.h>
#include <linux/mfd/wm8350/gpio.h> #include <linux/mfd/wm8350/gpio.h>
...@@ -66,6 +67,9 @@ ...@@ -66,6 +67,9 @@
#define WM8350_MAX_REGISTER 0xFF #define WM8350_MAX_REGISTER 0xFF
#define WM8350_UNLOCK_KEY 0x0013
#define WM8350_LOCK_KEY 0x0000
/* /*
* Field Definitions. * Field Definitions.
*/ */
...@@ -582,6 +586,7 @@ ...@@ -582,6 +586,7 @@
#define WM8350_NUM_IRQ_REGS 7 #define WM8350_NUM_IRQ_REGS 7
extern const struct regmap_config wm8350_regmap;
struct wm8350_reg_access { struct wm8350_reg_access {
u16 readable; /* Mask of readable bits */ u16 readable; /* Mask of readable bits */
u16 writable; /* Mask of writable bits */ u16 writable; /* Mask of writable bits */
...@@ -602,7 +607,6 @@ extern const u16 wm8352_mode2_defaults[]; ...@@ -602,7 +607,6 @@ extern const u16 wm8352_mode2_defaults[];
extern const u16 wm8352_mode3_defaults[]; extern const u16 wm8352_mode3_defaults[];
struct wm8350; struct wm8350;
struct regmap;
struct wm8350_hwmon { struct wm8350_hwmon {
struct platform_device *pdev; struct platform_device *pdev;
...@@ -615,6 +619,7 @@ struct wm8350 { ...@@ -615,6 +619,7 @@ struct wm8350 {
/* device IO */ /* device IO */
struct regmap *regmap; struct regmap *regmap;
u16 *reg_cache; u16 *reg_cache;
bool unlocked;
struct mutex auxadc_mutex; struct mutex auxadc_mutex;
struct completion auxadc_done; struct completion auxadc_done;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment