RE: [RFC PATCH 1/2] drm/msm/dp: enumerate edp panel during driver probe

2023-03-14 Thread Sankeerth Billakanti (QUIC)
t; development on top of the current msm-next. >>> >> >> Okay, I will move to the msm-next tip. >> >>>> >>>> So, we want to move the panel enumeration from bind to probe so that >>>> the probe defer is easier to handle and a

RE: [RFC PATCH 2/2] drm/msm/dp: enable pm_runtime support for dp driver

2023-03-01 Thread Sankeerth Billakanti (QUIC)
>> diff --git a/drivers/gpu/drm/msm/dp/dp_aux.c >> b/drivers/gpu/drm/msm/dp/dp_aux.c >[..] >> +static int dp_runtime_resume(struct device *dev) { >> +struct platform_device *pdev = to_platform_device(dev); >> +struct msm_dp *dp_display = platform_get_drvdata(pdev); >> +struct

RE: [RFC PATCH 0/2] drm/msm/dp: refactor the msm dp driver resources

2023-03-01 Thread Sankeerth Billakanti (QUIC)
a get/put sync functions. >> >> This is a RFC to verify with the community if the approach we are taking is >correct. >> >> https://partnerissuetracker.corp.google.com/issues/230631602 > >This link is useless, since its contents are not public. > Okay, I will remove it. >> &

RE: [RFC PATCH 1/2] drm/msm/dp: enumerate edp panel during driver probe

2023-03-01 Thread Sankeerth Billakanti (QUIC)
So, we want to move the panel enumeration from bind to probe so that >> the probe defer is easier to handle and also both the drivers appear >> consistent in panel enumeration. This change moves the DP driver >> panel-bridge enumeration to probe. >> >&

[RFC PATCH 2/2] drm/msm/dp: enable pm_runtime support for dp driver

2023-02-23 Thread Sankeerth Billakanti
solve this problem in DP driver by monitoring the resource enable disable calls. This change will enable support for the pm_runtime resume and suspend operations for DP driver. Signed-off-by: Sankeerth Billakanti --- drivers/gpu/drm/msm/dp/dp_aux.c | 6 ++ drivers/gpu/drm/msm/dp

[RFC PATCH 1/2] drm/msm/dp: enumerate edp panel during driver probe

2023-02-23 Thread Sankeerth Billakanti
-off-by: Sankeerth Billakanti --- drivers/gpu/drm/msm/dp/dp_aux.c | 149 ++-- drivers/gpu/drm/msm/dp/dp_catalog.c | 12 +++ drivers/gpu/drm/msm/dp/dp_catalog.h | 1 + drivers/gpu/drm/msm/dp/dp_display.c | 80 ++- 4 files changed, 182 insertions(+), 60

[RFC PATCH 0/2] drm/msm/dp: refactor the msm dp driver resources

2023-02-23 Thread Sankeerth Billakanti
. They are enabled or disabled via get/put sync functions. This is a RFC to verify with the community if the approach we are taking is correct. https://partnerissuetracker.corp.google.com/issues/230631602 Sankeerth Billakanti (2): drm/msm/dp: enumerate edp panel during driver probe drm/msm/dp: enable

RE: [PATCH v6 04/10] drm/msm/dp: Add basic PSR support for eDP

2022-10-12 Thread Sankeerth Billakanti
Hi Doug, I incorporated the comments in v7. >Hi, > >On Mon, Jul 11, 2022 at 5:57 AM Vinod Polimera > wrote: >> >> @@ -359,6 +367,24 @@ void dp_catalog_ctrl_lane_mapping(struct >dp_catalog *dp_catalog) >> ln_mapping); >> } >> >> +void

RE: [v3 3/5] drm/bridge: add psr support during panel bridge enable & disable sequence

2022-06-29 Thread Sankeerth Billakanti (QUIC)
Hi Dmitry, >On 21/06/2022 13:53, Vinod Polimera wrote: >> This change avoids panel prepare/unprepare based on self-refresh >> state. >> >> Signed-off-by: Sankeerth Billakanti >> Signed-off-by: Kalyan Thota >> Signed-off-by: Vinod Polimera >>

RE: [v3 1/5] drm/msm/dp: Add basic PSR support for eDP

2022-06-29 Thread Sankeerth Billakanti (QUIC)
Hi Dmitry, >On 21/06/2022 13:53, Vinod Polimera wrote: >> Add support for basic panel self refresh (PSR) feature for eDP. >> Add a new interface to set PSR state in the sink from DPU. >> Program the eDP controller to issue PSR enter and exit SDP to the >> sink. >

RE: [PATCH v2 1/4] drm/msm/dp: Add basic PSR support for eDP

2022-06-21 Thread Sankeerth Billakanti
Hi Bjorn, >> Add support for basic panel self refresh (PSR) feature for eDP. >> Add a new interface to set PSR state in the sink from DPU. >> Program the eDP controller to issue PSR enter and exit SDP to the >> sink. >> >> Signed-off-by: Sankeerth Billakanti

RE: [PATCH v2 1/4] drm/msm/dp: Add basic PSR support for eDP

2022-06-21 Thread Sankeerth Billakanti
the >> sink. >> >> Signed-off-by: Sankeerth Billakanti >> >> Changes in v2: >> - Use dp bridge to set psr entry/exit instead of dpu_enocder >> - Don't modify whitespaces >> - set self refresh aware from atomic_check >> - set self ref

RE: [PATCH 1/2] dt-bindings: msm/dp: List supplies in the bindings

2022-05-06 Thread Sankeerth Billakanti (QUIC)
>> >> Our internal power grid documents list the regulators as >> >> VDD_A_*_1P2 and VDD_A_*_0P9 for all the platforms. >> > >> >Do your internal power grid documents indicate what these supplies >> >are powering? The question is if these supplies power any of the >> >logic inside the eDP

RE: [PATCH 2/2] dt-bindings: phy: List supplies for qcom,edp-phy

2022-05-05 Thread Sankeerth Billakanti (QUIC)
> required: > - compatible > - reg >@@ -65,5 +68,8 @@ examples: > > #clock-cells = <1>; > #phy-cells = <0>; >+ >+ vdda-phy-supply = <_a_edp_0_1p2>; >+ vdda-pll-supply = <_a_edp_0_0p9>; > }; > ... >-- >2.36.0.rc2.479.g8af0fa9b8e-goog Reviewed-by: Sankeerth Billakanti

RE: [PATCH 1/2] dt-bindings: msm/dp: List supplies in the bindings

2022-05-05 Thread Sankeerth Billakanti (QUIC)
>Quoting Sankeerth Billakanti (2022-05-05 11:02:36) >> >> >> >> Quoting Douglas Anderson (2022-04-25 14:06:42) >> >> >> >> Having 'a' in 'vdda' typically means 'analog' for 'analog' >> >> circuits, so I'd expect this to only matter

RE: [PATCH 1/2] dt-bindings: msm/dp: List supplies in the bindings

2022-05-05 Thread Sankeerth Billakanti
;> clarify here. > >Interesting. Oddly enough, the sc7280 datasheet doesn't list the "_A". >It calls these "VDD_VREF_0P9" and "VDD_VREF_1P2". However, on the >schematic in front of me someone labeled these pins on the sc7280 with the >"A". ...and the driver looks for a supply with the "a". :-/ > >It would be good to get clarification from someone with better information. > >-Doug Our internal power grid documents list the regulators as VDD_A_*_1P2 and VDD_A_*_0P9 for all the platforms. So, as a practice, we put the same name in the DT files. Hence, Reviewed-by: Sankeerth Billakanti Thank you, Sankeerth

RE: [PATCH v9 2/4] drm/msm/dp: Support only IRQ_HPD and REPLUG interrupts for eDP

2022-04-25 Thread Sankeerth Billakanti (QUIC)
Hi Stephen, >> >> int dp_catalog_ctrl_get_interrupt(struct dp_catalog *dp_catalog) >> >> diff --git a/drivers/gpu/drm/msm/dp/dp_display.c >> >> b/drivers/gpu/drm/msm/dp/dp_display.c >> >> index 055681a..dea4de9 100644 >> >> --- a/drivers/gpu/drm/msm/dp/dp_display.c >> >> +++

RE: [PATCH v9 1/4] drm/msm/dp: Add eDP support via aux_bus

2022-04-25 Thread Sankeerth Billakanti (QUIC)
Hi Stephen, >>>> Quoting Sankeerth Billakanti (2022-04-22 02:11:03) >>>>> diff --git a/drivers/gpu/drm/msm/dp/dp_display.c >>>>> b/drivers/gpu/drm/msm/dp/dp_display.c >>>>> index d7a19d6..055681a 100644 >>>>> ---

[PATCH v10 4/4] drm/msm/dp: Support the eDP modes given by panel

2022-04-25 Thread Sankeerth Billakanti
The eDP controller does not have a reliable way keep panel powered on to read the sink capabilities. So, the controller driver cannot validate if a mode can be supported by the source. We will rely on the panel driver to populate only the supported modes for now. Signed-off-by: Sankeerth

[PATCH v10 3/4] drm/msm/dp: wait for hpd high before aux transaction

2022-04-25 Thread Sankeerth Billakanti
, the sink can be hotplugged and unplugged anytime. The hpd line gets asserted to indicate the sink is connected and ready. Hence there is no need to wait for the hpd line to be asserted for a DP sink. Signed-off-by: Sankeerth Billakanti Reviewed-by: Douglas Anderson Reviewed-by: Stephen Boyd

[PATCH v10 2/4] drm/msm/dp: Support only IRQ_HPD and REPLUG interrupts for eDP

2022-04-25 Thread Sankeerth Billakanti
and disconnect interrupts. Any disruption in the link status will be indicated via the IRQ_HPD interrupts. So, the eDP controller driver can just enable the IRQ_HPD and replug interrupts. The DP controller driver still needs to enable all the interrupts. Signed-off-by: Sankeerth Billakanti Reviewed

[PATCH v10 1/4] drm/msm/dp: Add eDP support via aux_bus

2022-04-25 Thread Sankeerth Billakanti
is initialized. The generic panel probe needs the controller resources to be enabled to support the aux transactions originating from the panel probe. Signed-off-by: Sankeerth Billakanti Reviewed-by: Douglas Anderson Reviewed-by: Stephen Boyd --- Changes in v10: - modify the error handling condition

[PATCH v10 0/4] Add support for the eDP panel over aux_bus

2022-04-25 Thread Sankeerth Billakanti
/20220211224006.1797846-6-dmitry.barysh...@linaro.org/ Sankeerth Billakanti (4): drm/msm/dp: Add eDP support via aux_bus drm/msm/dp: Support only IRQ_HPD and REPLUG interrupts for eDP drm/msm/dp: wait for hpd high before aux transaction drm/msm/dp: Support the eDP modes given by panel drivers/gpu/drm

RE: [PATCH v9 1/4] drm/msm/dp: Add eDP support via aux_bus

2022-04-25 Thread Sankeerth Billakanti (QUIC)
Hi Stephen, >Quoting Sankeerth Billakanti (2022-04-22 02:11:03) >> diff --git a/drivers/gpu/drm/msm/dp/dp_display.c >> b/drivers/gpu/drm/msm/dp/dp_display.c >> index d7a19d6..055681a 100644 >> --- a/drivers/gpu/drm/msm/dp/dp_display.c >> +++ b/drivers/gpu/drm/msm

RE: [PATCH v9 2/4] drm/msm/dp: Support only IRQ_HPD and REPLUG interrupts for eDP

2022-04-24 Thread Sankeerth Billakanti (QUIC)
Hi Stephen, >Quoting Sankeerth Billakanti (2022-04-22 02:11:04) >> The panel-edp enables the eDP panel power during probe, get_modes and >> pre-enable. The eDP connect and disconnect interrupts for the eDP/DP >> controller are directly dependent on panel power. As eDP disp

[PATCH v9 4/4] drm/msm/dp: Support the eDP modes given by panel

2022-04-22 Thread Sankeerth Billakanti
The eDP controller does not have a reliable way keep panel powered on to read the sink capabilities. So, the controller driver cannot validate if a mode can be supported by the source. We will rely on the panel driver to populate only the supported modes for now. Signed-off-by: Sankeerth

[PATCH v9 3/4] drm/msm/dp: wait for hpd high before aux transaction

2022-04-22 Thread Sankeerth Billakanti
, the sink can be hotplugged and unplugged anytime. The hpd line gets asserted to indicate the sink is connected and ready. Hence there is no need to wait for the hpd line to be asserted for a DP sink. Signed-off-by: Sankeerth Billakanti Reviewed-by: Douglas Anderson --- These changes may be handled

[PATCH v9 2/4] drm/msm/dp: Support only IRQ_HPD and REPLUG interrupts for eDP

2022-04-22 Thread Sankeerth Billakanti
and disconnect interrupts. Any disruption in the link status will be indicated via the IRQ_HPD interrupts. So, the eDP controller driver can just enable the IRQ_HPD and replug interrupts. The DP controller driver still needs to enable all the interrupts. Signed-off-by: Sankeerth Billakanti --- Changes in v9

[PATCH v9 1/4] drm/msm/dp: Add eDP support via aux_bus

2022-04-22 Thread Sankeerth Billakanti
is initialized. The generic panel probe needs the controller resources to be enabled to support the aux transactions originating from the panel probe. Signed-off-by: Sankeerth Billakanti --- Changes in v9: - add comments for panel probe - modify the error handling checks Changes in v8: - handle corner

[PATCH v9 0/4] Add support for the eDP panel over aux_bus

2022-04-22 Thread Sankeerth Billakanti
/20220211224006.1797846-6-dmitry.barysh...@linaro.org/ Sankeerth Billakanti (4): drm/msm/dp: Add eDP support via aux_bus drm/msm/dp: Support only IRQ_HPD and REPLUG interrupts for eDP drm/msm/dp: wait for hpd high before aux transaction drm/msm/dp: Support the eDP modes given by panel drivers/gpu/drm

RE: [PATCH v8 2/4] drm/msm/dp: Support only IRQ_HPD and REPLUG interrupts for eDP

2022-04-21 Thread Sankeerth Billakanti (QUIC)
Hi Doug, >On Thu, Apr 21, 2022 at 7:37 AM Sankeerth Billakanti > wrote: >> >> The panel-edp enables the eDP panel power during probe, get_modes and >> pre-enable. The eDP connect and disconnect interrupts for the eDP/DP >> controller are directly dependent on p

RE: [PATCH v8 1/4] drm/msm/dp: Add eDP support via aux_bus

2022-04-21 Thread Sankeerth Billakanti (QUIC)
Hi Doug, >On Thu, Apr 21, 2022 at 7:37 AM Sankeerth Billakanti > wrote: >> >> @@ -1530,6 +1532,61 @@ void msm_dp_debugfs_init(struct msm_dp >*dp_display, struct drm_minor *minor) >> } >> } >> >> +static int dp_display_get_next_

[PATCH v8 4/4] drm/msm/dp: Support the eDP modes given by panel

2022-04-21 Thread Sankeerth Billakanti
The eDP controller does not have a reliable way keep panel powered on to read the sink capabilities. So, the controller driver cannot validate if a mode can be supported by the source. We will rely on the panel driver to populate only the supported modes for now. Signed-off-by: Sankeerth

[PATCH v8 3/4] drm/msm/dp: wait for hpd high before aux transaction

2022-04-21 Thread Sankeerth Billakanti
, the sink can be hotplugged and unplugged anytime. The hpd line gets asserted to indicate the sink is connected and ready. Hence there is no need to wait for the hpd line to be asserted for a DP sink. Signed-off-by: Sankeerth Billakanti Reviewed-by: Douglas Anderson --- These changes may be handled

[PATCH v8 2/4] drm/msm/dp: Support only IRQ_HPD and REPLUG interrupts for eDP

2022-04-21 Thread Sankeerth Billakanti
and disconnect interrupts. Any disruption in the link status will be indicated via the IRQ_HPD interrupts. So, the eDP controller driver can just enable the IRQ_HPD and replug interrupts. The DP controller driver still needs to enable all the interrupts. Signed-off-by: Sankeerth Billakanti --- Changes in v8

[PATCH v8 1/4] drm/msm/dp: Add eDP support via aux_bus

2022-04-21 Thread Sankeerth Billakanti
is initialized. The generic panel probe needs the controller resources to be enabled to support the aux transactions originating from the panel probe. Signed-off-by: Sankeerth Billakanti --- Changes in v8: - handle corner cases - add comment for the bridge ops Changes in v7: - aux_bus is mandatory

[PATCH v8 0/4] Add support for the eDP panel over aux_bus

2022-04-21 Thread Sankeerth Billakanti
/20220211224006.1797846-6-dmitry.barysh...@linaro.org/ Sankeerth Billakanti (4): drm/msm/dp: Add eDP support via aux_bus drm/msm/dp: Support only IRQ_HPD and REPLUG interrupts for eDP drm/msm/dp: wait for hpd high before aux transaction drm/msm/dp: Support the eDP modes given by panel drivers/gpu/drm

[PATCH v7 4/4] Support the eDP modes given by panel

2022-04-14 Thread Sankeerth Billakanti
The eDP controller does not have a reliable way keep panel powered on to read the sink capabilities. So, the controller driver cannot validate if a mode can be supported by the source. We will rely on the panel driver to populate only the supported modes for now. Signed-off-by: Sankeerth

[PATCH v7 3/4] drm/msm/dp: wait for hpd high before aux transaction

2022-04-14 Thread Sankeerth Billakanti
, the sink can be hotplugged and unplugged anytime. The hpd line gets asserted to indicate the sink is connected and ready. Hence there is no need to wait for the hpd line to be asserted for a DP sink. Signed-off-by: Sankeerth Billakanti --- Changes in v7: - add a comment to say why the wait si done

[PATCH v7 2/4] drm/msm/dp: Support only IRQ_HPD and REPLUG interrupts for eDP

2022-04-14 Thread Sankeerth Billakanti
interrupts. Any disruption in the link status will be indicated via the IRQ_HPD interrupts. So, the eDP controller driver can just enable the IRQ_HPD and replug interrupts. The DP controller driver still needs to enable all the interrupts. Signed-off-by: Sankeerth Billakanti --- Changes in v7

[PATCH v7 1/4] drm/msm/dp: Add eDP support via aux_bus

2022-04-14 Thread Sankeerth Billakanti
is initialized. The generic panel probe needs the controller resources to be enabled to support the aux transactions originating from the panel probe. Signed-off-by: Sankeerth Billakanti --- Changes in v7: - aux_bus is mandatory for eDP - connector type check modified to just check for eDP Changes in v6

[PATCH v7 0/4] Add support for the eDP panel over aux_bus

2022-04-14 Thread Sankeerth Billakanti
This series adds support for generic eDP panel over aux_bus. These changes are dependent on the following series: https://patchwork.kernel.org/project/linux-arm-msm/list/?series=613654=* Sankeerth Billakanti (4): drm/msm/dp: Add eDP support via aux_bus drm/msm/dp: Support only IRQ_HPD

RE: [PATCH v6 8/8] drm/msm/dp: Handle eDP mode_valid differently from dp

2022-04-08 Thread Sankeerth Billakanti
> > > > > > > > > On Wed, 30 Mar 2022 at 19:04, Sankeerth Billakanti > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > The panel-edp driver modes needs to be val

RE: [PATCH v6 5/8] drm/msm/dp: prevent multiple votes for dp resources

2022-04-08 Thread Sankeerth Billakanti
> > On Wed, Mar 30, 2022 at 9:04 AM Sankeerth Billakanti > > wrote: > >> > >> The aux_bus support with the dp_display driver will enable the dp > >> resources during msm_dp_modeset_init. The host_init has to return > >> early if th

RE: [PATCH v6 8/8] drm/msm/dp: Handle eDP mode_valid differently from dp

2022-04-08 Thread Sankeerth Billakanti
Hi Dmitry, > > > > > > > On Wed, 30 Mar 2022 at 19:04, Sankeerth Billakanti > > > > > > > wrote: > > > > > > > > > > > > > > > > The panel-edp driver modes needs to be validated > > > &g

RE: [PATCH v6 1/8] drm/msm/dp: Add eDP support via aux_bus

2022-04-07 Thread Sankeerth Billakanti (QUIC)
Hi Dmitry and Doug, > Hi, > > On Tue, Apr 5, 2022 at 10:36 AM Dmitry Baryshkov > wrote: > > > > On 05/04/2022 20:02, Doug Anderson wrote: > > > Hi, > > > > > > On Tue, Apr 5, 2022 at 5:54 AM Dmitry Baryshkov > > > wrote: > > >>> 3. For DP and eDP HPD means something a little different. > > >>>

RE: [PATCH v6 8/8] drm/msm/dp: Handle eDP mode_valid differently from dp

2022-04-07 Thread Sankeerth Billakanti (QUIC)
Hi Dmitry, > > > > > On Wed, 30 Mar 2022 at 19:04, Sankeerth Billakanti > > > > > wrote: > > > > > > > > > > > > The panel-edp driver modes needs to be validated differently > > > > > > from DP because the link c

RE: [PATCH v6 7/8] drm/msm/dp: Support edp/dp without hpd

2022-04-07 Thread Sankeerth Billakanti (QUIC)
> > > On Wed, Mar 30, 2022 at 9:04 AM Sankeerth Billakanti > > > wrote: > > > > > > > > Some eDP sinks or platform boards will not support hpd. > > > > This patch adds support for those cases. > > > > > > You could say m

RE: [PATCH v6 6/8] drm/msm/dp: remove unnecessary delay during boot

2022-04-07 Thread Sankeerth Billakanti (QUIC)
Hi Dmitry, > > > On Wed, Mar 30, 2022 at 9:04 AM Sankeerth Billakanti > > > wrote: > > > > > > > > Remove the unnecessary delay in executing the EV_HPD_INIT_SETUP > > > event. > > > > > > Tell me more and put it in the commit m

RE: [PATCH v6 7/8] drm/msm/dp: Support edp/dp without hpd

2022-04-04 Thread Sankeerth Billakanti (QUIC)
Hi Doug, > On Wed, Mar 30, 2022 at 9:04 AM Sankeerth Billakanti > wrote: > > > > Some eDP sinks or platform boards will not support hpd. > > This patch adds support for those cases. > > You could say more, like: > > If we're not using HPD then _both_ t

RE: [PATCH v6 8/8] drm/msm/dp: Handle eDP mode_valid differently from dp

2022-04-04 Thread Sankeerth Billakanti (QUIC)
Hi Doug, > On Wed, Mar 30, 2022 at 11:02 PM Sankeerth Billakanti (QUIC) > wrote: > > > > Hi Dmitry, > > > > > On Wed, 30 Mar 2022 at 19:04, Sankeerth Billakanti > > > wrote: > > > > > > > > The panel-edp driver modes

RE: [PATCH v6 4/8] drm/msm/dp: avoid handling masked interrupts

2022-04-04 Thread Sankeerth Billakanti (QUIC)
> On Thu, 31 Mar 2022 at 14:05, Sankeerth Billakanti > wrote: > > > > Hi Dmitry, > > > > > On 31/03/2022 08:53, Sankeerth Billakanti (QUIC) wrote: > > > > Hi Dmitry, > > > > > > > >> On Wed, 30 Mar 2022 at 19:03, Sankeert

RE: [PATCH v6 6/8] drm/msm/dp: remove unnecessary delay during boot

2022-04-04 Thread Sankeerth Billakanti (QUIC)
Hi Doug, > On Wed, Mar 30, 2022 at 9:04 AM Sankeerth Billakanti > wrote: > > > > Remove the unnecessary delay in executing the EV_HPD_INIT_SETUP > event. > > Tell me more and put it in the commit message! Why did it used to be > necessary and why is it no longer n

RE: [PATCH v6 3/8] drm/msm/dp: Support only IRQ_HPD and REPLUG interrupts for eDP

2022-04-04 Thread Sankeerth Billakanti (QUIC)
Hi Doug, > On Wed, Mar 30, 2022 at 9:03 AM Sankeerth Billakanti > wrote: > > > > @@ -1374,6 +1382,12 @@ static int dp_pm_resume(struct device *dev) > > dp_catalog_ctrl_hpd_config(dp->catalog); > > > > > > + if (dp->dp_display.co

RE: [PATCH v6 2/8] drm/msm/dp: wait for hpd high before aux transaction

2022-04-04 Thread Sankeerth Billakanti (QUIC)
Hi Doug, > On Wed, Mar 30, 2022 at 9:03 AM Sankeerth Billakanti > wrote: > > > > The source device should ensure the sink is ready before proceeding to > > read the sink capability or performing any aux transactions. The sink > > s/performing/perform >

RE: [PATCH v6 4/8] drm/msm/dp: avoid handling masked interrupts

2022-03-31 Thread Sankeerth Billakanti
Hi Dmitry, > On 31/03/2022 08:53, Sankeerth Billakanti (QUIC) wrote: > > Hi Dmitry, > > > >> On Wed, 30 Mar 2022 at 19:03, Sankeerth Billakanti > >> wrote: > >>> > >>> The interrupt register will still reflect the connect and disconne

RE: [PATCH v6 8/8] drm/msm/dp: Handle eDP mode_valid differently from dp

2022-03-31 Thread Sankeerth Billakanti (QUIC)
Hi Dmitry, > On Wed, 30 Mar 2022 at 19:04, Sankeerth Billakanti > wrote: > > > > The panel-edp driver modes needs to be validated differently from DP > > because the link capabilities are not available for EDP by that time. > > > > Signed-off-by: Sa

RE: [PATCH v6 4/8] drm/msm/dp: avoid handling masked interrupts

2022-03-30 Thread Sankeerth Billakanti (QUIC)
Hi Dmitry, > On Wed, 30 Mar 2022 at 19:03, Sankeerth Billakanti > wrote: > > > > The interrupt register will still reflect the connect and disconnect > > interrupt status without generating an actual HW interrupt. > > The controller driver should not

[PATCH v6 8/8] drm/msm/dp: Handle eDP mode_valid differently from dp

2022-03-30 Thread Sankeerth Billakanti
The panel-edp driver modes needs to be validated differently from DP because the link capabilities are not available for EDP by that time. Signed-off-by: Sankeerth Billakanti --- drivers/gpu/drm/msm/dp/dp_display.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/drivers/gpu/drm/msm/dp

[PATCH v6 7/8] drm/msm/dp: Support edp/dp without hpd

2022-03-30 Thread Sankeerth Billakanti
Some eDP sinks or platform boards will not support hpd. This patch adds support for those cases. Signed-off-by: Sankeerth Billakanti --- drivers/gpu/drm/msm/dp/dp_catalog.c | 15 --- 1 file changed, 12 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/msm/dp/dp_catalog.c

[PATCH v6 6/8] drm/msm/dp: remove unnecessary delay during boot

2022-03-30 Thread Sankeerth Billakanti
Remove the unnecessary delay in executing the EV_HPD_INIT_SETUP event. Signed-off-by: Sankeerth Billakanti --- drivers/gpu/drm/msm/dp/dp_display.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/msm/dp/dp_display.c b/drivers/gpu/drm/msm/dp/dp_display.c index

[PATCH v6 5/8] drm/msm/dp: prevent multiple votes for dp resources

2022-03-30 Thread Sankeerth Billakanti
The aux_bus support with the dp_display driver will enable the dp resources during msm_dp_modeset_init. The host_init has to return early if the core is already initialized to prevent putting an additional vote for the dp controller resources. Signed-off-by: Sankeerth Billakanti --- drivers/gpu

[PATCH v6 4/8] drm/msm/dp: avoid handling masked interrupts

2022-03-30 Thread Sankeerth Billakanti
The interrupt register will still reflect the connect and disconnect interrupt status without generating an actual HW interrupt. The controller driver should not handle those masked interrupts. Signed-off-by: Sankeerth Billakanti --- drivers/gpu/drm/msm/dp/dp_catalog.c | 5 +++-- 1 file changed

[PATCH v6 3/8] drm/msm/dp: Support only IRQ_HPD and REPLUG interrupts for eDP

2022-03-30 Thread Sankeerth Billakanti
interrupts. Any disruption in the link status will be indicated via the IRQ_HPD interrupts. So, the eDP controller driver can just enable the IRQ_HPD and replug interrupts. The DP controller driver still needs to enable all the interrupts. Signed-off-by: Sankeerth Billakanti --- drivers/gpu/drm

[PATCH v6 2/8] drm/msm/dp: wait for hpd high before aux transaction

2022-03-30 Thread Sankeerth Billakanti
, the sink can be hotplugged and unplugged anytime. The hpd line gets asserted to indicate the sink is connected and ready. Hence there is no need to wait for the hpd line to be asserted for a DP sink. Signed-off-by: Sankeerth Billakanti --- Changes in v6: - Wait for hpd high only for eDP - Split

[PATCH v6 1/8] drm/msm/dp: Add eDP support via aux_bus

2022-03-30 Thread Sankeerth Billakanti
is initialized. The generic panel probe needs the controller resources to be enabled to support the aux transactions originating from the panel probe. Signed-off-by: Sankeerth Billakanti --- Changes in v6: - Remove initialization - Fix aux_bus node leak - Split the patches drivers/gpu/drm/msm/dp

[PATCH v6 0/8] Add support for the eDP panel over aux_bus

2022-03-30 Thread Sankeerth Billakanti
/project/linux-arm-msm/list/?series=613654=* Sankeerth Billakanti (8): drm/msm/dp: Add eDP support via aux_bus drm/msm/dp: wait for hpd high before aux transaction drm/msm/dp: Support only IRQ_HPD and REPLUG interrupts for eDP drm/msm/dp: avoid handling masked interrupts drm/msm/dp

RE: [PATCH v5 6/9] drm/msm/dp: wait for hpd high before any sink interaction

2022-03-25 Thread Sankeerth Billakanti
> -Original Message- > From: Doug Anderson > Sent: Friday, March 25, 2022 9:36 PM > To: Sankeerth Billakanti (QUIC) > Cc: Stephen Boyd ; David Airlie ; > dri-devel ; bjorn.anders...@linaro.org; > Thierry Reding ; Sam Ravnborg > ; Kuogee Hsieh (QUIC) ; >

RE: [PATCH v5 6/9] drm/msm/dp: wait for hpd high before any sink interaction

2022-03-25 Thread Sankeerth Billakanti (QUIC)
> -Original Message- > From: Doug Anderson > Sent: Saturday, March 19, 2022 5:26 AM > To: Stephen Boyd > Cc: Sankeerth Billakanti (QUIC) ; open list:OPEN > FIRMWARE AND FLATTENED DEVICE TREE BINDINGS > ; dri-devel ; > freedreno ; linux-arm-msm m...@vger.kerne

RE: [PATCH v5 5/9] drm/msm/dp: Add eDP support via aux_bus

2022-03-25 Thread Sankeerth Billakanti (QUIC)
> -Original Message- > From: Stephen Boyd > Sent: Friday, March 18, 2022 3:08 AM > To: Sankeerth Billakanti (QUIC) ; > devicet...@vger.kernel.org; dri-devel@lists.freedesktop.org; > freedr...@lists.freedesktop.org; linux-arm-...@vger.kernel.org; linux- > ker...

RE: [PATCH v5 2/9] arm64: dts: qcom: sc7280: Add support for eDP panel on CRD

2022-03-25 Thread Sankeerth Billakanti (QUIC)
> -Original Message- > From: Stephen Boyd > Sent: Friday, March 18, 2022 2:53 AM > To: Sankeerth Billakanti (QUIC) ; > devicet...@vger.kernel.org; dri-devel@lists.freedesktop.org; > freedr...@lists.freedesktop.org; linux-arm-...@vger.kernel.org; linux- > ker...

RE: [PATCH v5 2/9] arm64: dts: qcom: sc7280: Add support for eDP panel on CRD

2022-03-25 Thread Sankeerth Billakanti (QUIC)
> -Original Message- > From: Doug Anderson > Sent: Friday, March 18, 2022 10:51 PM > To: Sankeerth Billakanti (QUIC) > Cc: dri-devel ; linux-arm-msm m...@vger.kernel.org>; freedreno ; > LKML ; open list:OPEN FIRMWARE AND > FLATTENED DEVICE TREE BINDINGS ;

RE: [PATCH v5 3/9] arm64: dts: qcom: sc7280: Enable backlight for eDP panel

2022-03-25 Thread Sankeerth Billakanti (QUIC)
> -Original Message- > From: Stephen Boyd > Sent: Friday, March 18, 2022 2:58 AM > To: Sankeerth Billakanti (QUIC) ; > devicet...@vger.kernel.org; dri-devel@lists.freedesktop.org; > freedr...@lists.freedesktop.org; linux-arm-...@vger.kernel.org; linux- > ker...

[PATCH v5 9/9] drm/msm/dp: Support edp/dp without hpd

2022-03-16 Thread Sankeerth Billakanti
Some eDP sinks or platform boards will not support hpd. This patch adds support for those cases. Signed-off-by: Sankeerth Billakanti --- drivers/gpu/drm/msm/dp/dp_catalog.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/msm/dp/dp_catalog.c b/drivers

[PATCH v5 8/9] drm/msm/dp: Handle eDP mode_valid case

2022-03-16 Thread Sankeerth Billakanti
The panel-edp driver modes needs to be validated differently from DP because the link capabilities are not available for EDP by that time. Signed-off-by: Sankeerth Billakanti --- drivers/gpu/drm/msm/dp/dp_display.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/drivers/gpu/drm/msm/dp

[PATCH v5 7/9] drm/msm/dp: Support only IRQ_HPD and REPLUG interrupts for eDP

2022-03-16 Thread Sankeerth Billakanti
reflect the connect and disconnect interrupt status without generating an actual HW interrupt. The controller driver should not handle those masked interrupts. Signed-off-by: Sankeerth Billakanti --- drivers/gpu/drm/msm/dp/dp_catalog.c | 9 +++-- drivers/gpu/drm/msm/dp/dp_display.c | 24

[PATCH v5 6/9] drm/msm/dp: wait for hpd high before any sink interaction

2022-03-16 Thread Sankeerth Billakanti
after the panel is powered on. The panel power will be enabled from the panel-edp driver. The controller driver needs to wait for the hpd line to be asserted by the sink. Signed-off-by: Sankeerth Billakanti --- drivers/gpu/drm/msm/dp/dp_aux.c | 6 ++ drivers/gpu/drm/msm/dp

[PATCH v5 5/9] drm/msm/dp: Add eDP support via aux_bus

2022-03-16 Thread Sankeerth Billakanti
that the regulator and clock votes for the controller resources are balanced. EV_HPD_INIT_SETUP needs to execute immediately to enable the interrupts for the aux transactions from panel-edp to get the modes supported. Signed-off-by: Sankeerth Billakanti --- drivers/gpu/drm/msm/dp/dp_display.c

[PATCH v5 4/9] drm/panel-edp: add LQ140M1JW46 edp panel entry

2022-03-16 Thread Sankeerth Billakanti
Add panel identification entry for the sharp LQ140M1JW46 eDP panel with power sequencing delay information. Signed-off-by: Sankeerth Billakanti --- drivers/gpu/drm/panel/panel-edp.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/panel/panel-edp.c b/drivers/gpu/drm/panel

[PATCH v5 3/9] arm64: dts: qcom: sc7280: Enable backlight for eDP panel

2022-03-16 Thread Sankeerth Billakanti
Enable backlight support for eDP panel on CRD platform for sc7280. Signed-off-by: Sankeerth Billakanti --- Changes in v5: - Separate out backlight nodes arch/arm64/boot/dts/qcom/sc7280-crd.dts | 18 ++ 1 file changed, 18 insertions(+) diff --git a/arch/arm64/boot/dts/qcom

[PATCH v5 2/9] arm64: dts: qcom: sc7280: Add support for eDP panel on CRD

2022-03-16 Thread Sankeerth Billakanti
Enable support for eDP interface via aux_bus on CRD platform. Signed-off-by: Sankeerth Billakanti --- Changes in v5: - Change the order of patches - Remove the backlight nodes - Remove the bias setting - Fix compilation issue - Model VREG_EDP_BP for backlight power Changes in v4

[PATCH v5 1/9] arm64: dts: qcom: sc7280: rename edp_out label to mdss_edp_out

2022-03-16 Thread Sankeerth Billakanti
Rename the edp_out label in the sc7280 platform to mdss_edp_out so that the nodes related to mdss are all grouped together in the board specific files. Signed-off-by: Sankeerth Billakanti --- Changes in v5: - Change the order of patches - Modify commit text arch/arm64/boot/dts/qcom/sc7280

[PATCH v5 0/9] Add support for the eDP panel on sc7280 CRD

2022-03-16 Thread Sankeerth Billakanti
/project/linux-arm-msm/list/?series=613654=* Sankeerth Billakanti (9): arm64: dts: qcom: sc7280: rename edp_out label to mdss_edp_out arm64: dts: qcom: sc7280: Add support for eDP panel on CRD arm64: dts: qcom: sc7280: Enable backlight for eDP panel drm/panel-edp: add LQ140M1JW46 edp panel

RE: [RFC PATCH v2 4/5] drm/msm/dp: replace dp_connector with drm_bridge_connector

2022-03-16 Thread Sankeerth Billakanti (QUIC)
    enable-gpio = <_gpios 7 GPIO_ACTIVE_HIGH>; > >> > >>     pinctrl-names = "default"; > >>     pinctrl-0 = <_pwm_default>; > >>     }; > >> > >>         edp_panel: edp_panel { >

RE: [RFC PATCH v2 5/5] drm/msm/dp: remove extra wrappers and public functions

2022-03-16 Thread Sankeerth Billakanti (QUIC)
ing the latter functions from public space. > > Signed-off-by: Dmitry Baryshkov Tested-by: Sankeerth Billakanti > --- > drivers/gpu/drm/msm/dp/dp_display.c | 54 +++--- > drivers/gpu/drm/msm/dp/dp_display.h | 1 - > drivers/gpu/drm/msm/dp/dp_drm.c | 72 ++---

RE: [PATCH v3 3/4] drm/panel-edp: Add eDP sharp panel support

2022-02-10 Thread Sankeerth Billakanti
for the dp driver and hence we want to explore it as a separate task/feature after the basic PSR feature. Thank you, Sankeerth -Original Message- From: Dmitry Baryshkov Sent: Thursday, February 10, 2022 5:55 PM To: Sankeerth Billakanti (QUIC) ; dri-devel@lists.freedesktop.org; linux-arm

[PATCH v4 5/5] drm/msm/dp: Add driver support to utilize drm panel

2022-02-10 Thread Sankeerth Billakanti
is completed. Signed-off-by: Sankeerth Billakanti --- Changes in v4: - Remove obvious comments - Define separate connector_ops for eDP - Remove unnecessary checks Changes in v3: None drivers/gpu/drm/msm/dp/dp_display.c | 6 drivers/gpu/drm/msm/dp/dp_drm.c | 62

[PATCH v4 4/5] drm/panel-edp: Add eDP sharp panel support

2022-02-10 Thread Sankeerth Billakanti
Add support for the 14" sharp,lq140m1jw46 eDP panel. Signed-off-by: Sankeerth Billakanti --- 00 ff ff ff ff ff ff 00 4d 10 23 15 00 00 00 00 35 1e 01 04 a5 1f 11 78 07 de 50 a3 54 4c 99 26 0f 50 54 00 00 00 01 01 01 01 01 01 01 01 01 01 01 01 01 01 01 01 5a 87 80 a0 70 38 4d 40 30 20 35 00

[PATCH v4 3/5] arm64: dts: qcom: sc7280: rename edp_out label to mdss_edp_out

2022-02-10 Thread Sankeerth Billakanti
Rename the edp_out label in the sc7280 platform to mdss_edp_out. Signed-off-by: Sankeerth Billakanti --- arch/arm64/boot/dts/qcom/sc7280-crd.dts | 10 +- arch/arm64/boot/dts/qcom/sc7280.dtsi| 2 +- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/arch/arm64/boot/dts

[PATCH v4 2/5] arm64: dts: qcom: sc7280: Add support for eDP panel on CRD

2022-02-10 Thread Sankeerth Billakanti
Enable the eDP display panel support without HPD on sc7280 platform. Signed-off-by: Sankeerth Billakanti --- Changes in v4: - Create new patch for name changes - Remove output-low Changes in v3: - Sort the nodes alphabetically - Use - instead of _ as node names - Place the backlight

[PATCH v4 1/5] dt-bindings: display: simple: Add sharp LQ140M1JW46 panel

2022-02-10 Thread Sankeerth Billakanti
Add support for sharp LQ140M1JW46 display panel. It is a 14" eDP panel with 1920x1080 display resolution. Signed-off-by: Sankeerth Billakanti Acked-by: Rob Herring Reviewed-by: Stephen Boyd --- Changes in v4: None Changes in v3: None Documentation/devicetree/bindings/display/

[PATCH v4 0/5] Add support for the eDP panel on sc7280 CRD

2022-02-10 Thread Sankeerth Billakanti
on the following series: https://patchwork.kernel.org/project/linux-arm-msm/list/?series=586263=both=* https://patchwork.kernel.org/project/linux-arm-msm/list/?series=560587=%2A=both Sankeerth Billakanti (5): dt-bindings: display: simple: Add sharp LQ140M1JW46 panel arm64: dts: qcom: sc7280: Add support

RE: [PATCH v3 4/4] drm/msm/dp: Add driver support to utilize drm panel

2022-02-10 Thread Sankeerth Billakanti (QUIC)
Hi Stephen, Will make the changes. -Original Message- From: Stephen Boyd Sent: Thursday, February 10, 2022 6:52 AM To: Sankeerth Billakanti (QUIC) ; agr...@kernel.org; airl...@linux.ie; bjorn.anders...@linaro.org; dan...@ffwll.ch; devicet...@vger.kernel.org; diand...@chromium.org

RE: [PATCH v3 3/4] drm/panel-edp: Add eDP sharp panel support

2022-02-10 Thread Sankeerth Billakanti (QUIC)
Hi Stephen, Thank you for the review. I will share the new patch. -Original Message- From: Stephen Boyd Sent: Thursday, February 10, 2022 6:47 AM To: Sankeerth Billakanti (QUIC) ; agr...@kernel.org; airl...@linux.ie; bjorn.anders...@linaro.org; dan...@ffwll.ch; devicet

RE: [PATCH v3 2/4] arm64: dts: qcom: sc7280: Add support for eDP panel on CRD

2022-02-10 Thread Sankeerth Billakanti (QUIC)
Hi Stephen, Will implement all the suggested changes. Thank you, Sankeerth -Original Message- From: Stephen Boyd Sent: Thursday, February 10, 2022 6:45 AM To: Sankeerth Billakanti (QUIC) ; agr...@kernel.org; airl...@linux.ie; bjorn.anders...@linaro.org; dan...@ffwll.ch; devicet

[PATCH v3 1/4] dt-bindings: display: simple: Add sharp LQ140M1JW46 panel

2022-02-09 Thread Sankeerth Billakanti
Add support for sharp LQ140M1JW46 display panel. It is a 14" eDP panel with 1920x1080 display resolution. Signed-off-by: Sankeerth Billakanti --- Changes in v3: None Documentation/devicetree/bindings/display/panel/panel-simple.yaml | 2 ++ 1 file changed, 2 insertions(+) diff

[PATCH v3 4/4] drm/msm/dp: Add driver support to utilize drm panel

2022-02-09 Thread Sankeerth Billakanti
is completed. Signed-off-by: Sankeerth Billakanti --- Changes in v3: None drivers/gpu/drm/msm/dp/dp_display.c | 8 ++ drivers/gpu/drm/msm/dp/dp_drm.c | 54 + drivers/gpu/drm/msm/dp/dp_parser.h | 3 +++ 3 files changed, 60 insertions(+), 5 deletions

[PATCH v3 3/4] drm/panel-edp: Add eDP sharp panel support

2022-02-09 Thread Sankeerth Billakanti
Add support for the 14" sharp,lq140m1jw46 eDP panel. Signed-off-by: Sankeerth Billakanti --- Changes in v3: None drivers/gpu/drm/panel/panel-edp.c | 31 +++ 1 file changed, 31 insertions(+) diff --git a/drivers/gpu/drm/panel/panel-edp.c b/drivers/gpu/drm/

[PATCH v3 2/4] arm64: dts: qcom: sc7280: Add support for eDP panel on CRD

2022-02-09 Thread Sankeerth Billakanti
Enable the eDP display panel support without HPD on sc7280 platform. Signed-off-by: Sankeerth Billakanti --- Changes in v3: - Sort the nodes alphabetically - Use - instead of _ as node names - Place the backlight and panel nodes under root - Change the name of edp_out to mdss_edp_out

[PATCH v3 0/4] Add support for the eDP panel on sc7280 CRD

2022-02-09 Thread Sankeerth Billakanti
on the following series: https://patchwork.kernel.org/project/linux-arm-msm/list/?series=586263=both=* https://patchwork.kernel.org/project/linux-arm-msm/list/?series=560587=%2A=both Sankeerth Billakanti (4): dt-bindings: display: simple: Add sharp LQ140M1JW46 panel arm64: dts: qcom: sc7280: Add support

  1   2   >