From mboxrd@z Thu Jan 1 00:00:00 1970 From: Greg KH Date: Thu, 22 Jul 2010 18:45:57 +0000 Subject: Re: [PATCH 09/11] staging: comedi: formatting of pointers in Message-Id: <20100722184557.GC32116@kroah.com> List-Id: References: <1279176172-5609-1-git-send-email-segooon@gmail.com> <4C3EAEC5.8080908@gmail.com> In-Reply-To: <4C3EAEC5.8080908@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Jiri Slaby Cc: Kulikov Vasiliy , kernel-janitors@vger.kernel.org, devel@driverdev.osuosl.org, Javier Martinez Canillas , Jiri Kosina , Greg Kroah-Hartman , linux-kernel@vger.kernel.org, Dominik Brodowski , Alexander Kurz On Thu, Jul 15, 2010 at 08:46:29AM +0200, Jiri Slaby wrote: > On 07/15/2010 08:42 AM, Kulikov Vasiliy wrote: > > Use %p instead of %08x in printk(). > > > > Signed-off-by: Kulikov Vasiliy > > --- > > drivers/staging/comedi/drivers/quatech_daqp_cs.c | 2 +- > > 1 files changed, 1 insertions(+), 1 deletions(-) > > > > diff --git a/drivers/staging/comedi/drivers/quatech_daqp_cs.c b/drivers/staging/comedi/drivers/quatech_daqp_cs.c > > index 8d16380..8b49cb1 100644 > > --- a/drivers/staging/comedi/drivers/quatech_daqp_cs.c > > +++ b/drivers/staging/comedi/drivers/quatech_daqp_cs.c > > @@ -208,7 +208,7 @@ static void hex_dump(char *str, void *ptr, int len) > > The name and parameters suggest to remove the function completely. (And > use print_hex_dump_bytes instead.) I agree, Kulikov, can you make that chagne instead? thanks, greg k-h