Date: Mon, 14 May 2012 15:14:54 +0200
From: Wim Van Sebroeck <wim@iguana.be>
To: Randy Dunlap <rdunlap@xenotime.net>
Cc: Stephen Rothwell <sfr@canb.auug.org.au>, linux-next@vger.kernel.org,
        LKML <linux-kernel@vger.kernel.org>, linux-watchdog@vger.kernel.org
Subject: Re: [PATCH -next] via_wdt: depends on PCI
Message-ID: <20120514131454.GG3074@spo001.leaseweb.com>
References: <20120511162738.1d65b6efe099a54841be6313@canb.auug.org.au> <4FADA2B2.8070302@xenotime.net>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <4FADA2B2.8070302@xenotime.net>
User-Agent: Mutt/1.4.1i
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 784
Lines: 23

Hello Rancy,

> From: Randy Dunlap <rdunlap@xenotime.net>
> 
> via_wdt is a PCI driver so it should depend on PCI so that
> it will not cause build errors.
> 
> drivers/watchdog/via_wdt.c:256:1: warning: data definition has no type or storage class
> drivers/watchdog/via_wdt.c:256:1: warning: type defaults to 'int' in declaration of 'module_pci_driver'
> drivers/watchdog/via_wdt.c:256:1: warning: parameter names (without types) in function declaration
> 
> Signed-off-by: Randy Dunlap <rdunlap@xenotime.net>

Fix added.

Thanks,
Wim.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/