From patchwork Mon Sep 28 16:19:43 2009 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Filipe Rosset X-Patchwork-Id: 50412 Received: from vger.kernel.org (vger.kernel.org [209.132.176.167]) by demeter.kernel.org (8.14.2/8.14.2) with ESMTP id n8SGJ0Bb006800 for ; Mon, 28 Sep 2009 16:19:55 GMT Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752518AbZI1QTv (ORCPT ); Mon, 28 Sep 2009 12:19:51 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1752513AbZI1QTu (ORCPT ); Mon, 28 Sep 2009 12:19:50 -0400 Received: from qw-out-2122.google.com ([74.125.92.24]:55141 "EHLO qw-out-2122.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751904AbZI1QTu (ORCPT ); Mon, 28 Sep 2009 12:19:50 -0400 Received: by qw-out-2122.google.com with SMTP id 5so1599244qwd.37 for ; Mon, 28 Sep 2009 09:19:54 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:received:received:message-id :disposition-notification-to:date:from:reply-to:user-agent :mime-version:to:cc:subject:references:in-reply-to :x-enigmail-version:content-type; bh=8su9IEwXCx5qzcIkbWmEYHyUaJ9tbMQn0FTXUfGOxtE=; b=cYiD080U0cIq9pIJjOGImnBZC8WRwWMJSTzUolyBndxZJfiNc89sFZ5DL/2tJ1exTM hf85uV6InWJEOi1JaDZGnY7PVlMJvdNE8PQ6BNQo77k26VRDqaGxtj7vdutITh7/qLvF yN8NsoQS8kkYuHX6YDZMSQBNZKI/yRiiX36PQ= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=message-id:disposition-notification-to:date:from:reply-to :user-agent:mime-version:to:cc:subject:references:in-reply-to :x-enigmail-version:content-type; b=uECNteJXODxf0gujc0dNl06zX4gK/ZcxBkftXI1fLxIheppguNSfwc+wShMqoR+4r3 Ki83aQ8YJrIH9GI38Pr4ziq6OHJ6gnrmEu7jMbDrToI6B/8vCEqd0AC9OIlogS25nooF KzQhanC6iLncM7YEf+i95yoCQs3Zh/3Vlmhug= Received: by 10.224.74.146 with SMTP id u18mr2804036qaj.67.1254154793820; Mon, 28 Sep 2009 09:19:53 -0700 (PDT) Received: from ?192.168.1.26? (200-102-97-88.cslce701.dsl.brasiltelecom.net.br [200.102.97.88]) by mx.google.com with ESMTPS id 4sm19353qwe.6.2009.09.28.09.19.50 (version=SSLv3 cipher=RC4-MD5); Mon, 28 Sep 2009 09:19:52 -0700 (PDT) Message-ID: <4AC0E21F.1000301@gmail.com> Date: Mon, 28 Sep 2009 13:19:43 -0300 From: Filipe Rosset Reply-To: rosset.filipe@gmail.com User-Agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.4pre) Gecko/20090922 Fedora/3.0-2.7.b4.fc11 Lightning/1.0pre Thunderbird/3.0b4 MIME-Version: 1.0 To: Devin Heitmueller CC: linux-media@vger.kernel.org, Douglas Schilling Landgraf , Mauro Carvalho Chehab Subject: Re: [PATCH 2/2] em28xx: Convert printks to em28xx_err and em28xx_info References: <4AC0D05D.4060304@gmail.com> <829197380909280824q487c3effp64914d8430f16092@mail.gmail.com> In-Reply-To: <829197380909280824q487c3effp64914d8430f16092@mail.gmail.com> X-Enigmail-Version: 0.97a Sender: linux-media-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-media@vger.kernel.org Em 28-09-2009 12:24, Devin Heitmueller escreveu: > On Mon, Sep 28, 2009 at 11:03 AM, Filipe Rosset wrote: >> > > You should use the em28xx_errdev() instead of em28xx_err() if your > intent is to insert "dev->name" in front of the message. > > Devin > OK, modified patch. Filipe diff -r cbf8899e0fd4 linux/drivers/media/video/em28xx/em28xx-dvb.c --- a/linux/drivers/media/video/em28xx/em28xx-dvb.c Mon Sep 28 11:22:23 2009 -0300 +++ b/linux/drivers/media/video/em28xx/em28xx-dvb.c Mon Sep 28 12:56:48 2009 -0300 @@ -314,22 +314,22 @@ cfg.i2c_addr = addr; if (!dev->dvb->frontend) { - printk(KERN_ERR "%s/2: dvb frontend not attached. " + em28xx_errdev("%s/2: dvb frontend not attached. " "Can't attach xc3028\n", - dev->name); + dev->name); return -EINVAL; } fe = dvb_attach(xc2028_attach, dev->dvb->frontend, &cfg); if (!fe) { - printk(KERN_ERR "%s/2: xc3028 attach failed\n", - dev->name); + em28xx_errdev("%s/2: xc3028 attach failed\n", + dev->name); dvb_frontend_detach(dev->dvb->frontend); dev->dvb->frontend = NULL; return -EINVAL; } - printk(KERN_INFO "%s/2: xc3028 attached\n", dev->name); + em28xx_info("%s/2: xc3028 attached\n", dev->name); return 0; } @@ -464,7 +464,7 @@ dvb = kzalloc(sizeof(struct em28xx_dvb), GFP_KERNEL); if (dvb == NULL) { - printk(KERN_INFO "em28xx_dvb: memory allocation failed\n"); + em28xx_info("em28xx_dvb: memory allocation failed\n"); return -ENOMEM; } dev->dvb = dvb; @@ -570,15 +570,14 @@ } break; default: - printk(KERN_ERR "%s/2: The frontend of your DVB/ATSC card" + em28xx_errdev("%s/2: The frontend of your DVB/ATSC card" " isn't supported yet\n", - dev->name); + dev->name); break; } if (NULL == dvb->frontend) { - printk(KERN_ERR - "%s/2: frontend initialization failed\n", - dev->name); + em28xx_errdev("%s/2: frontend initialization failed\n", + dev->name); result = -EINVAL; goto out_free; } @@ -592,7 +591,7 @@ goto out_free; em28xx_set_mode(dev, EM28XX_SUSPEND); - printk(KERN_INFO "Successfully loaded em28xx-dvb\n"); + em28xx_info("Successfully loaded em28xx-dvb\n"); return 0; out_free: