lkml.org 
[lkml]   [2015]   [Dec]   [4]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH] Staging: cdrom: fix space required after that ','
From
Hi Yash,

2015-12-04 6:21 GMT-03:00 Shah, Yash (Y.) <yshah1@visteon.com>:
> From: Yash Shah<yshah1@visteon.com>
>
> fix space required after that ',' as detected by checkpatch.pl
>
> Signed-off-by: Yash Shah <yshah1@visteon.com>
> ---
> drivers/cdrom/cdrom.c | 12 ++++++------
> 1 file changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/cdrom/cdrom.c b/drivers/cdrom/cdrom.c
> index c206ccd..2d759b6 100644
> --- a/drivers/cdrom/cdrom.c
> +++ b/drivers/cdrom/cdrom.c
> @@ -536,7 +536,7 @@ static int cdrom_mrw_exit(struct cdrom_device_info *cdi)
> int ret;
>
> ret = cdrom_get_disc_info(cdi, &di);
> - if (ret < 0 || ret < (int)offsetof(typeof(di),disc_type))
> + if (ret < 0 || ret < (int)offsetof(typeof(di), disc_type))
> return 1;
>
> ret = 0;
> @@ -788,7 +788,7 @@ static int cdrom_mrw_open_write(struct cdrom_device_info *cdi)
> }
>
> ret = cdrom_get_disc_info(cdi, &di);
> - if (ret < 0 || ret < offsetof(typeof(di),disc_type))
> + if (ret < 0 || ret < offsetof(typeof(di), disc_type))
> return 1;
>
> if (!di.erasable)
> @@ -1050,7 +1050,7 @@ int open_for_data(struct cdrom_device_info *cdi)
> if (CDROM_CAN(CDC_CLOSE_TRAY) &&
> cdi->options & CDO_AUTO_CLOSE) {
> cd_dbg(CD_OPEN, "trying to close the tray\n");
> - ret=cdo->tray_move(cdi,0);
> + ret=cdo->tray_move(cdi, 0);

What do you think to add spaces between equal assignment too?

> if (ret) {
> cd_dbg(CD_OPEN, "bummer. tried to close the tray but failed.\n");
> /* Ignore the error from the low
> @@ -1215,7 +1215,7 @@ static int check_for_audio_disc(struct cdrom_device_info * cdi,
> if (CDROM_CAN(CDC_CLOSE_TRAY) &&
> cdi->options & CDO_AUTO_CLOSE) {
> cd_dbg(CD_OPEN, "trying to close the tray\n");
> - ret=cdo->tray_move(cdi,0);
> + ret=cdo->tray_move(cdi, 0);
> if (ret) {
> cd_dbg(CD_OPEN, "bummer. tried to close tray but failed.\n");
> /* Ignore the error from the low
> @@ -1591,8 +1591,8 @@ void init_cdrom_command(struct packet_command *cgc, void *buf, int len,
>
> /* DVD handling */
>
> -#define copy_key(dest,src) memcpy((dest), (src), sizeof(dvd_key))
> -#define copy_chal(dest,src) memcpy((dest), (src), sizeof(dvd_challenge))
> +#define copy_key(dest, src) memcpy((dest), (src), sizeof(dvd_key))
> +#define copy_chal(dest, :src) memcpy((dest), (src), sizeof(dvd_challenge))
>
> static void setup_report_key(struct packet_command *cgc, unsigned agid, unsigned type)
> {
> --
> 1.9.1
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/



--
Regards,

Geyslan G. Bem
hackingbits.com


\
 
 \ /
  Last update: 2015-12-04 12:01    [W:0.052 / U:1.056 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site