From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753277Ab2GZWNZ (ORCPT ); Thu, 26 Jul 2012 18:13:25 -0400 Received: from proofpoint-cluster.metrocast.net ([65.175.128.136]:56716 "EHLO proofpoint-cluster.metrocast.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752390Ab2GZWNX (ORCPT ); Thu, 26 Jul 2012 18:13:23 -0400 Subject: Re: [PATCH v2] cx18: Declare MODULE_FIRMWARE usage From: Andy Walls To: Tim Gardner Cc: linux-kernel@vger.kernel.org, Mauro Carvalho Chehab , ivtv-devel@ivtvdriver.org, linux-media@vger.kernel.org Date: Thu, 26 Jul 2012 18:12:47 -0400 In-Reply-To: <50117C91.8090207@canonical.com> References: <1343322358-128310-1-git-send-email-tim.gardner@canonical.com> <50117C91.8090207@canonical.com> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.0.3 (3.0.3-1.fc15) Content-Transfer-Encoding: 7bit Message-ID: <1343340768.2575.14.camel@palomino.walls.org> Mime-Version: 1.0 X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10432:5.7.7855,1.0.260,0.0.0000 definitions=2012-07-26_06:2012-07-26,2012-07-26,1970-01-01 signatures=0 X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 spamscore=0 ipscore=0 suspectscore=0 phishscore=0 bulkscore=0 adultscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=6.0.2-1203120001 definitions=main-1207260265 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, 2012-07-26 at 11:21 -0600, Tim Gardner wrote: > Missed a firmware file in cx18-av-firmware.c > > rtg Please send patches in-line vs. an attachment. This is still missing the firmware file in cx18-dvb.c. Regards, Andy > From 9b4be013f173efc12bb2776394bf6a5abb8725b6 Mon Sep 17 00:00:00 2001 > From: Tim Gardner > Date: Thu, 26 Jul 2012 11:03:51 -0600 > Subject: [PATCH v2] cx18: Declare MODULE_FIRMWARE usage > > Cc: Andy Walls > Cc: Mauro Carvalho Chehab > Cc: ivtv-devel@ivtvdriver.org > Cc: linux-media@vger.kernel.org > Signed-off-by: Tim Gardner > --- > drivers/media/video/cx18/cx18-av-firmware.c | 2 ++ > drivers/media/video/cx18/cx18-driver.c | 1 + > drivers/media/video/cx18/cx18-firmware.c | 10 ++++++++-- > 3 files changed, 11 insertions(+), 2 deletions(-) > > diff --git a/drivers/media/video/cx18/cx18-av-firmware.c > b/drivers/media/video/cx18/cx18-av-firmware.c > index 280aa4d..a34fd08 100644 > --- a/drivers/media/video/cx18/cx18-av-firmware.c > +++ b/drivers/media/video/cx18/cx18-av-firmware.c > @@ -221,3 +221,5 @@ int cx18_av_loadfw(struct cx18 *cx) > release_firmware(fw); > return 0; > } > + > +MODULE_FIRMWARE(FWFILE); > diff --git a/drivers/media/video/cx18/cx18-driver.c > b/drivers/media/video/cx18/cx18-driver.c > index 7e5ffd6..c67733d 100644 > --- a/drivers/media/video/cx18/cx18-driver.c > +++ b/drivers/media/video/cx18/cx18-driver.c > @@ -1357,3 +1357,4 @@ static void __exit module_cleanup(void) > > module_init(module_start); > module_exit(module_cleanup); > +MODULE_FIRMWARE(XC2028_DEFAULT_FIRMWARE); > diff --git a/drivers/media/video/cx18/cx18-firmware.c > b/drivers/media/video/cx18/cx18-firmware.c > index b85c292..a1c1cec 100644 > --- a/drivers/media/video/cx18/cx18-firmware.c > +++ b/drivers/media/video/cx18/cx18-firmware.c > @@ -376,6 +376,9 @@ void cx18_init_memory(struct cx18 *cx) > cx18_write_reg(cx, 0x00000101, CX18_WMB_CLIENT14); /* AVO */ > } > > +#define CX18_CPU_FIRMWARE "v4l-cx23418-cpu.fw" > +#define CX18_APU_FIRMWARE "v4l-cx23418-apu.fw" > + > int cx18_firmware_init(struct cx18 *cx) > { > u32 fw_entry_addr; > @@ -400,7 +403,7 @@ int cx18_firmware_init(struct cx18 *cx) > cx18_sw1_irq_enable(cx, IRQ_CPU_TO_EPU | IRQ_APU_TO_EPU); > cx18_sw2_irq_enable(cx, IRQ_CPU_TO_EPU_ACK | > IRQ_APU_TO_EPU_ACK); > > - sz = load_cpu_fw_direct("v4l-cx23418-cpu.fw", cx->enc_mem, > cx); > + sz = load_cpu_fw_direct(CX18_CPU_FIRMWARE, cx->enc_mem, cx); > if (sz <= 0) > return sz; > > @@ -408,7 +411,7 @@ int cx18_firmware_init(struct cx18 *cx) > cx18_init_scb(cx); > > fw_entry_addr = 0; > - sz = load_apu_fw_direct("v4l-cx23418-apu.fw", cx->enc_mem, cx, > + sz = load_apu_fw_direct(CX18_APU_FIRMWARE, cx->enc_mem, cx, > &fw_entry_addr); > if (sz <= 0) > return sz; > @@ -451,3 +454,6 @@ int cx18_firmware_init(struct cx18 *cx) > cx18_write_reg_expect(cx, 0x14001400, 0xc78110, 0x00001400, > 0x14001400); > return 0; > } > + > +MODULE_FIRMWARE(CX18_CPU_FIRMWARE); > +MODULE_FIRMWARE(CX18_APU_FIRMWARE); > -- > 1.7.9.5 > >