[PATCH 1/9] mtd: amd76xrom: use DEFINE_PCI_DEVICE_TABLE macro

Jingoo Han jg1.han at samsung.com
Wed Nov 27 23:36:57 EST 2013



> -----Original Message-----
> From: Jingoo Han [mailto:jg1.han at samsung.com]
> Sent: Thursday, November 28, 2013 11:32 AM
> To: 'Brian Norris'
> Cc: 'Artem Bityutskiy'; 'David Woodhouse'; linux-mtd at lists.infradead.org; 'Jingoo Han'
> Subject: [PATCH 1/9] mtd: amd76xrom: use DEFINE_PCI_DEVICE_TABLE macro
> 
> This macro is used to create a struct pci_device_id array.
> 
> Signed-off-by: Jingoo Han <jg1.han at samsung.com>

Please, ignore these patches.
According to the Greg Kroah-Hartman, 

"Yeah, and it's a horrid macro that deserves to be removed, please don't
use it in more places.

Actually, if you could just remove it, that would be best, sorry, I'm
not going to take these patches."

So, I will send the patch to remove 'DEFINE_PCI_DEVICE_TABLE' instead.
Sorry for annoying. :-)

Best regards,
Jingoo Han

> ---
>  drivers/mtd/maps/amd76xrom.c |    4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/mtd/maps/amd76xrom.c b/drivers/mtd/maps/amd76xrom.c
> index f7207b0..6a0582e 100644
> --- a/drivers/mtd/maps/amd76xrom.c
> +++ b/drivers/mtd/maps/amd76xrom.c
> @@ -296,7 +296,7 @@ static void amd76xrom_remove_one(struct pci_dev *pdev)
>  	amd76xrom_cleanup(window);
>  }
> 
> -static struct pci_device_id amd76xrom_pci_tbl[] = {
> +static DEFINE_PCI_DEVICE_TABLE(amd76xrom_pci_tbl) = {
>  	{ PCI_VENDOR_ID_AMD, PCI_DEVICE_ID_AMD_VIPER_7410,
>  		PCI_ANY_ID, PCI_ANY_ID, },
>  	{ PCI_VENDOR_ID_AMD, PCI_DEVICE_ID_AMD_VIPER_7440,
> @@ -319,7 +319,7 @@ static struct pci_driver amd76xrom_driver = {
>  static int __init init_amd76xrom(void)
>  {
>  	struct pci_dev *pdev;
> -	struct pci_device_id *id;
> +	const struct pci_device_id *id;
>  	pdev = NULL;
>  	for(id = amd76xrom_pci_tbl; id->vendor; id++) {
>  		pdev = pci_get_device(id->vendor, id->device, NULL);
> --
> 1.7.10.4





More information about the linux-mtd mailing list