From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
        id S1030527AbdDZVYs (ORCPT <rfc822;w@1wt.eu>);
        Wed, 26 Apr 2017 17:24:48 -0400
Received: from mail-pf0-f193.google.com ([209.85.192.193]:36065 "EHLO
        mail-pf0-f193.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
        with ESMTP id S1030467AbdDZVYi (ORCPT
        <rfc822;linux-kernel@vger.kernel.org>);
        Wed, 26 Apr 2017 17:24:38 -0400
Date: Wed, 26 Apr 2017 14:24:32 -0700 (PDT)
From: Chandra Annamaneni <chandra627@gmail.com>
X-X-Sender: chandra@panther
To: Dan Carpenter <dan.carpenter@oracle.com>
cc: Chandra Annamaneni <chandra627@gmail.com>,
        christian.gromm@microchip.com, andrey.shvetsov@k2l.de,
        devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v4] Staging: most: use __func__ instead of the function
 name
In-Reply-To: <20170426140856.6ncu47o7klwxllj5@mwanda>
Message-ID: <alpine.DEB.2.20.1704261424060.22604@panther>
References: <alpine.DEB.2.20.1704260559320.27185@turtle> <20170426140856.6ncu47o7klwxllj5@mwanda>
User-Agent: Alpine 2.20 (DEB 67 2015-01-07)
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Sorry!
And thanks for pointing it out.

regards
Chandra

On Wed, 26 Apr 2017, Dan Carpenter wrote:

> On Wed, Apr 26, 2017 at 06:02:29AM -0700, Chandra Annamaneni wrote:
> > 
> > Change video.c to use %s, __func__ instead of function names.
> > Warnings flagged by checkpatch.pl
> > 
> > Signed-off-by: Chandra Annamaneni <chandra627@gmail.com>
> > 
> > diff --git a/drivers/staging/most/aim-v4l2/video.c b/drivers/staging/most/aim-v4l2/video.c
> > index 59e861e..e074841 100644
> > --- a/drivers/staging/most/aim-v4l2/video.c
> > +++ b/drivers/staging/most/aim-v4l2/video.c
> > @@ -79,7 +79,7 @@ static int aim_vdev_open(struct file *filp)
> >  	struct most_video_dev *mdev = video_drvdata(filp);
> >  	struct aim_fh *fh;
> > 
> > -	v4l2_info(&mdev->v4l2_dev, "%s()\n", __func__);
> > +	v4l2_info(&mdev->v4l2_dev, "aim_vdev_open()\n");
> 
> You sent the reversed patch.
> 
> regards,
> dan carpenter
> 
>