public inbox for linux-media@vger.kernel.org
 help / color / mirror / Atom feed
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(-)
> 


      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