[PATCH 0/2] drm: bridge: adv7511: CEC support for ADV7535

Alvin Šipraga posted 2 patches 4 years, 3 months ago
There is a newer version of this series
drivers/gpu/drm/bridge/adv7511/adv7511.h     |  27 ++++-
drivers/gpu/drm/bridge/adv7511/adv7511_cec.c | 116 +++++++++++++------
drivers/gpu/drm/bridge/adv7511/adv7511_drv.c |  22 +++-
3 files changed, 119 insertions(+), 46 deletions(-)
[PATCH 0/2] drm: bridge: adv7511: CEC support for ADV7535
Posted by Alvin Šipraga 4 years, 3 months ago
From: Alvin Šipraga <alsi@bang-olufsen.dk>

We have an ADV7535 which is nominally supported by this driver. These
two patches fix up the driver to get CEC working too.

The first adds the basic support by correcting some register offsets.

The second addresses an issue we saw with CEC RX on the ADV7535. It
hasn't been tested with the other chips (e.g. ADV7533), although it
should be compatible. I'm sending it against drm-misc-next because the
issue wasn't reported for other chips, and ADV7535 didn't have CEC
support before. But feel free to take it into -fixes instead.

Alvin Šipraga (2):
  drm: bridge: adv7511: enable CEC support for ADV7535
  drm: bridge: adv7511: use non-legacy mode for CEC RX

 drivers/gpu/drm/bridge/adv7511/adv7511.h     |  27 ++++-
 drivers/gpu/drm/bridge/adv7511/adv7511_cec.c | 116 +++++++++++++------
 drivers/gpu/drm/bridge/adv7511/adv7511_drv.c |  22 +++-
 3 files changed, 119 insertions(+), 46 deletions(-)

-- 
2.35.1

Re: [PATCH 0/2] drm: bridge: adv7511: CEC support for ADV7535
Posted by Robert Foss 3 years, 9 months ago
Hi Alvin,

Sorry about being slow to get to this series.

Can you rebase it on drm-misc-next and send out the next version?

On Sat, 19 Mar 2022 at 16:10, Alvin Šipraga <alvin@pqrs.dk> wrote:
>
> From: Alvin Šipraga <alsi@bang-olufsen.dk>
>
> We have an ADV7535 which is nominally supported by this driver. These
> two patches fix up the driver to get CEC working too.
>
> The first adds the basic support by correcting some register offsets.
>
> The second addresses an issue we saw with CEC RX on the ADV7535. It
> hasn't been tested with the other chips (e.g. ADV7533), although it
> should be compatible. I'm sending it against drm-misc-next because the
> issue wasn't reported for other chips, and ADV7535 didn't have CEC
> support before. But feel free to take it into -fixes instead.
>
> Alvin Šipraga (2):
>   drm: bridge: adv7511: enable CEC support for ADV7535
>   drm: bridge: adv7511: use non-legacy mode for CEC RX
>
>  drivers/gpu/drm/bridge/adv7511/adv7511.h     |  27 ++++-
>  drivers/gpu/drm/bridge/adv7511/adv7511_cec.c | 116 +++++++++++++------
>  drivers/gpu/drm/bridge/adv7511/adv7511_drv.c |  22 +++-
>  3 files changed, 119 insertions(+), 46 deletions(-)
>
> --
> 2.35.1
>
Re: [PATCH 0/2] drm: bridge: adv7511: CEC support for ADV7535
Posted by Alvin Šipraga 3 years, 9 months ago
Hi Robert,

On Mon, Aug 29, 2022 at 06:02:17PM +0200, Robert Foss wrote:
> Hi Alvin,
> 
> Sorry about being slow to get to this series.

Nothing to be sorry about - in fact it was already applied! ;-)

ab0af093bf90 drm: bridge: adv7511: use non-legacy mode for CEC RX
0aae7623b495 drm: bridge: adv7511: enable CEC support for ADV7535

The one patch I am still waiting for feedback on is here:

https://lore.kernel.org/all/20220319152932.995904-1-alvin@pqrs.dk/

But it is an RFC patch and not of an urgent nature.

Thanks!

Kind regards,
Alvin

> 
> Can you rebase it on drm-misc-next and send out the next version?
> 
> On Sat, 19 Mar 2022 at 16:10, Alvin Šipraga <alvin@pqrs.dk> wrote:
> >
> > From: Alvin Šipraga <alsi@bang-olufsen.dk>
> >
> > We have an ADV7535 which is nominally supported by this driver. These
> > two patches fix up the driver to get CEC working too.
> >
> > The first adds the basic support by correcting some register offsets.
> >
> > The second addresses an issue we saw with CEC RX on the ADV7535. It
> > hasn't been tested with the other chips (e.g. ADV7533), although it
> > should be compatible. I'm sending it against drm-misc-next because the
> > issue wasn't reported for other chips, and ADV7535 didn't have CEC
> > support before. But feel free to take it into -fixes instead.
> >
> > Alvin Šipraga (2):
> >   drm: bridge: adv7511: enable CEC support for ADV7535
> >   drm: bridge: adv7511: use non-legacy mode for CEC RX
> >
> >  drivers/gpu/drm/bridge/adv7511/adv7511.h     |  27 ++++-
> >  drivers/gpu/drm/bridge/adv7511/adv7511_cec.c | 116 +++++++++++++------
> >  drivers/gpu/drm/bridge/adv7511/adv7511_drv.c |  22 +++-
> >  3 files changed, 119 insertions(+), 46 deletions(-)
> >
> > --
> > 2.35.1
> >