linux-arch.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Arnd Bergmann <arnd@arndb.de>
To: Sam Ravnborg <sam@ravnborg.org>
Cc: Guan Xuetao <gxt@mprc.pku.edu.cn>, Michal Marek <mmarek@suse.cz>,
	'lkml' <linux-kernel@vger.kernel.org>,
	'linux-kbuild' <linux-kbuild@vger.kernel.org>,
	'linux arch' <linux-arch@vger.kernel.org>,
	'Guan Xuetao' <guanxuetao@mprc.pku.edu.cn>,
	Remis Lima Baima <remis.developer@googlemail.com>
Subject: Re: [PATCH v4] kbuild: asm-generic support
Date: Wed, 27 Apr 2011 21:52:37 +0200	[thread overview]
Message-ID: <201104272152.37889.arnd@arndb.de> (raw)
In-Reply-To: <20110427194647.GA6621@merkur.ravnborg.org>

On Wednesday 27 April 2011 21:46:47 Sam Ravnborg wrote:
> From 8546953f0c87d6b1c37c25df73511780c86ad89d Mon Sep 17 00:00:00 2001
> From: Sam Ravnborg <sam@ravnborg.org>
> Date: Fri, 22 Apr 2011 14:45:13 +0200
> Subject: [PATCH] kbuild: asm-generic support
> 
> There is an increasing amount of header files
> shared between individual architectures in asm-generic.
> To avoid a lot of dummy wrapper files that just
> include the corresponding file in asm-generic provide
> some basic support in kbuild for this.
> 
> With the following patch an architecture can maintain
> a list of files in the file arch/$(ARCH)/include/asm/Kbuild
> 
> To use a generic file just add:
> 
>         generic-y += <name-of-header-file.h>
> 
> For each file listed kbuild will generate the necessary
> wrapper in arch/$(ARCH)/include/generated/asm.
> 
> When installing userspace headers a wrapper is likewise created.
> 
> The original inspiration for this came from the unicore32
> patchset - although a different method is used.
> 
> The patch includes several improvements from Arnd Bergmann.
> Michael Marek contributed Makefile.asm-generic.
> 
> Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
> Acked-by: Guan Xuetao <guanxuetao@mprc.pku.edu.cn>
> Tested-by: Guan Xuetao <guanxuetao@mprc.pku.edu.cn>
> Cc: Arnd Bergmann <arnd@arndb.de>

Acked-by: Arnd Bergmann <arnd@arndb.de>

I'm glad to see this finally happen.

Maybe you can mention Remis Baima in the changelog as well,
he did an earlier implementation back in the days when some
architectures were still using include/asm-${ARCH}, which IIRC
did not get merged because of the complexity associated with
that.

	Arnd

  parent reply	other threads:[~2011-04-27 19:52 UTC|newest]

Thread overview: 58+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-01-08 13:03 [RFC] kbuild: generic support for asm-generic Sam Ravnborg
2011-01-08 13:03 ` Sam Ravnborg
2011-01-08 13:04 ` [PATCH 1/2] kbuild: asm-generic support Sam Ravnborg
2011-01-08 13:04   ` Sam Ravnborg
2011-01-08 13:45   ` Stephen Rothwell
2011-01-08 14:03     ` Sam Ravnborg
2011-01-08 14:03       ` Sam Ravnborg
2011-01-08 14:10   ` Guan Xuetao
2011-01-08 14:10     ` Guan Xuetao
2011-01-08 13:05 ` [EXAMPLE PATCH 2/2] x86: start to utilize kbuild " Sam Ravnborg
2011-01-08 13:05   ` Sam Ravnborg
2011-01-08 20:53 ` [RFC] kbuild: generic support for asm-generic Arnd Bergmann
2011-01-08 21:33   ` Sam Ravnborg
2011-01-08 21:33     ` Sam Ravnborg
2011-01-09  0:15     ` Arnd Bergmann
2011-01-09  8:28 ` [RFC v2] " Sam Ravnborg
2011-01-09  8:28   ` Sam Ravnborg
2011-01-09  8:31   ` [PATCH 1/2] kbuild: asm-generic support Sam Ravnborg
2011-01-09  8:31     ` Sam Ravnborg
2011-01-09 14:03     ` Arnd Bergmann
2011-01-09 15:10       ` Sam Ravnborg
2011-01-09 16:13         ` Arnd Bergmann
2011-01-09 16:13           ` Arnd Bergmann
2011-01-09  8:32   ` [EXAMPLE PATCH 2/2] x86: start to utilize kbuild " Sam Ravnborg
2011-01-09  8:32     ` Sam Ravnborg
2011-01-09 19:27 ` [RFC v3] kbuild: generic support for asm-generic Sam Ravnborg
2011-01-09 19:27   ` Sam Ravnborg
2011-01-09 19:29   ` [PATCH 1/2] kbuild: asm-generic support Sam Ravnborg
2011-01-09 19:29     ` Sam Ravnborg
2011-01-09 20:31     ` Arnd Bergmann
2011-01-09 20:31       ` Arnd Bergmann
2011-01-09 21:24       ` Sam Ravnborg
2011-01-09 21:24         ` Sam Ravnborg
2011-01-10 13:14     ` Guan Xuetao
2011-01-10 13:14       ` Guan Xuetao
2011-01-10 16:26       ` Sam Ravnborg
2011-01-10 16:26         ` Sam Ravnborg
2011-01-11  1:39         ` Guan Xuetao
2011-01-10 13:31     ` Guan Xuetao
2011-01-10 13:31       ` Guan Xuetao
2011-01-13 16:14     ` Michal Marek
2011-01-13 17:01       ` Sam Ravnborg
2011-01-13 17:01         ` Sam Ravnborg
2011-01-14 14:43         ` Michal Marek
2011-04-22 15:53           ` [PATCH v3] " Sam Ravnborg
2011-04-22 15:53             ` Sam Ravnborg
2011-04-25  1:29             ` Guan Xuetao
2011-04-27 19:42               ` Sam Ravnborg
2011-04-27 19:42                 ` Sam Ravnborg
2011-04-27 19:46                 ` [PATCH v4] " Sam Ravnborg
2011-04-27 19:46                   ` Sam Ravnborg
2011-04-27 19:52                   ` Arnd Bergmann [this message]
2011-04-27 19:52                     ` Arnd Bergmann
2011-04-27 20:29                     ` [PATCH v5] " Sam Ravnborg
2011-04-27 20:29                       ` Sam Ravnborg
2011-04-28 16:16                       ` Michal Marek
2011-01-09 19:29   ` [EXAMPLE PATCH 2/2] x86: start to utilize kbuild " Sam Ravnborg
2011-01-09 19:29     ` Sam Ravnborg

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=201104272152.37889.arnd@arndb.de \
    --to=arnd@arndb.de \
    --cc=guanxuetao@mprc.pku.edu.cn \
    --cc=gxt@mprc.pku.edu.cn \
    --cc=linux-arch@vger.kernel.org \
    --cc=linux-kbuild@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mmarek@suse.cz \
    --cc=remis.developer@googlemail.com \
    --cc=sam@ravnborg.org \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).