[PATCH 0/3] firmware: xilinx: Clean up series

linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed

* [PATCH 0/3] firmware: xilinx: Clean up series
@ 2020-12-01 15:50 Michal Simek
  2020-12-01 15:50 ` [PATCH 1/3] firmware: xilinx: Remove additional newline Michal Simek
                   ` (2 more replies)
  0 siblings, 3 replies; 6+ messages in thread
From: Michal Simek @ 2020-12-01 15:50 UTC (permalink / raw)
  To: linux-kernel, monstr, michal.simek, git
  Cc: Greg Kroah-Hartman, Jolly Shah, Rajan Vaja, Tejas Patel,
	linux-arm-kernel

Hi,

for being able to review new changes more effectively it is good to get rid
of existing kernel-doc and checkpatch violations.
That's why this small clean up series.

Based on
https://lore.kernel.org/r/e606233d15bfdc594535dd34eb85472b42f61830.1606832997.git.michal.simek@xilinx.com

Thanks,
Michal


Michal Simek (3):
  firmware: xilinx: Remove additional newline
  firmware: xilinx: Add a blank line after function declaration
  firmware: xilinx: Properly align function parameter

 include/linux/firmware/xlnx-zynqmp.h | 45 ++++++++++++++++++++++++----
 1 file changed, 40 insertions(+), 5 deletions(-)

-- 
2.29.2


^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH 1/3] firmware: xilinx: Remove additional newline
  2020-12-01 15:50 [PATCH 0/3] firmware: xilinx: Clean up series Michal Simek
@ 2020-12-01 15:50 ` Michal Simek
  2020-12-01 15:50 ` [PATCH 2/3] firmware: xilinx: Add a blank line after function declaration Michal Simek
  2020-12-01 15:50 ` [PATCH 3/3] firmware: xilinx: Properly align function parameter Michal Simek
  2 siblings, 0 replies; 6+ messages in thread
From: Michal Simek @ 2020-12-01 15:50 UTC (permalink / raw)
  To: linux-kernel, monstr, michal.simek, git
  Cc: Greg Kroah-Hartman, Jolly Shah, Rajan Vaja, Tejas Patel,
	linux-arm-kernel

This additional newline is useless and also reported by checkpatch
--strict.

Signed-off-by: Michal Simek <michal.simek@xilinx.com>
---

 include/linux/firmware/xlnx-zynqmp.h | 1 -
 1 file changed, 1 deletion(-)

diff --git a/include/linux/firmware/xlnx-zynqmp.h b/include/linux/firmware/xlnx-zynqmp.h
index f84244ea633b..0db9005782d6 100644
--- a/include/linux/firmware/xlnx-zynqmp.h
+++ b/include/linux/firmware/xlnx-zynqmp.h
@@ -316,7 +316,6 @@ struct zynqmp_pm_query_data {
 	u32 arg3;
 };
 
-
 int zynqmp_pm_invoke_fn(u32 pm_api_id, u32 arg0, u32 arg1,
 			u32 arg2, u32 arg3, u32 *ret_payload);
 
-- 
2.29.2


^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [PATCH 2/3] firmware: xilinx: Add a blank line after function declaration
  2020-12-01 15:50 [PATCH 0/3] firmware: xilinx: Clean up series Michal Simek
  2020-12-01 15:50 ` [PATCH 1/3] firmware: xilinx: Remove additional newline Michal Simek
@ 2020-12-01 15:50 ` Michal Simek
  2020-12-01 15:50 ` [PATCH 3/3] firmware: xilinx: Properly align function parameter Michal Simek
  2 siblings, 0 replies; 6+ messages in thread
From: Michal Simek @ 2020-12-01 15:50 UTC (permalink / raw)
  To: linux-kernel, monstr, michal.simek, git
  Cc: Greg Kroah-Hartman, Jolly Shah, Rajan Vaja, Tejas Patel,
	linux-arm-kernel

Fix all these issues which are also reported by checkpatch --strict.

Signed-off-by: Michal Simek <michal.simek@xilinx.com>
---

 include/linux/firmware/xlnx-zynqmp.h | 34 ++++++++++++++++++++++++++++
 1 file changed, 34 insertions(+)

diff --git a/include/linux/firmware/xlnx-zynqmp.h b/include/linux/firmware/xlnx-zynqmp.h
index 0db9005782d6..0e7e72650ed3 100644
--- a/include/linux/firmware/xlnx-zynqmp.h
+++ b/include/linux/firmware/xlnx-zynqmp.h
@@ -363,107 +363,132 @@ static inline struct zynqmp_eemi_ops *zynqmp_pm_get_eemi_ops(void)
 {
 	return ERR_PTR(-ENODEV);
 }
+
 static inline int zynqmp_pm_get_api_version(u32 *version)
 {
 	return -ENODEV;
 }
+
 static inline int zynqmp_pm_get_chipid(u32 *idcode, u32 *version)
 {
 	return -ENODEV;
 }
+
 static inline int zynqmp_pm_query_data(struct zynqmp_pm_query_data qdata,
 				       u32 *out)
 {
 	return -ENODEV;
 }
+
 static inline int zynqmp_pm_clock_enable(u32 clock_id)
 {
 	return -ENODEV;
 }
+
 static inline int zynqmp_pm_clock_disable(u32 clock_id)
 {
 	return -ENODEV;
 }
+
 static inline int zynqmp_pm_clock_getstate(u32 clock_id, u32 *state)
 {
 	return -ENODEV;
 }
+
 static inline int zynqmp_pm_clock_setdivider(u32 clock_id, u32 divider)
 {
 	return -ENODEV;
 }
+
 static inline int zynqmp_pm_clock_getdivider(u32 clock_id, u32 *divider)
 {
 	return -ENODEV;
 }
+
 static inline int zynqmp_pm_clock_setrate(u32 clock_id, u64 rate)
 {
 	return -ENODEV;
 }
+
 static inline int zynqmp_pm_clock_getrate(u32 clock_id, u64 *rate)
 {
 	return -ENODEV;
 }
+
 static inline int zynqmp_pm_clock_setparent(u32 clock_id, u32 parent_id)
 {
 	return -ENODEV;
 }
+
 static inline int zynqmp_pm_clock_getparent(u32 clock_id, u32 *parent_id)
 {
 	return -ENODEV;
 }
+
 static inline int zynqmp_pm_set_pll_frac_mode(u32 clk_id, u32 mode)
 {
 	return -ENODEV;
 }
+
 static inline int zynqmp_pm_get_pll_frac_mode(u32 clk_id, u32 *mode)
 {
 	return -ENODEV;
 }
+
 static inline int zynqmp_pm_set_pll_frac_data(u32 clk_id, u32 data)
 {
 	return -ENODEV;
 }
+
 static inline int zynqmp_pm_get_pll_frac_data(u32 clk_id, u32 *data)
 {
 	return -ENODEV;
 }
+
 static inline int zynqmp_pm_set_sd_tapdelay(u32 node_id, u32 type, u32 value)
 {
 	return -ENODEV;
 }
+
 static inline int zynqmp_pm_sd_dll_reset(u32 node_id, u32 type)
 {
 	return -ENODEV;
 }
+
 static inline int zynqmp_pm_reset_assert(const enum zynqmp_pm_reset reset,
 			   const enum zynqmp_pm_reset_action assert_flag)
 {
 	return -ENODEV;
 }
+
 static inline int zynqmp_pm_reset_get_status(const enum zynqmp_pm_reset reset,
 					     u32 *status)
 {
 	return -ENODEV;
 }
+
 static inline int zynqmp_pm_init_finalize(void)
 {
 	return -ENODEV;
 }
+
 static inline int zynqmp_pm_set_suspend_mode(u32 mode)
 {
 	return -ENODEV;
 }
+
 static inline int zynqmp_pm_request_node(const u32 node, const u32 capabilities,
 					 const u32 qos,
 					 const enum zynqmp_pm_request_ack ack)
 {
 	return -ENODEV;
 }
+
 static inline int zynqmp_pm_release_node(const u32 node)
 {
 	return -ENODEV;
 }
+
 static inline int zynqmp_pm_set_requirement(const u32 node,
 					const u32 capabilities,
 					const u32 qos,
@@ -471,39 +496,48 @@ static inline int zynqmp_pm_set_requirement(const u32 node,
 {
 	return -ENODEV;
 }
+
 static inline int zynqmp_pm_aes_engine(const u64 address, u32 *out)
 {
 	return -ENODEV;
 }
+
 static inline int zynqmp_pm_fpga_load(const u64 address, const u32 size,
 				      const u32 flags)
 {
 	return -ENODEV;
 }
+
 static inline int zynqmp_pm_fpga_get_status(u32 *value)
 {
 	return -ENODEV;
 }
+
 static inline int zynqmp_pm_write_ggs(u32 index, u32 value)
 {
 	return -ENODEV;
 }
+
 static inline int zynqmp_pm_read_ggs(u32 index, u32 *value)
 {
 	return -ENODEV;
 }
+
 static inline int zynqmp_pm_write_pggs(u32 index, u32 value)
 {
 	return -ENODEV;
 }
+
 static inline int zynqmp_pm_read_pggs(u32 index, u32 *value)
 {
 	return -ENODEV;
 }
+
 static inline int zynqmp_pm_system_shutdown(const u32 type, const u32 subtype)
 {
 	return -ENODEV;
 }
+
 static inline int zynqmp_pm_set_boot_health_status(u32 value)
 {
 	return -ENODEV;
-- 
2.29.2


^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [PATCH 3/3] firmware: xilinx: Properly align function parameter
  2020-12-01 15:50 [PATCH 0/3] firmware: xilinx: Clean up series Michal Simek
  2020-12-01 15:50 ` [PATCH 1/3] firmware: xilinx: Remove additional newline Michal Simek
  2020-12-01 15:50 ` [PATCH 2/3] firmware: xilinx: Add a blank line after function declaration Michal Simek
@ 2020-12-01 15:50 ` Michal Simek
  2020-12-01 19:42   ` Joe Perches
  2 siblings, 1 reply; 6+ messages in thread
From: Michal Simek @ 2020-12-01 15:50 UTC (permalink / raw)
  To: linux-kernel, monstr, michal.simek, git
  Cc: Greg Kroah-Hartman, Jolly Shah, Rajan Vaja, Tejas Patel,
	linux-arm-kernel

Fix parameters alignment reported by checkpatch --strict.

Signed-off-by: Michal Simek <michal.simek@xilinx.com>
---

 include/linux/firmware/xlnx-zynqmp.h | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/include/linux/firmware/xlnx-zynqmp.h b/include/linux/firmware/xlnx-zynqmp.h
index 0e7e72650ed3..c5cdc68e431f 100644
--- a/include/linux/firmware/xlnx-zynqmp.h
+++ b/include/linux/firmware/xlnx-zynqmp.h
@@ -456,7 +456,8 @@ static inline int zynqmp_pm_sd_dll_reset(u32 node_id, u32 type)
 }
 
 static inline int zynqmp_pm_reset_assert(const enum zynqmp_pm_reset reset,
-			   const enum zynqmp_pm_reset_action assert_flag)
+					 const enum zynqmp_pm_reset_action
+					 assert_flag)
 {
 	return -ENODEV;
 }
@@ -490,9 +491,10 @@ static inline int zynqmp_pm_release_node(const u32 node)
 }
 
 static inline int zynqmp_pm_set_requirement(const u32 node,
-					const u32 capabilities,
-					const u32 qos,
-					const enum zynqmp_pm_request_ack ack)
+					    const u32 capabilities,
+					    const u32 qos,
+					    const enum zynqmp_pm_request_ack
+					    ack)
 {
 	return -ENODEV;
 }
-- 
2.29.2


^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [PATCH 3/3] firmware: xilinx: Properly align function parameter
  2020-12-01 15:50 ` [PATCH 3/3] firmware: xilinx: Properly align function parameter Michal Simek
@ 2020-12-01 19:42   ` Joe Perches
  2020-12-02  7:38     ` Michal Simek
  0 siblings, 1 reply; 6+ messages in thread
From: Joe Perches @ 2020-12-01 19:42 UTC (permalink / raw)
  To: Michal Simek, linux-kernel, monstr, git
  Cc: Greg Kroah-Hartman, Jolly Shah, Rajan Vaja, Tejas Patel,
	linux-arm-kernel

On Tue, 2020-12-01 at 16:50 +0100, Michal Simek wrote:
> Fix parameters alignment reported by checkpatch --strict.

Please use a newer checkpatch as the 80 column warning
isn't enforced quite the same way.

> diff --git a/include/linux/firmware/xlnx-zynqmp.h b/include/linux/firmware/xlnx-zynqmp.h
[]
> @@ -456,7 +456,8 @@ static inline int zynqmp_pm_sd_dll_reset(u32 node_id, u32 type)
>  }
>  
> 
>  static inline int zynqmp_pm_reset_assert(const enum zynqmp_pm_reset reset,
> -			   const enum zynqmp_pm_reset_action assert_flag)
> +					 const enum zynqmp_pm_reset_action
> +					 assert_flag)

I think these changes aren't very good as it separates
the type from the variable name.

> @@ -490,9 +491,10 @@ static inline int zynqmp_pm_release_node(const u32 node)
>  }
>  
> 
>  static inline int zynqmp_pm_set_requirement(const u32 node,
> -					const u32 capabilities,
> -					const u32 qos,
> -					const enum zynqmp_pm_request_ack ack)
> +					    const u32 capabilities,
> +					    const u32 qos,
> +					    const enum zynqmp_pm_request_ack
> +					    ack)




^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 3/3] firmware: xilinx: Properly align function parameter
  2020-12-01 19:42   ` Joe Perches
@ 2020-12-02  7:38     ` Michal Simek
  0 siblings, 0 replies; 6+ messages in thread
From: Michal Simek @ 2020-12-02  7:38 UTC (permalink / raw)
  To: Joe Perches, Michal Simek, linux-kernel, monstr, git
  Cc: Greg Kroah-Hartman, Jolly Shah, Rajan Vaja, Tejas Patel,
	linux-arm-kernel



On 01. 12. 20 20:42, Joe Perches wrote:
> On Tue, 2020-12-01 at 16:50 +0100, Michal Simek wrote:
>> Fix parameters alignment reported by checkpatch --strict.
> 
> Please use a newer checkpatch as the 80 column warning
> isn't enforced quite the same way.

I was using it but saw 80 lines breakage in editor that's why I put it
there.

> 
>> diff --git a/include/linux/firmware/xlnx-zynqmp.h b/include/linux/firmware/xlnx-zynqmp.h
> []
>> @@ -456,7 +456,8 @@ static inline int zynqmp_pm_sd_dll_reset(u32 node_id, u32 type)
>>  }
>>  
>>
>>  static inline int zynqmp_pm_reset_assert(const enum zynqmp_pm_reset reset,
>> -			   const enum zynqmp_pm_reset_action assert_flag)
>> +					 const enum zynqmp_pm_reset_action
>> +					 assert_flag)
> 
> I think these changes aren't very good as it separates
> the type from the variable name.

I have retest it and I can't see any warning when they are on the same
line. Will send v2 with fixing this.

Thanks,
Michal

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2020-12-02  7:39 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-12-01 15:50 [PATCH 0/3] firmware: xilinx: Clean up series Michal Simek
2020-12-01 15:50 ` [PATCH 1/3] firmware: xilinx: Remove additional newline Michal Simek
2020-12-01 15:50 ` [PATCH 2/3] firmware: xilinx: Add a blank line after function declaration Michal Simek
2020-12-01 15:50 ` [PATCH 3/3] firmware: xilinx: Properly align function parameter Michal Simek
2020-12-01 19:42   ` Joe Perches
2020-12-02  7:38     ` Michal Simek

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).