* [PATCH] mmc: dw_mmc: Fix some coding style
@ 2017-01-04 12:52 ` Ziyuan Xu
2017-01-04 13:03 ` Andy Shevchenko
` (2 more replies)
0 siblings, 3 replies; 6+ messages in thread
From: Ziyuan Xu @ 2017-01-04 12:52 UTC (permalink / raw)
To: ulf.hansson, jh80.chung; +Cc: linux-mmc, Ziyuan Xu, linux-kernel
Let's fix the warnings from checkpatch.pl:
- line over 80 characters;
- block comments should align the * on each Lines;
- statements not starting on a tabstop.
Signed-off-by: Ziyuan Xu <xzy.xu@rock-chips.com>
---
drivers/mmc/host/dw_mmc.c | 33 +++++++++++++++++----------------
1 file changed, 17 insertions(+), 16 deletions(-)
diff --git a/drivers/mmc/host/dw_mmc.c b/drivers/mmc/host/dw_mmc.c
index ed63237..a8efe3f 100644
--- a/drivers/mmc/host/dw_mmc.c
+++ b/drivers/mmc/host/dw_mmc.c
@@ -94,7 +94,8 @@ struct idmac_desc {
__le32 des1; /* Buffer sizes */
#define IDMAC_SET_BUFFER1_SIZE(d, s) \
- ((d)->des1 = ((d)->des1 & cpu_to_le32(0x03ffe000)) | (cpu_to_le32((s) & 0x1fff)))
+ ((d)->des1 = ((d)->des1 & cpu_to_le32(0x03ffe000)) | \
+ (cpu_to_le32((s) & 0x1fff)))
__le32 des2; /* buffer 1 physical address */
@@ -2733,16 +2734,16 @@ static void dw_mci_init_dma(struct dw_mci *host)
struct device_node *np = dev->of_node;
/*
- * Check tansfer mode from HCON[17:16]
- * Clear the ambiguous description of dw_mmc databook:
- * 2b'00: No DMA Interface -> Actually means using Internal DMA block
- * 2b'01: DesignWare DMA Interface -> Synopsys DW-DMA block
- * 2b'10: Generic DMA Interface -> non-Synopsys generic DMA block
- * 2b'11: Non DW DMA Interface -> pio only
- * Compared to DesignWare DMA Interface, Generic DMA Interface has a
- * simpler request/acknowledge handshake mechanism and both of them
- * are regarded as external dma master for dw_mmc.
- */
+ * Check tansfer mode from HCON[17:16]
+ * Clear the ambiguous description of dw_mmc databook:
+ * 2b'00: No DMA Interface -> Actually means using Internal DMA block
+ * 2b'01: DesignWare DMA Interface -> Synopsys DW-DMA block
+ * 2b'10: Generic DMA Interface -> non-Synopsys generic DMA block
+ * 2b'11: Non DW DMA Interface -> pio only
+ * Compared to DesignWare DMA Interface, Generic DMA Interface has a
+ * simpler request/acknowledge handshake mechanism and both of them
+ * are regarded as external dma master for dw_mmc.
+ */
host->use_dma = SDMMC_GET_TRANS_MODE(mci_readl(host, HCON));
if (host->use_dma == DMA_INTERFACE_IDMA) {
host->use_dma = TRANS_MODE_IDMAC;
@@ -2756,9 +2757,9 @@ static void dw_mci_init_dma(struct dw_mci *host)
/* Determine which DMA interface to use */
if (host->use_dma == TRANS_MODE_IDMAC) {
/*
- * Check ADDR_CONFIG bit in HCON to find
- * IDMAC address bus width
- */
+ * Check ADDR_CONFIG bit in HCON to find
+ * IDMAC address bus width
+ */
addr_config = SDMMC_GET_ADDR_CONFIG(mci_readl(host, HCON));
if (addr_config == 1) {
@@ -3337,8 +3338,8 @@ int dw_mci_runtime_resume(struct device *dev)
* Restore the initial value at FIFOTH register
* And Invalidate the prev_blksz with zero
*/
- mci_writel(host, FIFOTH, host->fifoth_val);
- host->prev_blksz = 0;
+ mci_writel(host, FIFOTH, host->fifoth_val);
+ host->prev_blksz = 0;
/* Put in max timeout */
mci_writel(host, TMOUT, 0xFFFFFFFF);
--
2.7.4
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] mmc: dw_mmc: Fix some coding style
2017-01-04 12:52 ` [PATCH] mmc: dw_mmc: Fix some coding style Ziyuan Xu
@ 2017-01-04 13:03 ` Andy Shevchenko
2017-01-04 18:32 ` Joe Perches
2017-01-05 1:19 ` Jaehoon Chung
2 siblings, 0 replies; 6+ messages in thread
From: Andy Shevchenko @ 2017-01-04 13:03 UTC (permalink / raw)
To: Ziyuan Xu; +Cc: Ulf Hansson, Jaehoon Chung, linux-mmc, linux-kernel
On Wed, Jan 4, 2017 at 2:52 PM, Ziyuan Xu <xzy.xu@rock-chips.com> wrote:
> Let's fix the warnings from checkpatch.pl:
>
> - line over 80 characters;
> - block comments should align the * on each Lines;
> - statements not starting on a tabstop.
> __le32 des1; /* Buffer sizes */
> #define IDMAC_SET_BUFFER1_SIZE(d, s) \
> - ((d)->des1 = ((d)->des1 & cpu_to_le32(0x03ffe000)) | (cpu_to_le32((s) & 0x1fff)))
> + ((d)->des1 = ((d)->des1 & cpu_to_le32(0x03ffe000)) | \
> + (cpu_to_le32((s) & 0x1fff)))
I don't think there is a significant value to split. How many
characters it fits right now?
Up to 90 for such a code is okay for my point of view. It makes it
more readable than split variant.
--
With Best Regards,
Andy Shevchenko
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] mmc: dw_mmc: Fix some coding style
2017-01-04 12:52 ` [PATCH] mmc: dw_mmc: Fix some coding style Ziyuan Xu
2017-01-04 13:03 ` Andy Shevchenko
@ 2017-01-04 18:32 ` Joe Perches
2017-01-05 7:30 ` Ziyuan
2017-01-05 1:19 ` Jaehoon Chung
2 siblings, 1 reply; 6+ messages in thread
From: Joe Perches @ 2017-01-04 18:32 UTC (permalink / raw)
To: Ziyuan Xu, ulf.hansson, jh80.chung; +Cc: linux-mmc, linux-kernel
On Wed, 2017-01-04 at 20:52 +0800, Ziyuan Xu wrote:
> Let's fix the warnings from checkpatch.pl:
>
> - line over 80 characters;
> - block comments should align the * on each Lines;
> - statements not starting on a tabstop.
>
> Signed-off-by: Ziyuan Xu <xzy.xu@rock-chips.com>
> ---
>
> drivers/mmc/host/dw_mmc.c | 33 +++++++++++++++++----------------
> 1 file changed, 17 insertions(+), 16 deletions(-)
>
> diff --git a/drivers/mmc/host/dw_mmc.c b/drivers/mmc/host/dw_mmc.c
[]
> @@ -94,7 +94,8 @@ struct idmac_desc {
>
> __le32 des1; /* Buffer sizes */
> #define IDMAC_SET_BUFFER1_SIZE(d, s) \
> - ((d)->des1 = ((d)->des1 & cpu_to_le32(0x03ffe000)) | (cpu_to_le32((s) & 0x1fff)))
> + ((d)->des1 = ((d)->des1 & cpu_to_le32(0x03ffe000)) | \
> + (cpu_to_le32((s) & 0x1fff)))
Please look to improve code rather than just shut up
the brainless checkpatch script.
If this is really valuable, it'd probably be better as
an inline function, or as it's only used once, just
as direct code in that one place.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] mmc: dw_mmc: Fix some coding style
2017-01-04 12:52 ` [PATCH] mmc: dw_mmc: Fix some coding style Ziyuan Xu
2017-01-04 13:03 ` Andy Shevchenko
2017-01-04 18:32 ` Joe Perches
@ 2017-01-05 1:19 ` Jaehoon Chung
2017-01-05 2:24 ` Shawn Lin
2 siblings, 1 reply; 6+ messages in thread
From: Jaehoon Chung @ 2017-01-05 1:19 UTC (permalink / raw)
To: Ziyuan Xu, ulf.hansson; +Cc: linux-mmc, linux-kernel
Hi,
On 01/04/2017 09:52 PM, Ziyuan Xu wrote:
> Let's fix the warnings from checkpatch.pl:
>
> - line over 80 characters;
> - block comments should align the * on each Lines;
> - statements not starting on a tabstop.
If there is no critical problem,
I think that current codes are keeping better than changing.
When someone contribute the patches relevant to these, it can be also changed,
(It's more meaningful.)
Best Regards,
Jaehoon Chung
>
> Signed-off-by: Ziyuan Xu <xzy.xu@rock-chips.com>
> ---
>
> drivers/mmc/host/dw_mmc.c | 33 +++++++++++++++++----------------
> 1 file changed, 17 insertions(+), 16 deletions(-)
>
> diff --git a/drivers/mmc/host/dw_mmc.c b/drivers/mmc/host/dw_mmc.c
> index ed63237..a8efe3f 100644
> --- a/drivers/mmc/host/dw_mmc.c
> +++ b/drivers/mmc/host/dw_mmc.c
> @@ -94,7 +94,8 @@ struct idmac_desc {
>
> __le32 des1; /* Buffer sizes */
> #define IDMAC_SET_BUFFER1_SIZE(d, s) \
> - ((d)->des1 = ((d)->des1 & cpu_to_le32(0x03ffe000)) | (cpu_to_le32((s) & 0x1fff)))
> + ((d)->des1 = ((d)->des1 & cpu_to_le32(0x03ffe000)) | \
> + (cpu_to_le32((s) & 0x1fff)))
>
> __le32 des2; /* buffer 1 physical address */
>
> @@ -2733,16 +2734,16 @@ static void dw_mci_init_dma(struct dw_mci *host)
> struct device_node *np = dev->of_node;
>
> /*
> - * Check tansfer mode from HCON[17:16]
> - * Clear the ambiguous description of dw_mmc databook:
> - * 2b'00: No DMA Interface -> Actually means using Internal DMA block
> - * 2b'01: DesignWare DMA Interface -> Synopsys DW-DMA block
> - * 2b'10: Generic DMA Interface -> non-Synopsys generic DMA block
> - * 2b'11: Non DW DMA Interface -> pio only
> - * Compared to DesignWare DMA Interface, Generic DMA Interface has a
> - * simpler request/acknowledge handshake mechanism and both of them
> - * are regarded as external dma master for dw_mmc.
> - */
> + * Check tansfer mode from HCON[17:16]
> + * Clear the ambiguous description of dw_mmc databook:
> + * 2b'00: No DMA Interface -> Actually means using Internal DMA block
> + * 2b'01: DesignWare DMA Interface -> Synopsys DW-DMA block
> + * 2b'10: Generic DMA Interface -> non-Synopsys generic DMA block
> + * 2b'11: Non DW DMA Interface -> pio only
> + * Compared to DesignWare DMA Interface, Generic DMA Interface has a
> + * simpler request/acknowledge handshake mechanism and both of them
> + * are regarded as external dma master for dw_mmc.
> + */
> host->use_dma = SDMMC_GET_TRANS_MODE(mci_readl(host, HCON));
> if (host->use_dma == DMA_INTERFACE_IDMA) {
> host->use_dma = TRANS_MODE_IDMAC;
> @@ -2756,9 +2757,9 @@ static void dw_mci_init_dma(struct dw_mci *host)
> /* Determine which DMA interface to use */
> if (host->use_dma == TRANS_MODE_IDMAC) {
> /*
> - * Check ADDR_CONFIG bit in HCON to find
> - * IDMAC address bus width
> - */
> + * Check ADDR_CONFIG bit in HCON to find
> + * IDMAC address bus width
> + */
> addr_config = SDMMC_GET_ADDR_CONFIG(mci_readl(host, HCON));
>
> if (addr_config == 1) {
> @@ -3337,8 +3338,8 @@ int dw_mci_runtime_resume(struct device *dev)
> * Restore the initial value at FIFOTH register
> * And Invalidate the prev_blksz with zero
> */
> - mci_writel(host, FIFOTH, host->fifoth_val);
> - host->prev_blksz = 0;
> + mci_writel(host, FIFOTH, host->fifoth_val);
> + host->prev_blksz = 0;
>
> /* Put in max timeout */
> mci_writel(host, TMOUT, 0xFFFFFFFF);
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] mmc: dw_mmc: Fix some coding style
2017-01-05 1:19 ` Jaehoon Chung
@ 2017-01-05 2:24 ` Shawn Lin
0 siblings, 0 replies; 6+ messages in thread
From: Shawn Lin @ 2017-01-05 2:24 UTC (permalink / raw)
To: Jaehoon Chung, Ziyuan Xu, ulf.hansson; +Cc: shawn.lin, linux-mmc, linux-kernel
On 2017/1/5 9:19, Jaehoon Chung wrote:
> Hi,
>
> On 01/04/2017 09:52 PM, Ziyuan Xu wrote:
>> Let's fix the warnings from checkpatch.pl:
>>
>> - line over 80 characters;
>> - block comments should align the * on each Lines;
>> - statements not starting on a tabstop.
>
> If there is no critical problem,
> I think that current codes are keeping better than changing.
> When someone contribute the patches relevant to these, it can be also changed,
> (It's more meaningful.)
>
Indeed.
Moreover, it breaks the git-blame history when we need to review
how the code had been developing..
> Best Regards,
> Jaehoon Chung
>
>>
>> Signed-off-by: Ziyuan Xu <xzy.xu@rock-chips.com>
>> ---
>>
>> drivers/mmc/host/dw_mmc.c | 33 +++++++++++++++++----------------
>> 1 file changed, 17 insertions(+), 16 deletions(-)
>>
>> diff --git a/drivers/mmc/host/dw_mmc.c b/drivers/mmc/host/dw_mmc.c
>> index ed63237..a8efe3f 100644
>> --- a/drivers/mmc/host/dw_mmc.c
>> +++ b/drivers/mmc/host/dw_mmc.c
>> @@ -94,7 +94,8 @@ struct idmac_desc {
>>
>> __le32 des1; /* Buffer sizes */
>> #define IDMAC_SET_BUFFER1_SIZE(d, s) \
>> - ((d)->des1 = ((d)->des1 & cpu_to_le32(0x03ffe000)) | (cpu_to_le32((s) & 0x1fff)))
>> + ((d)->des1 = ((d)->des1 & cpu_to_le32(0x03ffe000)) | \
>> + (cpu_to_le32((s) & 0x1fff)))
>>
>> __le32 des2; /* buffer 1 physical address */
>>
>> @@ -2733,16 +2734,16 @@ static void dw_mci_init_dma(struct dw_mci *host)
>> struct device_node *np = dev->of_node;
>>
>> /*
>> - * Check tansfer mode from HCON[17:16]
>> - * Clear the ambiguous description of dw_mmc databook:
>> - * 2b'00: No DMA Interface -> Actually means using Internal DMA block
>> - * 2b'01: DesignWare DMA Interface -> Synopsys DW-DMA block
>> - * 2b'10: Generic DMA Interface -> non-Synopsys generic DMA block
>> - * 2b'11: Non DW DMA Interface -> pio only
>> - * Compared to DesignWare DMA Interface, Generic DMA Interface has a
>> - * simpler request/acknowledge handshake mechanism and both of them
>> - * are regarded as external dma master for dw_mmc.
>> - */
>> + * Check tansfer mode from HCON[17:16]
>> + * Clear the ambiguous description of dw_mmc databook:
>> + * 2b'00: No DMA Interface -> Actually means using Internal DMA block
>> + * 2b'01: DesignWare DMA Interface -> Synopsys DW-DMA block
>> + * 2b'10: Generic DMA Interface -> non-Synopsys generic DMA block
>> + * 2b'11: Non DW DMA Interface -> pio only
>> + * Compared to DesignWare DMA Interface, Generic DMA Interface has a
>> + * simpler request/acknowledge handshake mechanism and both of them
>> + * are regarded as external dma master for dw_mmc.
>> + */
>> host->use_dma = SDMMC_GET_TRANS_MODE(mci_readl(host, HCON));
>> if (host->use_dma == DMA_INTERFACE_IDMA) {
>> host->use_dma = TRANS_MODE_IDMAC;
>> @@ -2756,9 +2757,9 @@ static void dw_mci_init_dma(struct dw_mci *host)
>> /* Determine which DMA interface to use */
>> if (host->use_dma == TRANS_MODE_IDMAC) {
>> /*
>> - * Check ADDR_CONFIG bit in HCON to find
>> - * IDMAC address bus width
>> - */
>> + * Check ADDR_CONFIG bit in HCON to find
>> + * IDMAC address bus width
>> + */
>> addr_config = SDMMC_GET_ADDR_CONFIG(mci_readl(host, HCON));
>>
>> if (addr_config == 1) {
>> @@ -3337,8 +3338,8 @@ int dw_mci_runtime_resume(struct device *dev)
>> * Restore the initial value at FIFOTH register
>> * And Invalidate the prev_blksz with zero
>> */
>> - mci_writel(host, FIFOTH, host->fifoth_val);
>> - host->prev_blksz = 0;
>> + mci_writel(host, FIFOTH, host->fifoth_val);
>> + host->prev_blksz = 0;
>>
>> /* Put in max timeout */
>> mci_writel(host, TMOUT, 0xFFFFFFFF);
>>
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
--
Best Regards
Shawn Lin
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] mmc: dw_mmc: Fix some coding style
2017-01-04 18:32 ` Joe Perches
@ 2017-01-05 7:30 ` Ziyuan
0 siblings, 0 replies; 6+ messages in thread
From: Ziyuan @ 2017-01-05 7:30 UTC (permalink / raw)
To: Joe Perches, ulf.hansson, jh80.chung; +Cc: linux-mmc, linux-kernel
On 01/05/2017 02:32 AM, Joe Perches wrote:
> On Wed, 2017-01-04 at 20:52 +0800, Ziyuan Xu wrote:
>> Let's fix the warnings from checkpatch.pl:
>>
>> - line over 80 characters;
>> - block comments should align the * on each Lines;
>> - statements not starting on a tabstop.
>>
>> Signed-off-by: Ziyuan Xu <xzy.xu@rock-chips.com>
>> ---
>>
>> drivers/mmc/host/dw_mmc.c | 33 +++++++++++++++++----------------
>> 1 file changed, 17 insertions(+), 16 deletions(-)
>>
>> diff --git a/drivers/mmc/host/dw_mmc.c b/drivers/mmc/host/dw_mmc.c
> []
>> @@ -94,7 +94,8 @@ struct idmac_desc {
>>
>> __le32 des1; /* Buffer sizes */
>> #define IDMAC_SET_BUFFER1_SIZE(d, s) \
>> - ((d)->des1 = ((d)->des1 & cpu_to_le32(0x03ffe000)) | (cpu_to_le32((s) & 0x1fff)))
>> + ((d)->des1 = ((d)->des1 & cpu_to_le32(0x03ffe000)) | \
>> + (cpu_to_le32((s) & 0x1fff)))
> Please look to improve code rather than just shut up
> the brainless checkpatch script.
>
> If this is really valuable, it'd probably be better as
> an inline function, or as it's only used once, just
> as direct code in that one place.
Fine, I get it. Thanks for the advice.:-)
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
>
>
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2017-01-05 7:31 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <CGME20170104125752epcas4p2a801fce6fd376d02d7440ad4bfe1716b@epcas4p2.samsung.com>
2017-01-04 12:52 ` [PATCH] mmc: dw_mmc: Fix some coding style Ziyuan Xu
2017-01-04 13:03 ` Andy Shevchenko
2017-01-04 18:32 ` Joe Perches
2017-01-05 7:30 ` Ziyuan
2017-01-05 1:19 ` Jaehoon Chung
2017-01-05 2:24 ` Shawn Lin
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).