linux-next.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Paul Gortmaker <paul.gortmaker@windriver.com>
To: Randy Dunlap <rdunlap@xenotime.net>
Cc: Heiko Carstens <heiko.carstens@de.ibm.com>,
	linux-next@vger.kernel.org,
	Stephen Rothwell <sfr@canb.auug.org.au>
Subject: Re: [PATCH -next] s390: add missing module.h/export.h includes
Date: Sat, 30 Jul 2011 14:50:32 -0400	[thread overview]
Message-ID: <4E345278.4060807@windriver.com> (raw)
In-Reply-To: <20110730113135.1f93e253.rdunlap@xenotime.net>

On 11-07-30 02:31 PM, Randy Dunlap wrote:
> On Sat, 30 Jul 2011 13:41:52 -0400 Paul Gortmaker wrote:
> 
>> On 11-07-30 03:25 AM, Heiko Carstens wrote:
>>> Subject: [PATCH] s390: add missing module.h/export.h includes
>>>
>>> From: Heiko Carstens <heiko.carstens@de.ibm.com>
>>>
>>> Fix several compile errors on s390 caused by splitting module.h.
>>
>> Thanks, I'll put this on the branch today.
> 
> Hi Paul,
> 
> Did you see the 10 (I think) patches that I posted in reply to
> Stephen's linux-next 20110729 announcement?
> [that I did not copy you on :( ]
> 
> They add linux/module.h or linux/export.h (and 1 for linux/sysfs.h)
> to multiple files to fix build errors.
> 
> You can see them as replies to https://lkml.org/lkml/2011/7/29/52
> or I can send them directly to you if you want me to do that...

I'll grab them from existing e-mails as appropriate.  For any ones
that I flag as new files only existing in next, we'll need the maintainer
to do the appropriate module.h addition.  Since their tree won't have
an export.h, they won't have any choice but to take the larger header
for now.  I plan on doing later sweeps to see if there are any lingering
optimizations via "downsizing" more module.h users, so we'll convert
any lingering few like this later.

Thanks for chipping in, it turned out to be quite the task in the end.
Paul.

> 
> ---
> ~Randy
> *** Remember to use Documentation/SubmitChecklist when testing your code ***

      reply	other threads:[~2011-07-30 18:51 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-07-30  7:25 [PATCH -next] s390: add missing module.h/export.h includes Heiko Carstens
2011-07-30 17:41 ` Paul Gortmaker
2011-07-30 18:31   ` Randy Dunlap
2011-07-30 18:50     ` Paul Gortmaker [this message]

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=4E345278.4060807@windriver.com \
    --to=paul.gortmaker@windriver.com \
    --cc=heiko.carstens@de.ibm.com \
    --cc=linux-next@vger.kernel.org \
    --cc=rdunlap@xenotime.net \
    --cc=sfr@canb.auug.org.au \
    /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).