Date: Fri, 30 Jun 2017 11:39:27 -0700
From: Mark Rogers <mail@mark-rogers.org>
To: Frans Klaver <fransklaver@gmail.com>
Cc: gregkh@linuxfoundation.org, me@tobin.cc, karniksayli1995@gmail.com,
        devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] staging: ks7010: fix styling WARNINGs
Message-ID: <20170630183927.GA13036@ubuntulaptop>
References: <1498798361-9965-1-git-send-email-mail@mark-rogers.org>
 <CAH6sp9OD+TGLNYa3H5h8=rccCT841AaX4vXR8wHChbpyi2azWw@mail.gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <CAH6sp9OD+TGLNYa3H5h8=rccCT841AaX4vXR8wHChbpyi2azWw@mail.gmail.com>
User-Agent: Mutt/1.5.24 (2015-08-30)
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 577
Lines: 16

Thank you for your feedback. I guess when making this patch I had the
preferred coding style in mind, but didn't ask myself if making the code
conform to it would truly improve readability.

I agree with all of your comments. Do you think the best course of
action is to create a new patch with this change alone and forget the
rest?

-       DPRINTK(1, "ks7010_sdio_remove()\n");
+       DPRINTK(1, "%s()\n", __func__);

Sorry about the newbie questions and bad patch, I will do better with
the next one. Thanks again for your time and feedback, I really
appreciate it.

Mark