Commit 9adddb01 authored by Alexey Firago's avatar Alexey Firago Committed by Stephen Boyd

clk: vc5: Add structure to describe particular chip features

Introduce vc5_chip_info structure to describe features of a particular
VC5 chip (id, number of FODs, number of outputs, flags).
For now flags are only used to indicate if chip has internal XTAL.
vc5_chip_info is set on probe from the matched of_device_id->data.

Also add defines to specify maximum number of FODs and clock outputs
supported by the driver.

With these changes it should be easier to extend driver to support
more VC5 models.
Signed-off-by: default avatarAlexey Firago <alexey_firago@mentor.com>
Reviewed-by: default avatarMarek Vasut <marek.vasut@gmail.com>
Signed-off-by: default avatarStephen Boyd <sboyd@codeaurora.org>
parent 8062b4aa
...@@ -113,12 +113,30 @@ ...@@ -113,12 +113,30 @@
#define VC5_MUX_IN_XIN BIT(0) #define VC5_MUX_IN_XIN BIT(0)
#define VC5_MUX_IN_CLKIN BIT(1) #define VC5_MUX_IN_CLKIN BIT(1)
/* Maximum number of clk_out supported by this driver */
#define VC5_MAX_CLK_OUT_NUM 3
/* Maximum number of FODs supported by this driver */
#define VC5_MAX_FOD_NUM 2
/* flags to describe chip features */
/* chip has built-in oscilator */
#define VC5_HAS_INTERNAL_XTAL BIT(0)
/* Supported IDT VC5 models. */ /* Supported IDT VC5 models. */
enum vc5_model { enum vc5_model {
IDT_VC5_5P49V5923, IDT_VC5_5P49V5923,
IDT_VC5_5P49V5933, IDT_VC5_5P49V5933,
}; };
/* Structure to describe features of a particular VC5 model */
struct vc5_chip_info {
const enum vc5_model model;
const unsigned int clk_fod_cnt;
const unsigned int clk_out_cnt;
const u32 flags;
};
struct vc5_driver_data; struct vc5_driver_data;
struct vc5_hw_data { struct vc5_hw_data {
...@@ -132,15 +150,15 @@ struct vc5_hw_data { ...@@ -132,15 +150,15 @@ struct vc5_hw_data {
struct vc5_driver_data { struct vc5_driver_data {
struct i2c_client *client; struct i2c_client *client;
struct regmap *regmap; struct regmap *regmap;
enum vc5_model model; const struct vc5_chip_info *chip_info;
struct clk *pin_xin; struct clk *pin_xin;
struct clk *pin_clkin; struct clk *pin_clkin;
unsigned char clk_mux_ins; unsigned char clk_mux_ins;
struct clk_hw clk_mux; struct clk_hw clk_mux;
struct vc5_hw_data clk_pll; struct vc5_hw_data clk_pll;
struct vc5_hw_data clk_fod[2]; struct vc5_hw_data clk_fod[VC5_MAX_FOD_NUM];
struct vc5_hw_data clk_out[3]; struct vc5_hw_data clk_out[VC5_MAX_CLK_OUT_NUM];
}; };
static const char * const vc5_mux_names[] = { static const char * const vc5_mux_names[] = {
...@@ -563,7 +581,7 @@ static struct clk_hw *vc5_of_clk_get(struct of_phandle_args *clkspec, ...@@ -563,7 +581,7 @@ static struct clk_hw *vc5_of_clk_get(struct of_phandle_args *clkspec,
struct vc5_driver_data *vc5 = data; struct vc5_driver_data *vc5 = data;
unsigned int idx = clkspec->args[0]; unsigned int idx = clkspec->args[0];
if (idx > 2) if (idx >= vc5->chip_info->clk_out_cnt)
return ERR_PTR(-EINVAL); return ERR_PTR(-EINVAL);
return &vc5->clk_out[idx].hw; return &vc5->clk_out[idx].hw;
...@@ -586,12 +604,10 @@ static const struct of_device_id clk_vc5_of_match[]; ...@@ -586,12 +604,10 @@ static const struct of_device_id clk_vc5_of_match[];
static int vc5_probe(struct i2c_client *client, static int vc5_probe(struct i2c_client *client,
const struct i2c_device_id *id) const struct i2c_device_id *id)
{ {
const struct of_device_id *of_id =
of_match_device(clk_vc5_of_match, &client->dev);
struct vc5_driver_data *vc5; struct vc5_driver_data *vc5;
struct clk_init_data init; struct clk_init_data init;
const char *parent_names[2]; const char *parent_names[2];
unsigned int n, idx; unsigned int n, idx = 0;
int ret; int ret;
vc5 = devm_kzalloc(&client->dev, sizeof(*vc5), GFP_KERNEL); vc5 = devm_kzalloc(&client->dev, sizeof(*vc5), GFP_KERNEL);
...@@ -600,7 +616,7 @@ static int vc5_probe(struct i2c_client *client, ...@@ -600,7 +616,7 @@ static int vc5_probe(struct i2c_client *client,
i2c_set_clientdata(client, vc5); i2c_set_clientdata(client, vc5);
vc5->client = client; vc5->client = client;
vc5->model = (enum vc5_model)of_id->data; vc5->chip_info = of_device_get_match_data(&client->dev);
vc5->pin_xin = devm_clk_get(&client->dev, "xin"); vc5->pin_xin = devm_clk_get(&client->dev, "xin");
if (PTR_ERR(vc5->pin_xin) == -EPROBE_DEFER) if (PTR_ERR(vc5->pin_xin) == -EPROBE_DEFER)
...@@ -622,8 +638,7 @@ static int vc5_probe(struct i2c_client *client, ...@@ -622,8 +638,7 @@ static int vc5_probe(struct i2c_client *client,
if (!IS_ERR(vc5->pin_xin)) { if (!IS_ERR(vc5->pin_xin)) {
vc5->clk_mux_ins |= VC5_MUX_IN_XIN; vc5->clk_mux_ins |= VC5_MUX_IN_XIN;
parent_names[init.num_parents++] = __clk_get_name(vc5->pin_xin); parent_names[init.num_parents++] = __clk_get_name(vc5->pin_xin);
} else if (vc5->model == IDT_VC5_5P49V5933) { } else if (vc5->chip_info->flags & VC5_HAS_INTERNAL_XTAL) {
/* IDT VC5 5P49V5933 has built-in oscilator. */
vc5->pin_xin = clk_register_fixed_rate(&client->dev, vc5->pin_xin = clk_register_fixed_rate(&client->dev,
"internal-xtal", NULL, "internal-xtal", NULL,
0, 25000000); 0, 25000000);
...@@ -672,8 +687,8 @@ static int vc5_probe(struct i2c_client *client, ...@@ -672,8 +687,8 @@ static int vc5_probe(struct i2c_client *client,
} }
/* Register FODs */ /* Register FODs */
for (n = 0; n < 2; n++) { for (n = 0; n < vc5->chip_info->clk_fod_cnt; n++) {
idx = vc5_map_index_to_output(vc5->model, n); idx = vc5_map_index_to_output(vc5->chip_info->model, n);
memset(&init, 0, sizeof(init)); memset(&init, 0, sizeof(init));
init.name = vc5_fod_names[idx]; init.name = vc5_fod_names[idx];
init.ops = &vc5_fod_ops; init.ops = &vc5_fod_ops;
...@@ -709,8 +724,8 @@ static int vc5_probe(struct i2c_client *client, ...@@ -709,8 +724,8 @@ static int vc5_probe(struct i2c_client *client,
} }
/* Register FOD-connected OUTx outputs */ /* Register FOD-connected OUTx outputs */
for (n = 1; n < 3; n++) { for (n = 1; n < vc5->chip_info->clk_out_cnt; n++) {
idx = vc5_map_index_to_output(vc5->model, n - 1); idx = vc5_map_index_to_output(vc5->chip_info->model, n - 1);
parent_names[0] = vc5_fod_names[idx]; parent_names[0] = vc5_fod_names[idx];
if (n == 1) if (n == 1)
parent_names[1] = vc5_mux_names[0]; parent_names[1] = vc5_mux_names[0];
...@@ -744,7 +759,7 @@ static int vc5_probe(struct i2c_client *client, ...@@ -744,7 +759,7 @@ static int vc5_probe(struct i2c_client *client,
return 0; return 0;
err_clk: err_clk:
if (vc5->model == IDT_VC5_5P49V5933) if (vc5->chip_info->flags & VC5_HAS_INTERNAL_XTAL)
clk_unregister_fixed_rate(vc5->pin_xin); clk_unregister_fixed_rate(vc5->pin_xin);
return ret; return ret;
} }
...@@ -755,12 +770,26 @@ static int vc5_remove(struct i2c_client *client) ...@@ -755,12 +770,26 @@ static int vc5_remove(struct i2c_client *client)
of_clk_del_provider(client->dev.of_node); of_clk_del_provider(client->dev.of_node);
if (vc5->model == IDT_VC5_5P49V5933) if (vc5->chip_info->flags & VC5_HAS_INTERNAL_XTAL)
clk_unregister_fixed_rate(vc5->pin_xin); clk_unregister_fixed_rate(vc5->pin_xin);
return 0; return 0;
} }
static const struct vc5_chip_info idt_5p49v5923_info = {
.model = IDT_VC5_5P49V5923,
.clk_fod_cnt = 2,
.clk_out_cnt = 3,
.flags = 0,
};
static const struct vc5_chip_info idt_5p49v5933_info = {
.model = IDT_VC5_5P49V5933,
.clk_fod_cnt = 2,
.clk_out_cnt = 3,
.flags = VC5_HAS_INTERNAL_XTAL,
};
static const struct i2c_device_id vc5_id[] = { static const struct i2c_device_id vc5_id[] = {
{ "5p49v5923", .driver_data = IDT_VC5_5P49V5923 }, { "5p49v5923", .driver_data = IDT_VC5_5P49V5923 },
{ "5p49v5933", .driver_data = IDT_VC5_5P49V5933 }, { "5p49v5933", .driver_data = IDT_VC5_5P49V5933 },
...@@ -769,8 +798,8 @@ static const struct i2c_device_id vc5_id[] = { ...@@ -769,8 +798,8 @@ static const struct i2c_device_id vc5_id[] = {
MODULE_DEVICE_TABLE(i2c, vc5_id); MODULE_DEVICE_TABLE(i2c, vc5_id);
static const struct of_device_id clk_vc5_of_match[] = { static const struct of_device_id clk_vc5_of_match[] = {
{ .compatible = "idt,5p49v5923", .data = (void *)IDT_VC5_5P49V5923 }, { .compatible = "idt,5p49v5923", .data = &idt_5p49v5923_info },
{ .compatible = "idt,5p49v5933", .data = (void *)IDT_VC5_5P49V5933 }, { .compatible = "idt,5p49v5933", .data = &idt_5p49v5933_info },
{ }, { },
}; };
MODULE_DEVICE_TABLE(of, clk_vc5_of_match); MODULE_DEVICE_TABLE(of, clk_vc5_of_match);
......
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