[PATCH] vgaarb: use bridges to control VGA routing where possible.

Mark Marshall mark.marshall at csr.com
Thu Dec 16 04:03:07 PST 2010


On 16/12/2010 05:54, Dave Airlie wrote:
> From: Dave Airlie<airlied at redhat.com>
>   int pci_set_vga_state(struct pci_dev *dev, bool decode,
> -		      unsigned int command_bits, bool change_bridge)
> +		      unsigned int command_bits, u32 flags)
>   {
>   	struct pci_bus *bus;
>   	struct pci_dev *bridge;
>   	u16 cmd;
>   	int rc;
>
> -	WARN_ON(command_bits&  ~(PCI_COMMAND_IO|PCI_COMMAND_MEMORY));
> +	WARN_ON((flags&  PCI_VGA_STATE_CHANGE_DECODES)&  (command_bits&  ~(PCI_COMMAND_IO|PCI_COMMAND_MEMORY)));

Minor thing, but I think the middle & should be a &&?

MM



More information about the dri-devel mailing list