* [PATCH] staging: media: atomisp: Fix typos and comments alignment issue in include/linux/* header files
@ 2025-11-15 15:18 Hamdan Khan
0 siblings, 0 replies; 3+ messages in thread
From: Hamdan Khan @ 2025-11-15 15:18 UTC (permalink / raw)
To: hansg, mchehab, sakari.ailus, andy, gregkh, linux-media,
linux-staging, linux-kernel
Cc: Hamdan Khan
This patch fixes minor comment and code alignment issues in
atomisp/include/linux/*.h header files. Some of them were warnings
generated by checkpatch.pl (missing * at the start of the line or */
present in the same line)
The changes are code-style related only and don't affect the
functionality.
Signed-off-by: Hamdan Khan <hamdankhan212@gmail.com>
---
.../media/atomisp/include/linux/atomisp.h | 30 +++++++++++--------
.../include/linux/atomisp_gmin_platform.h | 7 ++---
.../atomisp/include/linux/atomisp_platform.h | 21 +++++++------
3 files changed, 32 insertions(+), 26 deletions(-)
diff --git a/drivers/staging/media/atomisp/include/linux/atomisp.h b/drivers/staging/media/atomisp/include/linux/atomisp.h
index 3c8fa3f5808d..bce4e5f1bfc7 100644
--- a/drivers/staging/media/atomisp/include/linux/atomisp.h
+++ b/drivers/staging/media/atomisp/include/linux/atomisp.h
@@ -146,8 +146,7 @@ struct atomisp_3a_config {
unsigned int ae_y_coef_r; /* [gain] Weight of R for Y */
unsigned int ae_y_coef_g; /* [gain] Weight of G for Y */
unsigned int ae_y_coef_b; /* [gain] Weight of B for Y */
- unsigned int awb_lg_high_raw; /* [intensity]
- AWB level gate high for raw */
+ unsigned int awb_lg_high_raw; /* [intensity] AWB level gate high for raw */
unsigned int awb_lg_low; /* [intensity] AWB level gate low */
unsigned int awb_lg_high; /* [intensity] AWB level gate high */
int af_fir1_coef[7]; /* [factor] AF FIR coefficients of fir1 */
@@ -272,8 +271,10 @@ struct atomisp_wb_config {
/* Color Space Conversion settings */
struct atomisp_cc_config {
unsigned int fraction_bits;
- int matrix[3 * 3]; /* RGB2YUV Color matrix, signed
- <13-fraction_bits>.<fraction_bits> */
+ /* RGB2YUV Color matrix, signed
+ * <13-fraction_bits>.<fraction_bits>
+ */
+ int matrix[3 * 3];
};
/* De pixel noise configuration */
@@ -425,10 +426,8 @@ struct atomisp_parameters {
struct atomisp_3a_config *a3a_config; /* 3A Statistics config */
struct atomisp_xnr_config *xnr_config; /* eXtra Noise Reduction */
struct atomisp_dz_config *dz_config; /* Digital Zoom */
- struct atomisp_cc_config *yuv2rgb_cc_config; /* Color
- Correction config */
- struct atomisp_cc_config *rgb2yuv_cc_config; /* Color
- Correction config */
+ struct atomisp_cc_config *yuv2rgb_cc_config; /* Color Correction config */
+ struct atomisp_cc_config *rgb2yuv_cc_config; /* Color Correction config */
struct atomisp_macc_table *macc_table;
struct atomisp_gamma_table *gamma_table;
struct atomisp_ctc_table *ctc_table;
@@ -538,7 +537,8 @@ struct atomisp_ctc_table {
/* Parameter for overlay image loading */
struct atomisp_overlay {
- /* the frame containing the overlay data The overlay frame width should
+ /*
+ * the frame containing the overlay data The overlay frame width should
* be the multiples of 2*ISP_VEC_NELEMS. The overlay frame height
* should be the multiples of 2.
*/
@@ -562,10 +562,12 @@ struct atomisp_overlay {
/* the blending percent of overlay data for V subpixels */
unsigned char blend_overlay_perc_v;
/* the overlay start x pixel position on output frame It should be the
- multiples of 2*ISP_VEC_NELEMS. */
+ * multiples of 2*ISP_VEC_NELEMS.
+ */
unsigned int overlay_start_x;
/* the overlay start y pixel position on output frame It should be the
- multiples of 2. */
+ * multiples of 2.
+ */
unsigned int overlay_start_y;
};
@@ -836,8 +838,10 @@ enum atomisp_burst_capture_options {
#define V4L2_CID_ATOMISP_LOW_LIGHT \
(V4L2_CID_PRIVATE_BASE + 5)
-/* Camera class:
- * Exposure, Flash and privacy (indicator) light controls, to be upstreamed */
+/*
+ * Camera class:
+ * Exposure, Flash and privacy (indicator) light controls, to be upstreamed
+ */
#define V4L2_CID_CAMERA_LASTP1 (V4L2_CID_CAMERA_CLASS_BASE + 1024)
#define V4L2_CID_RUN_MODE (V4L2_CID_CAMERA_LASTP1 + 20)
diff --git a/drivers/staging/media/atomisp/include/linux/atomisp_gmin_platform.h b/drivers/staging/media/atomisp/include/linux/atomisp_gmin_platform.h
index 426c5ee4ec18..415b265505c7 100644
--- a/drivers/staging/media/atomisp/include/linux/atomisp_gmin_platform.h
+++ b/drivers/staging/media/atomisp/include/linux/atomisp_gmin_platform.h
@@ -15,8 +15,7 @@ int atomisp_gmin_remove_subdev(struct v4l2_subdev *sd);
int gmin_get_var_int(struct device *dev, bool is_gmin,
const char *var, int def);
struct camera_sensor_platform_data *
-gmin_camera_platform_data(
- struct v4l2_subdev *subdev,
- enum atomisp_input_format csi_format,
- enum atomisp_bayer_order csi_bayer);
+gmin_camera_platform_data(struct v4l2_subdev *subdev,
+ enum atomisp_input_format csi_format,
+ enum atomisp_bayer_order csi_bayer);
#endif
diff --git a/drivers/staging/media/atomisp/include/linux/atomisp_platform.h b/drivers/staging/media/atomisp/include/linux/atomisp_platform.h
index 6146555fe9cf..2378a21ee961 100644
--- a/drivers/staging/media/atomisp/include/linux/atomisp_platform.h
+++ b/drivers/staging/media/atomisp/include/linux/atomisp_platform.h
@@ -72,10 +72,12 @@ enum atomisp_input_format {
/* CSI2-MIPI specific format: YUV data.
*/
- ATOMISP_INPUT_FORMAT_YUV420_8_SHIFT, /* YUV420 8-bit (Chroma Shifted
- Pixel Sampling) */
- ATOMISP_INPUT_FORMAT_YUV420_10_SHIFT, /* YUV420 8-bit (Chroma Shifted
- Pixel Sampling) */
+ ATOMISP_INPUT_FORMAT_YUV420_8_SHIFT, /* YUV420 8-bit (Chroma Shifted
+ * Pixel Sampling)
+ */
+ ATOMISP_INPUT_FORMAT_YUV420_10_SHIFT, /* YUV420 10-bit (Chroma Shifted
+ * Pixel Sampling)
+ */
/* CSI2-MIPI specific format: Generic long packet data
*/
@@ -105,7 +107,7 @@ struct intel_v4l2_subdev_table {
};
/*
- * Sensor of external ISP can send multiple streams with different mipi data
+ * Sensor of external ISP can send multiple streams with different mipi data
* type in the same virtual channel. This information needs to come from the
* sensor or external ISP
*/
@@ -119,13 +121,14 @@ struct atomisp_input_stream_info {
enum atomisp_input_stream_id stream;
u8 enable;
/* Sensor driver fills ch_id with the id
- of the virtual channel. */
+ * of the virtual channel.
+ */
u8 ch_id;
/* Tells how many streams in this virtual channel. If 0 ignore rest
- * and the input format will be from mipi_info */
+ * and the input format will be from mipi_info
+ */
u8 isys_configs;
- /*
- * if more isys_configs is more than 0, sensor needs to configure the
+ /* If isys_configs is more than 0, sensor needs to configure the
* input format differently. width and height can be 0. If width and
* height is not zero, then the corresponding data needs to be set
*/
--
2.43.0
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [PATCH] staging: media: atomisp: Fix typos and comments alignment issue in include/linux/* header files
@ 2025-11-29 6:12 Hamdan Khan
2025-11-29 13:46 ` Andy Shevchenko
0 siblings, 1 reply; 3+ messages in thread
From: Hamdan Khan @ 2025-11-29 6:12 UTC (permalink / raw)
To: hansg, mchehab, sakari.ailus, andy, gregkh, linux-media,
linux-staging, linux-kernel
Cc: Hamdan Khan
This patch fixes minor comment and code alignment issues in
atomisp/include/linux/*.h header files. Some of them were warnings
generated by checkpatch.pl (missing * at the start of the line or */
present in the same line)
The changes are code-style related only and don't affect the
functionality.
Signed-off-by: Hamdan Khan <hamdankhan212@gmail.com>
---
.../media/atomisp/include/linux/atomisp.h | 30 +++++++++++--------
.../include/linux/atomisp_gmin_platform.h | 7 ++---
.../atomisp/include/linux/atomisp_platform.h | 21 +++++++------
3 files changed, 32 insertions(+), 26 deletions(-)
diff --git a/drivers/staging/media/atomisp/include/linux/atomisp.h b/drivers/staging/media/atomisp/include/linux/atomisp.h
index 3c8fa3f5808d..bce4e5f1bfc7 100644
--- a/drivers/staging/media/atomisp/include/linux/atomisp.h
+++ b/drivers/staging/media/atomisp/include/linux/atomisp.h
@@ -146,8 +146,7 @@ struct atomisp_3a_config {
unsigned int ae_y_coef_r; /* [gain] Weight of R for Y */
unsigned int ae_y_coef_g; /* [gain] Weight of G for Y */
unsigned int ae_y_coef_b; /* [gain] Weight of B for Y */
- unsigned int awb_lg_high_raw; /* [intensity]
- AWB level gate high for raw */
+ unsigned int awb_lg_high_raw; /* [intensity] AWB level gate high for raw */
unsigned int awb_lg_low; /* [intensity] AWB level gate low */
unsigned int awb_lg_high; /* [intensity] AWB level gate high */
int af_fir1_coef[7]; /* [factor] AF FIR coefficients of fir1 */
@@ -272,8 +271,10 @@ struct atomisp_wb_config {
/* Color Space Conversion settings */
struct atomisp_cc_config {
unsigned int fraction_bits;
- int matrix[3 * 3]; /* RGB2YUV Color matrix, signed
- <13-fraction_bits>.<fraction_bits> */
+ /* RGB2YUV Color matrix, signed
+ * <13-fraction_bits>.<fraction_bits>
+ */
+ int matrix[3 * 3];
};
/* De pixel noise configuration */
@@ -425,10 +426,8 @@ struct atomisp_parameters {
struct atomisp_3a_config *a3a_config; /* 3A Statistics config */
struct atomisp_xnr_config *xnr_config; /* eXtra Noise Reduction */
struct atomisp_dz_config *dz_config; /* Digital Zoom */
- struct atomisp_cc_config *yuv2rgb_cc_config; /* Color
- Correction config */
- struct atomisp_cc_config *rgb2yuv_cc_config; /* Color
- Correction config */
+ struct atomisp_cc_config *yuv2rgb_cc_config; /* Color Correction config */
+ struct atomisp_cc_config *rgb2yuv_cc_config; /* Color Correction config */
struct atomisp_macc_table *macc_table;
struct atomisp_gamma_table *gamma_table;
struct atomisp_ctc_table *ctc_table;
@@ -538,7 +537,8 @@ struct atomisp_ctc_table {
/* Parameter for overlay image loading */
struct atomisp_overlay {
- /* the frame containing the overlay data The overlay frame width should
+ /*
+ * the frame containing the overlay data The overlay frame width should
* be the multiples of 2*ISP_VEC_NELEMS. The overlay frame height
* should be the multiples of 2.
*/
@@ -562,10 +562,12 @@ struct atomisp_overlay {
/* the blending percent of overlay data for V subpixels */
unsigned char blend_overlay_perc_v;
/* the overlay start x pixel position on output frame It should be the
- multiples of 2*ISP_VEC_NELEMS. */
+ * multiples of 2*ISP_VEC_NELEMS.
+ */
unsigned int overlay_start_x;
/* the overlay start y pixel position on output frame It should be the
- multiples of 2. */
+ * multiples of 2.
+ */
unsigned int overlay_start_y;
};
@@ -836,8 +838,10 @@ enum atomisp_burst_capture_options {
#define V4L2_CID_ATOMISP_LOW_LIGHT \
(V4L2_CID_PRIVATE_BASE + 5)
-/* Camera class:
- * Exposure, Flash and privacy (indicator) light controls, to be upstreamed */
+/*
+ * Camera class:
+ * Exposure, Flash and privacy (indicator) light controls, to be upstreamed
+ */
#define V4L2_CID_CAMERA_LASTP1 (V4L2_CID_CAMERA_CLASS_BASE + 1024)
#define V4L2_CID_RUN_MODE (V4L2_CID_CAMERA_LASTP1 + 20)
diff --git a/drivers/staging/media/atomisp/include/linux/atomisp_gmin_platform.h b/drivers/staging/media/atomisp/include/linux/atomisp_gmin_platform.h
index 426c5ee4ec18..415b265505c7 100644
--- a/drivers/staging/media/atomisp/include/linux/atomisp_gmin_platform.h
+++ b/drivers/staging/media/atomisp/include/linux/atomisp_gmin_platform.h
@@ -15,8 +15,7 @@ int atomisp_gmin_remove_subdev(struct v4l2_subdev *sd);
int gmin_get_var_int(struct device *dev, bool is_gmin,
const char *var, int def);
struct camera_sensor_platform_data *
-gmin_camera_platform_data(
- struct v4l2_subdev *subdev,
- enum atomisp_input_format csi_format,
- enum atomisp_bayer_order csi_bayer);
+gmin_camera_platform_data(struct v4l2_subdev *subdev,
+ enum atomisp_input_format csi_format,
+ enum atomisp_bayer_order csi_bayer);
#endif
diff --git a/drivers/staging/media/atomisp/include/linux/atomisp_platform.h b/drivers/staging/media/atomisp/include/linux/atomisp_platform.h
index 6146555fe9cf..2378a21ee961 100644
--- a/drivers/staging/media/atomisp/include/linux/atomisp_platform.h
+++ b/drivers/staging/media/atomisp/include/linux/atomisp_platform.h
@@ -72,10 +72,12 @@ enum atomisp_input_format {
/* CSI2-MIPI specific format: YUV data.
*/
- ATOMISP_INPUT_FORMAT_YUV420_8_SHIFT, /* YUV420 8-bit (Chroma Shifted
- Pixel Sampling) */
- ATOMISP_INPUT_FORMAT_YUV420_10_SHIFT, /* YUV420 8-bit (Chroma Shifted
- Pixel Sampling) */
+ ATOMISP_INPUT_FORMAT_YUV420_8_SHIFT, /* YUV420 8-bit (Chroma Shifted
+ * Pixel Sampling)
+ */
+ ATOMISP_INPUT_FORMAT_YUV420_10_SHIFT, /* YUV420 10-bit (Chroma Shifted
+ * Pixel Sampling)
+ */
/* CSI2-MIPI specific format: Generic long packet data
*/
@@ -105,7 +107,7 @@ struct intel_v4l2_subdev_table {
};
/*
- * Sensor of external ISP can send multiple streams with different mipi data
+ * Sensor of external ISP can send multiple streams with different mipi data
* type in the same virtual channel. This information needs to come from the
* sensor or external ISP
*/
@@ -119,13 +121,14 @@ struct atomisp_input_stream_info {
enum atomisp_input_stream_id stream;
u8 enable;
/* Sensor driver fills ch_id with the id
- of the virtual channel. */
+ * of the virtual channel.
+ */
u8 ch_id;
/* Tells how many streams in this virtual channel. If 0 ignore rest
- * and the input format will be from mipi_info */
+ * and the input format will be from mipi_info
+ */
u8 isys_configs;
- /*
- * if more isys_configs is more than 0, sensor needs to configure the
+ /* If isys_configs is more than 0, sensor needs to configure the
* input format differently. width and height can be 0. If width and
* height is not zero, then the corresponding data needs to be set
*/
--
2.43.0
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] staging: media: atomisp: Fix typos and comments alignment issue in include/linux/* header files
2025-11-29 6:12 Hamdan Khan
@ 2025-11-29 13:46 ` Andy Shevchenko
0 siblings, 0 replies; 3+ messages in thread
From: Andy Shevchenko @ 2025-11-29 13:46 UTC (permalink / raw)
To: Hamdan Khan
Cc: hansg, mchehab, sakari.ailus, andy, gregkh, linux-media,
linux-staging, linux-kernel
On Sat, Nov 29, 2025 at 11:12:28AM +0500, Hamdan Khan wrote:
> This patch fixes minor comment and code alignment issues in
> atomisp/include/linux/*.h header files. Some of them were warnings
> generated by checkpatch.pl (missing * at the start of the line or */
> present in the same line)
...
> - int matrix[3 * 3]; /* RGB2YUV Color matrix, signed
> - <13-fraction_bits>.<fraction_bits> */
> + /* RGB2YUV Color matrix, signed
> + * <13-fraction_bits>.<fraction_bits>
> + */
/*
* This comment has to follow the common rules for
* multi-line comments. Use this as an example.
*/
...
> + /*
> + * the frame containing the overlay data The overlay frame width should
This becomes even worse. Fix the English text properly.
> * be the multiples of 2*ISP_VEC_NELEMS. The overlay frame height
> * should be the multiples of 2.
> */
...
> /* the overlay start x pixel position on output frame It should be the
> - multiples of 2*ISP_VEC_NELEMS. */
> + * multiples of 2*ISP_VEC_NELEMS.
> + */
The same as per two above comments.
...
> /* the overlay start y pixel position on output frame It should be the
> - multiples of 2. */
> + * multiples of 2.
> + */
Ditto.
> /* CSI2-MIPI specific format: YUV data.
> */
> - ATOMISP_INPUT_FORMAT_YUV420_8_SHIFT, /* YUV420 8-bit (Chroma Shifted
> - Pixel Sampling) */
> - ATOMISP_INPUT_FORMAT_YUV420_10_SHIFT, /* YUV420 8-bit (Chroma Shifted
> - Pixel Sampling) */
> + ATOMISP_INPUT_FORMAT_YUV420_8_SHIFT, /* YUV420 8-bit (Chroma Shifted
> + * Pixel Sampling)
> + */
> + ATOMISP_INPUT_FORMAT_YUV420_10_SHIFT, /* YUV420 10-bit (Chroma Shifted
> + * Pixel Sampling)
> + */
Still not good. Make the comments to lead the definitions and also fix
/* CSI2-MIPI specific format: YUV data. */
...
/* YUV420 10-bit (Chroma Shifted Pixel Sampling) */
ATOMISP_INPUT_FORMAT_YUV420_10_SHIFT,
> /* CSI2-MIPI specific format: Generic long packet data
> */
As per above.
You need to fix _all_ of them, do not leave the job half done.
...
> /*
> - * Sensor of external ISP can send multiple streams with different mipi data
> + * Sensor of external ISP can send multiple streams with different mipi data
MIPI
> * type in the same virtual channel. This information needs to come from the
> * sensor or external ISP
The sentence misses period at the end.
> */
...
> /* Sensor driver fills ch_id with the id
> - of the virtual channel. */
> + * of the virtual channel.
> + */
> /* Tells how many streams in this virtual channel. If 0 ignore rest
> - * and the input format will be from mipi_info */
> + * and the input format will be from mipi_info
> + */
> - /*
> - * if more isys_configs is more than 0, sensor needs to configure the
> + /* If isys_configs is more than 0, sensor needs to configure the
> * input format differently. width and height can be 0. If width and
> * height is not zero, then the corresponding data needs to be set
> */
All the above must be fixed properly.
...
TL:DR; this patch needs much more work. Take your time, the driver
is not so small. OTOH you can take one part of it and do this there.
I dunno if fixing one header subfolder is good enough, we may start
from it, but again, fix all problems, don't leave it in semi-baked
state.
--
With Best Regards,
Andy Shevchenko
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2025-11-29 13:46 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-11-15 15:18 [PATCH] staging: media: atomisp: Fix typos and comments alignment issue in include/linux/* header files Hamdan Khan
-- strict thread matches above, loose matches on Subject: below --
2025-11-29 6:12 Hamdan Khan
2025-11-29 13:46 ` Andy Shevchenko
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).