From patchwork Sat Oct 6 18:23:00 2012 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Sylwester Nawrocki X-Patchwork-Id: 1558811 Return-Path: X-Original-To: patchwork-linux-media@patchwork.kernel.org Delivered-To: patchwork-process-083081@patchwork1.kernel.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by patchwork1.kernel.org (Postfix) with ESMTP id 60FD93FD9C for ; Sat, 6 Oct 2012 18:23:09 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752165Ab2JFSXG (ORCPT ); Sat, 6 Oct 2012 14:23:06 -0400 Received: from mail-ee0-f46.google.com ([74.125.83.46]:46867 "EHLO mail-ee0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751924Ab2JFSXF (ORCPT ); Sat, 6 Oct 2012 14:23:05 -0400 Received: by mail-ee0-f46.google.com with SMTP id b15so2032750eek.19 for ; Sat, 06 Oct 2012 11:23:03 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=message-id:date:from:user-agent:mime-version:to:cc:subject :references:in-reply-to:content-type:content-transfer-encoding; bh=+Qgyd+mDxMOphccgMJ24mSdwFbn0t9F9Mcc0pxH6qqg=; b=AGZd/KdjNfNnAZZbbPS4JKBwmFkbpe9eCSntD3dnwysovbmDDdgemWOgT/F1KBRGum R66SrPXsI6n/Zc2CtddzMO+vxA0+IAHRzIfUrO2vBe/mLYxPHFTfdrfRe4HsIzXp/f96 8N/QBHW6hBmg+doEN4zy/1rQZQZUYl+3mrVcNUILY0YaAi56jwC4uKotSZQ6bNpmMc3X MtqX69CNYXE966QlOcJCqTlWtQLrGt4r7DHAWUE7Z6kNH7+zMDXAO697mKzI/w1IBeYa WCJj3l64rGJiyl9IwzFZZnNTXVzi3kse/rxv4NfFSeufuGzeezYb+Xll2QOPH9zeE0kP Fjxw== Received: by 10.14.225.73 with SMTP id y49mr1376687eep.25.1349547783607; Sat, 06 Oct 2012 11:23:03 -0700 (PDT) Received: from [192.168.1.110] (031011252076.warszawa.vectranet.pl. [31.11.252.76]) by mx.google.com with ESMTPS id i1sm23385164eeo.8.2012.10.06.11.23.01 (version=SSLv3 cipher=OTHER); Sat, 06 Oct 2012 11:23:02 -0700 (PDT) Message-ID: <50707704.5030402@gmail.com> Date: Sat, 06 Oct 2012 20:23:00 +0200 From: Sylwester Nawrocki User-Agent: Mozilla/5.0 (X11; Linux i686; rv:11.0) Gecko/20120412 Thunderbird/11.0.1 MIME-Version: 1.0 To: Jan Hoogenraad CC: Sylwester Nawrocki , linux-media@vger.kernel.org, a.hajda@samsung.com, sakari.ailus@iki.fi, laurent.pinchart@ideasonboard.com, hverkuil@xs4all.nl, kyungmin.park@samsung.com, sw0312.kim@samsung.com Subject: Re: Media_build broken by [PATCH RFC v3 5/5] m5mols: Implement .get_frame_desc subdev callback References: <1348674853-24596-1-git-send-email-s.nawrocki@samsung.com> <1348674853-24596-6-git-send-email-s.nawrocki@samsung.com> <50704D26.9020201@hoogenraad.net> In-Reply-To: <50704D26.9020201@hoogenraad.net> Sender: linux-media-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-media@vger.kernel.org Hello Jan, On 10/06/2012 05:24 PM, Jan Hoogenraad wrote: > On my ubuntu 10.4 system > > Linux 2.6.32-43-generic-pae #97-Ubuntu SMP Wed Sep 5 16:59:17 UTC 2012 > i686 GNU/Linux > > this patch breaks compilation of media_build. > The constant SZ_1M is not defined in the includes on my system > > Do you know what can be done about this ? > > --- > > /home/jhh/dvb/media_build/v4l/m5mols_core.c: In function > 'm5mols_set_frame_desc': > /home/jhh/dvb/media_build/v4l/m5mols_core.c:636: error: 'SZ_1M' > undeclared (first use in this function) > /home/jhh/dvb/media_build/v4l/m5mols_core.c:636: error: (Each undeclared > identifier is reported only once > /home/jhh/dvb/media_build/v4l/m5mols_core.c:636: error: for each > function it appears in.) Thanks for reporting this issue. You most likely don't need the M-5MOLS camera sensor driver on you system so one option is to just disable it at kernel config. Make sure CONFIG_VIDEO_M5MOLS is not set, it can be unselected at menuconfig -> Device Drivers -> Multimedia -> Encoders, decoders, sensors and other helper chips < > Fujitsu M-5MOLS 8MP sensor support The below patch which is intended to fix this issue won't work for media drivers backport builds on kernels older than 3.6, so m5mols driver should not be built for kernel versions < 3.6. 8<------------------------------------------------------------------- From 3e138ea603c9e5102452554cb14e4b404ce306e0 Mon Sep 17 00:00:00 2001 From: Sylwester Nawrocki Date: Sat, 6 Oct 2012 20:04:40 +0200 Subject: [PATCH] m5mols: Add missing #include Include header that is missing after commit ab7ef22419927 "[media] m5mols: Implement .get_frame_desc subdev callback". It prevents possible build errors due to undefined SZ_1M. Reported-by: Jan Hoogenraad Signed-off-by: Sylwester Nawrocki --- drivers/media/i2c/m5mols/m5mols.h | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/drivers/media/i2c/m5mols/m5mols.h b/drivers/media/i2c/m5mols/m5mols.h index 4ab8b37..90a6c52 100644 --- a/drivers/media/i2c/m5mols/m5mols.h +++ b/drivers/media/i2c/m5mols/m5mols.h @@ -16,6 +16,7 @@ #ifndef M5MOLS_H #define M5MOLS_H +#include #include #include "m5mols_reg.h"