Skip to content

Commit

Permalink
i2c: bcm2835: Ensure clock exists when probing
Browse files Browse the repository at this point in the history
Commit 9de93b0 upstream.

Probe function fails to recognize that upstream clock actually
doesn't yet exist because clock driver has not been initialized.
Actually try to go get the clock and test for its existence
before trying to set up a downstream clock based upon it.

This fixes a bug that causes the i2c driver not to work with
monolithic kernels.

Fixes: bebff81 ("i2c: bcm2835: Model Divider in CCF")
Signed-off-by: Annaliese McDermond <nh6z@nh6z.net>
Acked-by: Stefan Wahren <wahrenst@gmx.net>
Signed-off-by: Wolfram Sang <wsa@the-dreams.de>
  • Loading branch information
mcdermj authored and pelwell committed Jul 5, 2019
1 parent b689c4c commit 8ea4810
Showing 1 changed file with 12 additions and 4 deletions.
16 changes: 12 additions & 4 deletions drivers/i2c/busses/i2c-bcm2835.c
Original file line number Diff line number Diff line change
Expand Up @@ -244,15 +244,18 @@ static const struct clk_ops clk_bcm2835_i2c_ops = {
};

static struct clk *bcm2835_i2c_register_div(struct device *dev,
const char *mclk_name,
struct clk *mclk,
struct bcm2835_i2c_dev *i2c_dev)
{
struct clk_init_data init;
struct clk_bcm2835_i2c *priv;
char name[32];
const char *mclk_name;

snprintf(name, sizeof(name), "%s_div", dev_name(dev));

mclk_name = __clk_get_name(mclk);

init.ops = &clk_bcm2835_i2c_ops;
init.name = name;
init.parent_names = (const char* []) { mclk_name };
Expand Down Expand Up @@ -505,8 +508,8 @@ static int bcm2835_i2c_probe(struct platform_device *pdev)
struct resource *mem, *irq;
int ret;
struct i2c_adapter *adap;
const char *mclk_name;
struct clk *bus_clk;
struct clk *mclk;
u32 bus_clk_rate;

i2c_dev = devm_kzalloc(&pdev->dev, sizeof(*i2c_dev), GFP_KERNEL);
Expand All @@ -521,9 +524,14 @@ static int bcm2835_i2c_probe(struct platform_device *pdev)
if (IS_ERR(i2c_dev->regs))
return PTR_ERR(i2c_dev->regs);

mclk_name = of_clk_get_parent_name(pdev->dev.of_node, 0);
mclk = devm_clk_get(&pdev->dev, NULL);
if (IS_ERR(mclk)) {
if (PTR_ERR(mclk) != -EPROBE_DEFER)
dev_err(&pdev->dev, "Could not get clock\n");
return PTR_ERR(mclk);
}

bus_clk = bcm2835_i2c_register_div(&pdev->dev, mclk_name, i2c_dev);
bus_clk = bcm2835_i2c_register_div(&pdev->dev, mclk, i2c_dev);

if (IS_ERR(bus_clk)) {
dev_err(&pdev->dev, "Could not register clock\n");
Expand Down

0 comments on commit 8ea4810

Please sign in to comment.