* [PATCH] staging: media: av7110: remove redundant space after cast
@ 2022-04-18 17:09 Aliya Rahmani
2022-04-18 19:22 ` Joe Perches
0 siblings, 1 reply; 2+ messages in thread
From: Aliya Rahmani @ 2022-04-18 17:09 UTC (permalink / raw)
To: mchehab; +Cc: gregkh, linux-staging, outreachy, linux-kernel, Aliya Rahmani
Remove the unnecessary space immediately after a cast. Identified by
checkpatch: CHECK: No space is necessary after a cast.
Signed-off-by: Aliya Rahmani <aliyarahmani786@gmail.com>
---
drivers/staging/media/av7110/av7110_av.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/media/av7110/av7110_av.c b/drivers/staging/media/av7110/av7110_av.c
index 91f4866c7e59..df41d2ce1ed7 100644
--- a/drivers/staging/media/av7110/av7110_av.c
+++ b/drivers/staging/media/av7110/av7110_av.c
@@ -93,7 +93,7 @@ int av7110_record_cb(struct dvb_filter_pes2ts *p2t, u8 *buf, size_t len)
static int dvb_filter_pes2ts_cb(void *priv, unsigned char *data)
{
- struct dvb_demux_feed *dvbdmxfeed = (struct dvb_demux_feed *) priv;
+ struct dvb_demux_feed *dvbdmxfeed = (struct dvb_demux_feed *)priv;
dvbdmxfeed->cb.ts(data, 188, NULL, 0,
&dvbdmxfeed->feed.ts, NULL);
--
2.25.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] staging: media: av7110: remove redundant space after cast
2022-04-18 17:09 [PATCH] staging: media: av7110: remove redundant space after cast Aliya Rahmani
@ 2022-04-18 19:22 ` Joe Perches
0 siblings, 0 replies; 2+ messages in thread
From: Joe Perches @ 2022-04-18 19:22 UTC (permalink / raw)
To: Aliya Rahmani, mchehab; +Cc: gregkh, linux-staging, outreachy, linux-kernel
On Mon, 2022-04-18 at 22:39 +0530, Aliya Rahmani wrote:
> Remove the unnecessary space immediately after a cast. Identified by
> checkpatch: CHECK: No space is necessary after a cast.
[]
> diff --git a/drivers/staging/media/av7110/av7110_av.c b/drivers/staging/media/av7110/av7110_av.c
[]
> @@ -93,7 +93,7 @@ int av7110_record_cb(struct dvb_filter_pes2ts *p2t, u8 *buf, size_t len)
>
> static int dvb_filter_pes2ts_cb(void *priv, unsigned char *data)
> {
> - struct dvb_demux_feed *dvbdmxfeed = (struct dvb_demux_feed *) priv;
> + struct dvb_demux_feed *dvbdmxfeed = (struct dvb_demux_feed *)priv;
It's also unnecessary to cast void pointers so the cast could be removed.
struct dvb_demux_feed *dvbdmxfeed = priv;
And the code here uses dvbdmxfeed and feed rather interchangeably for
uses of a struct dvb_demux_feed *.
It might be better just to use feed and change the other uses too.
Do consider this just a coding exercise as the driver is for some
hardware that is effectively dead/unavailable/non-functional.
Look at:
commit 989cf18ed08f8b6efd1d1592d1d0108fa09b98f5
Author: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
Date: Mon May 31 17:27:09 2021 +0200
media: av7110: move driver to staging
This driver is really old, from devices that aren't
manufactured anymore for more than a decade.
Maybe instead consider adding a MAINTAINERS section
suggesting this is obsolete and/or a TODO entry in this
directory saying "Delete this" after some time.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2022-04-18 20:38 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-04-18 17:09 [PATCH] staging: media: av7110: remove redundant space after cast Aliya Rahmani
2022-04-18 19:22 ` Joe Perches
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.