| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <20060801093314.GB15168@pazke.donpac.ru>
Date: Tue, 1 Aug 2006 13:33:14 +0400
From: Andrey Panin <pazke@...pac.ru>
To: Edgar Hucek <hostmaster@...soft.at>
Cc: LKML <linux-kernel@...r.kernel.org>,
Linus Torvalds <torvalds@...l.org>, akpm@...l.org
Subject: Re: [PATCH 1/3] add-imacfb-docu-and-detection.patch
On 212, 07 31, 2006 at 10:25:01AM +0200, Edgar Hucek wrote:
> This Patch add basic Machine detection to imacfb and
> some Ducumentation bits for imacfb.
>
> Signed-off-by: Edgar Hucek <hostmaster@...soft.at>
>
>
> diff -uNr linux-2.6.18-rc2/Documentation/fb/imacfb.txt linux-2.6.18-rc2.mactel/Documentation/fb/imacfb.txt
> --- linux-2.6.18-rc2/Documentation/fb/imacfb.txt 1970-01-01 01:00:00.000000000 +0100
> +++ linux-2.6.18-rc2.mactel/Documentation/fb/imacfb.txt 2006-07-26 20:54:07.000000000 +0200
> @@ -0,0 +1,31 @@
> +
> +What is imacfb?
> +===============
> +
> +This is a generic EFI platform driver for Intel based Apple computers.
> +Imacfb is only for EFI booted Intel Macs.
> +
> +Supported Hardware
> +==================
> +
> +iMac 17"/20"
> +Macbook
> +Macbook Pro 15"/17"
> +MacMini
> +
> +How to use it?
> +==============
> +
> +Imacfb does not have any kind of autodetection of your machine.
> +You have to add the fillowing kernel parameters in your elilo.conf:
> + Macbook :
> + video=imacfb:macbook
> + MacMini :
> + video=imacfb:mini
> + Macbook Pro 15", iMac 17" :
> + video=imacfb:i17
> + Macbook Pro 17", iMac 20" :
> + video=imacfb:i20
> +
> +--
> +Edgar Hucek <gimli@...k-green.com>
> diff -uNr linux-2.6.18-rc2/drivers/video/imacfb.c linux-2.6.18-rc2.mactel/drivers/video/imacfb.c
> --- linux-2.6.18-rc2/drivers/video/imacfb.c 2006-07-16 10:38:27.000000000 +0200
> +++ linux-2.6.18-rc2.mactel/drivers/video/imacfb.c 2006-07-25 13:53:35.000000000 +0200
> @@ -18,6 +18,8 @@
> #include <linux/screen_info.h>
> #include <linux/slab.h>
> #include <linux/string.h>
> +#include <linux/dmi.h>
> +#include <linux/efi.h>
>
> #include <asm/io.h>
>
> @@ -28,7 +30,7 @@
> M_I20,
> M_MINI,
> M_MACBOOK,
> - M_NEW
> + M_UNKNOWN
> } MAC_TYPE;
>
> /* --------------------------------------------------------------------- */
> @@ -52,10 +54,36 @@
> };
>
> static int inverse;
> -static int model = M_NEW;
> +static int model = M_UNKNOWN;
> static int manual_height;
> static int manual_width;
>
> +static int set_system(struct dmi_system_id *id)
Missing __init ?
> +{
> + printk(KERN_INFO "imacfb: %s detected - set system to %ld\n",
> + id->ident, (long)id->driver_data);
> +
> + model = (long)id->driver_data;
> +
> + return 0;
> +}
> +
> +static struct dmi_system_id __initdata dmi_system_table[] = {
> + { set_system, "iMac4,1", {
> + DMI_MATCH(DMI_BIOS_VENDOR,"Apple Computer, Inc."),
> + DMI_MATCH(DMI_BIOS_VERSION,"iMac4,1") }, (void*)M_I17},
> + { set_system, "MacBookPro1,1", {
> + DMI_MATCH(DMI_BIOS_VENDOR,"Apple Computer, Inc."),
> + DMI_MATCH(DMI_BIOS_VERSION,"MacBookPro1,1") }, (void*)M_I17},
> + { set_system, "MacBook1,1", {
> + DMI_MATCH(DMI_BIOS_VENDOR,"Apple Computer, Inc."),
> + DMI_MATCH(DMI_PRODUCT_NAME,"MacBook1,1")}, (void *)M_MACBOOK},
> + { set_system, "Macmini1,1", {
> + DMI_MATCH(DMI_BIOS_VENDOR,"Apple Computer, Inc."),
> + DMI_MATCH(DMI_PRODUCT_NAME,"Macmini1,1")}, (void *)M_MINI},
> + {},
> +};
> +
> #define DEFAULT_FB_MEM 1024*1024*16
>
> /* --------------------------------------------------------------------- */
> @@ -149,7 +177,6 @@
> screen_info.lfb_linelength = 1472 * 4;
> screen_info.lfb_base = 0x80010000;
> break;
> - case M_NEW:
> case M_I20:
> screen_info.lfb_width = 1680;
> screen_info.lfb_height = 1050;
> @@ -207,6 +234,10 @@
> size_remap = size_total;
> imacfb_fix.smem_len = size_remap;
>
> +#ifndef __i386__
> + screen_info.imacpm_seg = 0;
> +#endif
> +
> if (!request_mem_region(imacfb_fix.smem_start, size_total, "imacfb")) {
> printk(KERN_WARNING
> "imacfb: cannot reserve video memory at 0x%lx\n",
> @@ -324,8 +355,16 @@
> int ret;
> char *option = NULL;
>
> - /* ignore error return of fb_get_options */
> - fb_get_options("imacfb", &option);
> + if (!efi_enabled)
> + return -ENODEV;
> + if (!dmi_check_system(dmi_system_table))
> + return -ENODEV;
> + if (model == M_UNKNOWN)
> + return -ENODEV;
> +
> + if (fb_get_options("imacfb", &option))
> + return -ENODEV;
> +
> imacfb_setup(option);
> ret = platform_driver_register(&imacfb_driver);
>
> diff -uNr linux-2.6.18-rc2/drivers/video/Kconfig linux-2.6.18-rc2.mactel/drivers/video/Kconfig
> --- linux-2.6.18-rc2/drivers/video/Kconfig 2006-07-16 10:38:27.000000000 +0200
> +++ linux-2.6.18-rc2.mactel/drivers/video/Kconfig 2006-07-25 13:30:55.000000000 +0200
> @@ -552,7 +552,7 @@
>
> config FB_IMAC
> bool "Intel-based Macintosh Framebuffer Support"
> - depends on (FB = y) && X86
> + depends on (FB = y) && X86 && EFI
> select FB_CFB_FILLRECT
> select FB_CFB_COPYAREA
> select FB_CFB_IMAGEBLIT
>
>
> -
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@...r.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
>
--
Andrey Panin | Linux and UNIX system administrator
pazke@...pac.ru | PGP key: wwwkeys.pgp.net
Download attachment "signature.asc" of type "application/pgp-signature" (190 bytes)
Powered by blists - more mailing lists