Date: Wed, 14 Jul 2010 15:07:39 -0700 (PDT)
Message-Id: <20100714.150739.193711125.davem@davemloft.net>
To: segooon@gmail.com
Cc: kernel-janitors@vger.kernel.org, grundler@parisc-linux.org,
        kyle@mcmartin.ca, joe@perches.com, jpirko@redhat.com,
        ben@decadent.org.uk, grant.likely@secretlab.ca, netdev@vger.kernel.org,
        linux-kernel@vger.kernel.org, devicetree-discuss@lists.ozlabs.org
Subject: Re: [PATCH 05/11] tulip: formatting of pointers in printk()
From: David Miller <davem@davemloft.net>
In-Reply-To: <1279130568-10857-1-git-send-email-segooon@gmail.com>
References: <1279130568-10857-1-git-send-email-segooon@gmail.com>
Mime-Version: 1.0
Content-Type: Text/Plain; charset=us-ascii
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 702
Lines: 20

From: Kulikov Vasiliy <segooon@gmail.com>
Date: Wed, 14 Jul 2010 22:02:47 +0400

> Use %p instead of %08x in printk().
> 
> Signed-off-by: Kulikov Vasiliy <segooon@gmail.com>

Since patch #5 and #6 are doing the same change to different
files in the same driver, I combined them into one commit.

There is no need to split things up with so much granularity.
If it's all in the same driver, doing the same transformation,
keep it all in one patch.

Thanks.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/