On 10/26/12 19:35, Jordan Justen wrote:

> +/**
> +  Sets the kernel command line parameter within the setup image.
> +
> +  @param[in,out] KernelSetup - The kernel setup image
> +  @param[in]     CommandLine - The kernel command line
> +
> +  @retval    EFI_SUCCESS - The Linux kernel setup is valid and supported
> +  @retval    EFI_INVALID_PARAMETER - KernelSetup was NULL
> +  @retval    EFI_UNSUPPORTED - The Linux kernel is not supported
> +
> +**/
> +EFI_STATUS
> +EFIAPI
> +LoadLinuxSetCommandLine (
> +  IN OUT VOID    *KernelSetup,
> +  IN CHAR8       *CommandLine
> +  );
> +
> +
> +/**
> +  Sets the kernel command line parameter within the setup image.

This is a leftover from the previous function -- this one sets the
initrd. (But I think you can fix this when committing, if at all.)

> +
> +  @param[in,out] KernelSetup - The kernel setup image
> +  @param[in]     Initrd - The initial root disk image
> +  @param[in]     InitrdSize - The initial root disk image size

BTW doesn't "initrd" stand for "initial RAM disk"? (Makes sense either way.)

Reviewed-by: Laszlo Ersek <ler...@redhat.com>

------------------------------------------------------------------------------
The Windows 8 Center - In partnership with Sourceforge
Your idea - your app - 30 days.
Get started!
http://windows8center.sourceforge.net/
what-html-developers-need-to-know-about-coding-windows-8-metro-style-apps/
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/edk2-devel

Reply via email to