Re: [PATCH V9 1/2] serial: exar: Revert "serial: exar: Add support for Sealevel 7xxxC serial cards"

["gregkh@linuxfoundation.org" <gregkh@linuxfoundation.org>]

On Fri, Sep 29, 2023 at 05:39:49PM +0000, Matthew Howell wrote:
> From: Matthew Howell <matthew.howell@sealevel.com>
> 
> Hardware ID of Sealevel 7xxxC cards changed prior to release.
> This has rendered 14ee78d5932a redundant.
> 
> This reverts commit 14ee78d5932afeb710c8305196a676a715bfdea8.
> 
> Fixes: 14ee78d5932a ("Add support for Sealevel 7xxxC serial cards")
> Signed-off-by: Matthew Howell <matthew.howell@sealevel.com>
> Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
> ---
> diff --git a/drivers/tty/serial/8250/8250_exar.c b/drivers/tty/serial/8250/8250_exar.c
> index 077c3ba3539e..3886f78ecbbf 100644
> --- a/drivers/tty/serial/8250/8250_exar.c
> +++ b/drivers/tty/serial/8250/8250_exar.c
> @@ -47,12 +47,6 @@
>  #define PCI_SUBDEVICE_ID_USR_2980		0x0128
>  #define PCI_SUBDEVICE_ID_USR_2981		0x0129
> 
> -#define PCI_DEVICE_ID_SEALEVEL_710xC		0x1001
> -#define PCI_DEVICE_ID_SEALEVEL_720xC		0x1002
> -#define PCI_DEVICE_ID_SEALEVEL_740xC		0x1004
> -#define PCI_DEVICE_ID_SEALEVEL_780xC		0x1008
> -#define PCI_DEVICE_ID_SEALEVEL_716xC		0x1010
> -
>  #define UART_EXAR_INT0		0x80
>  #define UART_EXAR_8XMODE	0x88	/* 8X sampling rate select */
>  #define UART_EXAR_SLEEP		0x8b	/* Sleep mode */
> @@ -652,8 +646,6 @@ exar_pci_probe(struct pci_dev *pcidev, const struct pci_device_id *ent)
>  		nr_ports = BIT(((pcidev->device & 0x38) >> 3) - 1);
>  	else if (board->num_ports)
>  		nr_ports = board->num_ports;
> -	else if (pcidev->vendor == PCI_VENDOR_ID_SEALEVEL)
> -		nr_ports = pcidev->device & 0xff;
>  	else
>  		nr_ports = pcidev->device & 0x0f;
> 
> @@ -893,12 +885,6 @@ static const struct pci_device_id exar_pci_tbl[] = {
>  	EXAR_DEVICE(COMMTECH, 4224PCI335, pbn_fastcom335_4),
>  	EXAR_DEVICE(COMMTECH, 2324PCI335, pbn_fastcom335_4),
>  	EXAR_DEVICE(COMMTECH, 2328PCI335, pbn_fastcom335_8),
> -
> -	EXAR_DEVICE(SEALEVEL, 710xC, pbn_exar_XR17V35x),
> -	EXAR_DEVICE(SEALEVEL, 720xC, pbn_exar_XR17V35x),
> -	EXAR_DEVICE(SEALEVEL, 740xC, pbn_exar_XR17V35x),
> -	EXAR_DEVICE(SEALEVEL, 780xC, pbn_exar_XR17V35x),
> -	EXAR_DEVICE(SEALEVEL, 716xC, pbn_exar_XR17V35x),
>  	{ 0, }
>  };
>  MODULE_DEVICE_TABLE(pci, exar_pci_tbl);
> 

Hi,

This is the friendly patch-bot of Greg Kroah-Hartman.  You have sent him
a patch that has triggered this response.  He used to manually respond
to these common problems, but in order to save his sanity (he kept
writing the same thing over and over, yet to different people), I was
created.  Hopefully you will not take offence and will fix the problem
in your patch and resubmit it so that it can be accepted into the Linux
kernel tree.

You are receiving this message because of the following common error(s)
as indicated below:

- This looks like a new version of a previously submitted patch, but you
  did not list below the --- line any changes from the previous version.
  Please read the section entitled "The canonical patch format" in the
  kernel file, Documentation/process/submitting-patches.rst for what
  needs to be done here to properly describe this.

If you wish to discuss this problem further, or you have questions about
how to resolve this issue, please feel free to respond to this email and
Greg will reply once he has dug out from the pending patches received
from other developers.

thanks,

greg k-h's patch email bot