Re: [PATCH] drm: bridge: adv7511: get edid in hpd_work to update CEC phys address

2023-10-25 Thread Alvin Šipraga
Hi Jani, On Mon, Oct 23, 2023 at 02:42:56PM +0300, Jani Nikula wrote: > On Sat, 14 Oct 2023, Alvin Šipraga wrote: > > @@ -457,6 +460,9 @@ static void adv7511_hpd_work(struct work_struct *work) > > if (adv7511->connector.dev) { > > if (status == connector_status_dis

Re: [PATCH] drm: bridge: adv7511: get edid in hpd_work to update CEC phys address

2023-10-23 Thread Jani Nikula
On Sat, 14 Oct 2023, Alvin Šipraga wrote: > From: Alvin Šipraga > > The adv7511 driver is solely responsible for setting the physical > address of its CEC adapter. To do this, it must read the EDID. However, > EDID is only read when either the drm_bridge_funcs :: get_edid or > drm_connector_helpe

Re: [PATCH] drm: bridge: adv7511: get edid in hpd_work to update CEC phys address

2023-10-16 Thread Alvin Šipraga
Hi Laurent, On Mon, Oct 16, 2023 at 11:17:49AM +0300, Laurent Pinchart wrote: > Hi Alvin, > > Thank you for the patch. > > CC'ing Hans Verkuil, to review the CEC side. > > On Sat, Oct 14, 2023 at 09:43:01PM +0200, Alvin Šipraga wrote: > > From: Alvin Šipraga > > > > The adv7511 driver is sole

Re: [PATCH] drm: bridge: adv7511: get edid in hpd_work to update CEC phys address

2023-10-16 Thread Laurent Pinchart
Hi Alvin, Thank you for the patch. CC'ing Hans Verkuil, to review the CEC side. On Sat, Oct 14, 2023 at 09:43:01PM +0200, Alvin Šipraga wrote: > From: Alvin Šipraga > > The adv7511 driver is solely responsible for setting the physical > address of its CEC adapter. To do this, it must read the

[PATCH] drm: bridge: adv7511: get edid in hpd_work to update CEC phys address

2023-10-15 Thread Alvin Šipraga
From: Alvin Šipraga The adv7511 driver is solely responsible for setting the physical address of its CEC adapter. To do this, it must read the EDID. However, EDID is only read when either the drm_bridge_funcs :: get_edid or drm_connector_helper_funcs :: get_modes ops are called. Without loss of g

Re: [PATCH] drm: bridge: adv7511: get edid in hpd_work to update CEC phys address

2023-10-14 Thread Alvin Šipraga
On Sat, Oct 14, 2023 at 09:43:01PM +0200, Alvin Šipraga wrote: > From: Alvin Šipraga > > The adv7511 driver is solely responsible for setting the physical > address of its CEC adapter. To do this, it must read the EDID. However, > EDID is only read when either the drm_bridge_funcs :: get_edid or

[RFC PATCH] drm: bridge: adv7511: get edid in hpd_work to update CEC phys address

2022-03-19 Thread Alvin Šipraga
From: Alvin Šipraga The adv7511 driver is solely responsible for setting the physical address of its CEC adapter. To do this, it must read the EDID. However, EDID is only read when either the drm_bridge_funcs :: get_edid or drm_connector_helper_funcs :: get_modes ops are called. Without loss of g