[RFC 0/6] pcmcia: separate 16-bit support from cardbus

Oliver Hartkopp socketcan at hartkopp.net
Mon Feb 27 12:32:40 PST 2023


On 27.02.23 20:53, Arnd Bergmann wrote:
> On Mon, Feb 27, 2023, at 20:07, Oliver Hartkopp wrote:

>> I assume these CAN bus PCMCIA interfaces won't work after your patch
>> set, right?
> 
> Correct, the patch series in its current form breaks this since
> your laptop is cardbus compatible. The options I can see are:
> 
> - abandon my series and keep everything unchanged, possibly removing
>    some of the pcmcia drivers that Dominik identified as candidates
> 
> - decide on a future timeline for when you are comfortable with
>    discontinuing this setup and require any CAN users with cardbus
>    laptops to move to USB or cardbus CAN adapters, apply the series
>    then
> 
> - duplicate the yenta_socket driver to have two variants of that,
>    require the user to choose between the cardbus and the pcmcia
>    variant depending on what card is going to be used.
> 
> Can you give more background on who is using the EMS PCMCIA card?
> I.e. are there reasons to use this device on modern kernels with
> machines that could also support the USB, expresscard or cardbus
> variants, or are you likely the only one doing this for the
> purpose of maintaining the driver?

Haha, good point.

In fact the EMS PCMCIA, the PEAK PCMCIA (PCAN PC Card) and the supported 
Softing PCMCIA cards are nearly 20 year old designs and they are all 
discontinued for some time now. Today you can easily get a high 
performance Classical CAN USB adapter with an excellent OSS firmware for 
~13 EUR.

The only other laptop CAN "Cards" I'm aware of are PCIe "ExpressCard" 
34/54 from PEAK System which use the PCI subsystem.

Maybe you are right and we should simply drop the support for those old 
PCMCIA drivers which will still be supported for the LTS 6.1 lifetime then.

@Marc Kleine-Budde: What do you think about removing these three drivers?

Best regards,
Oliver



More information about the linux-arm-kernel mailing list