From mboxrd@z Thu Jan 1 00:00:00 1970 From: Joe Perches Subject: Re: [PATCH v2 2/2] adi: adapt to eth_hw_addr_random() and changes in arch/blackfin Date: Fri, 17 Feb 2012 15:27:43 -0800 Message-ID: <1329521263.584.50.camel@joe2Laptop> References: <1329412171-20704-1-git-send-email-danny.kukawka@bisect.de> <201202171157.55585.danny.kukawka@bisect.de> <201202171815.59846.vapier@gentoo.org> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Cc: Danny Kukawka , "David S. Miller" , Sonic Zhang , Jeff Kirsher , Bob Liu , uclinux-dist-devel@blackfin.uclinux.org, netdev@vger.kernel.org, linux-kernel@vger.kernel.org To: Mike Frysinger Return-path: In-Reply-To: <201202171815.59846.vapier@gentoo.org> Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org On Fri, 2012-02-17 at 18:15 -0500, Mike Frysinger wrote: > On Friday 17 February 2012 05:57:54 Danny Kukawka wrote: > > Maybe. Many drivers already print information about setting a random mac > > address to a netdevice. Some with printk, some via > > netdev_warn/netdev_info/netdev_dbg or netif_dbg and other with costum > > methods. I could unify this, in a next step with a netdev_info in > > eth_hw_addr_random(), if it's okay for David. > i think unification in the core makes sense. it enforces consistency and > removes duplication. It's a good idea.