diff options
| author | Hans Verkuil <hans.verkuil@cisco.com> | 2016-09-11 05:57:30 -0300 |
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab@s-opensource.com> | 2016-09-12 06:42:44 -0300 |
| commit | c3b809834db8b1a8891c7ff873a216eac119628d (patch) | |
| tree | 3c9e45263676784095ac96100c8afbc17c6f68c1 | |
| parent | 34b27b13a2cd43e0d27e3ce13b96119efbbbedbc (diff) | |
[media] pulse8-cec: fix compiler warning
pulse8-cec.c: In function 'pulse8_connect':
pulse8-cec.c:447:2: warning: 'pa' may be used uninitialized in this function [-Wmaybe-uninitialized]
cec_s_phys_addr(pulse8->adap, pa, false);
^
pulse8-cec.c:609:6: note: 'pa' was declared here
u16 pa;
^
As far as I can tell, this can't actually happen. Still, it is better to just
initialize it.
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>
| -rw-r--r-- | drivers/staging/media/pulse8-cec/pulse8-cec.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/staging/media/pulse8-cec/pulse8-cec.c b/drivers/staging/media/pulse8-cec/pulse8-cec.c index cb6a2a3dcc68..13e8f61c0a34 100644 --- a/drivers/staging/media/pulse8-cec/pulse8-cec.c +++ b/drivers/staging/media/pulse8-cec/pulse8-cec.c @@ -606,7 +606,7 @@ static int pulse8_connect(struct serio *serio, struct serio_driver *drv) struct pulse8 *pulse8; int err = -ENOMEM; struct cec_log_addrs log_addrs = {}; - u16 pa; + u16 pa = CEC_PHYS_ADDR_INVALID; pulse8 = kzalloc(sizeof(*pulse8), GFP_KERNEL); |