All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] media:av7110:av7110_av.c remove blank line
@ 2022-03-29 19:01 Alaa Mohamed
  2022-03-29 20:10 ` Alison Schofield
  0 siblings, 1 reply; 2+ messages in thread
From: Alaa Mohamed @ 2022-03-29 19:01 UTC (permalink / raw)
  To: outreachy
  Cc: mchehab, gregkh, linux-media, linux-staging, linux-kernel,
	eng.alaamohamedsoliman.am

according to checkpatch.pl results
"CHECK: Please don't use multiple blank lines"
after lines 73 and 78 ,I removed the blank lines

Signed-off-by: Alaa Mohamed <eng.alaamohamedsoliman.am@gmail.com>
---
 drivers/staging/media/av7110/av7110_av.c | 2 --
 1 file changed, 2 deletions(-)

diff --git a/drivers/staging/media/av7110/av7110_av.c b/drivers/staging/media/av7110/av7110_av.c
index 91f4866c7e59..1544f120e8b1 100644
--- a/drivers/staging/media/av7110/av7110_av.c
+++ b/drivers/staging/media/av7110/av7110_av.c
@@ -70,12 +70,10 @@
 #define PIECE_RATE	 0x40
 #define SEAM_SPLICE	 0x20
 
-
 static void p_to_t(u8 const *buf, long int length, u16 pid,
 		   u8 *counter, struct dvb_demux_feed *feed);
 static int write_ts_to_decoder(struct av7110 *av7110, int type, const u8 *buf, size_t len);
 
-
 int av7110_record_cb(struct dvb_filter_pes2ts *p2t, u8 *buf, size_t len)
 {
 	struct dvb_demux_feed *dvbdmxfeed = (struct dvb_demux_feed *) p2t->priv;
-- 
2.35.1


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

* Re: [PATCH] media:av7110:av7110_av.c remove blank line
  2022-03-29 19:01 [PATCH] media:av7110:av7110_av.c remove blank line Alaa Mohamed
@ 2022-03-29 20:10 ` Alison Schofield
  0 siblings, 0 replies; 2+ messages in thread
From: Alison Schofield @ 2022-03-29 20:10 UTC (permalink / raw)
  To: Alaa Mohamed
  Cc: outreachy, mchehab, gregkh, linux-media, linux-staging, linux-kernel

On Tue, Mar 29, 2022 at 09:01:33PM +0200, Alaa Mohamed wrote:
> according to checkpatch.pl results
> "CHECK: Please don't use multiple blank lines"
> after lines 73 and 78 ,I removed the blank lines
> 
> Signed-off-by: Alaa Mohamed <eng.alaamohamedsoliman.am@gmail.com>
> ---
>  drivers/staging/media/av7110/av7110_av.c | 2 --

Hi Alaa,

We are not taking cleanup patches in drivers/staging/media.
It's documented in here https://kernelnewbies.org/Outreachyfirstpatch
and I will repost the restriction on this list as the discussion
occurred a few days before the contribution period began.

If you look at the git history on this file, you'll see it is
on it's way out of the kernel, not on it's way in!

Please look at my feedback to Rebecca and Sevinj recent checkpatch
cleanups as guidance for your next (non-media) cleanup patch.

Thanks!
Alison





>  1 file changed, 2 deletions(-)
> 
> diff --git a/drivers/staging/media/av7110/av7110_av.c b/drivers/staging/media/av7110/av7110_av.c
> index 91f4866c7e59..1544f120e8b1 100644
> --- a/drivers/staging/media/av7110/av7110_av.c
> +++ b/drivers/staging/media/av7110/av7110_av.c
> @@ -70,12 +70,10 @@
>  #define PIECE_RATE	 0x40
>  #define SEAM_SPLICE	 0x20
>  
> -
>  static void p_to_t(u8 const *buf, long int length, u16 pid,
>  		   u8 *counter, struct dvb_demux_feed *feed);
>  static int write_ts_to_decoder(struct av7110 *av7110, int type, const u8 *buf, size_t len);
>  
> -
>  int av7110_record_cb(struct dvb_filter_pes2ts *p2t, u8 *buf, size_t len)
>  {
>  	struct dvb_demux_feed *dvbdmxfeed = (struct dvb_demux_feed *) p2t->priv;
> -- 
> 2.35.1
> 
> 

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

end of thread, other threads:[~2022-03-29 20:08 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-03-29 19:01 [PATCH] media:av7110:av7110_av.c remove blank line Alaa Mohamed
2022-03-29 20:10 ` Alison Schofield

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.