Re: [PATCH 1/7] Input: gscps2 - clean up driver code style and structure
From: Helge Deller
Date: Tue Sep 15 2026 - 05:53:39 EST
Hi Dmitry,
On 8/30/26 22:52, Dmitry Torokhov wrote:
Clean up code style issues and function ordering in the gscps2 driver:you may add for the whole series:
- Reorder functions to place gscps2_interrupt() before its callers,
allowing removal of its forward declaration.
- Change gscps2_enable() to accept a boolean parameter and remove the
ENABLE and DISABLE macro definitions.
- Convert printk() calls to dev_dbg() and dev_warn().
- Fix operator spacing and multi-variable assignment.
- Add spinlock comment and use cpu_relax() in spin-wait loop.
Assisted-by: LLM
Signed-off-by: Dmitry Torokhov <dmitry.torokhov@xxxxxxxxx>
---
drivers/input/serio/gscps2.c | 208 +++++++++++++++++++------------------------
1 file changed, 91 insertions(+), 117 deletions(-)
Acked-by: Helge Deller <deller@xxxxxx>
Tested-by: Helge Deller <deller@xxxxxx>
Thank you!
Helge