Re: [PATCH] powerpc: Pegasos keyboard detection

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