* [PATCH 01/16] memory: bt1-l2-ctl: Add blank lines after declarations
2020-07-24 18:23 [PATCH 00/16] memory: Further cleanups Krzysztof Kozlowski
@ 2020-07-24 18:23 ` Krzysztof Kozlowski
2020-07-27 8:16 ` Krzysztof Kozlowski
2020-07-24 18:23 ` [PATCH 02/16] memory: mvebu-devbus: Add missing braces to all arms of if statement Krzysztof Kozlowski
` (14 subsequent siblings)
15 siblings, 1 reply; 35+ messages in thread
From: Krzysztof Kozlowski @ 2020-07-24 18:23 UTC (permalink / raw)
To: Florian Fainelli, Ray Jui, Scott Branden,
bcm-kernel-feedback-list, Russell King, Catalin Marinas,
Will Deacon, Krzysztof Kozlowski, Markus Mayer, Roger Quadros,
Tony Lindgren, Vladimir Zapolskiy, Kukjin Kim, Lukasz Luba,
linux-arm-kernel, linux-kernel, linux-omap, linux-samsung-soc,
linux-pm
Cc: Olof Johansson, Arnd Bergmann
Add blank lines to improve code readability. No functional change.
Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
drivers/memory/bt1-l2-ctl.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/memory/bt1-l2-ctl.c b/drivers/memory/bt1-l2-ctl.c
index 633fea6a4edf..85965fa26e0b 100644
--- a/drivers/memory/bt1-l2-ctl.c
+++ b/drivers/memory/bt1-l2-ctl.c
@@ -66,6 +66,7 @@ struct l2_ctl_device_attribute {
struct device_attribute dev_attr;
enum l2_ctl_stall id;
};
+
#define to_l2_ctl_dev_attr(_dev_attr) \
container_of(_dev_attr, struct l2_ctl_device_attribute, dev_attr)
@@ -242,6 +243,7 @@ static ssize_t l2_ctl_latency_store(struct device *dev,
return count;
}
+
static L2_CTL_ATTR_RW(l2_ws_latency, l2_ctl_latency, L2_WS_STALL);
static L2_CTL_ATTR_RW(l2_tag_latency, l2_ctl_latency, L2_TAG_STALL);
static L2_CTL_ATTR_RW(l2_data_latency, l2_ctl_latency, L2_DATA_STALL);
--
2.17.1
^ permalink raw reply related [flat|nested] 35+ messages in thread
* Re: [PATCH 01/16] memory: bt1-l2-ctl: Add blank lines after declarations
2020-07-24 18:23 ` [PATCH 01/16] memory: bt1-l2-ctl: Add blank lines after declarations Krzysztof Kozlowski
@ 2020-07-27 8:16 ` Krzysztof Kozlowski
0 siblings, 0 replies; 35+ messages in thread
From: Krzysztof Kozlowski @ 2020-07-27 8:16 UTC (permalink / raw)
To: Florian Fainelli, Ray Jui, Scott Branden,
bcm-kernel-feedback-list, Russell King, Catalin Marinas,
Will Deacon, Markus Mayer, Roger Quadros, Tony Lindgren,
Vladimir Zapolskiy, Kukjin Kim, Lukasz Luba, linux-arm-kernel,
linux-kernel, linux-omap, linux-samsung-soc, linux-pm
Cc: Olof Johansson, Arnd Bergmann
On Fri, Jul 24, 2020 at 08:23:13PM +0200, Krzysztof Kozlowski wrote:
> Add blank lines to improve code readability. No functional change.
>
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> ---
> drivers/memory/bt1-l2-ctl.c | 2 ++
> 1 file changed, 2 insertions(+)
Applied to drivers/memory tree.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 35+ messages in thread
* [PATCH 02/16] memory: mvebu-devbus: Add missing braces to all arms of if statement
2020-07-24 18:23 [PATCH 00/16] memory: Further cleanups Krzysztof Kozlowski
2020-07-24 18:23 ` [PATCH 01/16] memory: bt1-l2-ctl: Add blank lines after declarations Krzysztof Kozlowski
@ 2020-07-24 18:23 ` Krzysztof Kozlowski
2020-07-27 8:16 ` Krzysztof Kozlowski
2020-07-24 18:23 ` [PATCH 03/16] memory: mvebu-devbus: Align with open parenthesis Krzysztof Kozlowski
` (13 subsequent siblings)
15 siblings, 1 reply; 35+ messages in thread
From: Krzysztof Kozlowski @ 2020-07-24 18:23 UTC (permalink / raw)
To: Florian Fainelli, Ray Jui, Scott Branden,
bcm-kernel-feedback-list, Russell King, Catalin Marinas,
Will Deacon, Krzysztof Kozlowski, Markus Mayer, Roger Quadros,
Tony Lindgren, Vladimir Zapolskiy, Kukjin Kim, Lukasz Luba,
linux-arm-kernel, linux-kernel, linux-omap, linux-samsung-soc,
linux-pm
Cc: Olof Johansson, Arnd Bergmann
Add missing braces to all arms of if statement to align with coding
convention.
Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
drivers/memory/mvebu-devbus.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/memory/mvebu-devbus.c b/drivers/memory/mvebu-devbus.c
index 886aea587276..76efc653dc7e 100644
--- a/drivers/memory/mvebu-devbus.c
+++ b/drivers/memory/mvebu-devbus.c
@@ -124,11 +124,11 @@ static int devbus_get_timing_params(struct devbus *devbus,
* The bus width is encoded into the register as 0 for 8 bits,
* and 1 for 16 bits, so we do the necessary conversion here.
*/
- if (r->bus_width == 8)
+ if (r->bus_width == 8) {
r->bus_width = 0;
- else if (r->bus_width == 16)
+ } else if (r->bus_width == 16) {
r->bus_width = 1;
- else {
+ } else {
dev_err(devbus->dev, "invalid bus width %d\n", r->bus_width);
return -EINVAL;
}
--
2.17.1
^ permalink raw reply related [flat|nested] 35+ messages in thread
* Re: [PATCH 02/16] memory: mvebu-devbus: Add missing braces to all arms of if statement
2020-07-24 18:23 ` [PATCH 02/16] memory: mvebu-devbus: Add missing braces to all arms of if statement Krzysztof Kozlowski
@ 2020-07-27 8:16 ` Krzysztof Kozlowski
0 siblings, 0 replies; 35+ messages in thread
From: Krzysztof Kozlowski @ 2020-07-27 8:16 UTC (permalink / raw)
To: Florian Fainelli, Ray Jui, Scott Branden,
bcm-kernel-feedback-list, Russell King, Catalin Marinas,
Will Deacon, Markus Mayer, Roger Quadros, Tony Lindgren,
Vladimir Zapolskiy, Kukjin Kim, Lukasz Luba, linux-arm-kernel,
linux-kernel, linux-omap, linux-samsung-soc, linux-pm
Cc: Olof Johansson, Arnd Bergmann
On Fri, Jul 24, 2020 at 08:23:14PM +0200, Krzysztof Kozlowski wrote:
> Add missing braces to all arms of if statement to align with coding
> convention.
>
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> ---
> drivers/memory/mvebu-devbus.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
Applied to drivers/memory tree.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 35+ messages in thread
* [PATCH 03/16] memory: mvebu-devbus: Align with open parenthesis
2020-07-24 18:23 [PATCH 00/16] memory: Further cleanups Krzysztof Kozlowski
2020-07-24 18:23 ` [PATCH 01/16] memory: bt1-l2-ctl: Add blank lines after declarations Krzysztof Kozlowski
2020-07-24 18:23 ` [PATCH 02/16] memory: mvebu-devbus: Add missing braces to all arms of if statement Krzysztof Kozlowski
@ 2020-07-24 18:23 ` Krzysztof Kozlowski
2020-07-27 8:16 ` Krzysztof Kozlowski
2020-07-24 18:23 ` [PATCH 04/16] memory: omap-gpmc: Correct kerneldoc Krzysztof Kozlowski
` (12 subsequent siblings)
15 siblings, 1 reply; 35+ messages in thread
From: Krzysztof Kozlowski @ 2020-07-24 18:23 UTC (permalink / raw)
To: Florian Fainelli, Ray Jui, Scott Branden,
bcm-kernel-feedback-list, Russell King, Catalin Marinas,
Will Deacon, Krzysztof Kozlowski, Markus Mayer, Roger Quadros,
Tony Lindgren, Vladimir Zapolskiy, Kukjin Kim, Lukasz Luba,
linux-arm-kernel, linux-kernel, linux-omap, linux-samsung-soc,
linux-pm
Cc: Olof Johansson, Arnd Bergmann
The line continuation contained spaces but still failed to properly
align with open parenthesis.
Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
drivers/memory/mvebu-devbus.c | 14 +++++++-------
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/drivers/memory/mvebu-devbus.c b/drivers/memory/mvebu-devbus.c
index 76efc653dc7e..8450638e8670 100644
--- a/drivers/memory/mvebu-devbus.c
+++ b/drivers/memory/mvebu-devbus.c
@@ -134,22 +134,22 @@ static int devbus_get_timing_params(struct devbus *devbus,
}
err = get_timing_param_ps(devbus, node, "devbus,badr-skew-ps",
- &r->badr_skew);
+ &r->badr_skew);
if (err < 0)
return err;
err = get_timing_param_ps(devbus, node, "devbus,turn-off-ps",
- &r->turn_off);
+ &r->turn_off);
if (err < 0)
return err;
err = get_timing_param_ps(devbus, node, "devbus,acc-first-ps",
- &r->acc_first);
+ &r->acc_first);
if (err < 0)
return err;
err = get_timing_param_ps(devbus, node, "devbus,acc-next-ps",
- &r->acc_next);
+ &r->acc_next);
if (err < 0)
return err;
@@ -175,17 +175,17 @@ static int devbus_get_timing_params(struct devbus *devbus,
}
err = get_timing_param_ps(devbus, node, "devbus,ale-wr-ps",
- &w->ale_wr);
+ &w->ale_wr);
if (err < 0)
return err;
err = get_timing_param_ps(devbus, node, "devbus,wr-low-ps",
- &w->wr_low);
+ &w->wr_low);
if (err < 0)
return err;
err = get_timing_param_ps(devbus, node, "devbus,wr-high-ps",
- &w->wr_high);
+ &w->wr_high);
if (err < 0)
return err;
--
2.17.1
^ permalink raw reply related [flat|nested] 35+ messages in thread
* Re: [PATCH 03/16] memory: mvebu-devbus: Align with open parenthesis
2020-07-24 18:23 ` [PATCH 03/16] memory: mvebu-devbus: Align with open parenthesis Krzysztof Kozlowski
@ 2020-07-27 8:16 ` Krzysztof Kozlowski
0 siblings, 0 replies; 35+ messages in thread
From: Krzysztof Kozlowski @ 2020-07-27 8:16 UTC (permalink / raw)
To: Florian Fainelli, Ray Jui, Scott Branden,
bcm-kernel-feedback-list, Russell King, Catalin Marinas,
Will Deacon, Markus Mayer, Roger Quadros, Tony Lindgren,
Vladimir Zapolskiy, Kukjin Kim, Lukasz Luba, linux-arm-kernel,
linux-kernel, linux-omap, linux-samsung-soc, linux-pm
Cc: Olof Johansson, Arnd Bergmann
On Fri, Jul 24, 2020 at 08:23:15PM +0200, Krzysztof Kozlowski wrote:
> The line continuation contained spaces but still failed to properly
> align with open parenthesis.
>
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> ---
> drivers/memory/mvebu-devbus.c | 14 +++++++-------
> 1 file changed, 7 insertions(+), 7 deletions(-)
Applied to drivers/memory tree.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 35+ messages in thread
* [PATCH 04/16] memory: omap-gpmc: Correct kerneldoc
2020-07-24 18:23 [PATCH 00/16] memory: Further cleanups Krzysztof Kozlowski
` (2 preceding siblings ...)
2020-07-24 18:23 ` [PATCH 03/16] memory: mvebu-devbus: Align with open parenthesis Krzysztof Kozlowski
@ 2020-07-24 18:23 ` Krzysztof Kozlowski
2020-07-27 8:16 ` Krzysztof Kozlowski
2020-07-24 18:23 ` [PATCH 05/16] memory: omap-gpmc: Enclose macro argument usage in parenthesis Krzysztof Kozlowski
` (11 subsequent siblings)
15 siblings, 1 reply; 35+ messages in thread
From: Krzysztof Kozlowski @ 2020-07-24 18:23 UTC (permalink / raw)
To: Florian Fainelli, Ray Jui, Scott Branden,
bcm-kernel-feedback-list, Russell King, Catalin Marinas,
Will Deacon, Krzysztof Kozlowski, Markus Mayer, Roger Quadros,
Tony Lindgren, Vladimir Zapolskiy, Kukjin Kim, Lukasz Luba,
linux-arm-kernel, linux-kernel, linux-omap, linux-samsung-soc,
linux-pm
Cc: Olof Johansson, Arnd Bergmann
Use proper kerneldoc to fix GCC warnings like:
drivers/memory/omap-gpmc.c:299: warning: Function parameter or member 'cs' not described in 'gpmc_get_clk_period'
drivers/memory/omap-gpmc.c:432: warning: Excess function parameter 'ma' description in 'get_gpmc_timing_reg'
Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
drivers/memory/omap-gpmc.c | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/memory/omap-gpmc.c b/drivers/memory/omap-gpmc.c
index 2675ea4e37fe..c91a021ac421 100644
--- a/drivers/memory/omap-gpmc.c
+++ b/drivers/memory/omap-gpmc.c
@@ -289,8 +289,8 @@ static unsigned long gpmc_get_fclk_period(void)
/**
* gpmc_get_clk_period - get period of selected clock domain in ps
- * @cs Chip Select Region.
- * @cd Clock Domain.
+ * @cs: Chip Select Region.
+ * @cd: Clock Domain.
*
* GPMC_CS_CONFIG1 GPMCFCLKDIVIDER for cs has to be setup
* prior to calling this function with GPMC_CD_CLK.
@@ -409,7 +409,7 @@ static void gpmc_cs_bool_timings(int cs, const struct gpmc_bool_timings *p)
* @reg: GPMC_CS_CONFIGn register offset.
* @st_bit: Start Bit
* @end_bit: End Bit. Must be >= @st_bit.
- * @ma:x Maximum parameter value (before optional @shift).
+ * @max: Maximum parameter value (before optional @shift).
* If 0, maximum is as high as @st_bit and @end_bit allow.
* @name: DTS node name, w/o "gpmc,"
* @cd: Clock Domain of timing parameter.
@@ -1134,7 +1134,7 @@ static struct gpmc_nand_ops nand_ops = {
/**
* gpmc_omap_get_nand_ops - Get the GPMC NAND interface
- * @regs: the GPMC NAND register map exclusive for NAND use.
+ * @reg: the GPMC NAND register map exclusive for NAND use.
* @cs: GPMC chip select number on which the NAND sits. The
* register map returned will be specific to this chip select.
*
--
2.17.1
^ permalink raw reply related [flat|nested] 35+ messages in thread
* Re: [PATCH 04/16] memory: omap-gpmc: Correct kerneldoc
2020-07-24 18:23 ` [PATCH 04/16] memory: omap-gpmc: Correct kerneldoc Krzysztof Kozlowski
@ 2020-07-27 8:16 ` Krzysztof Kozlowski
0 siblings, 0 replies; 35+ messages in thread
From: Krzysztof Kozlowski @ 2020-07-27 8:16 UTC (permalink / raw)
To: Florian Fainelli, Ray Jui, Scott Branden,
bcm-kernel-feedback-list, Russell King, Catalin Marinas,
Will Deacon, Markus Mayer, Roger Quadros, Tony Lindgren,
Vladimir Zapolskiy, Kukjin Kim, Lukasz Luba, linux-arm-kernel,
linux-kernel, linux-omap, linux-samsung-soc, linux-pm
Cc: Olof Johansson, Arnd Bergmann
On Fri, Jul 24, 2020 at 08:23:16PM +0200, Krzysztof Kozlowski wrote:
> Use proper kerneldoc to fix GCC warnings like:
>
> drivers/memory/omap-gpmc.c:299: warning: Function parameter or member 'cs' not described in 'gpmc_get_clk_period'
> drivers/memory/omap-gpmc.c:432: warning: Excess function parameter 'ma' description in 'get_gpmc_timing_reg'
>
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> ---
> drivers/memory/omap-gpmc.c | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
Applied to drivers/memory tree.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 35+ messages in thread
* [PATCH 05/16] memory: omap-gpmc: Enclose macro argument usage in parenthesis
2020-07-24 18:23 [PATCH 00/16] memory: Further cleanups Krzysztof Kozlowski
` (3 preceding siblings ...)
2020-07-24 18:23 ` [PATCH 04/16] memory: omap-gpmc: Correct kerneldoc Krzysztof Kozlowski
@ 2020-07-24 18:23 ` Krzysztof Kozlowski
2020-07-27 8:16 ` Krzysztof Kozlowski
2020-07-24 18:23 ` [PATCH 06/16] memory: omap-gpmc: Use 'unsigned int' for consistency Krzysztof Kozlowski
` (10 subsequent siblings)
15 siblings, 1 reply; 35+ messages in thread
From: Krzysztof Kozlowski @ 2020-07-24 18:23 UTC (permalink / raw)
To: Florian Fainelli, Ray Jui, Scott Branden,
bcm-kernel-feedback-list, Russell King, Catalin Marinas,
Will Deacon, Krzysztof Kozlowski, Markus Mayer, Roger Quadros,
Tony Lindgren, Vladimir Zapolskiy, Kukjin Kim, Lukasz Luba,
linux-arm-kernel, linux-kernel, linux-omap, linux-samsung-soc,
linux-pm
Cc: Olof Johansson, Arnd Bergmann
Macros arguments should be enclosed by parenthesis for safety.
Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
drivers/memory/omap-gpmc.c | 20 ++++++++++----------
1 file changed, 10 insertions(+), 10 deletions(-)
diff --git a/drivers/memory/omap-gpmc.c b/drivers/memory/omap-gpmc.c
index c91a021ac421..62577255ba0f 100644
--- a/drivers/memory/omap-gpmc.c
+++ b/drivers/memory/omap-gpmc.c
@@ -107,8 +107,8 @@
#define ENABLE_PREFETCH (0x1 << 7)
#define DMA_MPU_MODE 2
-#define GPMC_REVISION_MAJOR(l) ((l >> 4) & 0xf)
-#define GPMC_REVISION_MINOR(l) (l & 0xf)
+#define GPMC_REVISION_MAJOR(l) (((l) >> 4) & 0xf)
+#define GPMC_REVISION_MINOR(l) ((l) & 0xf)
#define GPMC_HAS_WR_ACCESS 0x1
#define GPMC_HAS_WR_DATA_MUX_BUS 0x2
@@ -139,27 +139,27 @@
#define GPMC_CONFIG1_WRITEMULTIPLE_SUPP (1 << 28)
#define GPMC_CONFIG1_WRITETYPE_ASYNC (0 << 27)
#define GPMC_CONFIG1_WRITETYPE_SYNC (1 << 27)
-#define GPMC_CONFIG1_CLKACTIVATIONTIME(val) ((val & 3) << 25)
+#define GPMC_CONFIG1_CLKACTIVATIONTIME(val) (((val) & 3) << 25)
/** CLKACTIVATIONTIME Max Ticks */
#define GPMC_CONFIG1_CLKACTIVATIONTIME_MAX 2
-#define GPMC_CONFIG1_PAGE_LEN(val) ((val & 3) << 23)
+#define GPMC_CONFIG1_PAGE_LEN(val) (((val) & 3) << 23)
/** ATTACHEDDEVICEPAGELENGTH Max Value */
#define GPMC_CONFIG1_ATTACHEDDEVICEPAGELENGTH_MAX 2
#define GPMC_CONFIG1_WAIT_READ_MON (1 << 22)
#define GPMC_CONFIG1_WAIT_WRITE_MON (1 << 21)
-#define GPMC_CONFIG1_WAIT_MON_TIME(val) ((val & 3) << 18)
+#define GPMC_CONFIG1_WAIT_MON_TIME(val) (((val) & 3) << 18)
/** WAITMONITORINGTIME Max Ticks */
#define GPMC_CONFIG1_WAITMONITORINGTIME_MAX 2
-#define GPMC_CONFIG1_WAIT_PIN_SEL(val) ((val & 3) << 16)
-#define GPMC_CONFIG1_DEVICESIZE(val) ((val & 3) << 12)
+#define GPMC_CONFIG1_WAIT_PIN_SEL(val) (((val) & 3) << 16)
+#define GPMC_CONFIG1_DEVICESIZE(val) (((val) & 3) << 12)
#define GPMC_CONFIG1_DEVICESIZE_16 GPMC_CONFIG1_DEVICESIZE(1)
/** DEVICESIZE Max Value */
#define GPMC_CONFIG1_DEVICESIZE_MAX 1
-#define GPMC_CONFIG1_DEVICETYPE(val) ((val & 3) << 10)
+#define GPMC_CONFIG1_DEVICETYPE(val) (((val) & 3) << 10)
#define GPMC_CONFIG1_DEVICETYPE_NOR GPMC_CONFIG1_DEVICETYPE(0)
-#define GPMC_CONFIG1_MUXTYPE(val) ((val & 3) << 8)
+#define GPMC_CONFIG1_MUXTYPE(val) (((val) & 3) << 8)
#define GPMC_CONFIG1_TIME_PARA_GRAN (1 << 4)
-#define GPMC_CONFIG1_FCLK_DIV(val) (val & 3)
+#define GPMC_CONFIG1_FCLK_DIV(val) ((val) & 3)
#define GPMC_CONFIG1_FCLK_DIV2 (GPMC_CONFIG1_FCLK_DIV(1))
#define GPMC_CONFIG1_FCLK_DIV3 (GPMC_CONFIG1_FCLK_DIV(2))
#define GPMC_CONFIG1_FCLK_DIV4 (GPMC_CONFIG1_FCLK_DIV(3))
--
2.17.1
^ permalink raw reply related [flat|nested] 35+ messages in thread
* Re: [PATCH 05/16] memory: omap-gpmc: Enclose macro argument usage in parenthesis
2020-07-24 18:23 ` [PATCH 05/16] memory: omap-gpmc: Enclose macro argument usage in parenthesis Krzysztof Kozlowski
@ 2020-07-27 8:16 ` Krzysztof Kozlowski
0 siblings, 0 replies; 35+ messages in thread
From: Krzysztof Kozlowski @ 2020-07-27 8:16 UTC (permalink / raw)
To: Florian Fainelli, Ray Jui, Scott Branden,
bcm-kernel-feedback-list, Russell King, Catalin Marinas,
Will Deacon, Markus Mayer, Roger Quadros, Tony Lindgren,
Vladimir Zapolskiy, Kukjin Kim, Lukasz Luba, linux-arm-kernel,
linux-kernel, linux-omap, linux-samsung-soc, linux-pm
Cc: Olof Johansson, Arnd Bergmann
On Fri, Jul 24, 2020 at 08:23:17PM +0200, Krzysztof Kozlowski wrote:
> Macros arguments should be enclosed by parenthesis for safety.
>
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> ---
> drivers/memory/omap-gpmc.c | 20 ++++++++++----------
> 1 file changed, 10 insertions(+), 10 deletions(-)
Applied to drivers/memory tree.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 35+ messages in thread
* [PATCH 06/16] memory: omap-gpmc: Use 'unsigned int' for consistency
2020-07-24 18:23 [PATCH 00/16] memory: Further cleanups Krzysztof Kozlowski
` (4 preceding siblings ...)
2020-07-24 18:23 ` [PATCH 05/16] memory: omap-gpmc: Enclose macro argument usage in parenthesis Krzysztof Kozlowski
@ 2020-07-24 18:23 ` Krzysztof Kozlowski
2020-07-27 8:15 ` Krzysztof Kozlowski
2020-07-24 18:23 ` [PATCH 07/16] memory: omap-gpmc: Correct white space issues Krzysztof Kozlowski
` (9 subsequent siblings)
15 siblings, 1 reply; 35+ messages in thread
From: Krzysztof Kozlowski @ 2020-07-24 18:23 UTC (permalink / raw)
To: Florian Fainelli, Ray Jui, Scott Branden,
bcm-kernel-feedback-list, Russell King, Catalin Marinas,
Will Deacon, Krzysztof Kozlowski, Markus Mayer, Roger Quadros,
Tony Lindgren, Vladimir Zapolskiy, Kukjin Kim, Lukasz Luba,
linux-arm-kernel, linux-kernel, linux-omap, linux-samsung-soc,
linux-pm
Cc: Olof Johansson, Arnd Bergmann
Driver uses 'unsigned int' in other places instead of 'unsigned'.
Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
drivers/memory/omap-gpmc.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/memory/omap-gpmc.c b/drivers/memory/omap-gpmc.c
index 62577255ba0f..e4d2f359bcfa 100644
--- a/drivers/memory/omap-gpmc.c
+++ b/drivers/memory/omap-gpmc.c
@@ -243,7 +243,7 @@ static DEFINE_SPINLOCK(gpmc_mem_lock);
/* Define chip-selects as reserved by default until probe completes */
static unsigned int gpmc_cs_num = GPMC_CS_NUM;
static unsigned int gpmc_nr_waitpins;
-static unsigned gpmc_capability;
+static unsigned int gpmc_capability;
static void __iomem *gpmc_base;
static struct clk *gpmc_l3_clk;
@@ -1289,7 +1289,7 @@ int gpmc_omap_onenand_set_timings(struct device *dev, int cs, int freq,
}
EXPORT_SYMBOL_GPL(gpmc_omap_onenand_set_timings);
-int gpmc_get_client_irq(unsigned irq_config)
+int gpmc_get_client_irq(unsigned int irq_config)
{
if (!gpmc_irq_domain) {
pr_warn("%s called before GPMC IRQ domain available\n",
--
2.17.1
^ permalink raw reply related [flat|nested] 35+ messages in thread
* Re: [PATCH 06/16] memory: omap-gpmc: Use 'unsigned int' for consistency
2020-07-24 18:23 ` [PATCH 06/16] memory: omap-gpmc: Use 'unsigned int' for consistency Krzysztof Kozlowski
@ 2020-07-27 8:15 ` Krzysztof Kozlowski
0 siblings, 0 replies; 35+ messages in thread
From: Krzysztof Kozlowski @ 2020-07-27 8:15 UTC (permalink / raw)
To: Florian Fainelli, Ray Jui, Scott Branden,
bcm-kernel-feedback-list, Russell King, Catalin Marinas,
Will Deacon, Markus Mayer, Roger Quadros, Tony Lindgren,
Vladimir Zapolskiy, Kukjin Kim, Lukasz Luba, linux-arm-kernel,
linux-kernel, linux-omap, linux-samsung-soc, linux-pm
Cc: Olof Johansson, Arnd Bergmann
On Fri, Jul 24, 2020 at 08:23:18PM +0200, Krzysztof Kozlowski wrote:
> Driver uses 'unsigned int' in other places instead of 'unsigned'.
>
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> ---
> drivers/memory/omap-gpmc.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
Applied to drivers/memory tree.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 35+ messages in thread
* [PATCH 07/16] memory: omap-gpmc: Correct white space issues
2020-07-24 18:23 [PATCH 00/16] memory: Further cleanups Krzysztof Kozlowski
` (5 preceding siblings ...)
2020-07-24 18:23 ` [PATCH 06/16] memory: omap-gpmc: Use 'unsigned int' for consistency Krzysztof Kozlowski
@ 2020-07-24 18:23 ` Krzysztof Kozlowski
2020-07-27 8:15 ` Krzysztof Kozlowski
2020-07-24 18:23 ` [RFT 08/16] memory: omap-gpmc: Use WARN() instead of BUG() on wrong free Krzysztof Kozlowski
` (8 subsequent siblings)
15 siblings, 1 reply; 35+ messages in thread
From: Krzysztof Kozlowski @ 2020-07-24 18:23 UTC (permalink / raw)
To: Florian Fainelli, Ray Jui, Scott Branden,
bcm-kernel-feedback-list, Russell King, Catalin Marinas,
Will Deacon, Krzysztof Kozlowski, Markus Mayer, Roger Quadros,
Tony Lindgren, Vladimir Zapolskiy, Kukjin Kim, Lukasz Luba,
linux-arm-kernel, linux-kernel, linux-omap, linux-samsung-soc,
linux-pm
Cc: Olof Johansson, Arnd Bergmann
Remove some unneeded blank lines, align indentation with open
parenthesis (or fix existing alignment).
Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
drivers/memory/omap-gpmc.c | 24 +++++++-----------------
1 file changed, 7 insertions(+), 17 deletions(-)
diff --git a/drivers/memory/omap-gpmc.c b/drivers/memory/omap-gpmc.c
index e4d2f359bcfa..9339a7c75fdd 100644
--- a/drivers/memory/omap-gpmc.c
+++ b/drivers/memory/omap-gpmc.c
@@ -297,7 +297,6 @@ static unsigned long gpmc_get_fclk_period(void)
*/
static unsigned long gpmc_get_clk_period(int cs, enum gpmc_clk_domain cd)
{
-
unsigned long tick_ps = gpmc_get_fclk_period();
u32 l;
int div;
@@ -317,7 +316,6 @@ static unsigned long gpmc_get_clk_period(int cs, enum gpmc_clk_domain cd)
}
return tick_ps;
-
}
static unsigned int gpmc_ns_to_clk_ticks(unsigned int time_ns, int cs,
@@ -509,7 +507,7 @@ static void gpmc_cs_show_timings(int cs, const char *desc)
GPMC_GET_RAW_BOOL(GPMC_CS_CONFIG1, 4, 4, "time-para-granularity");
GPMC_GET_RAW(GPMC_CS_CONFIG1, 8, 9, "mux-add-data");
GPMC_GET_RAW_SHIFT_MAX(GPMC_CS_CONFIG1, 12, 13, 1,
- GPMC_CONFIG1_DEVICESIZE_MAX, "device-width");
+ GPMC_CONFIG1_DEVICESIZE_MAX, "device-width");
GPMC_GET_RAW(GPMC_CS_CONFIG1, 16, 17, "wait-pin");
GPMC_GET_RAW_BOOL(GPMC_CS_CONFIG1, 21, 21, "wait-on-write");
GPMC_GET_RAW_BOOL(GPMC_CS_CONFIG1, 22, 22, "wait-on-read");
@@ -623,9 +621,8 @@ static int set_gpmc_timing_reg(int cs, int reg, int st_bit, int end_bit, int max
l = gpmc_cs_read_reg(cs, reg);
#ifdef CONFIG_OMAP_GPMC_DEBUG
- pr_info(
- "GPMC CS%d: %-17s: %3d ticks, %3lu ns (was %3i ticks) %3d ns\n",
- cs, name, ticks, gpmc_get_clk_period(cs, cd) * ticks / 1000,
+ pr_info("GPMC CS%d: %-17s: %3d ticks, %3lu ns (was %3i ticks) %3d ns\n",
+ cs, name, ticks, gpmc_get_clk_period(cs, cd) * ticks / 1000,
(l >> st_bit) & mask, time);
#endif
l &= ~(mask << st_bit);
@@ -652,7 +649,6 @@ static int set_gpmc_timing_reg(int cs, int reg, int st_bit, int end_bit, int max
*/
static int gpmc_calc_waitmonitoring_divider(unsigned int wait_monitoring)
{
-
int div = gpmc_ns_to_ticks(wait_monitoring);
div += GPMC_CONFIG1_WAITMONITORINGTIME_MAX - 1;
@@ -664,7 +660,6 @@ static int gpmc_calc_waitmonitoring_divider(unsigned int wait_monitoring)
div = 1;
return div;
-
}
/**
@@ -718,7 +713,6 @@ int gpmc_cs_set_timings(int cs, const struct gpmc_timings *t,
if (!s->sync_read && !s->sync_write &&
(s->wait_on_read || s->wait_on_write)
) {
-
div = gpmc_calc_waitmonitoring_divider(t->wait_monitoring);
if (div < 0) {
pr_err("%s: waitmonitoringtime %3d ns too large for greatest gpmcfclkdivider.\n",
@@ -1512,7 +1506,6 @@ static void gpmc_mem_exit(void)
continue;
gpmc_cs_delete_mem(cs);
}
-
}
static void gpmc_mem_init(void)
@@ -1681,17 +1674,14 @@ static int gpmc_calc_async_read_timings(struct gpmc_timings *gpmc_t,
/* oe_on */
temp = dev_t->t_oeasu;
if (mux)
- temp = max_t(u32, temp,
- gpmc_t->adv_rd_off + dev_t->t_aavdh);
+ temp = max_t(u32, temp, gpmc_t->adv_rd_off + dev_t->t_aavdh);
gpmc_t->oe_on = gpmc_round_ps_to_ticks(temp);
/* access */
temp = max_t(u32, dev_t->t_iaa, /* XXX: remove t_iaa in async ? */
- gpmc_t->oe_on + dev_t->t_oe);
- temp = max_t(u32, temp,
- gpmc_t->cs_on + dev_t->t_ce);
- temp = max_t(u32, temp,
- gpmc_t->adv_on + dev_t->t_aa);
+ gpmc_t->oe_on + dev_t->t_oe);
+ temp = max_t(u32, temp, gpmc_t->cs_on + dev_t->t_ce);
+ temp = max_t(u32, temp, gpmc_t->adv_on + dev_t->t_aa);
gpmc_t->access = gpmc_round_ps_to_ticks(temp);
gpmc_t->oe_off = gpmc_t->access + gpmc_ticks_to_ps(1);
--
2.17.1
^ permalink raw reply related [flat|nested] 35+ messages in thread
* Re: [PATCH 07/16] memory: omap-gpmc: Correct white space issues
2020-07-24 18:23 ` [PATCH 07/16] memory: omap-gpmc: Correct white space issues Krzysztof Kozlowski
@ 2020-07-27 8:15 ` Krzysztof Kozlowski
0 siblings, 0 replies; 35+ messages in thread
From: Krzysztof Kozlowski @ 2020-07-27 8:15 UTC (permalink / raw)
To: Florian Fainelli, Ray Jui, Scott Branden,
bcm-kernel-feedback-list, Russell King, Catalin Marinas,
Will Deacon, Markus Mayer, Roger Quadros, Tony Lindgren,
Vladimir Zapolskiy, Kukjin Kim, Lukasz Luba, linux-arm-kernel,
linux-kernel, linux-omap, linux-samsung-soc, linux-pm
Cc: Olof Johansson, Arnd Bergmann
On Fri, Jul 24, 2020 at 08:23:19PM +0200, Krzysztof Kozlowski wrote:
> Remove some unneeded blank lines, align indentation with open
> parenthesis (or fix existing alignment).
>
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> ---
> drivers/memory/omap-gpmc.c | 24 +++++++-----------------
> 1 file changed, 7 insertions(+), 17 deletions(-)
Applied to drivers/memory tree.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 35+ messages in thread
* [RFT 08/16] memory: omap-gpmc: Use WARN() instead of BUG() on wrong free
2020-07-24 18:23 [PATCH 00/16] memory: Further cleanups Krzysztof Kozlowski
` (6 preceding siblings ...)
2020-07-24 18:23 ` [PATCH 07/16] memory: omap-gpmc: Correct white space issues Krzysztof Kozlowski
@ 2020-07-24 18:23 ` Krzysztof Kozlowski
2020-08-17 18:36 ` Krzysztof Kozlowski
2020-07-24 18:23 ` [PATCH 09/16] memory: omap-gpmc: Fix language typo Krzysztof Kozlowski
` (7 subsequent siblings)
15 siblings, 1 reply; 35+ messages in thread
From: Krzysztof Kozlowski @ 2020-07-24 18:23 UTC (permalink / raw)
To: Florian Fainelli, Ray Jui, Scott Branden,
bcm-kernel-feedback-list, Russell King, Catalin Marinas,
Will Deacon, Krzysztof Kozlowski, Markus Mayer, Roger Quadros,
Tony Lindgren, Vladimir Zapolskiy, Kukjin Kim, Lukasz Luba,
linux-arm-kernel, linux-kernel, linux-omap, linux-samsung-soc,
linux-pm
Cc: Olof Johansson, Arnd Bergmann
Since driver tracks reserved memory, freeing a non-reserved GPMC should
not be fatal and crash the system. Printing a warning is friendlier.
Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
Not tested on HW.
---
drivers/memory/omap-gpmc.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/memory/omap-gpmc.c b/drivers/memory/omap-gpmc.c
index 9339a7c75fdd..414bfb481266 100644
--- a/drivers/memory/omap-gpmc.c
+++ b/drivers/memory/omap-gpmc.c
@@ -1072,8 +1072,7 @@ void gpmc_cs_free(int cs)
spin_lock(&gpmc_mem_lock);
if (cs >= gpmc_cs_num || cs < 0 || !gpmc_cs_reserved(cs)) {
- printk(KERN_ERR "Trying to free non-reserved GPMC CS%d\n", cs);
- BUG();
+ WARN(1, "Trying to free non-reserved GPMC CS%d\n", cs);
spin_unlock(&gpmc_mem_lock);
return;
}
--
2.17.1
^ permalink raw reply related [flat|nested] 35+ messages in thread
* Re: [RFT 08/16] memory: omap-gpmc: Use WARN() instead of BUG() on wrong free
2020-07-24 18:23 ` [RFT 08/16] memory: omap-gpmc: Use WARN() instead of BUG() on wrong free Krzysztof Kozlowski
@ 2020-08-17 18:36 ` Krzysztof Kozlowski
0 siblings, 0 replies; 35+ messages in thread
From: Krzysztof Kozlowski @ 2020-08-17 18:36 UTC (permalink / raw)
To: Florian Fainelli, Ray Jui, Scott Branden,
bcm-kernel-feedback-list, Russell King, Catalin Marinas,
Will Deacon, Markus Mayer, Roger Quadros, Tony Lindgren,
Vladimir Zapolskiy, Kukjin Kim, Lukasz Luba, linux-arm-kernel,
linux-kernel, linux-omap, linux-samsung-soc, linux-pm
Cc: Olof Johansson, Arnd Bergmann
On Fri, Jul 24, 2020 at 08:23:20PM +0200, Krzysztof Kozlowski wrote:
> Since driver tracks reserved memory, freeing a non-reserved GPMC should
> not be fatal and crash the system. Printing a warning is friendlier.
>
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
>
> ---
>
> Not tested on HW.
> ---
> drivers/memory/omap-gpmc.c | 3 +--
Applied.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 35+ messages in thread
* [PATCH 09/16] memory: omap-gpmc: Fix language typo
2020-07-24 18:23 [PATCH 00/16] memory: Further cleanups Krzysztof Kozlowski
` (7 preceding siblings ...)
2020-07-24 18:23 ` [RFT 08/16] memory: omap-gpmc: Use WARN() instead of BUG() on wrong free Krzysztof Kozlowski
@ 2020-07-24 18:23 ` Krzysztof Kozlowski
2020-07-27 8:15 ` Krzysztof Kozlowski
2020-07-24 18:23 ` [PATCH 10/16] memory: omap-gpmc: Consistently use !res for NULL checks Krzysztof Kozlowski
` (6 subsequent siblings)
15 siblings, 1 reply; 35+ messages in thread
From: Krzysztof Kozlowski @ 2020-07-24 18:23 UTC (permalink / raw)
To: Florian Fainelli, Ray Jui, Scott Branden,
bcm-kernel-feedback-list, Russell King, Catalin Marinas,
Will Deacon, Krzysztof Kozlowski, Markus Mayer, Roger Quadros,
Tony Lindgren, Vladimir Zapolskiy, Kukjin Kim, Lukasz Luba,
linux-arm-kernel, linux-kernel, linux-omap, linux-samsung-soc,
linux-pm
Cc: Olof Johansson, Arnd Bergmann
Fix arbitary -> arbitrary.
Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
drivers/memory/omap-gpmc.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/memory/omap-gpmc.c b/drivers/memory/omap-gpmc.c
index 414bfb481266..76a9c700cbd5 100644
--- a/drivers/memory/omap-gpmc.c
+++ b/drivers/memory/omap-gpmc.c
@@ -2126,7 +2126,7 @@ static int gpmc_probe_generic_child(struct platform_device *pdev,
gpmc_cs_disable_mem(cs);
/*
- * FIXME: gpmc_cs_request() will map the CS to an arbitary
+ * FIXME: gpmc_cs_request() will map the CS to an arbitrary
* location in the gpmc address space. When booting with
* device-tree we want the NOR flash to be mapped to the
* location specified in the device-tree blob. So remap the
--
2.17.1
^ permalink raw reply related [flat|nested] 35+ messages in thread
* Re: [PATCH 09/16] memory: omap-gpmc: Fix language typo
2020-07-24 18:23 ` [PATCH 09/16] memory: omap-gpmc: Fix language typo Krzysztof Kozlowski
@ 2020-07-27 8:15 ` Krzysztof Kozlowski
0 siblings, 0 replies; 35+ messages in thread
From: Krzysztof Kozlowski @ 2020-07-27 8:15 UTC (permalink / raw)
To: Florian Fainelli, Ray Jui, Scott Branden,
bcm-kernel-feedback-list, Russell King, Catalin Marinas,
Will Deacon, Markus Mayer, Roger Quadros, Tony Lindgren,
Vladimir Zapolskiy, Kukjin Kim, Lukasz Luba, linux-arm-kernel,
linux-kernel, linux-omap, linux-samsung-soc, linux-pm
Cc: Olof Johansson, Arnd Bergmann
On Fri, Jul 24, 2020 at 08:23:21PM +0200, Krzysztof Kozlowski wrote:
> Fix arbitary -> arbitrary.
>
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> ---
> drivers/memory/omap-gpmc.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Applied to drivers/memory tree.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 35+ messages in thread
* [PATCH 10/16] memory: omap-gpmc: Consistently use !res for NULL checks
2020-07-24 18:23 [PATCH 00/16] memory: Further cleanups Krzysztof Kozlowski
` (8 preceding siblings ...)
2020-07-24 18:23 ` [PATCH 09/16] memory: omap-gpmc: Fix language typo Krzysztof Kozlowski
@ 2020-07-24 18:23 ` Krzysztof Kozlowski
2020-08-17 18:37 ` Krzysztof Kozlowski
2020-07-24 18:23 ` [PATCH 11/16] memory: brcmstb_dpfe: Fix language typo Krzysztof Kozlowski
` (5 subsequent siblings)
15 siblings, 1 reply; 35+ messages in thread
From: Krzysztof Kozlowski @ 2020-07-24 18:23 UTC (permalink / raw)
To: Florian Fainelli, Ray Jui, Scott Branden,
bcm-kernel-feedback-list, Russell King, Catalin Marinas,
Will Deacon, Krzysztof Kozlowski, Markus Mayer, Roger Quadros,
Tony Lindgren, Vladimir Zapolskiy, Kukjin Kim, Lukasz Luba,
linux-arm-kernel, linux-kernel, linux-omap, linux-samsung-soc,
linux-pm
Cc: Olof Johansson, Arnd Bergmann
The driver already uses 'if (!res)' pattern in the probe function so be
consistent.
Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
drivers/memory/omap-gpmc.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/memory/omap-gpmc.c b/drivers/memory/omap-gpmc.c
index 76a9c700cbd5..ce0e7e2d7cff 100644
--- a/drivers/memory/omap-gpmc.c
+++ b/drivers/memory/omap-gpmc.c
@@ -2393,7 +2393,7 @@ static int gpmc_probe(struct platform_device *pdev)
platform_set_drvdata(pdev, gpmc);
res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
- if (res == NULL)
+ if (!res)
return -ENOENT;
gpmc_base = devm_ioremap_resource(&pdev->dev, res);
--
2.17.1
^ permalink raw reply related [flat|nested] 35+ messages in thread
* Re: [PATCH 10/16] memory: omap-gpmc: Consistently use !res for NULL checks
2020-07-24 18:23 ` [PATCH 10/16] memory: omap-gpmc: Consistently use !res for NULL checks Krzysztof Kozlowski
@ 2020-08-17 18:37 ` Krzysztof Kozlowski
0 siblings, 0 replies; 35+ messages in thread
From: Krzysztof Kozlowski @ 2020-08-17 18:37 UTC (permalink / raw)
To: Florian Fainelli, Ray Jui, Scott Branden,
bcm-kernel-feedback-list, Russell King, Catalin Marinas,
Will Deacon, Markus Mayer, Roger Quadros, Tony Lindgren,
Vladimir Zapolskiy, Kukjin Kim, Lukasz Luba, linux-arm-kernel,
linux-kernel, linux-omap, linux-samsung-soc, linux-pm
Cc: Olof Johansson, Arnd Bergmann
On Fri, Jul 24, 2020 at 08:23:22PM +0200, Krzysztof Kozlowski wrote:
> The driver already uses 'if (!res)' pattern in the probe function so be
> consistent.
>
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> ---
> drivers/memory/omap-gpmc.c | 2 +-
Applied.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 35+ messages in thread
* [PATCH 11/16] memory: brcmstb_dpfe: Fix language typo
2020-07-24 18:23 [PATCH 00/16] memory: Further cleanups Krzysztof Kozlowski
` (9 preceding siblings ...)
2020-07-24 18:23 ` [PATCH 10/16] memory: omap-gpmc: Consistently use !res for NULL checks Krzysztof Kozlowski
@ 2020-07-24 18:23 ` Krzysztof Kozlowski
2020-07-24 18:38 ` Florian Fainelli
2020-07-24 18:23 ` [RFT 12/16] memory: brcmstb_dpfe: Add separate entry for compile test Krzysztof Kozlowski
` (4 subsequent siblings)
15 siblings, 1 reply; 35+ messages in thread
From: Krzysztof Kozlowski @ 2020-07-24 18:23 UTC (permalink / raw)
To: Florian Fainelli, Ray Jui, Scott Branden,
bcm-kernel-feedback-list, Russell King, Catalin Marinas,
Will Deacon, Krzysztof Kozlowski, Markus Mayer, Roger Quadros,
Tony Lindgren, Vladimir Zapolskiy, Kukjin Kim, Lukasz Luba,
linux-arm-kernel, linux-kernel, linux-omap, linux-samsung-soc,
linux-pm
Cc: Olof Johansson, Arnd Bergmann
Fix firwmare -> firmware.
Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
drivers/memory/brcmstb_dpfe.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/memory/brcmstb_dpfe.c b/drivers/memory/brcmstb_dpfe.c
index 604f0e3fb112..60e8633b1175 100644
--- a/drivers/memory/brcmstb_dpfe.c
+++ b/drivers/memory/brcmstb_dpfe.c
@@ -23,7 +23,7 @@
* - BE kernel + LE firmware image
* - BE kernel + BE firmware image
*
- * The DPCU always runs in big endian mode. The firwmare image, however, can
+ * The DPCU always runs in big endian mode. The firmware image, however, can
* be in either format. Also, communication between host CPU and DCPU is
* always in little endian.
*/
--
2.17.1
^ permalink raw reply related [flat|nested] 35+ messages in thread
* Re: [PATCH 11/16] memory: brcmstb_dpfe: Fix language typo
2020-07-24 18:23 ` [PATCH 11/16] memory: brcmstb_dpfe: Fix language typo Krzysztof Kozlowski
@ 2020-07-24 18:38 ` Florian Fainelli
2020-07-27 8:14 ` Krzysztof Kozlowski
0 siblings, 1 reply; 35+ messages in thread
From: Florian Fainelli @ 2020-07-24 18:38 UTC (permalink / raw)
To: Krzysztof Kozlowski, Ray Jui, Scott Branden,
bcm-kernel-feedback-list, Russell King, Catalin Marinas,
Will Deacon, Markus Mayer, Roger Quadros, Tony Lindgren,
Vladimir Zapolskiy, Kukjin Kim, Lukasz Luba, linux-arm-kernel,
linux-kernel, linux-omap, linux-samsung-soc, linux-pm
Cc: Olof Johansson, Arnd Bergmann
On 7/24/20 11:23 AM, Krzysztof Kozlowski wrote:
> Fix firwmare -> firmware.
>
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
Acked-by: Florian Fainelli <f.fainelli@gmail.com>
--
Florian
^ permalink raw reply [flat|nested] 35+ messages in thread
* Re: [PATCH 11/16] memory: brcmstb_dpfe: Fix language typo
2020-07-24 18:38 ` Florian Fainelli
@ 2020-07-27 8:14 ` Krzysztof Kozlowski
0 siblings, 0 replies; 35+ messages in thread
From: Krzysztof Kozlowski @ 2020-07-27 8:14 UTC (permalink / raw)
To: Florian Fainelli
Cc: Ray Jui, Scott Branden, bcm-kernel-feedback-list, Russell King,
Catalin Marinas, Will Deacon, Markus Mayer, Roger Quadros,
Tony Lindgren, Vladimir Zapolskiy, Kukjin Kim, Lukasz Luba,
linux-arm-kernel, linux-kernel, linux-omap, linux-samsung-soc,
linux-pm, Olof Johansson, Arnd Bergmann
On Fri, Jul 24, 2020 at 11:38:57AM -0700, Florian Fainelli wrote:
> On 7/24/20 11:23 AM, Krzysztof Kozlowski wrote:
> > Fix firwmare -> firmware.
> >
> > Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
>
> Acked-by: Florian Fainelli <f.fainelli@gmail.com>
Thanks!
Applied to drivers/memory tree.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 35+ messages in thread
* [RFT 12/16] memory: brcmstb_dpfe: Add separate entry for compile test
2020-07-24 18:23 [PATCH 00/16] memory: Further cleanups Krzysztof Kozlowski
` (10 preceding siblings ...)
2020-07-24 18:23 ` [PATCH 11/16] memory: brcmstb_dpfe: Fix language typo Krzysztof Kozlowski
@ 2020-07-24 18:23 ` Krzysztof Kozlowski
2020-07-24 18:40 ` Florian Fainelli
2020-07-24 18:23 ` [PATCH 13/16] memory: of: Correct kerneldoc Krzysztof Kozlowski
` (3 subsequent siblings)
15 siblings, 1 reply; 35+ messages in thread
From: Krzysztof Kozlowski @ 2020-07-24 18:23 UTC (permalink / raw)
To: Florian Fainelli, Ray Jui, Scott Branden,
bcm-kernel-feedback-list, Russell King, Catalin Marinas,
Will Deacon, Krzysztof Kozlowski, Markus Mayer, Roger Quadros,
Tony Lindgren, Vladimir Zapolskiy, Kukjin Kim, Lukasz Luba,
linux-arm-kernel, linux-kernel, linux-omap, linux-samsung-soc,
linux-pm
Cc: Olof Johansson, Arnd Bergmann
Add separate entry for Broadcom STB DPFE driver, selected by platforms
by default. This allows further customization and compile testing.
Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
I have other changes around drivers/memory/Kconfig pending. Sending them
together would avoid conflicts.
---
arch/arm/mach-bcm/Kconfig | 1 +
arch/arm64/Kconfig.platforms | 1 +
drivers/memory/Kconfig | 11 +++++++++++
drivers/memory/Makefile | 2 +-
4 files changed, 14 insertions(+), 1 deletion(-)
diff --git a/arch/arm/mach-bcm/Kconfig b/arch/arm/mach-bcm/Kconfig
index 1df0ee01ee02..b0c45238e69b 100644
--- a/arch/arm/mach-bcm/Kconfig
+++ b/arch/arm/mach-bcm/Kconfig
@@ -208,6 +208,7 @@ config ARCH_BRCMSTB
select ARM_GIC
select ARM_ERRATA_798181 if SMP
select HAVE_ARM_ARCH_TIMER
+ select BRCMSTB_DPFE
select BRCMSTB_L2_IRQ
select BCM7120_L2_IRQ
select ARCH_HAS_HOLES_MEMORYMODEL
diff --git a/arch/arm64/Kconfig.platforms b/arch/arm64/Kconfig.platforms
index ddc3bf75257f..f7791ed0ed6d 100644
--- a/arch/arm64/Kconfig.platforms
+++ b/arch/arm64/Kconfig.platforms
@@ -68,6 +68,7 @@ config ARCH_BRCMSTB
bool "Broadcom Set-Top-Box SoCs"
select ARCH_HAS_RESET_CONTROLLER
select BCM7038_L1_IRQ
+ select BRCMSTB_DPFE
select BRCMSTB_L2_IRQ
select GENERIC_IRQ_CHIP
select PINCTRL
diff --git a/drivers/memory/Kconfig b/drivers/memory/Kconfig
index cd4fc93b50df..fef6f0873fda 100644
--- a/drivers/memory/Kconfig
+++ b/drivers/memory/Kconfig
@@ -54,6 +54,17 @@ config ATMEL_EBI
tree is used. This bus supports NANDs, external ethernet controller,
SRAMs, ATA devices, etc.
+config BRCMSTB_DPFE
+ bool "Broadcom STB DPFE driver" if COMPILE_TEST
+ depends on ARCH_BRCMSTB || COMPILE_TEST
+ help
+ This driver provides access to the DPFE interface of Broadcom
+ STB SoCs. The firmware running on the DCPU inside the DDR PHY can
+ provide current information about the system's RAM, for instance
+ the DRAM refresh rate. This can be used as an indirect indicator
+ for the DRAM's temperature. Slower refresh rate means cooler RAM,
+ higher refresh rate means hotter RAM.
+
config BT1_L2_CTL
bool "Baikal-T1 CM2 L2-RAM Cache Control Block"
depends on MIPS_BAIKAL_T1 || COMPILE_TEST
diff --git a/drivers/memory/Makefile b/drivers/memory/Makefile
index b4533ffff2bc..e71cf7b99641 100644
--- a/drivers/memory/Makefile
+++ b/drivers/memory/Makefile
@@ -10,7 +10,7 @@ endif
obj-$(CONFIG_ARM_PL172_MPMC) += pl172.o
obj-$(CONFIG_ATMEL_SDRAMC) += atmel-sdramc.o
obj-$(CONFIG_ATMEL_EBI) += atmel-ebi.o
-obj-$(CONFIG_ARCH_BRCMSTB) += brcmstb_dpfe.o
+obj-$(CONFIG_BRCMSTB_DPFE) += brcmstb_dpfe.o
obj-$(CONFIG_BT1_L2_CTL) += bt1-l2-ctl.o
obj-$(CONFIG_TI_AEMIF) += ti-aemif.o
obj-$(CONFIG_TI_EMIF) += emif.o
--
2.17.1
^ permalink raw reply related [flat|nested] 35+ messages in thread
* Re: [RFT 12/16] memory: brcmstb_dpfe: Add separate entry for compile test
2020-07-24 18:23 ` [RFT 12/16] memory: brcmstb_dpfe: Add separate entry for compile test Krzysztof Kozlowski
@ 2020-07-24 18:40 ` Florian Fainelli
2020-07-24 18:42 ` Krzysztof Kozlowski
2020-08-17 18:38 ` Krzysztof Kozlowski
0 siblings, 2 replies; 35+ messages in thread
From: Florian Fainelli @ 2020-07-24 18:40 UTC (permalink / raw)
To: Krzysztof Kozlowski, Ray Jui, Scott Branden,
bcm-kernel-feedback-list, Russell King, Catalin Marinas,
Will Deacon, Markus Mayer, Roger Quadros, Tony Lindgren,
Vladimir Zapolskiy, Kukjin Kim, Lukasz Luba, linux-arm-kernel,
linux-kernel, linux-omap, linux-samsung-soc, linux-pm
Cc: Olof Johansson, Arnd Bergmann
On 7/24/20 11:23 AM, Krzysztof Kozlowski wrote:
> Add separate entry for Broadcom STB DPFE driver, selected by platforms
> by default. This allows further customization and compile testing.
>
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> ---
>
> I have other changes around drivers/memory/Kconfig pending. Sending them
> together would avoid conflicts.
>
> ---
> arch/arm/mach-bcm/Kconfig | 1 +
> arch/arm64/Kconfig.platforms | 1 +
> drivers/memory/Kconfig | 11 +++++++++++
> drivers/memory/Makefile | 2 +-
> 4 files changed, 14 insertions(+), 1 deletion(-)
>
> diff --git a/arch/arm/mach-bcm/Kconfig b/arch/arm/mach-bcm/Kconfig
> index 1df0ee01ee02..b0c45238e69b 100644
> --- a/arch/arm/mach-bcm/Kconfig
> +++ b/arch/arm/mach-bcm/Kconfig
> @@ -208,6 +208,7 @@ config ARCH_BRCMSTB
> select ARM_GIC
> select ARM_ERRATA_798181 if SMP
> select HAVE_ARM_ARCH_TIMER
> + select BRCMSTB_DPFE
> select BRCMSTB_L2_IRQ
> select BCM7120_L2_IRQ
> select ARCH_HAS_HOLES_MEMORYMODEL
> diff --git a/arch/arm64/Kconfig.platforms b/arch/arm64/Kconfig.platforms
> index ddc3bf75257f..f7791ed0ed6d 100644
> --- a/arch/arm64/Kconfig.platforms
> +++ b/arch/arm64/Kconfig.platforms
> @@ -68,6 +68,7 @@ config ARCH_BRCMSTB
> bool "Broadcom Set-Top-Box SoCs"
> select ARCH_HAS_RESET_CONTROLLER
> select BCM7038_L1_IRQ
> + select BRCMSTB_DPFE
The select is probably not warranted, see below.
> select BRCMSTB_L2_IRQ
> select GENERIC_IRQ_CHIP
> select PINCTRL
> diff --git a/drivers/memory/Kconfig b/drivers/memory/Kconfig
> index cd4fc93b50df..fef6f0873fda 100644
> --- a/drivers/memory/Kconfig
> +++ b/drivers/memory/Kconfig
> @@ -54,6 +54,17 @@ config ATMEL_EBI
> tree is used. This bus supports NANDs, external ethernet controller,
> SRAMs, ATA devices, etc.
>
> +config BRCMSTB_DPFE
> + bool "Broadcom STB DPFE driver" if COMPILE_TEST
> + depends on ARCH_BRCMSTB || COMPILE_TEST
Can you make it default ARCH_BRCMSTB and remove the select above?
With that:
Acked-by: Florian Fainelli <f.fainelli@gmail.com>
> + help
> + This driver provides access to the DPFE interface of Broadcom
> + STB SoCs. The firmware running on the DCPU inside the DDR PHY can
> + provide current information about the system's RAM, for instance
> + the DRAM refresh rate. This can be used as an indirect indicator
> + for the DRAM's temperature. Slower refresh rate means cooler RAM,
> + higher refresh rate means hotter RAM.
> +
> config BT1_L2_CTL
> bool "Baikal-T1 CM2 L2-RAM Cache Control Block"
> depends on MIPS_BAIKAL_T1 || COMPILE_TEST
> diff --git a/drivers/memory/Makefile b/drivers/memory/Makefile
> index b4533ffff2bc..e71cf7b99641 100644
> --- a/drivers/memory/Makefile
> +++ b/drivers/memory/Makefile
> @@ -10,7 +10,7 @@ endif
> obj-$(CONFIG_ARM_PL172_MPMC) += pl172.o
> obj-$(CONFIG_ATMEL_SDRAMC) += atmel-sdramc.o
> obj-$(CONFIG_ATMEL_EBI) += atmel-ebi.o
> -obj-$(CONFIG_ARCH_BRCMSTB) += brcmstb_dpfe.o
> +obj-$(CONFIG_BRCMSTB_DPFE) += brcmstb_dpfe.o
> obj-$(CONFIG_BT1_L2_CTL) += bt1-l2-ctl.o
> obj-$(CONFIG_TI_AEMIF) += ti-aemif.o
> obj-$(CONFIG_TI_EMIF) += emif.o
>
--
Florian
^ permalink raw reply [flat|nested] 35+ messages in thread
* Re: [RFT 12/16] memory: brcmstb_dpfe: Add separate entry for compile test
2020-07-24 18:40 ` Florian Fainelli
@ 2020-07-24 18:42 ` Krzysztof Kozlowski
2020-08-17 18:38 ` Krzysztof Kozlowski
1 sibling, 0 replies; 35+ messages in thread
From: Krzysztof Kozlowski @ 2020-07-24 18:42 UTC (permalink / raw)
To: Florian Fainelli
Cc: Ray Jui, Scott Branden, bcm-kernel-feedback-list, Russell King,
Catalin Marinas, Will Deacon, Markus Mayer, Roger Quadros,
Tony Lindgren, Vladimir Zapolskiy, Kukjin Kim, Lukasz Luba,
linux-arm-kernel, linux-kernel, linux-omap, linux-samsung-soc,
linux-pm, Olof Johansson, Arnd Bergmann
On Fri, Jul 24, 2020 at 11:40:09AM -0700, Florian Fainelli wrote:
> On 7/24/20 11:23 AM, Krzysztof Kozlowski wrote:
> > Add separate entry for Broadcom STB DPFE driver, selected by platforms
> > by default. This allows further customization and compile testing.
> >
> > Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> > ---
> >
> > I have other changes around drivers/memory/Kconfig pending. Sending them
> > together would avoid conflicts.
> >
> > ---
> > arch/arm/mach-bcm/Kconfig | 1 +
> > arch/arm64/Kconfig.platforms | 1 +
> > drivers/memory/Kconfig | 11 +++++++++++
> > drivers/memory/Makefile | 2 +-
> > 4 files changed, 14 insertions(+), 1 deletion(-)
> >
> > diff --git a/arch/arm/mach-bcm/Kconfig b/arch/arm/mach-bcm/Kconfig
> > index 1df0ee01ee02..b0c45238e69b 100644
> > --- a/arch/arm/mach-bcm/Kconfig
> > +++ b/arch/arm/mach-bcm/Kconfig
> > @@ -208,6 +208,7 @@ config ARCH_BRCMSTB
> > select ARM_GIC
> > select ARM_ERRATA_798181 if SMP
> > select HAVE_ARM_ARCH_TIMER
> > + select BRCMSTB_DPFE
> > select BRCMSTB_L2_IRQ
> > select BCM7120_L2_IRQ
> > select ARCH_HAS_HOLES_MEMORYMODEL
> > diff --git a/arch/arm64/Kconfig.platforms b/arch/arm64/Kconfig.platforms
> > index ddc3bf75257f..f7791ed0ed6d 100644
> > --- a/arch/arm64/Kconfig.platforms
> > +++ b/arch/arm64/Kconfig.platforms
> > @@ -68,6 +68,7 @@ config ARCH_BRCMSTB
> > bool "Broadcom Set-Top-Box SoCs"
> > select ARCH_HAS_RESET_CONTROLLER
> > select BCM7038_L1_IRQ
> > + select BRCMSTB_DPFE
>
> The select is probably not warranted, see below.
>
> > select BRCMSTB_L2_IRQ
> > select GENERIC_IRQ_CHIP
> > select PINCTRL
> > diff --git a/drivers/memory/Kconfig b/drivers/memory/Kconfig
> > index cd4fc93b50df..fef6f0873fda 100644
> > --- a/drivers/memory/Kconfig
> > +++ b/drivers/memory/Kconfig
> > @@ -54,6 +54,17 @@ config ATMEL_EBI
> > tree is used. This bus supports NANDs, external ethernet controller,
> > SRAMs, ATA devices, etc.
> >
> > +config BRCMSTB_DPFE
> > + bool "Broadcom STB DPFE driver" if COMPILE_TEST
> > + depends on ARCH_BRCMSTB || COMPILE_TEST
>
> Can you make it default ARCH_BRCMSTB and remove the select above?
>
> With that:
>
> Acked-by: Florian Fainelli <f.fainelli@gmail.com>
Sure, I'll fix it up. Thanks for feedback.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 35+ messages in thread
* Re: [RFT 12/16] memory: brcmstb_dpfe: Add separate entry for compile test
2020-07-24 18:40 ` Florian Fainelli
2020-07-24 18:42 ` Krzysztof Kozlowski
@ 2020-08-17 18:38 ` Krzysztof Kozlowski
1 sibling, 0 replies; 35+ messages in thread
From: Krzysztof Kozlowski @ 2020-08-17 18:38 UTC (permalink / raw)
To: Florian Fainelli
Cc: Ray Jui, Scott Branden, bcm-kernel-feedback-list, Russell King,
Catalin Marinas, Will Deacon, Markus Mayer, Roger Quadros,
Tony Lindgren, Vladimir Zapolskiy, Kukjin Kim, Lukasz Luba,
linux-arm-kernel, linux-kernel, linux-omap, linux-samsung-soc,
linux-pm, Olof Johansson, Arnd Bergmann
On Fri, Jul 24, 2020 at 11:40:09AM -0700, Florian Fainelli wrote:
> On 7/24/20 11:23 AM, Krzysztof Kozlowski wrote:
> > Add separate entry for Broadcom STB DPFE driver, selected by platforms
> > by default. This allows further customization and compile testing.
> >
> > Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> > ---
> >
> > I have other changes around drivers/memory/Kconfig pending. Sending them
> > together would avoid conflicts.
> >
> > ---
> > arch/arm/mach-bcm/Kconfig | 1 +
> > arch/arm64/Kconfig.platforms | 1 +
> > drivers/memory/Kconfig | 11 +++++++++++
> > drivers/memory/Makefile | 2 +-
> > 4 files changed, 14 insertions(+), 1 deletion(-)
> >
> > diff --git a/arch/arm/mach-bcm/Kconfig b/arch/arm/mach-bcm/Kconfig
> > index 1df0ee01ee02..b0c45238e69b 100644
> > --- a/arch/arm/mach-bcm/Kconfig
> > +++ b/arch/arm/mach-bcm/Kconfig
> > @@ -208,6 +208,7 @@ config ARCH_BRCMSTB
> > select ARM_GIC
> > select ARM_ERRATA_798181 if SMP
> > select HAVE_ARM_ARCH_TIMER
> > + select BRCMSTB_DPFE
> > select BRCMSTB_L2_IRQ
> > select BCM7120_L2_IRQ
> > select ARCH_HAS_HOLES_MEMORYMODEL
> > diff --git a/arch/arm64/Kconfig.platforms b/arch/arm64/Kconfig.platforms
> > index ddc3bf75257f..f7791ed0ed6d 100644
> > --- a/arch/arm64/Kconfig.platforms
> > +++ b/arch/arm64/Kconfig.platforms
> > @@ -68,6 +68,7 @@ config ARCH_BRCMSTB
> > bool "Broadcom Set-Top-Box SoCs"
> > select ARCH_HAS_RESET_CONTROLLER
> > select BCM7038_L1_IRQ
> > + select BRCMSTB_DPFE
>
> The select is probably not warranted, see below.
>
> > select BRCMSTB_L2_IRQ
> > select GENERIC_IRQ_CHIP
> > select PINCTRL
> > diff --git a/drivers/memory/Kconfig b/drivers/memory/Kconfig
> > index cd4fc93b50df..fef6f0873fda 100644
> > --- a/drivers/memory/Kconfig
> > +++ b/drivers/memory/Kconfig
> > @@ -54,6 +54,17 @@ config ATMEL_EBI
> > tree is used. This bus supports NANDs, external ethernet controller,
> > SRAMs, ATA devices, etc.
> >
> > +config BRCMSTB_DPFE
> > + bool "Broadcom STB DPFE driver" if COMPILE_TEST
> > + depends on ARCH_BRCMSTB || COMPILE_TEST
>
> Can you make it default ARCH_BRCMSTB and remove the select above?
>
> With that:
>
> Acked-by: Florian Fainelli <f.fainelli@gmail.com>
Applied with mentioned change.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 35+ messages in thread
* [PATCH 13/16] memory: of: Correct kerneldoc
2020-07-24 18:23 [PATCH 00/16] memory: Further cleanups Krzysztof Kozlowski
` (11 preceding siblings ...)
2020-07-24 18:23 ` [RFT 12/16] memory: brcmstb_dpfe: Add separate entry for compile test Krzysztof Kozlowski
@ 2020-07-24 18:23 ` Krzysztof Kozlowski
2020-07-27 8:09 ` Krzysztof Kozlowski
2020-07-24 18:23 ` [PATCH 14/16] memory: pl172: Enclose macro argument usage in parenthesis Krzysztof Kozlowski
` (2 subsequent siblings)
15 siblings, 1 reply; 35+ messages in thread
From: Krzysztof Kozlowski @ 2020-07-24 18:23 UTC (permalink / raw)
To: Florian Fainelli, Ray Jui, Scott Branden,
bcm-kernel-feedback-list, Russell King, Catalin Marinas,
Will Deacon, Krzysztof Kozlowski, Markus Mayer, Roger Quadros,
Tony Lindgren, Vladimir Zapolskiy, Kukjin Kim, Lukasz Luba,
linux-arm-kernel, linux-kernel, linux-omap, linux-samsung-soc,
linux-pm
Cc: Olof Johansson, Arnd Bergmann
Use proper kerneldoc to fix GCC warnings like:
drivers/memory/of_memory.c:30: warning: Function parameter or member 'dev' not described in 'of_get_min_tck'
Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
drivers/memory/of_memory.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/memory/of_memory.c b/drivers/memory/of_memory.c
index 67b811cb2cb9..d9f5437d3bce 100644
--- a/drivers/memory/of_memory.c
+++ b/drivers/memory/of_memory.c
@@ -18,7 +18,7 @@
/**
* of_get_min_tck() - extract min timing values for ddr
* @np: pointer to ddr device tree node
- * @device: device requesting for min timing values
+ * @dev: device requesting for min timing values
*
* Populates the lpddr2_min_tck structure by extracting data
* from device tree node. Returns a pointer to the populated
@@ -155,7 +155,7 @@ EXPORT_SYMBOL(of_get_ddr_timings);
/**
* of_lpddr3_get_min_tck() - extract min timing values for lpddr3
* @np: pointer to ddr device tree node
- * @device: device requesting for min timing values
+ * @dev: device requesting for min timing values
*
* Populates the lpddr3_min_tck structure by extracting data
* from device tree node. Returns a pointer to the populated
--
2.17.1
^ permalink raw reply related [flat|nested] 35+ messages in thread
* Re: [PATCH 13/16] memory: of: Correct kerneldoc
2020-07-24 18:23 ` [PATCH 13/16] memory: of: Correct kerneldoc Krzysztof Kozlowski
@ 2020-07-27 8:09 ` Krzysztof Kozlowski
0 siblings, 0 replies; 35+ messages in thread
From: Krzysztof Kozlowski @ 2020-07-27 8:09 UTC (permalink / raw)
To: Florian Fainelli, Ray Jui, Scott Branden,
bcm-kernel-feedback-list, Russell King, Catalin Marinas,
Will Deacon, Markus Mayer, Roger Quadros, Tony Lindgren,
Vladimir Zapolskiy, Kukjin Kim, Lukasz Luba, linux-arm-kernel,
linux-kernel, linux-omap, linux-samsung-soc, linux-pm
Cc: Olof Johansson, Arnd Bergmann
On Fri, Jul 24, 2020 at 08:23:25PM +0200, Krzysztof Kozlowski wrote:
> Use proper kerneldoc to fix GCC warnings like:
>
> drivers/memory/of_memory.c:30: warning: Function parameter or member 'dev' not described in 'of_get_min_tck'
>
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> ---
> drivers/memory/of_memory.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
Applied to drivers/memory tree.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 35+ messages in thread
* [PATCH 14/16] memory: pl172: Enclose macro argument usage in parenthesis
2020-07-24 18:23 [PATCH 00/16] memory: Further cleanups Krzysztof Kozlowski
` (12 preceding siblings ...)
2020-07-24 18:23 ` [PATCH 13/16] memory: of: Correct kerneldoc Krzysztof Kozlowski
@ 2020-07-24 18:23 ` Krzysztof Kozlowski
2020-07-27 8:09 ` Krzysztof Kozlowski
2020-07-24 18:23 ` [PATCH 15/16] memory: samsung: exynos-srom: Correct alignment Krzysztof Kozlowski
2020-07-24 18:23 ` [PATCH 16/16] memory: samsung: exynos5422-dmc: Correct white space issues Krzysztof Kozlowski
15 siblings, 1 reply; 35+ messages in thread
From: Krzysztof Kozlowski @ 2020-07-24 18:23 UTC (permalink / raw)
To: Florian Fainelli, Ray Jui, Scott Branden,
bcm-kernel-feedback-list, Russell King, Catalin Marinas,
Will Deacon, Krzysztof Kozlowski, Markus Mayer, Roger Quadros,
Tony Lindgren, Vladimir Zapolskiy, Kukjin Kim, Lukasz Luba,
linux-arm-kernel, linux-kernel, linux-omap, linux-samsung-soc,
linux-pm
Cc: Olof Johansson, Arnd Bergmann
Macros arguments should be enclosed by parenthesis for safety.
Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
drivers/memory/pl172.c | 14 +++++++-------
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/drivers/memory/pl172.c b/drivers/memory/pl172.c
index 084891eb29cc..575fadbffa30 100644
--- a/drivers/memory/pl172.c
+++ b/drivers/memory/pl172.c
@@ -21,7 +21,7 @@
#include <linux/of_platform.h>
#include <linux/time.h>
-#define MPMC_STATIC_CFG(n) (0x200 + 0x20 * n)
+#define MPMC_STATIC_CFG(n) (0x200 + 0x20 * (n))
#define MPMC_STATIC_CFG_MW_8BIT 0x0
#define MPMC_STATIC_CFG_MW_16BIT 0x1
#define MPMC_STATIC_CFG_MW_32BIT 0x2
@@ -31,17 +31,17 @@
#define MPMC_STATIC_CFG_EW BIT(8)
#define MPMC_STATIC_CFG_B BIT(19)
#define MPMC_STATIC_CFG_P BIT(20)
-#define MPMC_STATIC_WAIT_WEN(n) (0x204 + 0x20 * n)
+#define MPMC_STATIC_WAIT_WEN(n) (0x204 + 0x20 * (n))
#define MPMC_STATIC_WAIT_WEN_MAX 0x0f
-#define MPMC_STATIC_WAIT_OEN(n) (0x208 + 0x20 * n)
+#define MPMC_STATIC_WAIT_OEN(n) (0x208 + 0x20 * (n))
#define MPMC_STATIC_WAIT_OEN_MAX 0x0f
-#define MPMC_STATIC_WAIT_RD(n) (0x20c + 0x20 * n)
+#define MPMC_STATIC_WAIT_RD(n) (0x20c + 0x20 * (n))
#define MPMC_STATIC_WAIT_RD_MAX 0x1f
-#define MPMC_STATIC_WAIT_PAGE(n) (0x210 + 0x20 * n)
+#define MPMC_STATIC_WAIT_PAGE(n) (0x210 + 0x20 * (n))
#define MPMC_STATIC_WAIT_PAGE_MAX 0x1f
-#define MPMC_STATIC_WAIT_WR(n) (0x214 + 0x20 * n)
+#define MPMC_STATIC_WAIT_WR(n) (0x214 + 0x20 * (n))
#define MPMC_STATIC_WAIT_WR_MAX 0x1f
-#define MPMC_STATIC_WAIT_TURN(n) (0x218 + 0x20 * n)
+#define MPMC_STATIC_WAIT_TURN(n) (0x218 + 0x20 * (n))
#define MPMC_STATIC_WAIT_TURN_MAX 0x0f
/* Maximum number of static chip selects */
--
2.17.1
^ permalink raw reply related [flat|nested] 35+ messages in thread
* Re: [PATCH 14/16] memory: pl172: Enclose macro argument usage in parenthesis
2020-07-24 18:23 ` [PATCH 14/16] memory: pl172: Enclose macro argument usage in parenthesis Krzysztof Kozlowski
@ 2020-07-27 8:09 ` Krzysztof Kozlowski
0 siblings, 0 replies; 35+ messages in thread
From: Krzysztof Kozlowski @ 2020-07-27 8:09 UTC (permalink / raw)
To: Florian Fainelli, Ray Jui, Scott Branden,
bcm-kernel-feedback-list, Russell King, Catalin Marinas,
Will Deacon, Markus Mayer, Roger Quadros, Tony Lindgren,
Vladimir Zapolskiy, Kukjin Kim, Lukasz Luba, linux-arm-kernel,
linux-kernel, linux-omap, linux-samsung-soc, linux-pm
Cc: Olof Johansson, Arnd Bergmann
On Fri, Jul 24, 2020 at 08:23:26PM +0200, Krzysztof Kozlowski wrote:
> Macros arguments should be enclosed by parenthesis for safety.
>
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> ---
> drivers/memory/pl172.c | 14 +++++++-------
> 1 file changed, 7 insertions(+), 7 deletions(-)
Applied to drivers/memory tree.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 35+ messages in thread
* [PATCH 15/16] memory: samsung: exynos-srom: Correct alignment
2020-07-24 18:23 [PATCH 00/16] memory: Further cleanups Krzysztof Kozlowski
` (13 preceding siblings ...)
2020-07-24 18:23 ` [PATCH 14/16] memory: pl172: Enclose macro argument usage in parenthesis Krzysztof Kozlowski
@ 2020-07-24 18:23 ` Krzysztof Kozlowski
2020-07-24 18:23 ` [PATCH 16/16] memory: samsung: exynos5422-dmc: Correct white space issues Krzysztof Kozlowski
15 siblings, 0 replies; 35+ messages in thread
From: Krzysztof Kozlowski @ 2020-07-24 18:23 UTC (permalink / raw)
To: Florian Fainelli, Ray Jui, Scott Branden,
bcm-kernel-feedback-list, Russell King, Catalin Marinas,
Will Deacon, Krzysztof Kozlowski, Markus Mayer, Roger Quadros,
Tony Lindgren, Vladimir Zapolskiy, Kukjin Kim, Lukasz Luba,
linux-arm-kernel, linux-kernel, linux-omap, linux-samsung-soc,
linux-pm
Cc: Olof Johansson, Arnd Bergmann
Align indentation with open parenthesis (or fix existing alignment).
Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
drivers/memory/samsung/exynos-srom.c | 22 +++++++++++-----------
1 file changed, 11 insertions(+), 11 deletions(-)
diff --git a/drivers/memory/samsung/exynos-srom.c b/drivers/memory/samsung/exynos-srom.c
index 6510d7bab217..e73dd330af47 100644
--- a/drivers/memory/samsung/exynos-srom.c
+++ b/drivers/memory/samsung/exynos-srom.c
@@ -47,9 +47,9 @@ struct exynos_srom {
struct exynos_srom_reg_dump *reg_offset;
};
-static struct exynos_srom_reg_dump *exynos_srom_alloc_reg_dump(
- const unsigned long *rdump,
- unsigned long nr_rdump)
+static struct exynos_srom_reg_dump *
+exynos_srom_alloc_reg_dump(const unsigned long *rdump,
+ unsigned long nr_rdump)
{
struct exynos_srom_reg_dump *rd;
unsigned int i;
@@ -116,7 +116,7 @@ static int exynos_srom_probe(struct platform_device *pdev)
}
srom = devm_kzalloc(&pdev->dev,
- sizeof(struct exynos_srom), GFP_KERNEL);
+ sizeof(struct exynos_srom), GFP_KERNEL);
if (!srom)
return -ENOMEM;
@@ -130,7 +130,7 @@ static int exynos_srom_probe(struct platform_device *pdev)
platform_set_drvdata(pdev, srom);
srom->reg_offset = exynos_srom_alloc_reg_dump(exynos_srom_offsets,
- ARRAY_SIZE(exynos_srom_offsets));
+ ARRAY_SIZE(exynos_srom_offsets));
if (!srom->reg_offset) {
iounmap(srom->reg_base);
return -ENOMEM;
@@ -157,16 +157,16 @@ static int exynos_srom_probe(struct platform_device *pdev)
#ifdef CONFIG_PM_SLEEP
static void exynos_srom_save(void __iomem *base,
- struct exynos_srom_reg_dump *rd,
- unsigned int num_regs)
+ struct exynos_srom_reg_dump *rd,
+ unsigned int num_regs)
{
for (; num_regs > 0; --num_regs, ++rd)
rd->value = readl(base + rd->offset);
}
static void exynos_srom_restore(void __iomem *base,
- const struct exynos_srom_reg_dump *rd,
- unsigned int num_regs)
+ const struct exynos_srom_reg_dump *rd,
+ unsigned int num_regs)
{
for (; num_regs > 0; --num_regs, ++rd)
writel(rd->value, base + rd->offset);
@@ -177,7 +177,7 @@ static int exynos_srom_suspend(struct device *dev)
struct exynos_srom *srom = dev_get_drvdata(dev);
exynos_srom_save(srom->reg_base, srom->reg_offset,
- ARRAY_SIZE(exynos_srom_offsets));
+ ARRAY_SIZE(exynos_srom_offsets));
return 0;
}
@@ -186,7 +186,7 @@ static int exynos_srom_resume(struct device *dev)
struct exynos_srom *srom = dev_get_drvdata(dev);
exynos_srom_restore(srom->reg_base, srom->reg_offset,
- ARRAY_SIZE(exynos_srom_offsets));
+ ARRAY_SIZE(exynos_srom_offsets));
return 0;
}
#endif
--
2.17.1
^ permalink raw reply related [flat|nested] 35+ messages in thread
* [PATCH 16/16] memory: samsung: exynos5422-dmc: Correct white space issues
2020-07-24 18:23 [PATCH 00/16] memory: Further cleanups Krzysztof Kozlowski
` (14 preceding siblings ...)
2020-07-24 18:23 ` [PATCH 15/16] memory: samsung: exynos-srom: Correct alignment Krzysztof Kozlowski
@ 2020-07-24 18:23 ` Krzysztof Kozlowski
2020-08-18 8:22 ` Lukasz Luba
15 siblings, 1 reply; 35+ messages in thread
From: Krzysztof Kozlowski @ 2020-07-24 18:23 UTC (permalink / raw)
To: Florian Fainelli, Ray Jui, Scott Branden,
bcm-kernel-feedback-list, Russell King, Catalin Marinas,
Will Deacon, Krzysztof Kozlowski, Markus Mayer, Roger Quadros,
Tony Lindgren, Vladimir Zapolskiy, Kukjin Kim, Lukasz Luba,
linux-arm-kernel, linux-kernel, linux-omap, linux-samsung-soc,
linux-pm
Cc: Olof Johansson, Arnd Bergmann
Remove unneeded blank line and align indentation with open parenthesis.
Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
drivers/memory/samsung/exynos5422-dmc.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/memory/samsung/exynos5422-dmc.c b/drivers/memory/samsung/exynos5422-dmc.c
index 0388066a7d96..62a7193e3904 100644
--- a/drivers/memory/samsung/exynos5422-dmc.c
+++ b/drivers/memory/samsung/exynos5422-dmc.c
@@ -1394,7 +1394,7 @@ static int exynos5_dmc_probe(struct platform_device *pdev)
return PTR_ERR(dmc->base_drexi1);
dmc->clk_regmap = syscon_regmap_lookup_by_phandle(np,
- "samsung,syscon-clk");
+ "samsung,syscon-clk");
if (IS_ERR(dmc->clk_regmap))
return PTR_ERR(dmc->clk_regmap);
@@ -1473,7 +1473,6 @@ static int exynos5_dmc_probe(struct platform_device *pdev)
exynos5_dmc_df_profile.polling_ms = 500;
}
-
dmc->df = devm_devfreq_add_device(dev, &exynos5_dmc_df_profile,
DEVFREQ_GOV_SIMPLE_ONDEMAND,
&dmc->gov_data);
--
2.17.1
^ permalink raw reply related [flat|nested] 35+ messages in thread
* Re: [PATCH 16/16] memory: samsung: exynos5422-dmc: Correct white space issues
2020-07-24 18:23 ` [PATCH 16/16] memory: samsung: exynos5422-dmc: Correct white space issues Krzysztof Kozlowski
@ 2020-08-18 8:22 ` Lukasz Luba
0 siblings, 0 replies; 35+ messages in thread
From: Lukasz Luba @ 2020-08-18 8:22 UTC (permalink / raw)
To: Krzysztof Kozlowski, linux-samsung-soc, linux-pm
Cc: Florian Fainelli, Ray Jui, Scott Branden,
bcm-kernel-feedback-list, Russell King, Catalin Marinas,
Will Deacon, Markus Mayer, Roger Quadros, Tony Lindgren,
Vladimir Zapolskiy, Kukjin Kim, linux-arm-kernel, linux-kernel,
linux-omap, Olof Johansson, Arnd Bergmann
On 7/24/20 7:23 PM, Krzysztof Kozlowski wrote:
> Remove unneeded blank line and align indentation with open parenthesis.
>
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> ---
> drivers/memory/samsung/exynos5422-dmc.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/drivers/memory/samsung/exynos5422-dmc.c b/drivers/memory/samsung/exynos5422-dmc.c
> index 0388066a7d96..62a7193e3904 100644
> --- a/drivers/memory/samsung/exynos5422-dmc.c
> +++ b/drivers/memory/samsung/exynos5422-dmc.c
> @@ -1394,7 +1394,7 @@ static int exynos5_dmc_probe(struct platform_device *pdev)
> return PTR_ERR(dmc->base_drexi1);
>
> dmc->clk_regmap = syscon_regmap_lookup_by_phandle(np,
> - "samsung,syscon-clk");
> + "samsung,syscon-clk");
> if (IS_ERR(dmc->clk_regmap))
> return PTR_ERR(dmc->clk_regmap);
>
> @@ -1473,7 +1473,6 @@ static int exynos5_dmc_probe(struct platform_device *pdev)
> exynos5_dmc_df_profile.polling_ms = 500;
> }
>
> -
> dmc->df = devm_devfreq_add_device(dev, &exynos5_dmc_df_profile,
> DEVFREQ_GOV_SIMPLE_ONDEMAND,
> &dmc->gov_data);
>
LGTM
Acked-by: Lukasz Luba <lukasz.luba@arm.com>
Regards,
Lukasz
^ permalink raw reply [flat|nested] 35+ messages in thread