From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mike Frysinger Date: Tue, 21 Jul 2009 02:28:09 -0400 Subject: [U-Boot] [PATCH 2/2] document network driver framework In-Reply-To: <4A654D77.4070901@gmail.com> References: <4A5F6524.7040909@gmail.com> <1247965449-23375-2-git-send-email-vapier@gentoo.org> <4A654D77.4070901@gmail.com> Message-ID: <200907210228.09882.vapier@gentoo.org> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Tuesday 21 July 2009 01:09:11 Ben Warren wrote: > Mike Frysinger wrote: > > Signed-off-by: Mike Frysinger > > --- > > Ben: some things to note: > > - i adopted Jean's proposed naming scheme in the CONFIG section > > Is this a generally-accepted naming convention? I personally think it's > crap, and since there isn't a single driver that uses it yet, you might > say this is a bit ahead of the curve. some style needed to be suggested, and what Jean proposed is better than what we have today (which is nothing) > > - i deprecated calling the driver-specific entry point > > "xxx_initialization()" in favor of "xxx_register()" because the > > former is way too confusing with everyone also having "xxx_init()" > > That may be so, but since there isn't a single driver that uses this > naming convention, you're wishing something that ain't so. that's why i said "should", deprecated current naming, and noted existing practice. if you agree with the proposal, it's easy enough to run sed on a few files to fix one function name. you agree with my comment that today's behavior is confusing even if you stare and bang on the code day in and day out ? it's even worse for the occasional observer ... -mike -------------- next part -------------- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 836 bytes Desc: This is a digitally signed message part. Url : http://lists.denx.de/pipermail/u-boot/attachments/20090721/ac05a5f9/attachment.pgp