2) The fix was in the wrong place anyway, if it was going to be done
anywhere
at all it needs to be in
arch/powerpc/kernel/prom_init.c:fixup_device_tree_chrp()
like the ISA ranges breakage (which is on Briq) and IDE IRQ
misnumbering fix.
Not the keyboard platform driver.
Yeah. In the bootwrapper.
3) In any case this should be something that is fixed in the firmware,
as any
stalwart, stubborn Linux developer will rant at you about.
Sure, if you *can* get a fix for the firmware. Until every
user has this update, fixing it in the kernel wrapper helps
users.
As for Segher, bootwrapper not such a good place as that's still
mussing up
the kernel with these fixes. Let the boot loader do it for the OS, and
don't
mess up the OS with device-tree fixups.
Sure, the bootloader can do it too, but then we need to fix
every bootloader that's used with Linux on this platform.
Maybe that's just one, that would make things simple :-)
After all it may not just be Linux
that stumbles on it. Why have the same patch in every OS?
This is just pragmatics: Linux needs the workaround -> Linux
implements the workaround. Sure it is not a _proper_ fix, but
a correctly implemented workaround "fixes" it for all users,
forever.
With nvramrc, the fix is done for EVERY operating system from firmware
upwards.
But it's something every user has to do separately.
The semi-official Genesi line of support and what I have been told by
the board
designer is if you need to fix something in the device tree, that is
what nvramrc
is for, and that is why Open Firmware runs Forth scripts.
That's hardly the only reason. But yeah, that's one way to
implement the workaround, but _we_ (the Linux community) cannot
do it like that (easily) for all users.
Segher
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
