[coreboot] add unwanted_vpci parsing to cs5536 southbridge

Jordan Crouse jordan.crouse at amd.com
Thu Apr 17 18:32:36 CEST 2008


On 17/04/08 12:19 -0400, Ward Vandewege wrote:
> Updated version of this patch, as per Carl-Daniel's comments and taking into
> account the DTC changes have been merged.
> 
> Thanks,
> Ward.
> 
> -- 
> Ward Vandewege <ward at fsf.org>
> Free Software Foundation - Senior System Administrator

> Add unwanted_vpci parsing to AMD's cs5536 southbridge.
> 
> Signed-off-by: Ward Vandewege <ward at gnu.org>
Acked-by: Jordan Crouse <jordan.crouse at amd.com>

With the fix mention on IRC to move the variable declaration to the 
top of the function.

> Index: southbridge/amd/cs5536/cs5536.c
> ===================================================================
> --- southbridge/amd/cs5536/cs5536.c	(revision 656)
> +++ southbridge/amd/cs5536/cs5536.c	(working copy)
> @@ -648,6 +648,12 @@
>  
>  	enable_USB_port4(sb);
>  
> +	/* disable unwanted virtual PCI devices */
> +	int i;
> +	for (i = 0; 0 != sb->unwanted_vpci[i]; i++) {
> +		hide_vpci(sb->unwanted_vpci[i]);
> +	}
> +
>  	if (sb->enable_ide)
>  		ide_init(dev);
>  
> Index: southbridge/amd/cs5536/dts
> ===================================================================
> --- southbridge/amd/cs5536/dts	(revision 656)
> +++ southbridge/amd/cs5536/dts	(working copy)
> @@ -64,4 +64,8 @@
>  	 * probably not what you want. 
>  	 */
>  	power_button = "0";
> +
> +	/* vpci devices to be disabled */
> +	unwanted_vpci = < 0 >;
> +
>  };

> -- 
> coreboot mailing list
> coreboot at coreboot.org
> http://www.coreboot.org/mailman/listinfo/coreboot

-- 
Jordan Crouse
Systems Software Development Engineer 
Advanced Micro Devices, Inc.





More information about the coreboot mailing list