From: "Shah, Yash (Y.)" <yshah1@visteon.com>
To: "Geyslan G. Bem" <geyslan@gmail.com>
CC: "axboe@kernel.dk" <axboe@kernel.dk>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "Babu, Viswanathan (V.)" <vbabu3@visteon.com>
Subject: RE: [PATCH] Staging: cdrom: fix space required after that ','
Thread-Topic: [PATCH] Staging: cdrom: fix space required after that ','
Thread-Index: AQHRLnUt6bOr/COxr06Kqf8dlVqMfp66oqcAgAACQRA=
Date: Fri, 4 Dec 2015 10:49:39 +0000
Message-ID: <DBXPR06MB49492D3052BBE298B33AB8F930C0@DBXPR06MB494.eurprd06.prod.outlook.com>
References: <20151204092130.GA6979@ubuntu>
 <CAGG-pUQkWD1FFTbxzf5od3m7-5ygnk-P677nCsjQmgWpJ_2tpQ@mail.gmail.com>
In-Reply-To: <CAGG-pUQkWD1FFTbxzf5od3m7-5ygnk-P677nCsjQmgWpJ_2tpQ@mail.gmail.com>
Accept-Language: en-US
Content-Language: en-US
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-originalarrivaltime: 04 Dec 2015 10:49:39.6262
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 7a147aaf-01ec-498c-80a1-e34a8c63c548
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DBXPR06MB463
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
Content-Transfer-Encoding: 8bit
X-MIME-Autoconverted: from base64 to 8bit by mail.home.local id tB4AnnLT021078
Content-Length: 3860
Lines: 98

Hi,

Ok I will send all the spaces related error in one single patch.

Thanks,
Yash Shah

-----Original Message-----
From: Geyslan G. Bem [mailto:geyslan@gmail.com] 
Sent: Friday, December 04, 2015 4:08 PM
To: Shah, Yash (Y.)
Cc: axboe@kernel.dk; linux-kernel@vger.kernel.org; Babu, Viswanathan (V.)
Subject: Re: [PATCH] Staging: cdrom: fix space required after that ','

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
????{.n?+???????+%?????ݶ??w??{.n?+????{??G?????{ay?ʇڙ?,j??f???h?????????z_??(?階?ݢj"???m??????G????????????&???~???iO???z??v?^?m????????????I?