* jason schildt <[EMAIL PROTECTED]> [050903 00:03]:
> +/* We can reduce the size of code generated by romcc by
> + * changing all of the fixed size types that live in registers
> + * into simple unsigned variables. (ie s/uint8_t/unsigned/g)
> + */

Why is this? I would consider specifying an 8bit type to be more
space-safing than using some generic untyped integer value. If not this
should be fixed in romcc..

>       /* AMD K8 Unsupported 1Ghz? */
>       if (id == (PCI_VENDOR_ID_AMD | (0x1100 << 16))) {
> -             if (is_cpu_pre_e0())  // CK804 support 1G?

> +             device_t dev_2 = PCI_DEV(0,0x18,2);
> +                     if(pci_read_config32(dev_2,0x9c) < 0x20f00) {

The function call looks a lot more readable here. How much is the gain
of manual inlining here?

Stefan

-- 
LinuxBIOS mailing list
[email protected]
http://www.openbios.org/mailman/listinfo/linuxbios

Reply via email to