Re: [Letux-kernel] [PATCH v16 1/4] drm/bridge: dw-hdmi: introduce dw_hdmi_enable_poll()

2022-03-04 Thread H. Nikolaus Schaller
Hi Paul, > Am 04.03.2022 um 19:41 schrieb H. Nikolaus Schaller : > > > >> Am 04.03.2022 um 19:33 schrieb Paul Cercueil : >> >> >> >> Le ven., mars 4 2022 at 19:15:13 +0100, H. Nikolaus Schaller >> a écrit : >>> Hi Paul, Am 04.03.2022 um 19:04 schrieb Paul Cercueil : Le ven.,

Re: [Letux-kernel] [PATCH v16 1/4] drm/bridge: dw-hdmi: introduce dw_hdmi_enable_poll()

2022-03-04 Thread H. Nikolaus Schaller
> Am 04.03.2022 um 19:33 schrieb Paul Cercueil : > > > > Le ven., mars 4 2022 at 19:15:13 +0100, H. Nikolaus Schaller > a écrit : >> Hi Paul, >>> Am 04.03.2022 um 19:04 schrieb Paul Cercueil : >>> Le ven., mars 4 2022 at 18:51:14 +0100, H. Nikolaus Schaller >>> a écrit : Hi Paul,

Re: [Letux-kernel] [PATCH v16 1/4] drm/bridge: dw-hdmi: introduce dw_hdmi_enable_poll()

2022-03-04 Thread Paul Cercueil
Le ven., mars 4 2022 at 19:15:13 +0100, H. Nikolaus Schaller a écrit : Hi Paul, Am 04.03.2022 um 19:04 schrieb Paul Cercueil : Le ven., mars 4 2022 at 18:51:14 +0100, H. Nikolaus Schaller a écrit : Hi Paul, Neil, Am 04.03.2022 um 17:47 schrieb Paul Cercueil : From what I

Re: [Letux-kernel] [PATCH v16 1/4] drm/bridge: dw-hdmi: introduce dw_hdmi_enable_poll()

2022-03-04 Thread H. Nikolaus Schaller
Hi Paul, > Am 04.03.2022 um 19:04 schrieb Paul Cercueil : > > > > Le ven., mars 4 2022 at 18:51:14 +0100, H. Nikolaus Schaller > a écrit : >> Hi Paul, Neil, >>> Am 04.03.2022 um 17:47 schrieb Paul Cercueil : >>> From what I understood in Nikolaus' last message, HDMI hotplug is actually >>>

Re: [Letux-kernel] [PATCH v16 1/4] drm/bridge: dw-hdmi: introduce dw_hdmi_enable_poll()

2022-03-04 Thread Paul Cercueil
Le ven., mars 4 2022 at 18:51:14 +0100, H. Nikolaus Schaller a écrit : Hi Paul, Neil, Am 04.03.2022 um 17:47 schrieb Paul Cercueil : From what I understood in Nikolaus' last message, HDMI hotplug is actually correctly detected, so there's no need for polling. What is missing is the

Re: [Letux-kernel] [PATCH v16 1/4] drm/bridge: dw-hdmi: introduce dw_hdmi_enable_poll()

2022-03-04 Thread H. Nikolaus Schaller
Hi Paul, Neil, > Am 04.03.2022 um 17:47 schrieb Paul Cercueil : > > From what I understood in Nikolaus' last message, HDMI hotplug is actually > correctly detected, so there's no need for polling. What is missing is the > call to drm_kms_helper_hotplug_event *somewhere*, so that the

Re: [Letux-kernel] [PATCH v16 1/4] drm/bridge: dw-hdmi: introduce dw_hdmi_enable_poll()

2022-03-04 Thread Paul Cercueil
Hi Neil, Le ven., mars 4 2022 at 14:30:46 +0100, Neil Armstrong a écrit : Hi, On 03/03/2022 18:59, H. Nikolaus Schaller wrote: Hi Paul, Neil, Am 03.03.2022 um 18:20 schrieb Paul Cercueil : Hi Nikolaus, [snip] Well he said "the Ingenic DRM core" aka ingenic-drm-drv.c. You do have

Re: [Letux-kernel] [PATCH v16 1/4] drm/bridge: dw-hdmi: introduce dw_hdmi_enable_poll()

2022-03-04 Thread Neil Armstrong
Hi, On 03/03/2022 18:59, H. Nikolaus Schaller wrote: Hi Paul, Neil, Am 03.03.2022 um 18:20 schrieb Paul Cercueil : Hi Nikolaus, [snip] Well he said "the Ingenic DRM core" aka ingenic-drm-drv.c. You do have access to the main drm_device in the ingenic_drm_bind() function, so you can add

Re: [Letux-kernel] [PATCH v16 1/4] drm/bridge: dw-hdmi: introduce dw_hdmi_enable_poll()

2022-03-03 Thread H. Nikolaus Schaller
Hi Paul, Neil, > Am 03.03.2022 um 18:20 schrieb Paul Cercueil : > > Hi Nikolaus, > > [snip] > >>> Well he said "the Ingenic DRM core" aka ingenic-drm-drv.c. You do have >>> access to the main drm_device in the ingenic_drm_bind() function, so you >>> can add it there (with a cleanup function

Re: [Letux-kernel] [PATCH v16 1/4] drm/bridge: dw-hdmi: introduce dw_hdmi_enable_poll()

2022-03-03 Thread Paul Cercueil
Hi Nikolaus, [snip] Well he said "the Ingenic DRM core" aka ingenic-drm-drv.c. You do have access to the main drm_device in the ingenic_drm_bind() function, so you can add it there (with a cleanup function calling drm_kms_helper_poll_fini() registered with drmm_add_action_or_reset()).

Re: [Letux-kernel] [PATCH v16 1/4] drm/bridge: dw-hdmi: introduce dw_hdmi_enable_poll()

2022-03-03 Thread H. Nikolaus Schaller
Hi Paul, > Am 03.03.2022 um 17:51 schrieb Paul Cercueil : > > Hi Nikolaus, > > Le jeu., mars 3 2022 at 17:43:05 +0100, H. Nikolaus Schaller > a écrit : >> Hi Neil, >>> Am 03.03.2022 um 17:30 schrieb H. Nikolaus Schaller : >>> Hi Neil, Am 03.03.2022 um 17:23 schrieb Neil Armstrong :

Re: [Letux-kernel] [PATCH v16 1/4] drm/bridge: dw-hdmi: introduce dw_hdmi_enable_poll()

2022-03-03 Thread Paul Cercueil
Hi Nikolaus, Le jeu., mars 3 2022 at 17:43:05 +0100, H. Nikolaus Schaller a écrit : Hi Neil, Am 03.03.2022 um 17:30 schrieb H. Nikolaus Schaller : Hi Neil, Am 03.03.2022 um 17:23 schrieb Neil Armstrong : Hi, On 26/02/2022 18:12, H. Nikolaus Schaller wrote: so that

Re: [Letux-kernel] [PATCH v16 1/4] drm/bridge: dw-hdmi: introduce dw_hdmi_enable_poll()

2022-03-03 Thread H. Nikolaus Schaller
Hi Neil, > Am 03.03.2022 um 17:30 schrieb H. Nikolaus Schaller : > > Hi Neil, > >> Am 03.03.2022 um 17:23 schrieb Neil Armstrong : >> >> Hi, >> >> On 26/02/2022 18:12, H. Nikolaus Schaller wrote: >>> so that specialization drivers like ingenic-dw-hdmi can enable polling. >>> Signed-off-by: H.