Skip to content

Commit

Permalink
i2c: designware: Make HW init functions static
Browse files Browse the repository at this point in the history
Recent i2c-designware slave support patches use master or slave HW init
functions through the function pointer so we can declare them static.

While at it, rename i2c_dw_init() as i2c_dw_init_master().

Signed-off-by: Jarkko Nikula <jarkko.nikula@linux.intel.com>
Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Tested-by: Luis Oliveira <lolivei@synopsys.com>
Signed-off-by: Wolfram Sang <wsa@the-dreams.de>
  • Loading branch information
Jarkko Nikula authored and Wolfram Sang committed Jul 4, 2017
1 parent 9809cb8 commit 21bf440
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 8 deletions.
2 changes: 0 additions & 2 deletions drivers/i2c/busses/i2c-designware-core.h
Original file line number Diff line number Diff line change
Expand Up @@ -306,8 +306,6 @@ int i2c_dw_handle_tx_abort(struct dw_i2c_dev *dev);
u32 i2c_dw_func(struct i2c_adapter *adap);
void i2c_dw_disable(struct dw_i2c_dev *dev);
void i2c_dw_disable_int(struct dw_i2c_dev *dev);
int i2c_dw_init(struct dw_i2c_dev *dev);
int i2c_dw_init_slave(struct dw_i2c_dev *dev);

extern u32 i2c_dw_read_comp_param(struct dw_i2c_dev *dev);
extern int i2c_dw_probe(struct dw_i2c_dev *dev);
Expand Down
7 changes: 3 additions & 4 deletions drivers/i2c/busses/i2c-designware-master.c
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@ static void i2c_dw_configure_fifo_master(struct dw_i2c_dev *dev)
* This function is called during I2C init function, and in case of timeout at
* run time.
*/
int i2c_dw_init(struct dw_i2c_dev *dev)
static int i2c_dw_init_master(struct dw_i2c_dev *dev)
{
u32 hcnt, lcnt;
u32 reg, comp_param1;
Expand Down Expand Up @@ -171,7 +171,6 @@ int i2c_dw_init(struct dw_i2c_dev *dev)

return 0;
}
EXPORT_SYMBOL_GPL(i2c_dw_init);

static void i2c_dw_xfer_init(struct dw_i2c_dev *dev)
{
Expand Down Expand Up @@ -444,7 +443,7 @@ i2c_dw_xfer(struct i2c_adapter *adap, struct i2c_msg msgs[], int num)
if (!wait_for_completion_timeout(&dev->cmd_complete, adap->timeout)) {
dev_err(dev->dev, "controller timed out\n");
/* i2c_dw_init implicitly disables the adapter */
i2c_dw_init(dev);
i2c_dw_init_master(dev);
ret = -ETIMEDOUT;
goto done;
}
Expand Down Expand Up @@ -622,7 +621,7 @@ int i2c_dw_probe(struct dw_i2c_dev *dev)

init_completion(&dev->cmd_complete);

dev->init = i2c_dw_init;
dev->init = i2c_dw_init_master;
dev->disable = i2c_dw_disable;
dev->disable_int = i2c_dw_disable_int;

Expand Down
3 changes: 1 addition & 2 deletions drivers/i2c/busses/i2c-designware-slave.c
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ static void i2c_dw_configure_fifo_slave(struct dw_i2c_dev *dev)
* This function is called during I2C init function, and in case of timeout at
* run time.
*/
int i2c_dw_init_slave(struct dw_i2c_dev *dev)
static int i2c_dw_init_slave(struct dw_i2c_dev *dev)
{
u32 sda_falling_time, scl_falling_time;
u32 reg, comp_param1;
Expand Down Expand Up @@ -168,7 +168,6 @@ int i2c_dw_init_slave(struct dw_i2c_dev *dev)

return 0;
}
EXPORT_SYMBOL_GPL(i2c_dw_init_slave);

static int i2c_dw_reg_slave(struct i2c_client *slave)
{
Expand Down

0 comments on commit 21bf440

Please sign in to comment.