From: Mauro Carvalho Chehab <mchehab@redhat.com>
To: Hans Verkuil <hverkuil@xs4all.nl>
Cc: "linux-media" <linux-media@vger.kernel.org>
Subject: Re: Can you take a look at these dvb-apps warnings/errors?
Date: Tue, 21 May 2013 07:18:56 -0300 [thread overview]
Message-ID: <20130521071856.26588be9@redhat.com> (raw)
In-Reply-To: <201305211120.18566.hverkuil@xs4all.nl>
Em Tue, 21 May 2013 11:20:18 +0200
Hans Verkuil <hverkuil@xs4all.nl> escreveu:
> On Mon 20 May 2013 23:22:15 Mauro Carvalho Chehab wrote:
> > Hi Hans,
> >
> > Em Fri, 17 May 2013 10:30:57 +0200
> > Hans Verkuil <hverkuil@xs4all.nl> escreveu:
> >
> > > Hi Mauro,
> > >
> > > Can you take a look at these? The daily build is failing because of this.
> > >
> > > Thanks!
> > >
> > > Hans
> > >
> > > test_video.c:322:2: warning: format ‘%d’ expects argument of type ‘int’, but argument 2 has type ‘ssize_t’ [-Wformat]
> > > dvbscan.c:128:6: warning: variable ‘output_type’ set but not used [-Wunused-but-set-variable]
> > > dvbscan.c:126:6: warning: variable ‘uk_ordering’ set but not used [-Wunused-but-set-variable]
> > > dvbscan.c:124:32: warning: variable ‘inversion’ set but not used [-Wunused-but-set-variable]
> > > dvbscan_dvb.c:27:44: warning: unused parameter ‘fe’ [-Wunused-parameter]
> > > dvbscan_atsc.c:27:45: warning: unused parameter ‘fe’ [-Wunused-parameter]
> > > util.c:193:7: error: ‘SYS_DVBC_ANNEX_A’ undeclared (first use in this function)
> > > util.c:194:7: error: ‘SYS_DVBC_ANNEX_C’ undeclared (first use in this function)
> > > util.c:262:26: error: ‘DTV_ENUM_DELSYS’ undeclared (first use in this function)
> > > util.c:263:1: warning: control reaches end of non-void function [-Wreturn-type]
> > > make[2]: *** [util.o] Error 1
> > > make[1]: *** [all] Error 2
> > > make: *** [all] Error 2
> >
> > I'm not touching on dvb-apps for a long time. From my side, all I need in
> > terms of userspace apps for DVB is there at dvbv5/libdvbv5 on v4l-utils.
> >
> > That's said, from the above errors, it seems that it got added (partial)
> > support for DVB v5 but, somehow, it is compiling with an older
> > dvb/frontend.h header on your build.
> >
> > Regards,
> > Mauro
> >
>
> I've running a 3.8 kernel on my daily build server, so that's why those
> headers are old. I've manually updated the headers.
>
> Note that dvb-apps has an include directory with old headers as well, but
> those headers aren't used anyway it seems.
Yes, it has old headers on it, but I've no idea if it uses them or not.
> Should I bother with building dvb-apps at all?
It's your call. As I said, from my side, I don't bother about it anymore.
Regards,
Mauro
prev parent reply other threads:[~2013-05-21 10:19 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-05-17 8:30 Can you take a look at these dvb-apps warnings/errors? Hans Verkuil
2013-05-20 21:22 ` Mauro Carvalho Chehab
2013-05-21 9:20 ` Hans Verkuil
2013-05-21 10:18 ` Mauro Carvalho Chehab [this message]
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=20130521071856.26588be9@redhat.com \
--to=mchehab@redhat.com \
--cc=hverkuil@xs4all.nl \
--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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox