diff mbox series

media: dvb: remove spaces before '['

Message ID ad3e13dcf101ff613c7fc72c99db8dd2@208suo.com (mailing list archive)
State Changes Requested
Delegated to: Hans Verkuil
Headers
Series media: dvb: remove spaces before '[' |

Commit Message

sunran001@208suo.com July 14, 2023, 8:06 a.m. UTC 
  Remove spaces to clear checkpatch errors.

ERROR: space prohibited before open square bracket '['

Signed-off-by: Ran Sun <sunran001@208suo.com>
---
  drivers/media/dvb-frontends/tda826x.c | 6 +++---
  1 file changed, 3 insertions(+), 3 deletions(-)

      dprintk("%s:\n", __func__);
@@ -65,7 +65,7 @@ static int tda826x_set_params(struct dvb_frontend *fe)
      u32 div;
      u32 ksyms;
      u32 bandwidth;
-    u8 buf [11];
+    u8 buf[11];
      struct i2c_msg msg = { .addr = priv->i2c_address, .flags = 0, .buf 
= buf, .len = 11 };

      dprintk("%s:\n", __func__);
@@ -130,7 +130,7 @@ static const struct dvb_tuner_ops tda826x_tuner_ops 
= {
  struct dvb_frontend *tda826x_attach(struct dvb_frontend *fe, int addr, 
struct i2c_adapter *i2c, int has_loopthrough)
  {
      struct tda826x_priv *priv = NULL;
-    u8 b1 [] = { 0, 0 };
+    u8 b1[] = { 0, 0 };
      struct i2c_msg msg[2] = {
          { .addr = addr, .flags = 0,        .buf = NULL, .len = 0 },
          { .addr = addr, .flags = I2C_M_RD, .buf = b1, .len = 2 }

Comments

Hans Verkuil July 19, 2023, 6:05 a.m. UTC | #1 
Hi Ran Sun,

On 14/07/2023 10:06, sunran001@208suo.com wrote:
> Remove spaces to clear checkpatch errors.
> 
> ERROR: space prohibited before open square bracket '['
> 
> Signed-off-by: Ran Sun <sunran001@208suo.com>
> ---
>  drivers/media/dvb-frontends/tda826x.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/media/dvb-frontends/tda826x.c b/drivers/media/dvb-frontends/tda826x.c
> index f9703a1dd758..7f17efe17d11 100644
> --- a/drivers/media/dvb-frontends/tda826x.c
> +++ b/drivers/media/dvb-frontends/tda826x.c
> @@ -38,7 +38,7 @@ static int tda826x_sleep(struct dvb_frontend *fe)
>  {
>      struct tda826x_priv *priv = fe->tuner_priv;
>      int ret;
> -    u8 buf [] = { 0x00, 0x8d };
> +    u8 buf[] = { 0x00, 0x8d };

This patch is corrupt: TABs have been replaced by spaces.

It's not just this patch, but all 7 patches you posted to linux-media.

Also, please mention the name of the driver you fixed in the subject.
So: "media: dvb: tda826x: remove space before '['".

Regards,

	Hans

>      struct i2c_msg msg = { .addr = priv->i2c_address, .flags = 0, .buf = buf, .len = 2 };
> 
>      dprintk("%s:\n", __func__);
> @@ -65,7 +65,7 @@ static int tda826x_set_params(struct dvb_frontend *fe)
>      u32 div;
>      u32 ksyms;
>      u32 bandwidth;
> -    u8 buf [11];
> +    u8 buf[11];
>      struct i2c_msg msg = { .addr = priv->i2c_address, .flags = 0, .buf = buf, .len = 11 };
> 
>      dprintk("%s:\n", __func__);
> @@ -130,7 +130,7 @@ static const struct dvb_tuner_ops tda826x_tuner_ops = {
>  struct dvb_frontend *tda826x_attach(struct dvb_frontend *fe, int addr, struct i2c_adapter *i2c, int has_loopthrough)
>  {
>      struct tda826x_priv *priv = NULL;
> -    u8 b1 [] = { 0, 0 };
> +    u8 b1[] = { 0, 0 };
>      struct i2c_msg msg[2] = {
>          { .addr = addr, .flags = 0,        .buf = NULL, .len = 0 },
>          { .addr = addr, .flags = I2C_M_RD, .buf = b1, .len = 2 }
diff mbox series

Patch

diff --git a/drivers/media/dvb-frontends/tda826x.c 
b/drivers/media/dvb-frontends/tda826x.c
index f9703a1dd758..7f17efe17d11 100644
--- a/drivers/media/dvb-frontends/tda826x.c
+++ b/drivers/media/dvb-frontends/tda826x.c
@@ -38,7 +38,7 @@  static int tda826x_sleep(struct dvb_frontend *fe)
  {
      struct tda826x_priv *priv = fe->tuner_priv;
      int ret;
-    u8 buf [] = { 0x00, 0x8d };
+    u8 buf[] = { 0x00, 0x8d };
      struct i2c_msg msg = { .addr = priv->i2c_address, .flags = 0, .buf 
= buf, .len = 2 };