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 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.