All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mauro Carvalho Chehab <m.chehab@samsung.com>
To: James Hogan <james.hogan@imgtec.com>
Cc: linux-media@vger.kernel.org
Subject: Re: [PATCH v4 10/10] rc: img-ir: add Sanyo decoder module
Date: Wed, 12 Mar 2014 07:57:59 -0300	[thread overview]
Message-ID: <20140312075759.71eceec7@samsung.com> (raw)
In-Reply-To: <1393630140-31765-11-git-send-email-james.hogan@imgtec.com>

Hi James,

Em Fri, 28 Feb 2014 23:29:00 +0000
James Hogan <james.hogan@imgtec.com> escreveu:

> Add an img-ir module for decoding the Sanyo infrared protocol.

After applying this series, some new warnings are popping up,
when compiled with W=1:

drivers/media/rc/img-ir/img-ir-hw.c: In function 'img_ir_free_timing':
drivers/media/rc/img-ir/img-ir-hw.c:228:23: warning: variable 'maxlen' set but not used [-Wunused-but-set-variable]
  unsigned int minlen, maxlen, ft_min;
                       ^
drivers/media/rc/img-ir/img-ir-hw.c:228:15: warning: variable 'minlen' set but not used [-Wunused-but-set-variable]
  unsigned int minlen, maxlen, ft_min;
               ^
drivers/media/rc/img-ir/img-ir-jvc.c:76:3: warning: initialized field overwritten [-Woverride-init]
   },
   ^
drivers/media/rc/img-ir/img-ir-jvc.c:76:3: warning: (near initialization for 'img_ir_jvc.timings.s00') [-Woverride-init]
drivers/media/rc/img-ir/img-ir-jvc.c:81:3: warning: initialized field overwritten [-Woverride-init]
   },
   ^
drivers/media/rc/img-ir/img-ir-jvc.c:81:3: warning: (near initialization for 'img_ir_jvc.timings.s01') [-Woverride-init]

Please fix.

Regards,
Mauro

  reply	other threads:[~2014-03-12 10:58 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-02-28 23:28 [PATCH v4 00/10] media: rc: ImgTec IR decoder driver James Hogan
2014-02-28 23:28 ` [PATCH v4 01/10] dt: binding: add binding for ImgTec IR block James Hogan
2014-02-28 23:28 ` [PATCH v4 02/10] rc: img-ir: add base driver James Hogan
2014-02-28 23:28 ` [PATCH v4 03/10] rc: img-ir: add raw driver James Hogan
2014-02-28 23:28 ` [PATCH v4 04/10] rc: img-ir: add hardware decoder driver James Hogan
2014-02-28 23:28 ` [PATCH v4 05/10] rc: img-ir: add to build James Hogan
2014-02-28 23:28 ` [PATCH v4 06/10] rc: img-ir: add NEC decoder module James Hogan
2014-02-28 23:28 ` [PATCH v4 07/10] rc: img-ir: add JVC " James Hogan
2014-02-28 23:28 ` [PATCH v4 08/10] rc: img-ir: add Sony " James Hogan
2014-02-28 23:28 ` [PATCH v4 09/10] rc: img-ir: add Sharp " James Hogan
2014-02-28 23:29 ` [PATCH v4 10/10] rc: img-ir: add Sanyo " James Hogan
2014-03-12 10:57   ` Mauro Carvalho Chehab [this message]
2014-03-12 11:12     ` James Hogan
2014-03-25 23:53 ` [PATCH v4 00/10] media: rc: ImgTec IR decoder driver David Härdeman
2014-03-26  9:28   ` James Hogan
2014-03-26 13:33     ` David Härdeman

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20140312075759.71eceec7@samsung.com \
    --to=m.chehab@samsung.com \
    --cc=james.hogan@imgtec.com \
    --cc=linux-media@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.