* [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@vger.kernel.org, linux-kernel@vger.kernel.org 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 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
* 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
end of thread, other threads:[~2017-01-05 7:30 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