[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-Id: <20250711-new_atomisp-v9-0-a9dd62425ef6@gmail.com>
Date: Fri, 11 Jul 2025 19:54:35 +0800
From: LiangCheng Wang <zaq14760@...il.com>
To: Andy Shevchenko <andy@...nel.org>, Hans de Goede <hansg@...nel.org>,
Mauro Carvalho Chehab <mchehab@...nel.org>,
Sakari Ailus <sakari.ailus@...ux.intel.com>,
Greg Kroah-Hartman <gregkh@...uxfoundation.org>
Cc: linux-media@...r.kernel.org, linux-kernel@...r.kernel.org,
linux-staging@...ts.linux.dev, LiangCheng Wang <zaq14760@...il.com>
Subject: [PATCH v9 0/3] staging: media: atomisp: indentation cleanup per
directory
This patch series focuses on cleaning up indentation (spaces, tabs) in the
AtomISP driver under drivers/staging/media/atomisp/pci/isp/kernels/.
This version continues to address only indentation-related issues,
with patches split by subdirectory for clarity and easier review.
There is no functional logic change.
---
Changes in v9:
- Reformatted struct comments to follow kernel-doc format
- Reverted macro definition changes that removed alignment
- Reverted multi-line array initializations to use brace-on-own-line style
- Added trailing comma in small array initializations
- Adjusted function declarations to follow kernel-style indentation
- Removed unnecessary churn in cases where code was already style-compliant
- Link to v8: https://lore.kernel.org/r/20250704-new_atomisp-v8-0-2a8560cbd9be@gmail.com
Changes in v8:
- Reorganized cleanup by subdirectory (one commit per directory)
- Focused only on indentation fixes (spaces, tabs)
- Removed all clang-format involvement
- No functional changes
- Link to v7: https://lore.kernel.org/all/20250629113050.58138-1-zaq14760@gmail.com/
Changes in v7:
- Split previous monolithic patch into multiple smaller patches
- Applied clang-format to entire driver excluding i2c directory
- Fixed checkpatch.pl-reported ERRORs (parentheses in macros, unnecessary return parentheses, zero-initialized globals, spaces after unary minus)
- Left WARNINGS untouched for future cleanup
- No functional logic changes
- Link to v6: https://lore.kernel.org/r/20250627-bar-v6-1-b22b5ea3ced0@gmail.com
Changes in v6:
- Applied clang-format across the entire AtomISP driver
- Fixed all checkpatch.pl-reported ERRORs
- Added explanation of tooling and scope
- No functional logic modified
- Moved 'Suggested-by' and 'Link' tags above Signed-off-by
- Link to v5: https://lore.kernel.org/r/20250625-bar-v5-1-db960608b607@gmail.com
Changes in v5:
- Replaced space-based indentation with tabs in output_1.0 directory
- Used checkpatch.pl and grep to identify formatting issues
- No functional changes made
- This patch is now focused solely on tab/space issues
- Link to v4: https://lore.kernel.org/r/20250624-bar-v4-1-9f9f9ae9f868@gmail.com
Changes in v4:
- Moved assignment operator '=' to the same line for static struct definitions
- Remove unnecessary line breaks in function definitions
- Update commit message to reflect all the coding style fixes
- Link to v3: https://lore.kernel.org/r/20250622-bar-v3-1-4cc91ef01c3a@gmail.com
Changes in v3:
- Removed extra spaces between type and asterisk (e.g., `*to`) in function
declarations, as pointed out by Andy Shevchenko
- Update commit message to reflect all the coding style fixes
- Link to v2: https://lore.kernel.org/r/20250621-bar-v2-1-4e6cfc779614@gmail.com
Changes in v2:
- Fix patch subject prefix to "staging: media: atomisp:" to comply with media CI style.
- No other functional changes.
- Link to v1: https://lore.kernel.org/r/20250621-bar-v1-1-5a3e7004462c@gmail.com
Thanks for your previous feedback.
Best regards,
LiangCheng Wang <zaq14760@...il.com>
Signed-off-by: LiangCheng Wang <zaq14760@...il.com>
---
LiangCheng Wang (3):
staging: media: atomisp: fix indentation in aa
staging: media: atomisp: fix indentation in anr
staging: media: atomisp: fix indentation in bh
.../pci/isp/kernels/aa/aa_2/ia_css_aa2_types.h | 8 +++++---
.../pci/isp/kernels/anr/anr_1.0/ia_css_anr.host.c | 22 ++++++++--------------
.../pci/isp/kernels/anr/anr_1.0/ia_css_anr.host.h | 20 +++++++-------------
.../pci/isp/kernels/anr/anr_2/ia_css_anr2.host.c | 14 +++++---------
.../pci/isp/kernels/anr/anr_2/ia_css_anr2.host.h | 13 +++++--------
.../pci/isp/kernels/bh/bh_2/ia_css_bh.host.c | 14 +++++---------
.../pci/isp/kernels/bh/bh_2/ia_css_bh.host.h | 14 +++++---------
7 files changed, 40 insertions(+), 65 deletions(-)
---
base-commit: 8c2e52ebbe885c7eeaabd3b7ddcdc1246fc400d2
change-id: 20250704-new_atomisp-e73f471f3078
Best regards,
--
LiangCheng Wang <zaq14760@...il.com>
Powered by blists - more mailing lists