On Fri, 22 Feb 2008 10:22:52 -0600 Matt Domsch wrote:

>  Documentation/kernel-parameters.txt |    3 +--
>  arch/x86/boot/edd.c                 |   10 +++++++++-
>  drivers/firmware/Kconfig            |    9 +++++++++
>  3 files changed, 19 insertions(+), 3 deletions(-)
> 
> diff --git a/Documentation/kernel-parameters.txt 
> b/Documentation/kernel-parameters.txt
> index 9a5b665..3e6f0f5 100644
> --- a/Documentation/kernel-parameters.txt
> +++ b/Documentation/kernel-parameters.txt
> @@ -625,8 +625,7 @@ and is between 256 and 4096 characters. It is defined in 
> the file
>       eata=           [HW,SCSI]
>  
>       edd=            [EDD]
> -                     Format: {"of[f]" | "sk[ipmbr]"}
> -                     See comment in arch/i386/boot/edd.S
> +                     Format: {"off" | "on" | "sk[ipmbr]"}

Where is "sk" valid (parsed) as a value for "edd="?
I see "skip" or "skipmbr"....
or is that just a typo, should be "skip[mbr]"?


>       eisa_irq_edge=  [PARISC,HW]
>                       See header of drivers/parisc/eisa.c.
> diff --git a/arch/x86/boot/edd.c b/arch/x86/boot/edd.c
> index 8721dc4..4191805 100644
> --- a/arch/x86/boot/edd.c
> +++ b/arch/x86/boot/edd.c
> @@ -128,17 +128,25 @@ void query_edd(void)
>  {
>       char eddarg[8];
>       int do_mbr = 1;
> +#ifdef CONFIG_EDD_OFF
> +     int do_edd = 0;
> +#else
>       int do_edd = 1;
> +#endif
>       int be_quiet;
>       int devno;
>       struct edd_info ei, *edp;
>       u32 *mbrptr;
>  
>       if (cmdline_find_option("edd", eddarg, sizeof eddarg) > 0) {
> -             if (!strcmp(eddarg, "skipmbr") || !strcmp(eddarg, "skip"))
> +             if (!strcmp(eddarg, "skipmbr") || !strcmp(eddarg, "skip")) {
> +                     do_edd = 1;
>                       do_mbr = 0;
> +             }
>               else if (!strcmp(eddarg, "off"))
>                       do_edd = 0;
> +             else if (!strcmp(eddarg, "on"))
> +                     do_edd = 1;
>       }
>  
>       be_quiet = cmdline_find_option_bool("quiet");
> diff --git a/drivers/firmware/Kconfig b/drivers/firmware/Kconfig
> index 05f02a3..c28e33f 100644
> --- a/drivers/firmware/Kconfig
> +++ b/drivers/firmware/Kconfig
> @@ -17,6 +17,15 @@ config EDD
>            obscure configurations. Most disk controller BIOS vendors do
>            not yet implement this feature.
>  
> +config EDD_OFF
> +     bool "Sets default behavior for EDD detection to off"
> +     depends on EDD
> +     default n
> +     help
> +       Say Y if you want EDD disabled by default, even though it is compiled 
> into the
> +       kernel. Say N if you want EDD enabled by default. EDD can be 
> dynamically set
> +       using the kernel parameter 'edd={on|skipmbr|off}'.
> +
>  config EFI_VARS
>       tristate "EFI Variable Support via sysfs"
>       depends on EFI

---
~Randy
--
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/

Reply via email to