From mboxrd@z Thu Jan 1 00:00:00 1970 From: Joe Perches Subject: Re: [PATCH 1/2] vsprintf: Add %pMbt, bluetooth mac address Date: Sat, 04 Dec 2010 09:48:51 -0800 Message-ID: <1291484931.31274.21.camel@Joe-Laptop> References: Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Cc: Marcel Holtmann , "Gustavo F. Padovan" , linux-kernel@vger.kernel.org, netdev@vger.kernel.org To: =?UTF-8?Q?Micha=C5=82_Miros=C5=82aw?= Return-path: In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org On Sat, 2010-12-04 at 12:03 +0100, Micha=C5=82 Miros=C5=82aw wrote: > 2010/12/4 Joe Perches : > > Bluetooth output the MAC address in reverse order. > > Adding %pMbt allows the batostr function to be removed. > Just a nitpick: > You could call it %pMR, as in 'Reverse', so it sounds better when/if > some other subsystem uses it. It would also be a hint of what is this > doing instead of where it came from. I considered that but believe %pMbt is clearer as most likely no other subsystem will be quite so far (out to lunch? in left field? north? :) enough to do that again. If any maintainer wants it changed, it's not any sort of problem to me, say so and I'll resubmit it.