Linux Media Controller development
 help / color / mirror / Atom feed
* [PATCH v3 0/4]
@ 2026-03-29 18:48 Serban-Pascu Robert
  2026-03-29 18:48 ` [PATCH v3 1/4] staging: media: atomisp: fix SPDX license identifier style Serban-Pascu Robert
                   ` (3 more replies)
  0 siblings, 4 replies; 5+ messages in thread
From: Serban-Pascu Robert @ 2026-03-29 18:48 UTC (permalink / raw)
  To: hansg, mchehab
  Cc: andy, sakari.ailus, gregkh, daniel.baluta, linux-media,
	linux-staging, linux-kernel, Serban-Pascu Robert

This series addresses several minor coding style issues 
in system_global.h as requested during review. The previous consolidated
patch has been  split into four separate logical changes: 
SPDX header style, blank  lines removal, block comment reformatting,
and a typo correction.

Serban-Pascu Robert (4):
  staging: media: atomisp: fix SPDX license identifier style
  staging: media: atomisp: fix typo 'uninterruptible' in comment
  staging: media: atomisp: reformat block comments
  staging: media: atomisp: remove redundant blank lines

 drivers/staging/media/atomisp/pci/system_global.h | 12 +++++-------
 1 file changed, 5 insertions(+), 7 deletions(-)

-- 
2.43.0


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

* [PATCH v3 1/4] staging: media: atomisp: fix SPDX license identifier style
  2026-03-29 18:48 [PATCH v3 0/4] Serban-Pascu Robert
@ 2026-03-29 18:48 ` Serban-Pascu Robert
  2026-03-29 18:48 ` [PATCH v3 2/4] staging: media: atomisp: fix typo Serban-Pascu Robert
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 5+ messages in thread
From: Serban-Pascu Robert @ 2026-03-29 18:48 UTC (permalink / raw)
  To: hansg, mchehab
  Cc: andy, sakari.ailus, gregkh, daniel.baluta, linux-media,
	linux-staging, linux-kernel, Serban-Pascu Robert

Use the C-style block comment for the SPDX license identifier in the
header file as per Linux kernel coding standards.

Signed-off-by: Serban-Pascu Robert <robyserbanpascu06@gmail.com>
---
 drivers/staging/media/atomisp/pci/system_global.h | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/drivers/staging/media/atomisp/pci/system_global.h b/drivers/staging/media/atomisp/pci/system_global.h
index e8a29f73d67a..c99fb649fb5f 100644
--- a/drivers/staging/media/atomisp/pci/system_global.h
+++ b/drivers/staging/media/atomisp/pci/system_global.h
@@ -1,5 +1,4 @@
-/* SPDX-License-Identifier: GPL-2.0 */
-// SPDX-License-Identifier: GPL-2.0-or-later
+/* SPDX-License-Identifier: GPL-2.0-or-later */
 /*
  *    (c) 2020 Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
  */
-- 
2.43.0


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

* [PATCH v3 2/4] staging: media: atomisp: fix typo
  2026-03-29 18:48 [PATCH v3 0/4] Serban-Pascu Robert
  2026-03-29 18:48 ` [PATCH v3 1/4] staging: media: atomisp: fix SPDX license identifier style Serban-Pascu Robert
@ 2026-03-29 18:48 ` Serban-Pascu Robert
  2026-03-29 18:48 ` [PATCH v3 3/4] staging: media: atomisp: reformat block comments Serban-Pascu Robert
  2026-03-29 18:48 ` [PATCH v3 4/4] staging: media: atomisp: remove redundant blank lines Serban-Pascu Robert
  3 siblings, 0 replies; 5+ messages in thread
From: Serban-Pascu Robert @ 2026-03-29 18:48 UTC (permalink / raw)
  To: hansg, mchehab
  Cc: andy, sakari.ailus, gregkh, daniel.baluta, linux-media,
	linux-staging, linux-kernel, Serban-Pascu Robert

Correct a spelling error in a comment: 
'uninteruptible' -> 'uninterruptible'.

Signed-off-by: Serban-Pascu Robert <robyserbanpascu06@gmail.com>
---
 drivers/staging/media/atomisp/pci/system_global.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/staging/media/atomisp/pci/system_global.h b/drivers/staging/media/atomisp/pci/system_global.h
index c99fb649fb5f..6420436d88f7 100644
--- a/drivers/staging/media/atomisp/pci/system_global.h
+++ b/drivers/staging/media/atomisp/pci/system_global.h
@@ -28,7 +28,7 @@
 #define DMA_DDR_TO_HMEM_WORKAROUND
 
 /*
- * The longest allowed (uninteruptible) bus transfer, does not
+ * The longest allowed (uninterruptible) bus transfer, does not
  * take stalling into account
  */
 #define HIVE_ISP_MAX_BURST_LENGTH	1024
-- 
2.43.0


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

* [PATCH v3 3/4] staging: media: atomisp: reformat block comments
  2026-03-29 18:48 [PATCH v3 0/4] Serban-Pascu Robert
  2026-03-29 18:48 ` [PATCH v3 1/4] staging: media: atomisp: fix SPDX license identifier style Serban-Pascu Robert
  2026-03-29 18:48 ` [PATCH v3 2/4] staging: media: atomisp: fix typo Serban-Pascu Robert
@ 2026-03-29 18:48 ` Serban-Pascu Robert
  2026-03-29 18:48 ` [PATCH v3 4/4] staging: media: atomisp: remove redundant blank lines Serban-Pascu Robert
  3 siblings, 0 replies; 5+ messages in thread
From: Serban-Pascu Robert @ 2026-03-29 18:48 UTC (permalink / raw)
  To: hansg, mchehab
  Cc: andy, sakari.ailus, gregkh, daniel.baluta, linux-media,
	linux-staging, linux-kernel, Serban-Pascu Robert

Add missing asterisks to block comments to conform with the
standard Linux kernel comment style.

Signed-off-by: Serban-Pascu Robert <robyserbanpascu06@gmail.com>
---
 drivers/staging/media/atomisp/pci/system_global.h | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/drivers/staging/media/atomisp/pci/system_global.h b/drivers/staging/media/atomisp/pci/system_global.h
index 6420436d88f7..c65a5be8445a 100644
--- a/drivers/staging/media/atomisp/pci/system_global.h
+++ b/drivers/staging/media/atomisp/pci/system_global.h
@@ -100,8 +100,9 @@ typedef enum {
 	N_GDC_ID
 } gdc_ID_t;
 
-/* this extra define is needed because we want to use it also
-   in the preprocessor, and that doesn't work with enums.
+/*
+ * this extra define is needed because we want to use it also
+ * in the preprocessor, and that doesn't work with enums.
  */
 #define N_GDC_ID_CPP 2
 
-- 
2.43.0


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

* [PATCH v3 4/4] staging: media: atomisp: remove redundant blank lines
  2026-03-29 18:48 [PATCH v3 0/4] Serban-Pascu Robert
                   ` (2 preceding siblings ...)
  2026-03-29 18:48 ` [PATCH v3 3/4] staging: media: atomisp: reformat block comments Serban-Pascu Robert
@ 2026-03-29 18:48 ` Serban-Pascu Robert
  3 siblings, 0 replies; 5+ messages in thread
From: Serban-Pascu Robert @ 2026-03-29 18:48 UTC (permalink / raw)
  To: hansg, mchehab
  Cc: andy, sakari.ailus, gregkh, daniel.baluta, linux-media,
	linux-staging, linux-kernel, Serban-Pascu Robert

Remove multiple consecutive blank lines to improve code readability
and follow kernel style guidelines.

Signed-off-by: Serban-Pascu Robert <robyserbanpascu06@gmail.com>
---
 drivers/staging/media/atomisp/pci/system_global.h | 2 --
 1 file changed, 2 deletions(-)

diff --git a/drivers/staging/media/atomisp/pci/system_global.h b/drivers/staging/media/atomisp/pci/system_global.h
index c65a5be8445a..d417230faff2 100644
--- a/drivers/staging/media/atomisp/pci/system_global.h
+++ b/drivers/staging/media/atomisp/pci/system_global.h
@@ -218,7 +218,6 @@ typedef enum {
 #define	N_ACQUISITION_UNIT_ID		1
 #define	N_CTRL_UNIT_ID			1
 
-
 enum ia_css_isp_memories {
 	IA_CSS_ISP_PMEM0 = 0,
 	IA_CSS_ISP_DMEM0,
@@ -248,7 +247,6 @@ typedef enum {
 	N_ISYS_IRQ_ID
 } isys_irq_ID_t;
 
-
 /*
  * Input-buffer Controller.
  */
-- 
2.43.0


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

end of thread, other threads:[~2026-03-29 18:49 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-03-29 18:48 [PATCH v3 0/4] Serban-Pascu Robert
2026-03-29 18:48 ` [PATCH v3 1/4] staging: media: atomisp: fix SPDX license identifier style Serban-Pascu Robert
2026-03-29 18:48 ` [PATCH v3 2/4] staging: media: atomisp: fix typo Serban-Pascu Robert
2026-03-29 18:48 ` [PATCH v3 3/4] staging: media: atomisp: reformat block comments Serban-Pascu Robert
2026-03-29 18:48 ` [PATCH v3 4/4] staging: media: atomisp: remove redundant blank lines Serban-Pascu Robert

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox