[PATCH 0/2] Add support for parallel port devices Netmos 9900 and Asix 99100

[PATCH 0/2] Add support for parallel port devices Netmos 9900 and Asix 99100

[Alexander Baehr]

Support for the established parallel port devices Netmos 9900 and Asix 99100 
was lacking. This patch queue adds support for them and was successfullly 
tested on the two devices.

Signed-off-by: Alexander Baehr <abaehr@osadl.org>  

[PATCH 1/2] Add parport support for Netmos device 9900

[Alexander Baehr]

The Netmos parallel port 9901 was already supported but the device 9900 was
not. This patch adds the required settings for it and was successfully 
tested with the Netmos device 9900.

Signed-off-by: Alexander Baehr <abaehr@osadl.org>
---
 drivers/parport/parport_pc.c |    4 ++++
 1 file changed, 4 insertions(+)

Index: linux-5.19.0-rt7/drivers/parport/parport_pc.c
===================================================================
--- linux-5.19.0-rt7.orig/drivers/parport/parport_pc.c
+++ linux-5.19.0-rt7/drivers/parport/parport_pc.c
@@ -2604,6 +2604,7 @@ enum parport_pc_pci_cards {
 	oxsemi_pcie_pport,
 	aks_0100,
 	mobility_pp,
+	netmos_9900,
 	netmos_9705,
 	netmos_9715,
 	netmos_9755,
@@ -2665,6 +2666,7 @@ static struct parport_pc_pci {
 	/* oxsemi_pcie_pport */		{ 1, { { 0, 1 }, } },
 	/* aks_0100 */                  { 1, { { 0, -1 }, } },
 	/* mobility_pp */		{ 1, { { 0, 1 }, } },
+	/* netmos_9900 */		{ 1, { { 0, -1 }, } },
 
 	/* The netmos entries below are untested */
 	/* netmos_9705 */               { 1, { { 0, -1 }, } },
@@ -2746,6 +2748,8 @@ static const struct pci_device_id parpor
 	  PCI_ANY_ID, PCI_ANY_ID, 0, 0, aks_0100 },
 	{ 0x14f2, 0x0121, PCI_ANY_ID, PCI_ANY_ID, 0, 0, mobility_pp },
 	/* NetMos communication controllers */
+	{ PCI_VENDOR_ID_NETMOS, PCI_DEVICE_ID_NETMOS_9900,
+	  0xA000, 0x2000, 0, 0, netmos_9900 },
 	{ PCI_VENDOR_ID_NETMOS, PCI_DEVICE_ID_NETMOS_9705,
 	  PCI_ANY_ID, PCI_ANY_ID, 0, 0, netmos_9705 },
 	{ PCI_VENDOR_ID_NETMOS, PCI_DEVICE_ID_NETMOS_9715,

[PATCH 2/2] Add parport support for Asix device 99100

[Alexander Baehr]

For some reason, Asix decided to produce a parallel port that is fully
compatible with Netmos device 9900, but named it 99100. It obviously has
another vendor ID, but the device ID is the same. This patch adds the 
required settings and was successfully tested with the Asix device. 

Signed-off-by: Alexander Baehr <abaehr@osadl.org>
---
 drivers/parport/parport_pc.c |    4 ++++
 include/linux/pci_ids.h      |    3 +++
 2 files changed, 7 insertions(+)

Index: linux/drivers/parport/parport_pc.c
===================================================================
--- linux.orig/drivers/parport/parport_pc.c
+++ linux/drivers/parport/parport_pc.c
@@ -2612,6 +2612,7 @@ enum parport_pc_pci_cards {
 	netmos_9815,
 	netmos_9901,
 	netmos_9865,
+	asix_ax99100,
 	quatech_sppxp100,
 	wch_ch382l,
 };
@@ -2766,6 +2767,9 @@ static const struct pci_device_id parpor
 	  0xA000, 0x1000, 0, 0, netmos_9865 },
 	{ PCI_VENDOR_ID_NETMOS, PCI_DEVICE_ID_NETMOS_9865,
 	  0xA000, 0x2000, 0, 0, netmos_9865 },
+	/* Asix AX99100 Parallel port PCIExpressCard */
+	{ PCI_VENDOR_ID_ASIX, PCI_DEVICE_ID_AX99100,
+	  0xA000, 0x2000, 0, 0, netmos_9900 },
 	/* Quatech SPPXP-100 Parallel port PCI ExpressCard */
 	{ PCI_VENDOR_ID_QUATECH, PCI_DEVICE_ID_QUATECH_SPPXP_100,
 	  PCI_ANY_ID, PCI_ANY_ID, 0, 0, quatech_sppxp100 },
Index: linux/include/linux/pci_ids.h
===================================================================
--- linux.orig/include/linux/pci_ids.h
+++ linux/include/linux/pci_ids.h
@@ -3108,4 +3108,7 @@
 
 #define PCI_VENDOR_ID_NCUBE		0x10ff
 
+#define PCI_VENDOR_ID_ASIX		0x125b
+#define PCI_DEVICE_ID_AX99100		0x9100
+
 #endif /* _LINUX_PCI_IDS_H */

Re: [PATCH 2/2] Add parport support for Asix device 99100

[Greg Kroah-Hartman]

On Sat, Aug 06, 2022 at 01:31:26PM +0200, Alexander Baehr wrote:
> For some reason, Asix decided to produce a parallel port that is fully
> compatible with Netmos device 9900, but named it 99100. It obviously has
> another vendor ID, but the device ID is the same. This patch adds the 
> required settings and was successfully tested with the Asix device. 
> 
> Signed-off-by: Alexander Baehr <abaehr@osadl.org>
> ---
>  drivers/parport/parport_pc.c |    4 ++++
>  include/linux/pci_ids.h      |    3 +++
>  2 files changed, 7 insertions(+)
> 
> Index: linux/drivers/parport/parport_pc.c
> ===================================================================
> --- linux.orig/drivers/parport/parport_pc.c
> +++ linux/drivers/parport/parport_pc.c
> @@ -2612,6 +2612,7 @@ enum parport_pc_pci_cards {
>  	netmos_9815,
>  	netmos_9901,
>  	netmos_9865,
> +	asix_ax99100,
>  	quatech_sppxp100,
>  	wch_ch382l,
>  };
> @@ -2766,6 +2767,9 @@ static const struct pci_device_id parpor
>  	  0xA000, 0x1000, 0, 0, netmos_9865 },
>  	{ PCI_VENDOR_ID_NETMOS, PCI_DEVICE_ID_NETMOS_9865,
>  	  0xA000, 0x2000, 0, 0, netmos_9865 },
> +	/* Asix AX99100 Parallel port PCIExpressCard */
> +	{ PCI_VENDOR_ID_ASIX, PCI_DEVICE_ID_AX99100,
> +	  0xA000, 0x2000, 0, 0, netmos_9900 },
>  	/* Quatech SPPXP-100 Parallel port PCI ExpressCard */
>  	{ PCI_VENDOR_ID_QUATECH, PCI_DEVICE_ID_QUATECH_SPPXP_100,
>  	  PCI_ANY_ID, PCI_ANY_ID, 0, 0, quatech_sppxp100 },
> Index: linux/include/linux/pci_ids.h
> ===================================================================
> --- linux.orig/include/linux/pci_ids.h
> +++ linux/include/linux/pci_ids.h
> @@ -3108,4 +3108,7 @@
>  
>  #define PCI_VENDOR_ID_NCUBE		0x10ff
>  
> +#define PCI_VENDOR_ID_ASIX		0x125b
> +#define PCI_DEVICE_ID_AX99100		0x9100

Please read the top of this file for why we shouldn't be added new
entries to it.

thanks,

greg k-h

Re: [PATCH 2/2] Add parport support for Asix device 99100

[Greg Kroah-Hartman]

On Sat, Aug 06, 2022 at 01:31:26PM +0200, Alexander Baehr wrote:
> For some reason, Asix decided to produce a parallel port that is fully
> compatible with Netmos device 9900, but named it 99100. It obviously has
> another vendor ID, but the device ID is the same. This patch adds the 
> required settings and was successfully tested with the Asix device. 
> 
> Signed-off-by: Alexander Baehr <abaehr@osadl.org>
> ---
>  drivers/parport/parport_pc.c |    4 ++++
>  include/linux/pci_ids.h      |    3 +++
>  2 files changed, 7 insertions(+)
> 
> Index: linux/drivers/parport/parport_pc.c
> ===================================================================
> --- linux.orig/drivers/parport/parport_pc.c
> +++ linux/drivers/parport/parport_pc.c
> @@ -2612,6 +2612,7 @@ enum parport_pc_pci_cards {
>  	netmos_9815,
>  	netmos_9901,
>  	netmos_9865,
> +	asix_ax99100,
>  	quatech_sppxp100,
>  	wch_ch382l,
>  };
> @@ -2766,6 +2767,9 @@ static const struct pci_device_id parpor
>  	  0xA000, 0x1000, 0, 0, netmos_9865 },
>  	{ PCI_VENDOR_ID_NETMOS, PCI_DEVICE_ID_NETMOS_9865,
>  	  0xA000, 0x2000, 0, 0, netmos_9865 },
> +	/* Asix AX99100 Parallel port PCIExpressCard */
> +	{ PCI_VENDOR_ID_ASIX, PCI_DEVICE_ID_AX99100,
> +	  0xA000, 0x2000, 0, 0, netmos_9900 },
>  	/* Quatech SPPXP-100 Parallel port PCI ExpressCard */
>  	{ PCI_VENDOR_ID_QUATECH, PCI_DEVICE_ID_QUATECH_SPPXP_100,
>  	  PCI_ANY_ID, PCI_ANY_ID, 0, 0, quatech_sppxp100 },
> Index: linux/include/linux/pci_ids.h
> ===================================================================
> --- linux.orig/include/linux/pci_ids.h
> +++ linux/include/linux/pci_ids.h
> @@ -3108,4 +3108,7 @@
>  
>  #define PCI_VENDOR_ID_NCUBE		0x10ff
>  
> +#define PCI_VENDOR_ID_ASIX		0x125b
> +#define PCI_DEVICE_ID_AX99100		0x9100

Please read the top of this file for why to not add new ids to it.

thanks,

greg k-h