| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <9f8b3018-2b5e-2471-f5d4-bac03e4ab259@xs4all.nl>
Date: Fri, 9 Apr 2021 12:53:35 +0200
From: Hans Verkuil <hverkuil@...all.nl>
To: Mitali Borkar <mitaliborkar810@...il.com>, clabbe@...libre.com,
mchehab@...nel.org, gregkh@...uxfoundation.org
Cc: linux-media@...r.kernel.org, linux-staging@...ts.linux.dev,
linux-kernel@...r.kernel.org, outreachy-kernel@...glegroups.com,
mitali_s@...iitr.ac.in
Subject: Re: [PATCH v2] staging: media: zoran: remove and add '*' in
long(multi-line) comments
On 09/04/2021 12:49, Mitali Borkar wrote:
> Added '*' before every line inside long(multi-line) comments. Removed
> '*/' from end of the comment line and added to next line as per linux
> kernel coding style. Aligned '*' accordingly to make code neater.
>
> Signed-off-by: Mitali Borkar <mitaliborkar810@...il.com>
> ---
>
> Changes from v1:- Changes made in code according to linux kernel coding
> style for long(multi-line) comments.
>
> drivers/staging/media/zoran/zr36050.c | 138 +++++++++++++++-----------
> 1 file changed, 81 insertions(+), 57 deletions(-)
>
> diff --git a/drivers/staging/media/zoran/zr36050.c b/drivers/staging/media/zoran/zr36050.c
> index 663ac2b3434e..703064009c6b 100644
> --- a/drivers/staging/media/zoran/zr36050.c
> +++ b/drivers/staging/media/zoran/zr36050.c
> @@ -25,7 +25,8 @@
> #include "videocodec.h"
>
> /* it doesn't make sense to have more than 20 or so,
The coding style says that /* is on a line of its own. So change that too.
Regards,
Hans
> - * just to prevent some unwanted loops */
> + * just to prevent some unwanted loops
> + */
> #define MAX_CODECS 20
>
> /* amount of chips attached via this driver */
> @@ -44,9 +45,10 @@ MODULE_PARM_DESC(debug, "Debug level (0-4)");
>
> /* =========================================================================
> * Local hardware I/O functions:
> -
> - read/write via codec layer (registers are located in the master device)
> - ========================================================================= */
> + *
> + * read/write via codec layer (registers are located in the master device)
> + * =========================================================================
> + */
>
> /* read and write functions */
> static u8 zr36050_read(struct zr36050 *ptr, u16 reg)
> @@ -81,9 +83,10 @@ static void zr36050_write(struct zr36050 *ptr, u16 reg, u8 value)
>
> /* =========================================================================
> * Local helper function:
> -
> - status read
> - ========================================================================= */
> + *
> + * status read
> + * =========================================================================
> + */
>
> /* status is kept in datastructure */
> static u8 zr36050_read_status1(struct zr36050 *ptr)
> @@ -96,9 +99,10 @@ static u8 zr36050_read_status1(struct zr36050 *ptr)
>
> /* =========================================================================
> * Local helper function:
> -
> - scale factor read
> - ========================================================================= */
> + *
> + * scale factor read
> + * =========================================================================
> + */
>
> /* scale factor is kept in datastructure */
> static u16 zr36050_read_scalefactor(struct zr36050 *ptr)
> @@ -113,9 +117,10 @@ static u16 zr36050_read_scalefactor(struct zr36050 *ptr)
>
> /* =========================================================================
> * Local helper function:
> -
> - wait if codec is ready to proceed (end of processing) or time is over
> - ========================================================================= */
> + *
> + * wait if codec is ready to proceed (end of processing) or time is over
> + * =========================================================================
> + */
>
> static void zr36050_wait_end(struct zr36050 *ptr)
> {
> @@ -134,9 +139,10 @@ static void zr36050_wait_end(struct zr36050 *ptr)
>
> /* =========================================================================
> * Local helper function:
> -
> - basic test of "connectivity", writes/reads to/from memory the SOF marker
> - ========================================================================= */
> + *
> + * basic test of "connectivity", writes/reads to/from memory the SOF marker
> + * =========================================================================
> + */
>
> static int zr36050_basic_test(struct zr36050 *ptr)
> {
> @@ -175,9 +181,10 @@ static int zr36050_basic_test(struct zr36050 *ptr)
>
> /* =========================================================================
> * Local helper function:
> -
> - simple loop for pushing the init datasets
> - ========================================================================= */
> + *
> + * simple loop for pushing the init datasets
> + * =========================================================================
> + */
>
> static int zr36050_pushit(struct zr36050 *ptr, u16 startreg, u16 len, const char *data)
> {
> @@ -193,14 +200,15 @@ static int zr36050_pushit(struct zr36050 *ptr, u16 startreg, u16 len, const char
>
> /* =========================================================================
> * Basic datasets:
> -
> - jpeg baseline setup data (you find it on lots places in internet, or just
> - extract it from any regular .jpg image...)
> -
> - Could be variable, but until it's not needed it they are just fixed to save
> - memory. Otherwise expand zr36050 structure with arrays, push the values to
> - it and initialize from there, as e.g. the linux zr36057/60 driver does it.
> - ========================================================================= */
> + *
> + * jpeg baseline setup data (you find it on lots places in internet, or just
> + * extract it from any regular .jpg image...)
> + *
> + * Could be variable, but until it's not needed it they are just fixed to save
> + * memory. Otherwise expand zr36050 structure with arrays, push the values to
> + * it and initialize from there, as e.g. the linux zr36057/60 driver does it.
> + * =========================================================================
> + */
>
> static const char zr36050_dqt[0x86] = {
> 0xff, 0xdb, //Marker: DQT
> @@ -295,15 +303,17 @@ static const char zr36050_decimation_v[8] = { 1, 1, 1, 0, 0, 0, 0, 0 };
>
> /* =========================================================================
> * Local helper functions:
> -
> - calculation and setup of parameter-dependent JPEG baseline segments
> - (needed for compression only)
> - ========================================================================= */
> + *
> + * calculation and setup of parameter-dependent JPEG baseline segments
> + * (needed for compression only)
> + * =========================================================================
> + */
>
> /* ------------------------------------------------------------------------- */
>
> /* SOF (start of frame) segment depends on width, height and sampling ratio
> - * of each color component */
> + * of each color component
> + */
>
> static int zr36050_set_sof(struct zr36050 *ptr)
> {
> @@ -334,7 +344,8 @@ static int zr36050_set_sof(struct zr36050 *ptr)
> /* ------------------------------------------------------------------------- */
>
> /* SOS (start of scan) segment depends on the used scan components
> - * of each color component */
> + * of each color component
> + */
>
> static int zr36050_set_sos(struct zr36050 *ptr)
> {
> @@ -379,12 +390,14 @@ static int zr36050_set_dri(struct zr36050 *ptr)
>
> /* =========================================================================
> * Setup function:
> + *
> + * Setup compression/decompression of Zoran's JPEG processor
> + * ( see also zoran 36050 manual )
> + *
> + * ... sorry for the spaghetti code ...
> + * =========================================================================
> + */
>
> - Setup compression/decompression of Zoran's JPEG processor
> - ( see also zoran 36050 manual )
> -
> - ... sorry for the spaghetti code ...
> - ========================================================================= */
> static void zr36050_init(struct zr36050 *ptr)
> {
> int sum = 0;
> @@ -420,7 +433,8 @@ static void zr36050_init(struct zr36050 *ptr)
> sum += zr36050_set_dri(ptr);
>
> /* setup the fixed jpeg tables - maybe variable, though -
> - * (see table init section above) */
> + * (see table init section above)
> + */
> dprintk(3, "%s: write DQT, DHT, APP\n", ptr->name);
> sum += zr36050_pushit(ptr, ZR050_DQT_IDX,
> sizeof(zr36050_dqt), zr36050_dqt);
> @@ -532,12 +546,15 @@ static void zr36050_init(struct zr36050 *ptr)
>
> /* =========================================================================
> * CODEC API FUNCTIONS
> -
> - this functions are accessed by the master via the API structure
> - ========================================================================= */
> + *
> + * this functions are accessed by the master via the API structure
> + * =========================================================================
> + */
>
> /* set compression/expansion mode and launches codec -
> - * this should be the last call from the master before starting processing */
> + * this should be the last call from the master before starting processing
> + */
> +
> static int zr36050_set_mode(struct videocodec *codec, int mode)
> {
> struct zr36050 *ptr = (struct zr36050 *)codec->data;
> @@ -566,7 +583,8 @@ static int zr36050_set_video(struct videocodec *codec, const struct tvnorm *norm
> cap->decimation, cap->quality);
> /* if () return -EINVAL;
> * trust the master driver that it knows what it does - so
> - * we allow invalid startx/y and norm for now ... */
> + * we allow invalid startx/y and norm for now ...
> + */
> ptr->width = cap->width / (cap->decimation & 0xff);
> ptr->height = cap->height / ((cap->decimation >> 8) & 0xff);
>
> @@ -586,7 +604,8 @@ static int zr36050_set_video(struct videocodec *codec, const struct tvnorm *norm
> ptr->real_code_vol = size >> 3; /* in bytes */
>
> /* Set max_block_vol here (previously in zr36050_init, moved
> - * here for consistency with zr36060 code */
> + * here for consistency with zr36060 code
> + */
> zr36050_write(ptr, ZR050_MBCV, ptr->max_block_vol);
>
> return 0;
> @@ -643,7 +662,8 @@ static int zr36050_control(struct videocodec *codec, int type, int size, void *d
> return -EFAULT;
> ptr->total_code_vol = *ival;
> /* (Kieran Morrissey)
> - * code copied from zr36060.c to ensure proper bitrate */
> + * code copied from zr36060.c to ensure proper bitrate
> + */
> ptr->real_code_vol = (ptr->total_code_vol * 6) >> 3;
> break;
>
> @@ -708,9 +728,10 @@ static int zr36050_control(struct videocodec *codec, int type, int size, void *d
>
> /* =========================================================================
> * Exit and unregister function:
> -
> - Deinitializes Zoran's JPEG processor
> - ========================================================================= */
> + *
> + * Deinitializes Zoran's JPEG processor
> + * =========================================================================
> + */
>
> static int zr36050_unset(struct videocodec *codec)
> {
> @@ -733,12 +754,13 @@ static int zr36050_unset(struct videocodec *codec)
>
> /* =========================================================================
> * Setup and registry function:
> -
> - Initializes Zoran's JPEG processor
> -
> - Also sets pixel size, average code size, mode (compr./decompr.)
> - (the given size is determined by the processor with the video interface)
> - ========================================================================= */
> + *
> + * Initializes Zoran's JPEG processor
> + *
> + * Also sets pixel size, average code size, mode (compr./decompr.)
> + * (the given size is determined by the processor with the video interface)
> + * =========================================================================
> + */
>
> static int zr36050_setup(struct videocodec *codec)
> {
> @@ -774,7 +796,8 @@ static int zr36050_setup(struct videocodec *codec)
> memcpy(ptr->v_samp_ratio, zr36050_decimation_v, 8);
>
> ptr->bitrate_ctrl = 0; /* 0 or 1 - fixed file size flag
> - * (what is the difference?) */
> + * (what is the difference?)
> + */
> ptr->mode = CODEC_DO_COMPRESSION;
> ptr->width = 384;
> ptr->height = 288;
> @@ -814,7 +837,8 @@ static const struct videocodec zr36050_codec = {
>
> /* =========================================================================
> * HOOK IN DRIVER AS KERNEL MODULE
> - ========================================================================= */
> + * =========================================================================
> + */
>
> static int __init zr36050_init_module(void)
> {
>
Powered by blists - more mailing lists