All of lore.kernel.org
 help / color / mirror / Atom feed
From: robherring2@gmail.com (Rob Herring)
To: linux-arm-kernel@lists.infradead.org
Subject: [RFC PATCH 0/3] ARM: Add default system.h and uncompress.h
Date: Thu, 28 Jul 2011 08:53:12 -0500	[thread overview]
Message-ID: <4E3169C8.2060406@gmail.com> (raw)
In-Reply-To: <alpine.LFD.2.00.1107280023140.12766@xanadu.home>

Nicolas,

On 07/27/2011 11:40 PM, Nicolas Pitre wrote:
> On Wed, 27 Jul 2011, Rob Herring wrote:
> 
>> From: Rob Herring <rob.herring@calxeda.com>
>>
>> This is a relatively simple way to remove the platform requirement for
>> some header files without making changes in every existing platform.
>> It's perhaps not a solution for the difficult cases, but can get the
>> simple cases out of the way making it clear what remains to be fixed.
>> I've done this for system.h and uncompress.h so far. timex.h and io.h
>> may be additional candidates.
> 
> I'm of two minds about this.  It is true that this makes things easy and 
> would avoid some of the #ifdef'ery that is otherwise necessary.  On the 
> other hand this gives us more opportunity for procrastinating and not 
> really do a complete and thorough job.  It would also be easier for some 
> platform to sneak in some of those header files to work around some 
> shortcomings in the generic version which is not helping maintainers 
> keeping a good discipline.  AS we are working on this, I might prefer 
> that mistakes do break the build rather than silently causing a 
> fallback with a dummy generic version to be picked up which might only 
> cause problems at run time much later.
> 
I just threw this out as an option.

I certainly view this as somewhat temporary, but just longer lived than
a single patch series and simpler that a bunch of temporary
HAVE_FOO_HEADER defines. Ultimately, the headers could just get rolled
into their parent asm header once all the platforms are converted over.
I don't view it as procrastinating, but allows platforms to be cleaned
up independently and spreads the work to others. The decision to move
gpio code into drivers/gpio is a good example. There was clear path and
it all went very quickly. It also allows focusing on the harder problems
while people are F2F.

> A Linaro event is taking place next week in England where a focused 
> effort will be deployed to work on this.  Let's see how much we'll be 
> able to clean up.
> 
Yes, I'm well aware of that. I'll participate remotely as much as I can.
Let me know how I can help.

Rob

  reply	other threads:[~2011-07-28 13:53 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-07-28  3:32 [RFC PATCH 0/3] ARM: Add default system.h and uncompress.h Rob Herring
2011-07-28  3:32 ` [RFC PATCH 1/3] ARM: include mach headers before common arm headers Rob Herring
2011-07-28  3:32 ` [RFC PATCH 2/3] ARM: add a default mach/uncompress.h header Rob Herring
2011-07-28  3:32 ` [RFC PATCH 3/3] ARM: add a default mach/system.h header Rob Herring
2011-07-28  4:40 ` [RFC PATCH 0/3] ARM: Add default system.h and uncompress.h Nicolas Pitre
2011-07-28 13:53   ` Rob Herring [this message]
2011-07-28  9:52 ` Russell King - ARM Linux
2011-07-28 13:21   ` Rob Herring
2011-07-28 15:01     ` Russell King - ARM Linux

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=4E3169C8.2060406@gmail.com \
    --to=robherring2@gmail.com \
    --cc=linux-arm-kernel@lists.infradead.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 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.