Gitweb:     
http://git.kernel.org/git/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=05dfa35e84331c6921ab394463069e9376f0bd76
Commit:     05dfa35e84331c6921ab394463069e9376f0bd76
Parent:     f4df73c2914a49bf6a55896aaecb0563c955e167
Author:     Truxton Fulton <[EMAIL PROTECTED]>
AuthorDate: Sat Nov 17 16:27:01 2007 +0100
Committer:  Thomas Gleixner <[EMAIL PROTECTED](none)>
CommitDate: Sat Nov 17 16:27:01 2007 +0100

    x86: fix reboot with no keyboard attached
    
    Attempt to fix http://bugzilla.kernel.org/show_bug.cgi?id=8378
    
    Hiroto Shibuya wrote to tell me that he has a VIA EPIA-EK10000 which
    suffers from the reboot problem when no keyboard is attached.  My first
    patch works for him:
    
      
http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=59f4e7d572980a521b7bdba74ab71b21f5995538
    
    But the latest patch does not work for him :
    
      
http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=8b93789808756bcc1e5c90c99f1b1ef52f839a51
    
    We found that it was necessary to also set the "disable keyboard" flag in
    the command byte, as the first patch was doing.  The second patch tries to
    minimally modify the command byte, but it is not enough.
    
    Please consider this simple one-line patch to help people with low end VIA
    motherboards reboot when no keyboard is attached.  Hiroto Shibuya has
    verified that this works for him (as I no longer have an afflicted
    machine).
    
    
    Additional discussion:
    
    Note that original patch from Truxton DOES
    disable keyboard and this has been in main tree since 2.6.14, thus it must 
have
    quite a bit of air time already.
    
    
http://git.kernel.org/?p=linux/kernel/git/stable/linux-2.6.14.y.git;a=commit;h=59f4e7d572980a521b7bdba74ab71b21f5995538
    
    Note that he only mention "System flag" in the description and comment, but
    in the code, "disable keyboard" flag is set.
    
      outb(0x14, 0x60);       /* set "System flag" */
    
    In 2.6.23, he made a change to read the current byte and then mask the 
flags,
    but along this change,  he only set the "System flag" and dropped the 
setting
    of "disable keyboard" flag.
    
    
http://git.kernel.org/?p=linux/kernel/git/stable/linux-2.6.23.y.git;a=commit;h=8b93789808756bcc1e5c90c99f1b1ef52f839a51
    
       outb(cmd | 0x04, 0x60); /* set "System flag" */
    
    So my request is to restore the setting of disable keyboard flag which has 
been
    there since 2.6.14 but disappeared in 2.6.23.
    
    Cc: Lee Garrett <[EMAIL PROTECTED]>
    Cc: "Hiroto Shibuya" <[EMAIL PROTECTED]>
    Cc: Natalie Protasevich <[EMAIL PROTECTED]>
    Cc: Dmitry Torokhov <[EMAIL PROTECTED]>
    Cc: Ingo Molnar <[EMAIL PROTECTED]>
    Cc: "H. Peter Anvin" <[EMAIL PROTECTED]>
    Cc: Aristeu Rozanski <[EMAIL PROTECTED]>
    Cc: <[EMAIL PROTECTED]>
    Signed-off-by: Andrew Morton <[EMAIL PROTECTED]>
    Signed-off-by: Thomas Gleixner <[EMAIL PROTECTED]>
---
 include/asm-x86/mach-default/mach_reboot.h |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/include/asm-x86/mach-default/mach_reboot.h 
b/include/asm-x86/mach-default/mach_reboot.h
index e23fd9f..6adee6a 100644
--- a/include/asm-x86/mach-default/mach_reboot.h
+++ b/include/asm-x86/mach-default/mach_reboot.h
@@ -49,7 +49,7 @@ static inline void mach_reboot(void)
                udelay(50);
                kb_wait();
                udelay(50);
-               outb(cmd | 0x04, 0x60); /* set "System flag" */
+               outb(cmd | 0x14, 0x60); /* set "System flag" and "Keyboard 
Disabled" */
                udelay(50);
                kb_wait();
                udelay(50);
-
To unsubscribe from this list: send the line "unsubscribe git-commits-head" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to