i2c: tegra: Fix missing blank lines after declarations

Checkpatch warns about missing blank lines after declarations in the
Tegra I2C driver and so fix these.

Note that the initialisation of 'val' to zero in tegra_dvc_init() is
unnecessary and so remove this.

Signed-off-by: Jon Hunter <jonathanh@nvidia.com>
Signed-off-by: Wolfram Sang <wsa@the-dreams.de>
This commit is contained in:
Jon Hunter 2016-08-26 14:08:59 +01:00 committed by Wolfram Sang
parent 2929be29e0
commit f5076685b3

View file

@ -245,15 +245,17 @@ static void i2c_readsl(struct tegra_i2c_dev *i2c_dev, void *data,
static void tegra_i2c_mask_irq(struct tegra_i2c_dev *i2c_dev, u32 mask) static void tegra_i2c_mask_irq(struct tegra_i2c_dev *i2c_dev, u32 mask)
{ {
u32 int_mask = i2c_readl(i2c_dev, I2C_INT_MASK); u32 int_mask;
int_mask &= ~mask;
int_mask = i2c_readl(i2c_dev, I2C_INT_MASK) & ~mask;
i2c_writel(i2c_dev, int_mask, I2C_INT_MASK); i2c_writel(i2c_dev, int_mask, I2C_INT_MASK);
} }
static void tegra_i2c_unmask_irq(struct tegra_i2c_dev *i2c_dev, u32 mask) static void tegra_i2c_unmask_irq(struct tegra_i2c_dev *i2c_dev, u32 mask)
{ {
u32 int_mask = i2c_readl(i2c_dev, I2C_INT_MASK); u32 int_mask;
int_mask |= mask;
int_mask = i2c_readl(i2c_dev, I2C_INT_MASK) | mask;
i2c_writel(i2c_dev, int_mask, I2C_INT_MASK); i2c_writel(i2c_dev, int_mask, I2C_INT_MASK);
} }
@ -261,6 +263,7 @@ static int tegra_i2c_flush_fifos(struct tegra_i2c_dev *i2c_dev)
{ {
unsigned long timeout = jiffies + HZ; unsigned long timeout = jiffies + HZ;
u32 val = i2c_readl(i2c_dev, I2C_FIFO_CONTROL); u32 val = i2c_readl(i2c_dev, I2C_FIFO_CONTROL);
val |= I2C_FIFO_CONTROL_TX_FLUSH | I2C_FIFO_CONTROL_RX_FLUSH; val |= I2C_FIFO_CONTROL_TX_FLUSH | I2C_FIFO_CONTROL_RX_FLUSH;
i2c_writel(i2c_dev, val, I2C_FIFO_CONTROL); i2c_writel(i2c_dev, val, I2C_FIFO_CONTROL);
@ -386,7 +389,8 @@ static int tegra_i2c_fill_tx_fifo(struct tegra_i2c_dev *i2c_dev)
*/ */
static void tegra_dvc_init(struct tegra_i2c_dev *i2c_dev) static void tegra_dvc_init(struct tegra_i2c_dev *i2c_dev)
{ {
u32 val = 0; u32 val;
val = dvc_readl(i2c_dev, DVC_CTRL_REG3); val = dvc_readl(i2c_dev, DVC_CTRL_REG3);
val |= DVC_CTRL_REG3_SW_PROG; val |= DVC_CTRL_REG3_SW_PROG;
val |= DVC_CTRL_REG3_I2C_DONE_INTR_EN; val |= DVC_CTRL_REG3_I2C_DONE_INTR_EN;
@ -400,6 +404,7 @@ static void tegra_dvc_init(struct tegra_i2c_dev *i2c_dev)
static inline int tegra_i2c_clock_enable(struct tegra_i2c_dev *i2c_dev) static inline int tegra_i2c_clock_enable(struct tegra_i2c_dev *i2c_dev)
{ {
int ret; int ret;
if (!i2c_dev->hw->has_single_clk_source) { if (!i2c_dev->hw->has_single_clk_source) {
ret = clk_enable(i2c_dev->fast_clk); ret = clk_enable(i2c_dev->fast_clk);
if (ret < 0) { if (ret < 0) {
@ -461,11 +466,11 @@ static int tegra_i2c_init(struct tegra_i2c_dev *i2c_dev)
if (!i2c_dev->is_dvc) { if (!i2c_dev->is_dvc) {
u32 sl_cfg = i2c_readl(i2c_dev, I2C_SL_CNFG); u32 sl_cfg = i2c_readl(i2c_dev, I2C_SL_CNFG);
sl_cfg |= I2C_SL_CNFG_NACK | I2C_SL_CNFG_NEWSL; sl_cfg |= I2C_SL_CNFG_NACK | I2C_SL_CNFG_NEWSL;
i2c_writel(i2c_dev, sl_cfg, I2C_SL_CNFG); i2c_writel(i2c_dev, sl_cfg, I2C_SL_CNFG);
i2c_writel(i2c_dev, 0xfc, I2C_SL_ADDR1); i2c_writel(i2c_dev, 0xfc, I2C_SL_ADDR1);
i2c_writel(i2c_dev, 0x00, I2C_SL_ADDR2); i2c_writel(i2c_dev, 0x00, I2C_SL_ADDR2);
} }
val = 7 << I2C_FIFO_CONTROL_TX_TRIG_SHIFT | val = 7 << I2C_FIFO_CONTROL_TX_TRIG_SHIFT |
@ -680,6 +685,7 @@ static int tegra_i2c_xfer(struct i2c_adapter *adap, struct i2c_msg msgs[],
for (i = 0; i < num; i++) { for (i = 0; i < num; i++) {
enum msg_end_type end_type = MSG_END_STOP; enum msg_end_type end_type = MSG_END_STOP;
if (i < (num - 1)) { if (i < (num - 1)) {
if (msgs[i + 1].flags & I2C_M_NOSTART) if (msgs[i + 1].flags & I2C_M_NOSTART)
end_type = MSG_END_CONTINUE; end_type = MSG_END_CONTINUE;
@ -956,6 +962,7 @@ static int tegra_i2c_probe(struct platform_device *pdev)
static int tegra_i2c_remove(struct platform_device *pdev) static int tegra_i2c_remove(struct platform_device *pdev)
{ {
struct tegra_i2c_dev *i2c_dev = platform_get_drvdata(pdev); struct tegra_i2c_dev *i2c_dev = platform_get_drvdata(pdev);
i2c_del_adapter(&i2c_dev->adapter); i2c_del_adapter(&i2c_dev->adapter);
if (i2c_dev->is_multimaster_mode) if (i2c_dev->is_multimaster_mode)