diff options
author | Philipp Zabel <p.zabel@pengutronix.de> | 2015-01-22 10:17:29 -0500 |
---|---|---|
committer | Wolfram Sang <wsa@the-dreams.de> | 2015-01-22 10:34:39 -0500 |
commit | 4e355f5128ff5f340b5de49015d4a25d9c6b38c4 (patch) | |
tree | ba96bb59a2e50747a9241e5183b1f90a0c0e3964 /drivers/i2c/busses/i2c-imx.c | |
parent | a5eb71b207ecefac07bc36fd50b43d83c047b0ce (diff) |
i2c: imx: whitespace and checkpatch cleanup
This patch fixes up some whitespace issues and addresses a few
checkpatch warnings.
Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>
Acked-by: Fugang Duan <B38611@freescale.com>
Signed-off-by: Wolfram Sang <wsa@the-dreams.de>
Diffstat (limited to 'drivers/i2c/busses/i2c-imx.c')
-rw-r--r-- | drivers/i2c/busses/i2c-imx.c | 23 |
1 files changed, 13 insertions, 10 deletions
diff --git a/drivers/i2c/busses/i2c-imx.c b/drivers/i2c/busses/i2c-imx.c index cb7c4b28b624..d7b26fc6f432 100644 --- a/drivers/i2c/busses/i2c-imx.c +++ b/drivers/i2c/busses/i2c-imx.c | |||
@@ -201,7 +201,7 @@ struct imx_i2c_struct { | |||
201 | void __iomem *base; | 201 | void __iomem *base; |
202 | wait_queue_head_t queue; | 202 | wait_queue_head_t queue; |
203 | unsigned long i2csr; | 203 | unsigned long i2csr; |
204 | unsigned int disable_delay; | 204 | unsigned int disable_delay; |
205 | int stopped; | 205 | int stopped; |
206 | unsigned int ifdr; /* IMX_I2C_IFDR */ | 206 | unsigned int ifdr; /* IMX_I2C_IFDR */ |
207 | unsigned int cur_clk; | 207 | unsigned int cur_clk; |
@@ -479,8 +479,8 @@ static void i2c_imx_set_clk(struct imx_i2c_struct *i2c_imx) | |||
479 | i2c_clk_rate = clk_get_rate(i2c_imx->clk); | 479 | i2c_clk_rate = clk_get_rate(i2c_imx->clk); |
480 | if (i2c_imx->cur_clk == i2c_clk_rate) | 480 | if (i2c_imx->cur_clk == i2c_clk_rate) |
481 | return; | 481 | return; |
482 | else | 482 | |
483 | i2c_imx->cur_clk = i2c_clk_rate; | 483 | i2c_imx->cur_clk = i2c_clk_rate; |
484 | 484 | ||
485 | div = (i2c_clk_rate + i2c_imx->bitrate - 1) / i2c_imx->bitrate; | 485 | div = (i2c_clk_rate + i2c_imx->bitrate - 1) / i2c_imx->bitrate; |
486 | if (div < i2c_clk_div[0].div) | 486 | if (div < i2c_clk_div[0].div) |
@@ -488,7 +488,8 @@ static void i2c_imx_set_clk(struct imx_i2c_struct *i2c_imx) | |||
488 | else if (div > i2c_clk_div[i2c_imx->hwdata->ndivs - 1].div) | 488 | else if (div > i2c_clk_div[i2c_imx->hwdata->ndivs - 1].div) |
489 | i = i2c_imx->hwdata->ndivs - 1; | 489 | i = i2c_imx->hwdata->ndivs - 1; |
490 | else | 490 | else |
491 | for (i = 0; i2c_clk_div[i].div < div; i++); | 491 | for (i = 0; i2c_clk_div[i].div < div; i++) |
492 | ; | ||
492 | 493 | ||
493 | /* Store divider value */ | 494 | /* Store divider value */ |
494 | i2c_imx->ifdr = i2c_clk_div[i].val; | 495 | i2c_imx->ifdr = i2c_clk_div[i].val; |
@@ -820,6 +821,7 @@ static int i2c_imx_read(struct imx_i2c_struct *i2c_imx, struct i2c_msg *msgs, bo | |||
820 | /* read data */ | 821 | /* read data */ |
821 | for (i = 0; i < msgs->len; i++) { | 822 | for (i = 0; i < msgs->len; i++) { |
822 | u8 len = 0; | 823 | u8 len = 0; |
824 | |||
823 | result = i2c_imx_trx_complete(i2c_imx); | 825 | result = i2c_imx_trx_complete(i2c_imx); |
824 | if (result) | 826 | if (result) |
825 | return result; | 827 | return result; |
@@ -915,15 +917,16 @@ static int i2c_imx_xfer(struct i2c_adapter *adapter, | |||
915 | /* write/read data */ | 917 | /* write/read data */ |
916 | #ifdef CONFIG_I2C_DEBUG_BUS | 918 | #ifdef CONFIG_I2C_DEBUG_BUS |
917 | temp = imx_i2c_read_reg(i2c_imx, IMX_I2C_I2CR); | 919 | temp = imx_i2c_read_reg(i2c_imx, IMX_I2C_I2CR); |
918 | dev_dbg(&i2c_imx->adapter.dev, "<%s> CONTROL: IEN=%d, IIEN=%d, " | 920 | dev_dbg(&i2c_imx->adapter.dev, |
919 | "MSTA=%d, MTX=%d, TXAK=%d, RSTA=%d\n", __func__, | 921 | "<%s> CONTROL: IEN=%d, IIEN=%d, MSTA=%d, MTX=%d, TXAK=%d, RSTA=%d\n", |
922 | __func__, | ||
920 | (temp & I2CR_IEN ? 1 : 0), (temp & I2CR_IIEN ? 1 : 0), | 923 | (temp & I2CR_IEN ? 1 : 0), (temp & I2CR_IIEN ? 1 : 0), |
921 | (temp & I2CR_MSTA ? 1 : 0), (temp & I2CR_MTX ? 1 : 0), | 924 | (temp & I2CR_MSTA ? 1 : 0), (temp & I2CR_MTX ? 1 : 0), |
922 | (temp & I2CR_TXAK ? 1 : 0), (temp & I2CR_RSTA ? 1 : 0)); | 925 | (temp & I2CR_TXAK ? 1 : 0), (temp & I2CR_RSTA ? 1 : 0)); |
923 | temp = imx_i2c_read_reg(i2c_imx, IMX_I2C_I2SR); | 926 | temp = imx_i2c_read_reg(i2c_imx, IMX_I2C_I2SR); |
924 | dev_dbg(&i2c_imx->adapter.dev, | 927 | dev_dbg(&i2c_imx->adapter.dev, |
925 | "<%s> STATUS: ICF=%d, IAAS=%d, IBB=%d, " | 928 | "<%s> STATUS: ICF=%d, IAAS=%d, IBB=%d, IAL=%d, SRW=%d, IIF=%d, RXAK=%d\n", |
926 | "IAL=%d, SRW=%d, IIF=%d, RXAK=%d\n", __func__, | 929 | __func__, |
927 | (temp & I2SR_ICF ? 1 : 0), (temp & I2SR_IAAS ? 1 : 0), | 930 | (temp & I2SR_ICF ? 1 : 0), (temp & I2SR_IAAS ? 1 : 0), |
928 | (temp & I2SR_IBB ? 1 : 0), (temp & I2SR_IAL ? 1 : 0), | 931 | (temp & I2SR_IBB ? 1 : 0), (temp & I2SR_IAL ? 1 : 0), |
929 | (temp & I2SR_SRW ? 1 : 0), (temp & I2SR_IIF ? 1 : 0), | 932 | (temp & I2SR_SRW ? 1 : 0), (temp & I2SR_IIF ? 1 : 0), |
@@ -1002,7 +1005,7 @@ static int i2c_imx_probe(struct platform_device *pdev) | |||
1002 | i2c_imx->adapter.owner = THIS_MODULE; | 1005 | i2c_imx->adapter.owner = THIS_MODULE; |
1003 | i2c_imx->adapter.algo = &i2c_imx_algo; | 1006 | i2c_imx->adapter.algo = &i2c_imx_algo; |
1004 | i2c_imx->adapter.dev.parent = &pdev->dev; | 1007 | i2c_imx->adapter.dev.parent = &pdev->dev; |
1005 | i2c_imx->adapter.nr = pdev->id; | 1008 | i2c_imx->adapter.nr = pdev->id; |
1006 | i2c_imx->adapter.dev.of_node = pdev->dev.of_node; | 1009 | i2c_imx->adapter.dev.of_node = pdev->dev.of_node; |
1007 | i2c_imx->base = base; | 1010 | i2c_imx->base = base; |
1008 | 1011 | ||
@@ -1061,7 +1064,7 @@ static int i2c_imx_probe(struct platform_device *pdev) | |||
1061 | i2c_imx->adapter.name); | 1064 | i2c_imx->adapter.name); |
1062 | dev_info(&i2c_imx->adapter.dev, "IMX I2C adapter registered\n"); | 1065 | dev_info(&i2c_imx->adapter.dev, "IMX I2C adapter registered\n"); |
1063 | 1066 | ||
1064 | /* Init DMA config if support*/ | 1067 | /* Init DMA config if supported */ |
1065 | i2c_imx_dma_request(i2c_imx, phy_addr); | 1068 | i2c_imx_dma_request(i2c_imx, phy_addr); |
1066 | 1069 | ||
1067 | return 0; /* Return OK */ | 1070 | return 0; /* Return OK */ |