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