From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Arend van Spriel" Subject: Re: [PATCH] net: change return values in mac_pton() function Date: Wed, 13 Jul 2011 13:21:46 +0200 Message-ID: <4E1D7FCA.1050203@broadcom.com> References: <1310549424-5484-1-git-send-email-arend@broadcom.com> <20110713.041253.1879038886370324446.davem@davemloft.net> Mime-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: "adobriyan@gmail.com" , "netdev@vger.kernel.org" , "linux-kernel@vger.kernel.org" To: "David Miller" Return-path: Received: from mms1.broadcom.com ([216.31.210.17]:2703 "EHLO mms1.broadcom.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S964915Ab1GMLW1 (ORCPT ); Wed, 13 Jul 2011 07:22:27 -0400 In-Reply-To: <20110713.041253.1879038886370324446.davem@davemloft.net> Sender: netdev-owner@vger.kernel.org List-ID: On 07/13/2011 01:12 PM, David Miller wrote: > From: Alexey Dobriyan > Date: Wed, 13 Jul 2011 13:09:03 +0300 > >> On Wed, Jul 13, 2011 at 12:30 PM, Arend van Spriel wrote: >>> The original commit adding this function noted a diverge from usual >>> 0=success/-E=fail, but no motivation for it. >> I thought it was obvious, but indeed wasn't explicitely >> mentioned in changelog. But see inet_pton(3). > Agreed it's better for mac_pton() to be consistent with existing, > similar, interfaces like inet_pton. I just liked the general approach of zero indicating success. But even in the realm called Linux not everything can be black and white, I guess :-D Please drop the patch. Gr. AvS -- Almost nobody dances sober, unless they happen to be insane. -- H.P. Lovecraft --