[PATCH 08/14] xfree86: delete useless "Primary device is not PCI" message
Tomas Carnecky
tom at dbservice.com
Tue Jun 1 09:29:28 PDT 2010
On 6/1/10 4:59 PM, Tiago Vignatti wrote:
> It's not PCI? So what?!
>
> Signed-off-by: Tiago Vignatti<tiago.vignatti at nokia.com>
> ---
> hw/xfree86/common/xf86pciBus.c | 6 ------
> 1 files changed, 0 insertions(+), 6 deletions(-)
>
> diff --git a/hw/xfree86/common/xf86pciBus.c b/hw/xfree86/common/xf86pciBus.c
> index a751427..b64bae3 100644
> --- a/hw/xfree86/common/xf86pciBus.c
> +++ b/hw/xfree86/common/xf86pciBus.c
> @@ -1318,14 +1318,8 @@ xf86PciMatchDriver(char* matches[], int nmatches) {
> }
>
> pci_iterator_destroy(iter);
> -
> - if (!info) {
> - ErrorF("Primary device is not PCI\n");
> - }
> #ifdef __linux__
> - else {
> matchDriverFromFiles(matches, info->vendor_id, info->device_id);
How much do Xorg developers care about indentation? This should be at
the same level as pci_iterator_destroy() a couple lines above.
> - }
> #endif /* __linux__ */
>
> for (i = 0; (i< nmatches)&& (matches[i]); i++) {
More information about the xorg-devel
mailing list