On 09/02/2021 01.46, Greg Kroah-Hartman wrote:
I HATE adding new PORT_ types, as I am almost positive no one uses them, but as they are in the uapi files, we can't delete them. So, just use an existing one, why do you want a new one? If you don't have a userspace tool that requires it, don't bother. Just use PORT_8250 and be done with it. I should force all new drivers to use that as well :)
Krzysztof: given this, I think it would be fair to add an enum to the driver to keep track of the actual hardware type (grouped into probably just 3, S3C24XX, S3C6400, APPLE), get rid of any code in there that cares about the uapi-visible port type (other than setting it correctly for those that do exist, to maintain current behavior), and just make everything else use PORT_8250 for that?
-- Hector Martin (marcan@xxxxxxxxx) Public Key: https://mrcn.st/pub
