[PATCH 0/2] drm/bridge: it66121: Add it66122 support

Nishanth Menon posted 2 patches 1 month, 3 weeks ago
There is a newer version of this series
.../devicetree/bindings/display/bridge/ite,it66121.yaml   | 1 +
drivers/gpu/drm/bridge/ite-it66121.c                      | 8 ++++++++
2 files changed, 9 insertions(+)
[PATCH 0/2] drm/bridge: it66121: Add it66122 support
Posted by Nishanth Menon 1 month, 3 weeks ago
Hi,

Add support for IT66122, which for all practical purposes is
drop in replacement for IT66121 except for the ID register
definition.

BeagleY-AI uses this new part as the old part is no longer in production
as far as I understand.

Now, BeaglePlay uses it66121 at the moment, but at some point, it might
end up flipping over to the new part.

An alternate implementation could be to drop the revision check or make
it66121 check include alternate ID check.. but that seems a little
non-standard.. Anyways, I suppose mediatek platforms will face this
problem as well at some point.

Nishanth Menon (2):
  dt-bindings: display: bridge: it66121: Add compatible string for
    IT66122
  drm/bridge: it66121: Add it66122 support

 .../devicetree/bindings/display/bridge/ite,it66121.yaml   | 1 +
 drivers/gpu/drm/bridge/ite-it66121.c                      | 8 ++++++++
 2 files changed, 9 insertions(+)

-- 
2.47.0
Re: [PATCH 0/2] drm/bridge: it66121: Add it66122 support
Posted by Andrew Davis 1 month, 3 weeks ago
On 8/13/25 2:08 PM, Nishanth Menon wrote:
> Hi,
> 
> Add support for IT66122, which for all practical purposes is
> drop in replacement for IT66121 except for the ID register
> definition.
> 
> BeagleY-AI uses this new part as the old part is no longer in production
> as far as I understand.
> 
> Now, BeaglePlay uses it66121 at the moment, but at some point, it might
> end up flipping over to the new part.
> 
> An alternate implementation could be to drop the revision check or make
> it66121 check include alternate ID check.. but that seems a little
> non-standard.. Anyways, I suppose mediatek platforms will face this
> problem as well at some point.
> 

Hmmm, since these boards will probably have to switch parts mid-production
it would cause us to need to have a new DT with the updated compatible
just for a otherwise transparent revision. Might be better to just
loosen the PID check so the alternative part work just the same.

Andrew

> Nishanth Menon (2):
>    dt-bindings: display: bridge: it66121: Add compatible string for
>      IT66122
>    drm/bridge: it66121: Add it66122 support
> 
>   .../devicetree/bindings/display/bridge/ite,it66121.yaml   | 1 +
>   drivers/gpu/drm/bridge/ite-it66121.c                      | 8 ++++++++
>   2 files changed, 9 insertions(+)
>
Re: [PATCH 0/2] drm/bridge: it66121: Add it66122 support
Posted by Nishanth Menon 1 month, 3 weeks ago
On 14:32-20250813, Andrew Davis wrote:
> On 8/13/25 2:08 PM, Nishanth Menon wrote:
> > Hi,
> > 
> > Add support for IT66122, which for all practical purposes is
> > drop in replacement for IT66121 except for the ID register
> > definition.
> > 
> > BeagleY-AI uses this new part as the old part is no longer in production
> > as far as I understand.
> > 
> > Now, BeaglePlay uses it66121 at the moment, but at some point, it might
> > end up flipping over to the new part.
> > 
> > An alternate implementation could be to drop the revision check or make
> > it66121 check include alternate ID check.. but that seems a little
> > non-standard.. Anyways, I suppose mediatek platforms will face this
> > problem as well at some point.
> > 
> 
> Hmmm, since these boards will probably have to switch parts mid-production
> it would cause us to need to have a new DT with the updated compatible
> just for a otherwise transparent revision. Might be better to just
> loosen the PID check so the alternative part work just the same.

I think we can get both world.. respinning this up in v2

-- 
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D
https://ti.com/opensource