Re: [PATCH 1/4] drm: Add I2C connector type
From: Javier Martinez Canillas
Date: Wed Feb 02 2022 - 03:46:14 EST
On 2/1/22 23:29, Simon Ser wrote:
> On Tuesday, February 1st, 2022 at 21:57, Sam Ravnborg <sam@xxxxxxxxxxxx> wrote:
>
>> As I wrote in another part of this thread(s) - typing the patch is easy.
>> But I do not understand the userspace implications so I need someone
>> else to say go.
>
> User-space shouldn't really use the connector for anything except making it
> easier for the user to understand where to plug the display cable. I think a
> generic "panel" connector type makes sense.
>
I'll drop this patch from the series. I didn't have all the context and thought
that was an opportunity to add an I2C type, since there was a SPI type already.
But seems to be more controversial than I expected and shouldn't really matter
for a tiny driver like this. I will just go ahead and use the Unknown type.
Best regards,
--
Javier Martinez Canillas
Linux Engineering
Red Hat