Re: [PATCH v3 1/1] drm/bridge: ps8640: Rework power state handling

2020-09-15 Thread Neil Armstrong
On 27/08/2020 10:59, Enric Balletbo i Serra wrote: > The get_edid() callback can be triggered anytime by an ioctl, i.e > > drm_mode_getconnector (ioctl) > -> drm_helper_probe_single_connector_modes >-> drm_bridge_connector_get_modes > -> ps8640_bridge_get_edid > > Actually

Re: [PATCH v3 1/1] drm/bridge: ps8640: Rework power state handling

2020-09-01 Thread Bilal Wasim
Hi Enric, On Thu, 27 Aug 2020 10:59:11 +0200 Enric Balletbo i Serra wrote: > The get_edid() callback can be triggered anytime by an ioctl, i.e > > drm_mode_getconnector (ioctl) > -> drm_helper_probe_single_connector_modes >-> drm_bridge_connector_get_modes > -> ps8640_

Re: [PATCH v3 1/1] drm/bridge: ps8640: Rework power state handling

2020-08-31 Thread Enric Balletbo i Serra
Hi, On 27/8/20 21:34, kernel test robot wrote: > Hi Enric, > > I love your patch! Perhaps something to improve: > > [auto build test WARNING on drm-tip/drm-tip] > [cannot apply to linux/master drm-intel/for-linux-next linus/master v5.9-rc2] > [If your patch is applied to the wrong git tree, kind

Re: [PATCH v3 1/1] drm/bridge: ps8640: Rework power state handling

2020-08-31 Thread Enric Balletbo i Serra
Hi Bilal, On 31/8/20 11:32, Bilal Wasim wrote: > > Hi Enric, > > On Thu, 27 Aug 2020 10:59:11 +0200 > Enric Balletbo i Serra wrote: > >> The get_edid() callback can be triggered anytime by an ioctl, i.e >> >> drm_mode_getconnector (ioctl) >> -> drm_helper_probe_single_connector_modes >>

Re: [PATCH v3 1/1] drm/bridge: ps8640: Rework power state handling

2020-08-31 Thread Bilal Wasim
Hi Enric, On Thu, 27 Aug 2020 10:59:11 +0200 Enric Balletbo i Serra wrote: > The get_edid() callback can be triggered anytime by an ioctl, i.e > > drm_mode_getconnector (ioctl) > -> drm_helper_probe_single_connector_modes >-> drm_bridge_connector_get_modes > -> ps8640_

Re: [PATCH v3 1/1] drm/bridge: ps8640: Rework power state handling

2020-08-27 Thread kernel test robot
Hi Enric, I love your patch! Perhaps something to improve: [auto build test WARNING on drm-tip/drm-tip] [cannot apply to linux/master drm-intel/for-linux-next linus/master v5.9-rc2] [If your patch is applied to the wrong git tree, kindly drop us a note. And when submitting patch, we suggest to us

[PATCH v3 1/1] drm/bridge: ps8640: Rework power state handling

2020-08-27 Thread Enric Balletbo i Serra
The get_edid() callback can be triggered anytime by an ioctl, i.e drm_mode_getconnector (ioctl) -> drm_helper_probe_single_connector_modes -> drm_bridge_connector_get_modes -> ps8640_bridge_get_edid Actually if the bridge pre_enable() function was not called before get_edid