On Wed, Oct 22, 2008 at 04:23:37PM +0530, nagaraj s k wrote:
> I tried making the changes as you rightly pointed out, please comment on
> this patch and let me know if this still needs some work.
I don't know what you're doing, but the original version of this file
has tabs. You seem to have converted this to 4-space indents somehow.
> @@ -29,14 +29,15 @@ static int via_fetch_size(void)
> values = A_SIZE_8(agp_bridge->driver->aperture_sizes);
> pci_read_config_byte(agp_bridge->dev, VIA_APSIZE, &temp);
> for (i = 0; i < agp_bridge->driver->num_aperture_sizes; i++) {
> - if (temp == values[i].size_value) {
> - agp_bridge->previous_size =
> - agp_bridge->current_size = (void *) (values + i);
> + if (temp == values[i].size_value)
> + continue;
> + agp_bridge->previous_size = agp_bridge->current_size =
> + values + i;
> agp_bridge->aperture_size_idx = i;
> return values[i].size;
> }
> - }
> - printk(KERN_ERR PFX "Unknown aperture size from AGP bridge (0x%x)\n",
> temp);
> + dev_err(&agp_bridge->dev->dev, "Unknown AGP aperture size 0x%x\n",
> + temp);
> return 0;
> }
>
> @@ -82,9 +83,9 @@ static void via_tlbflush(struct agp_memo
> u32 temp;
>
> pci_read_config_dword(agp_bridge->dev, VIA_GARTCTRL, &temp);
> - temp |= (1<<7);
> + temp |= (1 << 7);
> pci_write_config_dword(agp_bridge->dev, VIA_GARTCTRL, temp);
> - temp &= ~(1<<7);
> + temp &= ~(1 << 7);
> pci_write_config_dword(agp_bridge->dev, VIA_GARTCTRL, temp);
> }
>
> @@ -114,13 +115,15 @@ static int via_fetch_size_agp3(void)
> temp &= 0xfff;
>
> for (i = 0; i < agp_bridge->driver->num_aperture_sizes; i++) {
> - if (temp == values[i].size_value) {
> - agp_bridge->previous_size =
> - agp_bridge->current_size = (void *) (values + i);
> + if (temp == values[i].size_value)
> + continue;
> + agp_bridge->previous_size = agp_bridge->current_size =
> + values + i;
> agp_bridge->aperture_size_idx = i;
> return values[i].size;
> }
> - }
> + dev_err(&agp_bridge->dev->dev, "Unknown AGP aperture size 0x%x\n",
> + temp);
> return 0;
> }
>
> @@ -141,12 +144,13 @@ static int via_configure_agp3(void)
> agp_bridge->gatt_bus_addr & 0xfffff000);
>
> /* 1. Enable GTLB in RX90<7>, all AGP aperture access needs to fetch
> - * translation table first.
> - * 2. Enable AGP aperture in RX91<0>. This bit controls the enabling of
> the
> - * graphics AGP aperture for the AGP3.0 port.
> + * translation table first.
> + * 2. Enable AGP aperture in RX91<0>. This bit controls the
> + * enabling of the graphics AGP aperture for the AGP3.0 port.
You've deleted the spaces at the beginning of the line here. Don't do
that.
> */
> pci_read_config_dword(agp_bridge->dev, VIA_AGP3_GARTCTRL, &temp);
> - pci_write_config_dword(agp_bridge->dev, VIA_AGP3_GARTCTRL, temp |
> (3<<7));
> + pci_write_config_dword(agp_bridge->dev, VIA_AGP3_GARTCTRL,
> + temp | (3 << 7));
> return 0;
> }
>
> @@ -156,7 +160,8 @@ static void via_cleanup_agp3(void)
> struct aper_size_info_16 *previous_size;
>
> previous_size = A_SIZE_16(agp_bridge->previous_size);
> - pci_write_config_byte(agp_bridge->dev, VIA_APSIZE,
> previous_size->size_value);
> + pci_write_config_byte(agp_bridge->dev, VIA_APSIZE,
> + previous_size->size_value);
> }
>
>
> @@ -165,7 +170,8 @@ static void via_tlbflush_agp3(struct agp
> u32 temp;
>
> pci_read_config_dword(agp_bridge->dev, VIA_AGP3_GARTCTRL, &temp);
> - pci_write_config_dword(agp_bridge->dev, VIA_AGP3_GARTCTRL, temp &
> ~(1<<7));
> + pci_write_config_dword(agp_bridge->dev, VIA_AGP3_GARTCTRL,
> + temp & ~(1 << 7));
> pci_write_config_dword(agp_bridge->dev, VIA_AGP3_GARTCTRL, temp);
> }
>
> @@ -421,13 +427,13 @@ static struct agp_device_ids via_agp_dev
> * VIA's AGP3 chipsets do magick to put the AGP bridge compliant
> * with the same standards version as the graphics card.
> */
> -static void check_via_agp3 (struct agp_bridge_data *bridge)
> +static void check_via_agp3(struct agp_bridge_data *bridge)
> {
> u8 reg;
>
> pci_read_config_byte(bridge->dev, VIA_AGPSEL, ®);
> /* Check AGP 2.0 compatibility mode. */
> - if ((reg & (1<<1))==0)
> + if ((reg & (1 << 1)) == 0)
> bridge->driver = &via_agp3_driver;
> }
>
> @@ -445,7 +451,8 @@ static int __devinit agp_via_probe(struc
> return -ENODEV;
>
> j = ent - agp_via_pci_table;
> - printk (KERN_INFO PFX "Detected VIA %s chipset\n",
> devs[j].chipset_name);
> + printk(KERN_INFO PFX "Detected VIA %s chipset\n",
> + devs[j].chipset_name);
>
> bridge = agp_alloc_bridge();
> if (!bridge)
> @@ -461,7 +468,8 @@ static int __devinit agp_via_probe(struc
> if (pdev->device == PCI_DEVICE_ID_VIA_8367_0) {
> /* Is there a KT400 subsystem ? */
> if (pdev->subsystem_device == PCI_DEVICE_ID_VIA_8377_0) {
> - printk(KERN_INFO PFX "Found KT400 in disguise as a KT266.\n");
> + printk(KERN_INFO PFX "Found KT400 in disguise
> + as a KT266.\n");
No.
--
Matthew Wilcox Intel Open Source Technology Centre
"Bill, look, we understand that you're interested in selling us this
operating system, but compare it to ours. We can't possibly take such
a retrograde step."