Re: [PATCH] fix PCMCIA Iomega Clik!

Niklas Cassel
2025-11-18T14:56:11Z

Hello René,

On Mon, Nov 17, 2025 at 01:43:21PM +0100, René Rebe wrote:

Please change the subject to include: "ata: pata_pcmcia: " prefix.

I.e. something like:

ata: pata_pcmcia: Add Iomega Clik! PCMCIA ATA/ATAPI Adapter

Looking at:
https://en.wikipedia.org/wiki/PocketZip

It seems like this device was renamed to PocketZip.

So perhaps:
ata: pata_pcmcia: Add Iomega PocketZip Drive

> Add PCMCIA Iomega Clik! ID to pata_pcmcia.

Perhaps:
Add Iomega PocketZip Drive ID to pata_pcmcia.

>
> -- Signed-off-by: René Rebe <rene@exactco.de>

There should be no "-- " before the Signed-off-by, see:
https://docs.kernel.org/process/submitting-patches.html#developer-s-certificate-of-origin-1-1

>
> --- a/drivers/ata/pata_pcmcia.c 2025-02-27 18:51:20.641993816 +0100
> +++ b/drivers/ata/pata_pcmcia.c 2025-02-27 18:56:22.022992807 +0100
> @@ -362,6 +362,7 @@
> PCMCIA_DEVICE_PROD_ID12("STI", "Flash 5.0", 0xbf2df18d, 0x8cb57a0e),
> PCMCIA_MFC_DEVICE_PROD_ID12(1, "SanDisk", "ConnectPlus", 0x7a954bd9, 0x74be00c6),
> PCMCIA_DEVICE_PROD_ID2("Flash Card", 0x5a362506),
> + PCMCIA_DEVICE_PROD_ID2("PCMCIA ATA/ATAPI Adapter", 0x888D7B73),

I think this should be something like:
PCMCIA_DEVICE_PROD_ID2("Iomega PocketZip Drive", 0x888D7B73),

Also, make sure that you add this entry at the correct place, i.e. in
alphabetical order (Iomega, before IBM), please also make the hex ID
string in lower case letters to match the existing entries.

Kind regards,
Niklas