| Message ID | 20210416235336.1552102-1-sudipm.mukherjee@gmail.com |
|---|---|
| State | Accepted |
| Commit | e594cda5f8c8e34ac6444166418690c65bbda47f |
| Headers | show |
| Series | media: sp887x: drop unneeded assignment | expand |
diff --git a/drivers/media/dvb-frontends/sp887x.c b/drivers/media/dvb-frontends/sp887x.c index c89a91a3daf4..146e7f2dd3c5 100644 --- a/drivers/media/dvb-frontends/sp887x.c +++ b/drivers/media/dvb-frontends/sp887x.c @@ -140,7 +140,7 @@ static int sp887x_initial_setup (struct dvb_frontend* fe, const struct firmware u8 buf [BLOCKSIZE + 2]; int i; int fw_size = fw->size; - const unsigned char *mem = fw->data; + const unsigned char *mem = fw->data + 10; dprintk("%s\n", __func__); @@ -148,8 +148,6 @@ static int sp887x_initial_setup (struct dvb_frontend* fe, const struct firmware if (fw_size < FW_SIZE + 10) return -ENODEV; - mem = fw->data + 10; - /* soft reset */ sp887x_writereg(state, 0xf1a, 0x000);
The pointer 'mem' was initialized to 'fw->data' but immediately after that it was assigned 'fw->data + 10'. Lets remove the extra assignement and initialize the pointer to the address its going to use. Signed-off-by: Sudip Mukherjee <sudipm.mukherjee@gmail.com> --- drivers/media/dvb-frontends/sp887x.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-)