Re: [sodaville] [PATCH v2 01/15] x86/e820: remove conditional earlymapping in parse_e820_ext

From: H. Peter Anvin
Date: Wed Feb 02 2011 - 13:59:58 EST


On 01/27/2011 11:54 AM, Daniel Drake wrote:
>
> This patch will indeed cause problems for OLPC. Thanks for bringing it
> to our attention.
>
> On OLPC, the device tree is not used as a source of devices like on
> other platforms, it is simply used to present information to the
> kernel and userspace (in read-only fashion).
>
> If I understand it correctly, the above patch is saying: if we have a
> device tree, don't add the standard x86 RTC device.
>
> However, what we need it to say is: if we have a device tree *and* the
> device tree is being used as a source of devices, don't add the
> standard x86 RTC device.
>
> Therefore in the OLPC case, this particular bail-out condition will
> never be met, because the device tree is not being used as a source of
> devices.
>
> Does that make sense?
>

It makes sense in the abstract, but it sounds like OLPC is a total WTF
in this sense...

-hpa
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/