From: Hans Verkuil <hverkuil@xs4all.nl>
To: Mauro Carvalho Chehab <m.chehab@samsung.com>
Cc: Linux Media Mailing List <linux-media@vger.kernel.org>,
Mauro Carvalho Chehab <mchehab@infradead.org>
Subject: Re: [PATCH v3 00/29] Fix errors/warnings with allmodconfig/allyesconfig on non-x86 archs
Date: Tue, 05 Nov 2013 16:17:26 +0100 [thread overview]
Message-ID: <52790C06.1010806@xs4all.nl> (raw)
In-Reply-To: <1383645702-30636-1-git-send-email-m.chehab@samsung.com>
Hi Mauro,
For all patches except 22 (which got a newer revision):
Reviewed-by: Hans Verkuil <hans.verkuil@cisco.com>
Regards,
Hans
On 11/05/13 11:01, Mauro Carvalho Chehab wrote:
> To be sure that we're not introducing compilation regressions on media, I'm now
> using ktest to check for errors/warnings.
>
> My current setup is cross-building on several architectures:
> alpha, arm, avr32, cris (64), frv, i386, ia64, m32r, m68k, mips, openrisc, parisc, s390, sh, sparc, sparc64, uml, x86_64
>
> I tried to enable a few other archs:
> blackfin, cris (32), powerpc (32, 64), tile, xtensa
>
> but they fail to compile with allyesconfig due to non-media related issues.
>
> I'm still unsure about how often I'll be doing it, I intend to run it at least
> by the end of the subsystem merge window (by -rc6 or -rc7), and fix the
> issues found there.
>
> V3: contains fixes for the feedbacks received:
> - I2C client driver now returns -EINVAL;
> - I2C adapter drivers now returns -EOPNOSUPP;
> - a macro was added for the buffers;
> - check for failure on allocation at v4l2-async was added;
> - Used iguanair code from Sean, instead of my code;
> - Most buffer sizes changed to 64 bytes, to match max URB
> control size.
>
> Mauro Carvalho Chehab (28):
> [media] tda9887: remove an warning when compiling for alpha
> [media] radio-shark: remove a warning when CONFIG_PM is not defined
> [media] zoran: don't build it on alpha
> [media] cx18: struct i2c_client is too big for stack
> [media] tef6862: fix warning on avr32 arch
> [media] platform drivers: Fix build on frv arch
> [media] radio-si470x-i2c: fix a warning on ia64
> [media] rc: Fir warnings on m68k arch
> [media] uvc/lirc_serial: Fix some warnings on parisc arch
> [media] s5h1420: Don't use dynamic static allocation
> [media] dvb-frontends: Don't use dynamic static allocation
> [media] dvb-frontends: Don't use dynamic static allocation
> [media] stb0899_drv: Don't use dynamic static allocation
> [media] stv0367: Don't use dynamic static allocation
> [media] stv090x: Don't use dynamic static allocation
> [media] av7110_hw: Don't use dynamic static allocation
> [media] tuners: Don't use dynamic static allocation
> [media] tuner-xc2028: Don't use dynamic static allocation
> [media] cimax2: Don't use dynamic static allocation
> [media] v4l2-async: Don't use dynamic static allocation
> [media] cxusb: Don't use dynamic static allocation
> [media] dibusb-common: Don't use dynamic static allocation
> [media] dw2102: Don't use dynamic static allocation
> [media] af9015: Don't use dynamic static allocation
> [media] af9035: Don't use dynamic static allocation
> [media] mxl111sf: Don't use dynamic static allocation
> [media] lirc_zilog: Don't use dynamic static allocation
> [media] cx18: disable compilation on frv arch
>
> Sean Young (1):
> [media] iguanair: simplify calculation of carrier delay cycles
>
> drivers/media/dvb-frontends/af9013.c | 12 +++-
> drivers/media/dvb-frontends/af9033.c | 21 ++++++-
> drivers/media/dvb-frontends/bcm3510.c | 15 ++++-
> drivers/media/dvb-frontends/cxd2820r_core.c | 21 ++++++-
> drivers/media/dvb-frontends/itd1000.c | 13 +++-
> drivers/media/dvb-frontends/mt312.c | 10 ++-
> drivers/media/dvb-frontends/nxt200x.c | 11 +++-
> drivers/media/dvb-frontends/rtl2830.c | 12 +++-
> drivers/media/dvb-frontends/rtl2832.c | 12 +++-
> drivers/media/dvb-frontends/s5h1420.c | 9 ++-
> drivers/media/dvb-frontends/stb0899_drv.c | 12 +++-
> drivers/media/dvb-frontends/stb6100.c | 11 +++-
> drivers/media/dvb-frontends/stv0367.c | 13 +++-
> drivers/media/dvb-frontends/stv090x.c | 12 +++-
> drivers/media/dvb-frontends/stv6110.c | 12 +++-
> drivers/media/dvb-frontends/stv6110x.c | 13 +++-
> drivers/media/dvb-frontends/tda10071.c | 21 ++++++-
> drivers/media/dvb-frontends/tda18271c2dd.c | 14 ++++-
> drivers/media/dvb-frontends/zl10039.c | 12 +++-
> drivers/media/pci/cx18/Kconfig | 1 +
> drivers/media/pci/cx18/cx18-driver.c | 20 +++---
> drivers/media/pci/cx23885/cimax2.c | 13 +++-
> drivers/media/pci/ttpci/av7110_hw.c | 19 +++++-
> drivers/media/pci/zoran/Kconfig | 1 +
> drivers/media/platform/soc_camera/rcar_vin.c | 1 +
> drivers/media/radio/radio-shark.c | 2 +
> drivers/media/radio/radio-shark2.c | 2 +
> drivers/media/radio/si470x/radio-si470x-i2c.c | 4 +-
> drivers/media/radio/tef6862.c | 20 +++---
> drivers/media/rc/fintek-cir.h | 4 +-
> drivers/media/rc/iguanair.c | 22 ++-----
> drivers/media/rc/nuvoton-cir.h | 4 +-
> drivers/media/tuners/e4000.c | 21 ++++++-
> drivers/media/tuners/fc2580.c | 21 ++++++-
> drivers/media/tuners/tda18212.c | 21 ++++++-
> drivers/media/tuners/tda18218.c | 21 ++++++-
> drivers/media/tuners/tda9887.c | 4 +-
> drivers/media/tuners/tuner-xc2028.c | 8 ++-
> drivers/media/usb/dvb-usb-v2/af9015.c | 3 +-
> drivers/media/usb/dvb-usb-v2/af9035.c | 29 ++++++++-
> drivers/media/usb/dvb-usb-v2/mxl111sf.c | 10 ++-
> drivers/media/usb/dvb-usb/cxusb.c | 41 ++++++++++--
> drivers/media/usb/dvb-usb/dibusb-common.c | 10 ++-
> drivers/media/usb/dvb-usb/dw2102.c | 90 ++++++++++++++++++++++++---
> drivers/media/usb/uvc/uvc_video.c | 3 +-
> drivers/media/v4l2-core/v4l2-async.c | 31 ++++++++-
> drivers/staging/media/lirc/lirc_serial.c | 9 ++-
> drivers/staging/media/lirc/lirc_zilog.c | 12 +++-
> 48 files changed, 598 insertions(+), 105 deletions(-)
>
prev parent reply other threads:[~2013-11-05 15:18 UTC|newest]
Thread overview: 39+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-11-05 10:01 [PATCH v3 00/29] Fix errors/warnings with allmodconfig/allyesconfig on non-x86 archs Mauro Carvalho Chehab
2013-11-05 10:01 ` [PATCH v3 01/29] [media] tda9887: remove an warning when compiling for alpha Mauro Carvalho Chehab
2013-11-05 10:01 ` [PATCH v3 02/29] [media] radio-shark: remove a warning when CONFIG_PM is not defined Mauro Carvalho Chehab
2013-11-05 10:01 ` [PATCH v3 03/29] [media] zoran: don't build it on alpha Mauro Carvalho Chehab
2013-11-05 10:01 ` [PATCH v3 04/29] [media] cx18: struct i2c_client is too big for stack Mauro Carvalho Chehab
2013-11-06 0:19 ` Andy Walls
2013-11-05 10:01 ` [PATCH v3 05/29] [media] tef6862: fix warning on avr32 arch Mauro Carvalho Chehab
2013-11-05 10:01 ` [PATCH v3 06/29] [media] iguanair: simplify calculation of carrier delay cycles Mauro Carvalho Chehab
2013-11-05 10:01 ` [PATCH v3 07/29] [media] platform drivers: Fix build on frv arch Mauro Carvalho Chehab
2013-11-05 10:01 ` [PATCH v3 08/29] [media] radio-si470x-i2c: fix a warning on ia64 Mauro Carvalho Chehab
2013-11-05 10:01 ` [PATCH v3 09/29] [media] rc: Fir warnings on m68k arch Mauro Carvalho Chehab
2013-11-05 10:01 ` [PATCH v3 10/29] [media] uvc/lirc_serial: Fix some warnings on parisc arch Mauro Carvalho Chehab
2013-11-05 10:01 ` [PATCH v3 11/29] [media] s5h1420: Don't use dynamic static allocation Mauro Carvalho Chehab
2013-11-05 10:01 ` [PATCH v3 12/29] [media] dvb-frontends: " Mauro Carvalho Chehab
2013-11-05 10:01 ` [PATCH v3 13/29] " Mauro Carvalho Chehab
2013-11-05 17:00 ` Antti Palosaari
2013-11-05 10:01 ` [PATCH v3 14/29] [media] stb0899_drv: " Mauro Carvalho Chehab
2013-11-05 10:01 ` [PATCH v3 15/29] [media] stv0367: " Mauro Carvalho Chehab
2013-11-05 10:01 ` [PATCH v3 16/29] [media] stv090x: " Mauro Carvalho Chehab
2013-11-05 10:01 ` [PATCH v3 17/29] [media] av7110_hw: " Mauro Carvalho Chehab
2013-11-05 10:01 ` [PATCH v3 18/29] [media] tuners: " Mauro Carvalho Chehab
2013-11-05 17:11 ` Antti Palosaari
2013-11-07 18:55 ` Antti Palosaari
2013-11-07 21:13 ` Mauro Carvalho Chehab
2013-11-07 23:27 ` Antti Palosaari
2013-11-05 10:01 ` [PATCH v3 19/29] [media] tuner-xc2028: " Mauro Carvalho Chehab
2013-11-05 10:01 ` [PATCH v3 20/29] [media] cimax2: " Mauro Carvalho Chehab
2013-11-05 10:01 ` [PATCH v3 21/29] [media] v4l2-async: " Mauro Carvalho Chehab
2013-11-05 10:01 ` [PATCH v3 22/29] [media] cxusb: " Mauro Carvalho Chehab
2013-11-05 10:01 ` [PATCH v3 23/29] [media] dibusb-common: " Mauro Carvalho Chehab
2013-11-05 10:01 ` [PATCH v3 24/29] [media] dw2102: " Mauro Carvalho Chehab
2013-11-05 10:01 ` [PATCH v3 25/29] [media] af9015: " Mauro Carvalho Chehab
2013-11-05 17:14 ` Antti Palosaari
2013-11-05 10:01 ` [PATCH v3 26/29] [media] af9035: " Mauro Carvalho Chehab
2013-11-05 17:17 ` Antti Palosaari
2013-11-05 10:01 ` [PATCH v3 27/29] [media] mxl111sf: " Mauro Carvalho Chehab
2013-11-05 10:01 ` [PATCH v3 28/29] [media] lirc_zilog: " Mauro Carvalho Chehab
2013-11-05 10:01 ` [PATCH v3 29/29] [media] cx18: disable compilation on frv arch Mauro Carvalho Chehab
2013-11-05 15:17 ` Hans Verkuil [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=52790C06.1010806@xs4all.nl \
--to=hverkuil@xs4all.nl \
--cc=linux-media@vger.kernel.org \
--cc=m.chehab@samsung.com \
--cc=mchehab@infradead.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