All of lore.kernel.org
 help / color / mirror / Atom feed
From: Sam Ravnborg <sam@ravnborg.org>
To: Oleg Verych <olecom@flower.upol.cz>
Cc: LKML <linux-kernel@vger.kernel.org>, kbuild-devel@lists.sourceforge.net
Subject: Re: Rename asm-offsets tool or not? (Re: [patch 01/03] kbuild, asm-values: infrastructure)
Date: Sun, 16 Sep 2007 22:12:45 +0200	[thread overview]
Message-ID: <20070916201245.GA10000@uranus.ravnborg.org> (raw)
In-Reply-To: <20070916193258.GD13543@flower.upol.cz>

On Sun, Sep 16, 2007 at 09:32:58PM +0200, Oleg Verych wrote:
> On Sun, Sep 16, 2007 at 08:29:12PM +0200, Sam Ravnborg wrote:
> > Hi Oleg.
> 
> Hallo. Nice, you are bringing it back. I'll try to have LKML-like
> output this time, not a makefile mess and stuff:
> 
> []
> > I see no value in renaming from asm_offset to asm_value - please drop it.
> > Introducing the generic asm-values.h should wait and when you do
> > you should be preapred to update all architectures (ecasue otherwise it
> > will not happen).
> 
> All archs, are using same syntax. But.
> 
> Interesting exception is MIPS, where tokens are organized and implemented
> more nicely, that is where *asm-values* name coming from, actually. One
> can see all that text, size, constant tokens in addition to just offsets.
> 
> So, if name change isn't worth, then OK; filename compatibility check, as
> one in `linux/Kbuild', is easy to hack as well to remove.
> 
> But nice feature/tool with meaningful name and functionality is just
> another thing.
> 
> Opinions?
Everyone and their cousin these days knows that asm-offset is constants
generated from C and used in assembler.
If we benefit from more values in the asm-offset file - let's do it.
But there is no reason to change a name that has been used for this purpose
for as long as I can remember.

	Sam

  reply	other threads:[~2007-09-16 20:11 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-06-12 23:36 [patch 00/03] kbuild, asm-values: not only offsets, not only for $ARCH Oleg Verych
2007-06-12 23:36 ` [patch 01/03] kbuild, asm-values: infrastructure Oleg Verych
2007-06-14  7:41   ` Oleg Verych
2007-06-14  9:12     ` Sam Ravnborg
2007-06-14 11:16       ` Need opinions on asm-offsets cleanup (Re: [patch 01/03] kbuild, asm-values: infrastructure) Oleg Verych
2007-06-14 11:23         ` Sam Ravnborg
2007-09-16 18:29   ` [patch 01/03] kbuild, asm-values: infrastructure Sam Ravnborg
2007-09-16 19:32     ` Rename asm-offsets tool or not? (Re: [patch 01/03] kbuild, asm-values: infrastructure) Oleg Verych
2007-09-16 20:12       ` Sam Ravnborg [this message]
2007-06-12 23:36 ` [patch 02/03] kbuild, asm-values: successor of asm-offsets Oleg Verych
2007-06-13  8:55   ` Removing of dummy asm-offset files (Re: [patch 02/03] kbuild, asm-values: successor of asm-offsets) Oleg Verych
2007-06-12 23:36 ` [patch 03/03] kbuild, asm-values: private for lguest Oleg Verych

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20070916201245.GA10000@uranus.ravnborg.org \
    --to=sam@ravnborg.org \
    --cc=kbuild-devel@lists.sourceforge.net \
    --cc=linux-kernel@vger.kernel.org \
    --cc=olecom@flower.upol.cz \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.