mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
synced 2026-05-10 10:20:17 -04:00
media: i2c: imx258: Add regulator control
The device tree bindings define the relevant regulators for the sensor, so update the driver to request the regulators and control them at the appropriate times. Signed-off-by: Dave Stevenson <dave.stevenson@raspberrypi.com> Signed-off-by: Luis Garcia <git@luigi311.com> Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com> Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
This commit is contained in:
committed by
Hans Verkuil
parent
04392d73ae
commit
850b8acde0
@@ -7,6 +7,7 @@
|
||||
#include <linux/i2c.h>
|
||||
#include <linux/module.h>
|
||||
#include <linux/pm_runtime.h>
|
||||
#include <linux/regulator/consumer.h>
|
||||
#include <media/v4l2-ctrls.h>
|
||||
#include <media/v4l2-device.h>
|
||||
#include <media/v4l2-fwnode.h>
|
||||
@@ -507,6 +508,16 @@ static const char * const imx258_test_pattern_menu[] = {
|
||||
"Pseudorandom Sequence (PN9)",
|
||||
};
|
||||
|
||||
/* regulator supplies */
|
||||
static const char * const imx258_supply_name[] = {
|
||||
/* Supplies can be enabled in any order */
|
||||
"vana", /* Analog (2.8V) supply */
|
||||
"vdig", /* Digital Core (1.2V) supply */
|
||||
"vif", /* IF (1.8V) supply */
|
||||
};
|
||||
|
||||
#define IMX258_NUM_SUPPLIES ARRAY_SIZE(imx258_supply_name)
|
||||
|
||||
/* Configurations for supported link frequencies */
|
||||
#define IMX258_LINK_FREQ_634MHZ 633600000ULL
|
||||
#define IMX258_LINK_FREQ_320MHZ 320000000ULL
|
||||
@@ -611,6 +622,7 @@ struct imx258 {
|
||||
struct mutex mutex;
|
||||
|
||||
struct clk *clk;
|
||||
struct regulator_bulk_data supplies[IMX258_NUM_SUPPLIES];
|
||||
};
|
||||
|
||||
static inline struct imx258 *to_imx258(struct v4l2_subdev *_sd)
|
||||
@@ -995,9 +1007,19 @@ static int imx258_power_on(struct device *dev)
|
||||
struct imx258 *imx258 = to_imx258(sd);
|
||||
int ret;
|
||||
|
||||
ret = regulator_bulk_enable(IMX258_NUM_SUPPLIES,
|
||||
imx258->supplies);
|
||||
if (ret) {
|
||||
dev_err(dev, "%s: failed to enable regulators\n",
|
||||
__func__);
|
||||
return ret;
|
||||
}
|
||||
|
||||
ret = clk_prepare_enable(imx258->clk);
|
||||
if (ret)
|
||||
if (ret) {
|
||||
dev_err(dev, "failed to enable clock\n");
|
||||
regulator_bulk_disable(IMX258_NUM_SUPPLIES, imx258->supplies);
|
||||
}
|
||||
|
||||
return ret;
|
||||
}
|
||||
@@ -1008,6 +1030,7 @@ static int imx258_power_off(struct device *dev)
|
||||
struct imx258 *imx258 = to_imx258(sd);
|
||||
|
||||
clk_disable_unprepare(imx258->clk);
|
||||
regulator_bulk_disable(IMX258_NUM_SUPPLIES, imx258->supplies);
|
||||
|
||||
return 0;
|
||||
}
|
||||
@@ -1220,6 +1243,18 @@ static void imx258_free_controls(struct imx258 *imx258)
|
||||
mutex_destroy(&imx258->mutex);
|
||||
}
|
||||
|
||||
static int imx258_get_regulators(struct imx258 *imx258,
|
||||
struct i2c_client *client)
|
||||
{
|
||||
unsigned int i;
|
||||
|
||||
for (i = 0; i < IMX258_NUM_SUPPLIES; i++)
|
||||
imx258->supplies[i].supply = imx258_supply_name[i];
|
||||
|
||||
return devm_regulator_bulk_get(&client->dev,
|
||||
IMX258_NUM_SUPPLIES, imx258->supplies);
|
||||
}
|
||||
|
||||
static int imx258_probe(struct i2c_client *client)
|
||||
{
|
||||
struct imx258 *imx258;
|
||||
@@ -1230,6 +1265,11 @@ static int imx258_probe(struct i2c_client *client)
|
||||
if (!imx258)
|
||||
return -ENOMEM;
|
||||
|
||||
ret = imx258_get_regulators(imx258, client);
|
||||
if (ret)
|
||||
return dev_err_probe(&client->dev, ret,
|
||||
"failed to get regulators\n");
|
||||
|
||||
imx258->clk = devm_clk_get_optional(&client->dev, NULL);
|
||||
if (IS_ERR(imx258->clk))
|
||||
return dev_err_probe(&client->dev, PTR_ERR(imx258->clk),
|
||||
|
||||
Reference in New Issue
Block a user