[PATCH 0/5 v3] adv7511 EDID probing improvements

2017-01-11 Thread Archit Taneja
be appreciated! Tested on DB410c on 4.10-rc3. Works well for me. Thanks, Archit > > thanks > -john > > New in v3: > * Addressed naming improvements and drm_kms_helper_hotplug_event > usage corrections as suggested by Laurent. > > Cc: David Airlie > Cc: Archit Taneja

[PATCH v5 2/2] drm/bridge: adv7511: Initialize regulators

2017-01-11 Thread Archit Taneja
Maintain a table of regulator names expected by ADV7511 and ADV7533. Use regulator_bulk_* api to configure these. Initialize and enable the regulators during probe itself. Controlling these dynamically is left for later. Reviewed-by: Laurent Pinchart Signed-off-by: Archit Taneja --- drivers

[PATCH v5 1/2] dt-bindings: drm/bridge: adv7511: Add regulator bindings

2017-01-11 Thread Archit Taneja
Add the regulator supply properties needed by ADV7511 and ADV7533. Acked-by: Laurent Pinchart Acked-by: Rob Herring Signed-off-by: Archit Taneja --- v5: - Bring back supplies for individual pins - In v2, we had a v3p3-supply for DVDD_3V on ADV7511 and V3P3 pin on ADV7533. We don't really

[PATCH v5 0/2] drm/bridge: adv7511: Enable regulators

2017-01-11 Thread Archit Taneja
in the binding docs. - Update the driver to manage regulators for both ADV7511 and ADV7533. - Have separate supply entries for AVDD, DVDD, PVDD, A2VDD pins. - Use regulator_bulk_* API to configure regulators. Archit Taneja (2): dt-bindings: drm/bridge: adv7511: Add regulator bindings drm/bridge: adv7511

[PATCH] drm/msm/dsi: Set msm_dsi->encoders before initializing bridge

2017-01-11 Thread Archit Taneja
he encoder's bridge. That's now managed by the bridge API. Cc: Laurent Pinchart <laurent.pinchart+renesas at ideasonboard.com> Signed-off-by: Archit Taneja --- drivers/gpu/drm/msm/dsi/dsi.c | 8 +++- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/drivers/gpu/drm/msm/ds

[PATCH] drm/bridge: analogix dp: Fix runtime PM state on driver bind

2017-01-09 Thread Archit Taneja
On 01/05/2017 01:01 PM, Sean Paul wrote: > On Fri, Dec 30, 2016 at 4:57 AM, Marek Szyprowski > wrote: >> Analogix_dp_bind() can be called from component framework, which doesn't >> guarantee proper runtime PM state of the device during bind operation, >> so ensure that device is runtime active

[PATCH 10/10] drm/msm/dsi: Add PHY/PLL for 8x96

2017-01-07 Thread Archit Taneja
-by: Archit Taneja --- drivers/gpu/drm/msm/Kconfig|7 + drivers/gpu/drm/msm/Makefile |2 + drivers/gpu/drm/msm/dsi/dsi.h |8 + drivers/gpu/drm/msm/dsi/phy/dsi_phy.c |2 + drivers/gpu/drm/msm/dsi/phy/dsi_phy.h |2 + drivers/gpu/drm/msm

[PATCH 09/10] drm/msm/dsi: Add new method to calculate 14nm PHY timings

2017-01-07 Thread Archit Taneja
From: Hai Li <h...@codeaurora.org> The 14nm DSI PHY on 8x96 (called PHY v2 downstream) requires a different set of calculations for computing D-PHY timing params. Create a timing_calc_v2 func for the newer v2 PHYs. Signed-off-by: Hai Li Signed-off-by: Archit Taneja --- drivers/gpu/drm/m

[PATCH 08/10] drm/msm/dsi: Udpate generated headers for 14nm PHY and PLL

2017-01-07 Thread Archit Taneja
Signed-off-by: Archit Taneja --- drivers/gpu/drm/msm/dsi/dsi.xml.h | 252 ++ 1 file changed, 252 insertions(+) diff --git a/drivers/gpu/drm/msm/dsi/dsi.xml.h b/drivers/gpu/drm/msm/dsi/dsi.xml.h index 4958594..234b3b3 100644 --- a/drivers/gpu/drm/msm/dsi

[PATCH 07/10] drm/msm/dsi: Move PHY operations out of host

2017-01-07 Thread Archit Taneja
Hai Li Signed-off-by: Archit Taneja --- drivers/gpu/drm/msm/dsi/dsi.h | 18 ++- drivers/gpu/drm/msm/dsi/dsi_host.c | 46 +++--- drivers/gpu/drm/msm/dsi/dsi_manager.c | 178 ++-- drivers/gpu/drm/msm/dsi/phy/dsi_phy.c

[PATCH 06/10] drm/msm/dsi: Reset both PHYs before clock operation for dual DSI

2017-01-07 Thread Archit Taneja
by Archit Taneja ] Signed-off-by: Archit Taneja --- drivers/gpu/drm/msm/dsi/dsi.h | 1 + drivers/gpu/drm/msm/dsi/dsi_host.c| 25 + drivers/gpu/drm/msm/dsi/dsi_manager.c | 32 +--- 3 files changed, 43 insertions(+), 15 deletions

[PATCH 05/10] drm/msm/dsi: Pass down use case to PHY

2017-01-07 Thread Archit Taneja
From: Hai Li <h...@codeaurora.org> For some new types of DSI PHY, more settings depend on use cases controlled by DSI manager. This change allows DSI manager to setup PHY with a use case. Signed-off-by: Hai Li Signed-off-by: Archit Taneja --- drivers/gpu/drm/msm/dsi/dsi.h

[PATCH 04/10] drm/msm/dsi: Return more timings from PHY to host

2017-01-07 Thread Archit Taneja
From: Hai Li <h...@codeaurora.org> The DSI host is required to configure more timings calculated in PHY. By introducing a shared structure, this change allows more timing information passed from PHY to host. Signed-off-by: Hai Li Signed-off-by: Archit Taneja --- drivers/gpu/drm/msm/dsi

[PATCH 03/10] drm/msm/dsi: Add a PHY op that initializes version specific stuff

2017-01-07 Thread Archit Taneja
Create an init() op for dsi_phy which sets up things specific to a given DSI PHY. The dsi_phy driver probe expects every DSI version to get a "dsi_phy_regulator" mmio base. This isn't the case for 8x96. Creating an init() op will allow us to accommodate such differences. Signed-off-

[PATCH 02/10] drm/msm/dsi: Add 8x96 info in dsi_cfg

2017-01-07 Thread Archit Taneja
-off-by: Archit Taneja --- drivers/gpu/drm/msm/dsi/dsi_cfg.c | 25 + drivers/gpu/drm/msm/dsi/dsi_cfg.h | 1 + 2 files changed, 26 insertions(+) diff --git a/drivers/gpu/drm/msm/dsi/dsi_cfg.c b/drivers/gpu/drm/msm/dsi/dsi_cfg.c index 63436d8..a5d75c9 100644 --- a/drivers

[PATCH 01/10] drm/msm/dsi: Don't error if a DSI host doesn't have a device connected

2017-01-07 Thread Archit Taneja
evice even if it doesn't have a bridge/panel connected to it. Signed-off-by: Archit Taneja --- drivers/gpu/drm/msm/dsi/dsi_host.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/gpu/drm/msm/dsi/dsi_host.c b/drivers/gpu/drm/msm/dsi/dsi_host.c index 3538b7

[PATCH 00/10] drm/msm/dsi: Dual DSI and 8x96 PHY/PLL support

2017-01-07 Thread Archit Taneja
(single DSI) on DB820c. Archit Taneja (6): drm/msm/dsi: Don't error if a DSI host doesn't have a device connected drm/msm/dsi: Add 8x96 info in dsi_cfg drm/msm/dsi: Add a PHY op that initializes version specific stuff drm/msm/dsi: Reset both PHYs before clock operation for dual DSI drm

[PATCH] rnndb: dsi: Add DSI PHY 14nm domains

2017-01-07 Thread Archit Taneja
Add DSI PHY 14nm domains for DSI PHY common, DSI PHY lane and DSI PLL registers. Used in MSM8996. Signed-off-by: Archit Taneja --- rnndb/dsi/dsi.xml | 128 ++ 1 file changed, 128 insertions(+) diff --git a/rnndb/dsi/dsi.xml b/rnndb/dsi

[PATCH V7 3/4] drm/bridge: Add driver for GE B850v3 LVDS/DP++ Bridge

2017-01-05 Thread Archit Taneja
gt; Video output > > Cc: Martyn Welch > Cc: Martin Donnelly > Cc: Daniel Vetter > Cc: Enric Balletbo i Serra > Cc: Philipp Zabel > Cc: Rob Herring > Cc: Fabio Estevam > CC: David Airlie > CC: Thierry Reding > CC: Thierry Reding > CC: Archit Taneja > Re

[PATCH v7 3/5] drm: bridge: add support for TI ths8135

2017-01-03 Thread Archit Taneja
Hi Sekhar, On 1/2/2017 4:38 PM, Sekhar Nori wrote: > Hi Archit, > > On Wednesday 14 December 2016 10:35 AM, Archit Taneja wrote: >> >> >> On 12/13/2016 03:39 PM, Bartosz Golaszewski wrote: >>> THS8135 is a configurable video DAC, but no configuration is ac

[PATCH 13/17] drm/bridge: Use recommened kerneldoc for struct member refs

2017-01-02 Thread Archit Taneja
g). > > Also some minor drive-by polish where it makes sense, I read a lot > of docs ... > > Cc: Archit Taneja > Cc: Jani Nikula > Cc: Chris Wilson > Signed-off-by: Daniel Vetter > --- > drivers/gpu/drm/drm_bridge.c | 27 +++-

[RFC 8/9] HACK: drm/msm/mdp5: Add support for legacy cursor updates

2016-12-21 Thread Archit Taneja
On 12/20/2016 07:10 PM, Maarten Lankhorst wrote: > Op 20-12-16 om 07:23 schreef Archit Taneja: >> >> >> On 12/19/2016 06:20 PM, Maarten Lankhorst wrote: >>> Op 19-12-16 om 13:08 schreef Archit Taneja: >>>> This code has been more or less picked up f

[PATCH 2/2] rnndb: mdp5: Update mdp5_pipe

2016-12-20 Thread Archit Taneja
Update mdp5_pipe to incorporate SSPP_NONE and SSPP_CURSORx pipes Signed-off-by: Archit Taneja --- rnndb/mdp/mdp5.xml | 25 ++--- 1 file changed, 14 insertions(+), 11 deletions(-) diff --git a/rnndb/mdp/mdp5.xml b/rnndb/mdp/mdp5.xml index 49aa207..a5ae1e3 100644 --- a/rnndb

[PATCH 1/2] rnndb: mdp5: Add missing bitfields for MDP5_LM_BLEND_COLOR_OUT

2016-12-20 Thread Archit Taneja
Signed-off-by: Archit Taneja --- rnndb/mdp/mdp5.xml | 4 1 file changed, 4 insertions(+) diff --git a/rnndb/mdp/mdp5.xml b/rnndb/mdp/mdp5.xml index de9560b..49aa207 100644 --- a/rnndb/mdp/mdp5.xml +++ b/rnndb/mdp/mdp5.xml @@ -451,6 +451,10 @@ xsi:schemaLocation="

[PATCH 0/2] rnndb: mdp5: Cursor related reg updates

2016-12-20 Thread Archit Taneja
Some cursor SSPP related updates. Archit Taneja (2): rnndb: mdp5: Add missing bitfields for MDP5_LM_BLEND_COLOR_OUT rnndb: mdp5: Update mdp5_pipe rnndb/mdp/mdp5.xml | 29 ++--- 1 file changed, 18 insertions(+), 11 deletions(-) -- The Qualcomm Innovation Center, Inc

[RFC 8/9] HACK: drm/msm/mdp5: Add support for legacy cursor updates

2016-12-20 Thread Archit Taneja
On 12/19/2016 06:20 PM, Maarten Lankhorst wrote: > Op 19-12-16 om 13:08 schreef Archit Taneja: >> This code has been more or less picked up from the vc4 and intel >> implementations of update_plane() funcs for cursor planes. >> >> The upd

[PATCH 9/9] drm/msm/mdp5: Add cursor planes

2016-12-19 Thread Archit Taneja
lag. Signed-off-by: Archit Taneja --- drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c | 32 +--- drivers/gpu/drm/msm/msm_atomic.c| 5 - 2 files changed, 25 insertions(+), 12 deletions(-) diff --git a/drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c b/drivers/gpu/drm/msm/mdp/m

[RFC 8/9] HACK: drm/msm/mdp5: Add support for legacy cursor updates

2016-12-19 Thread Archit Taneja
for every new fb. The slow path will ensure that the fb is prepared/pinned etc. Cc: Signed-off-by: Archit Taneja --- - Don't know what to do for locking here :/ drivers/gpu/drm/msm/mdp/mdp5/mdp5_crtc.c | 7 ++ drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.h | 1 + drivers/gpu/drm/msm/mdp/mdp5

[PATCH 7/9] drm/msm/mdp5: Refactor mdp5_plane_atomic_check

2016-12-19 Thread Archit Taneja
, but crtc_state would need to be derived differently. Refactor mdp5_plane_atomic_check to mdp5_plane_atomic_check_with_state such that the latter takes crtc_state as an argument. This is similar to what the intel driver has done for async cursor updates. Signed-off-by: Archit Taneja --- drivers/gpu/drm/msm

[PATCH 6/9] drm/msm/mdp5: Misc cursor plane bits

2016-12-19 Thread Archit Taneja
in mdp5_plane_init instead of a bool telling whether plane is primary or not. Signed-off-by: Archit Taneja --- drivers/gpu/drm/msm/mdp/mdp5/mdp5_cfg.c | 10 + drivers/gpu/drm/msm/mdp/mdp5/mdp5_crtc.c | 34 +++ drivers/gpu/drm/msm/mdp/mdp5/mdp5_ctl.c | 10

[PATCH 5/9] drm/msm/mdp5: Configure COLOR3_OUT propagation

2016-12-19 Thread Archit Taneja
. This is borrowed from downstream MDP5 kernel driver. Without this, we don't see any cursor plane content. Signed-off-by: Archit Taneja --- drivers/gpu/drm/msm/mdp/mdp5/mdp5_crtc.c | 30 ++ 1 file changed, 30 insertions(+) diff --git a/drivers/gpu/drm/msm/mdp/mdp5/mdp5_crtc.c

[PATCH 4/9] drm/msm/mdp5: Use plane helpers to configure src/dst rectangles

2016-12-19 Thread Archit Taneja
lane), we would still see the plane in its last 'visible' configuration. Signed-off-by: Archit Taneja --- drivers/gpu/drm/msm/mdp/mdp5/mdp5_plane.c | 59 +++ drivers/gpu/drm/msm/msm_atomic.c | 21 +++ 2 files changed, 60 insertions(+), 20 deletions(-) d

[PATCH 3/9] drm/msm/mdp5: Prepare CRTC/LM for empty stages

2016-12-19 Thread Archit Taneja
to be staged at the topmost blender of the LM, which can result in empty stages in between 2) In the future, when we support multiple LMs per CRTC. We could have stages which don't have any pipe assigned to them. Signed-off-by: Archit Taneja --- drivers/gpu/drm/msm/mdp/mdp5/mdp5_crtc.c | 2 +- drivers/gpu

[PATCH 2/9] drm/msm/mdp5: Update generated headers

2016-12-19 Thread Archit Taneja
Signed-off-by: Archit Taneja --- drivers/gpu/drm/msm/mdp/mdp5/mdp5.xml.h | 30 -- 1 file changed, 20 insertions(+), 10 deletions(-) diff --git a/drivers/gpu/drm/msm/mdp/mdp5/mdp5.xml.h b/drivers/gpu/drm/msm/mdp/mdp5/mdp5.xml.h index 27d5371..6db1b8b 100644

[PATCH 1/9] drm/msm/mdp5: cfg: Add pipe_cursor block

2016-12-19 Thread Archit Taneja
Define the block in advance so that the generated mdp5.xml.h doesn't break build. Signed-off-by: Archit Taneja --- drivers/gpu/drm/msm/mdp/mdp5/mdp5_cfg.h | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/msm/mdp/mdp5/mdp5_cfg.h b/drivers/gpu/drm/msm/mdp/mdp5/mdp5_cfg.h index

[PATCH 0/9] drm/msm/mdp5: Cursor plane stuff

2016-12-19 Thread Archit Taneja
ms to work okay. Are there any tests that mix up atomic commits and legacy cursor updates a lot to identify issues? Archit Taneja (9): drm/msm/mdp5: cfg: Add pipe_cursor block drm/msm/mdp5: Update generated headers drm/msm/mdp5: Prepare CRTC/LM for empty stages drm/msm/mdp5: Use pla

[PATCH 6/6] drm/msm/mdp5: Create only as many CRTCs as we need

2016-12-19 Thread Archit Taneja
TCs we have. Signed-off-by: Archit Taneja --- drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c | 39 - 1 file changed, 29 insertions(+), 10 deletions(-) diff --git a/drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c b/drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c index 646f160..84ec

[PATCH 5/6] drm/msm/mdp5: cfg: Change count to unsigned int

2016-12-19 Thread Archit Taneja
Count can't be non-zero. Changing to uint will also prevent future warnings. Signed-off-by: Archit Taneja --- drivers/gpu/drm/msm/mdp/mdp5/mdp5_cfg.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/msm/mdp/mdp5/mdp5_cfg.h b/drivers/gpu/drm/msm/mdp/mdp5

[PATCH 4/6] drm/msm/mdp5: Create single encoder per interface (INTF)

2016-12-19 Thread Archit Taneja
. It can use the the kms func op set_encoder_mode to change the mode of operation, which in turn would configure the interface type for the INTF. In mdp5_cmd_encoder.c, we remove the redundant code, and make the commmand mode funcs as helpers that are used in mdp5_encoder.c Signed-off-by: Archit

[PATCH 3/6] drm/msm/mdp5: Prepare for merging video and command encoders

2016-12-19 Thread Archit Taneja
Rename the mdp5_encoder_* ops for active displays to mdp5_vid_encoder_* ops. Signed-off-by: Archit Taneja --- drivers/gpu/drm/msm/mdp/mdp5/mdp5_encoder.c | 31 ++--- drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.c | 3 ++- drivers/gpu/drm/msm/mdp/mdp5/mdp5_kms.h | 4

[PATCH 2/6] drm/msm: Set encoder's mode of operation using a kms func

2016-12-19 Thread Archit Taneja
device's mode flags are. Start with providing a way to set the mdp5_intf_mode using a kms func. Signed-off-by: Archit Taneja --- drivers/gpu/drm/msm/dsi/dsi.h | 1 + drivers/gpu/drm/msm/dsi/dsi_host.c | 2 ++ drivers/gpu/drm/msm/dsi/dsi_manager.c | 12

[PATCH 1/6] drm/msm: Construct only one encoder for DSI

2016-12-19 Thread Archit Taneja
that the encoder is configured only in video mode. Later, the same encoder would be usable in both modes. Signed-off-by: Archit Taneja --- drivers/gpu/drm/msm/dsi/dsi.c | 17 ++--- drivers/gpu/drm/msm/dsi/dsi.h | 4 ++-- drivers/gpu/drm/msm/dsi/dsi_manager.c | 22

[PATCH 0/6] drm/msm/mdp5: Encoder related cleanups

2016-12-19 Thread Archit Taneja
The MDP5 and DSI drivers created 2 drm_encoders for a DSI interface (one for each mode of operation). This patch fixes that. Now, with the # encoders equal to the # of displays, we can create the right # of CRTCs. We previously created LM # of CRTCs, which ate up too many primary planes. Archit

[PATCH v4 0/5] Cleanup DRM bridge attach/detach

2016-12-18 Thread Archit Taneja
On 12/14/2016 3:29 PM, Laurent Pinchart wrote: > Hello, > > This patch series is a respin of the DRM bridge attach/detach cleanup patches > that were previously part of "[PATCH v3 00/13] R-Car DU: Use drm bridge API". > As patches 1/5 and 3/5 touch a large number of drivers and are thus painful

[PATCH v4 1/2] dt-bindings: drm/bridge: adv7511: Add regulator bindings

2016-12-18 Thread Archit Taneja
On 12/10/2016 3:41 AM, Rob Herring wrote: > On Mon, Dec 05, 2016 at 01:23:54PM +0530, Archit Taneja wrote: >> Add the regulator supply properties needed by ADV7511 and ADV7533. >> >> Cc: devicetree at vger.kernel.org >> Acked-by: Laurent Pinchart >

[PATCH v4 3/5] drm: bridge: Link encoder and bridge in core code

2016-12-16 Thread Archit Taneja
Hi, On 12/14/2016 03:29 PM, Laurent Pinchart wrote: > Instead of linking encoders and bridges in every driver (and getting it > wrong half of the time, as many drivers forget to set the drm_bridge > encoder pointer), do so in core code. The drm_bridge_attach() function > needs the encoder and

[PATCH] drm/bridge: analogix_dp: set the DPCD600 during disabling the psr

2016-12-15 Thread Archit Taneja
On 12/14/2016 03:30 PM, Sean Paul wrote: > On Wed, Dec 14, 2016 at 12:03 AM, Archit Taneja > wrote: >> Hi, >> >> On 12/12/2016 08:28 PM, Sean Paul wrote: >>> >>> On Fri, Dec 9, 2016 at 9:49 PM, Caesar Wang wrote: >>>> >>>&g

[PATCH v7 3/5] drm: bridge: add support for TI ths8135

2016-12-14 Thread Archit Taneja
On 12/13/2016 03:39 PM, Bartosz Golaszewski wrote: > THS8135 is a configurable video DAC, but no configuration is actually > necessary to make it work. > > For now use the dumb-vga-dac driver to support it. Queued to drm-misc-next Archit > > Signed-off-by: Bartosz Golaszewski > Reviewed-by:

[PATCH v7 2/5] drm: bridge: add DT bindings for TI ths8135

2016-12-14 Thread Archit Taneja
Hi, On 12/13/2016 03:39 PM, Bartosz Golaszewski wrote: > THS8135 is a configurable video DAC. Add DT bindings for this chip. Queued to drm-misc-next > > Signed-off-by: Bartosz Golaszewski > Reviewed-by: Laurent Pinchart > Acked-by: Rob Herring > --- >

[PATCH] drm/bridge: analogix_dp: set the DPCD600 during disabling the psr

2016-12-14 Thread Archit Taneja
ink mode during enter and exit the psr, >> this issue is gone if switching the fast link to main link mode. >> > > Cc: Archit Taneja Do we want this as a fix in 4.10? Or is it okay to get it in 4.11? In other words, should this go to drm-misc-next or drm-misc-fixes? Thanks,

[PATCH v2 6/6] drm/i915: Add a cursor hack to allow converting legacy page flip to atomic, v3.

2016-12-14 Thread Archit Taneja
On 12/13/2016 07:22 PM, Maarten Lankhorst wrote: > Op 13-12-16 om 14:01 schreef Archit Taneja: >> Hi, >> >> On 12/12/2016 4:04 PM, Maarten Lankhorst wrote: >>> Do something similar to vc4, only allow updating the cursor state >>> in-place through a fast

[PATCH v2 6/6] drm/i915: Add a cursor hack to allow converting legacy page flip to atomic, v3.

2016-12-13 Thread Archit Taneja
Hi, On 12/12/2016 4:04 PM, Maarten Lankhorst wrote: > Do something similar to vc4, only allow updating the cursor state > in-place through a fastpath when the watermarks are unaffected. This > will allow cursor movement to be smooth, but changing cursor size or > showing/hiding cursor will still

[RFC][PATCH 0/5 v2] adv7511 EDID probing improvements

2016-12-12 Thread Archit Taneja
for ADV7511 too. Laurent, could we get an Ack for the series from you? Thanks, Archit > > thanks > -john > > Cc: David Airlie > Cc: Archit Taneja > Cc: Wolfram Sang <wsa+renesas at sang-engineering.com> > Cc: Lars-Peter Clausen > Cc: Laurent Pinchart > Cc:

[PATCH v2] drm/bridge: analogix: Don't return -EINVAL when panel not support PSR in PSR functions

2016-12-07 Thread Archit Taneja
On 12/07/2016 06:27 AM, zain wang wrote: > We will ignored PSR setting if panel not support it. So, in this case, we > should > return from analogix_dp_enable/disable_psr() without any error code. > Let's retrun 0 instead of -EINVAL when panel not support PSR in >

[PATCH v4 2/2] drm/bridge: adv7511: Initialize regulators

2016-12-05 Thread Archit Taneja
Maintain a table of regulator names expect by ADV7511 and ADV7533. Use regulator_bulk_* api to configure these. Initialize and enable the regulators during probe itself. Controlling these dynamically is left for later. Reviewed-by: Laurent Pinchart Signed-off-by: Archit Taneja --- drivers/gpu

[PATCH v4 1/2] dt-bindings: drm/bridge: adv7511: Add regulator bindings

2016-12-05 Thread Archit Taneja
Add the regulator supply properties needed by ADV7511 and ADV7533. Cc: devicetree at vger.kernel.org Acked-by: Laurent Pinchart Signed-off-by: Archit Taneja --- Documentation/devicetree/bindings/display/bridge/adi,adv7511.txt | 8 1 file changed, 8 insertions(+) diff --git

[PATCH v4 0/2] drm/bridge: adv7511: Enable regulators

2016-12-05 Thread Archit Taneja
separate supply entries for AVDD, DVDD, PVDD, A2VDD pins. - Use regulator_bulk_* API to configure regulators. Archit Taneja (2): dt-bindings: drm/bridge: adv7511: Add regulator bindings drm/bridge: adv7511: Initialize regulators .../bindings/display/bridge/adi,adv7511.txt| 8

[PATCH] drm/bridge: analogix: Don't return -EINVAL when panel not support PSR in PSR functions

2016-12-05 Thread Archit Taneja
On 12/02/2016 09:33 PM, Sean Paul wrote: > On Thu, Dec 1, 2016 at 10:54 PM, Archit Taneja > wrote: >> Hi, >> >> On 12/02/2016 08:02 AM, zain wang wrote: >>> >>> We will ignored PSR setting if panel not support it. So, in this case, we >>> sh

[PATCH] drm/bridge: analogix: Don't return -EINVAL when panel not support PSR in PSR functions

2016-12-02 Thread Archit Taneja
Hi, On 12/02/2016 08:02 AM, zain wang wrote: > We will ignored PSR setting if panel not support it. So, in this case, we > should > return from analogix_dp_enable/disable_psr() without any error code. > Let's retrun 0 instead of -EINVAL when panel not support PSR in >

[PATCH] drm/bridge: tc358767: don't warn if display side ASSR enable fails

2016-12-01 Thread Archit Taneja
On 11/30/2016 05:18 PM, Lucas Stach wrote: > ASSR is an optional feature, so it's a valid operating condition for > the display to reject ASSR enable. Demote the warning to the debug > level. Lgtm. Will pull it if Philipp or Andrey don't have any comments on it. Thanks, Archit > >

[PATCH v3 03/13] drm: bridge: Link encoder and bridge in core code

2016-11-30 Thread Archit Taneja
On 11/30/2016 4:35 PM, Laurent Pinchart wrote: > Hi Archit, > > On Wednesday 30 Nov 2016 16:30:53 Archit Taneja wrote: >> On 11/30/2016 03:53 PM, Laurent Pinchart wrote: >>> On Wednesday 30 Nov 2016 10:35:02 Archit Taneja wrote: >>>> On 11/29/2

[PATCH v3 03/13] drm: bridge: Link encoder and bridge in core code

2016-11-30 Thread Archit Taneja
On 11/30/2016 03:53 PM, Laurent Pinchart wrote: > Hi Archit, > > On Wednesday 30 Nov 2016 10:35:02 Archit Taneja wrote: >> On 11/29/2016 11:27 PM, Laurent Pinchart wrote: >>> On Tuesday 29 Nov 2016 15:57:06 Archit Taneja wrote: >>>> On 11/29/2016 02:34 PM, L

[PATCH v3 03/13] drm: bridge: Link encoder and bridge in core code

2016-11-30 Thread Archit Taneja
On 11/29/2016 11:27 PM, Laurent Pinchart wrote: > Hi Archit, > > On Tuesday 29 Nov 2016 15:57:06 Archit Taneja wrote: >> On 11/29/2016 02:34 PM, Laurent Pinchart wrote: >>> Instead of linking encoders and bridges in every driver (and getting it >>> wrong half of

[RFC][PATCH 5/5 v2] drm/bridge: adv7511: Reuse __adv7511_power_on/off() when probing EDID

2016-11-30 Thread Archit Taneja
etting lost. Using the logic > in __adv7511_power_on/off() which syncs and dirtys the cache > avoids this issue. > > Thus this patch changes the EDID probing logic so that we > re-use the __adv7511_power_on/off() calls. > > Cc: David Airlie > Cc: Archit Taneja > Cc: Wolf

[PATCH v3 04/13] drm: bridge: Detach bridge from encoder at encoder cleanup time

2016-11-29 Thread Archit Taneja
On 11/29/2016 02:34 PM, Laurent Pinchart wrote: > Most drivers that use bridges forgot to detach them at cleanup time. > Instead of fixing them one by one, detach the bridge in the core > drm_encoder_cleanup() function. > > Signed-off-by: Laurent Pinchart

[PATCH v3 03/13] drm: bridge: Link encoder and bridge in core code

2016-11-29 Thread Archit Taneja
On 11/29/2016 02:34 PM, Laurent Pinchart wrote: > Instead of linking encoders and bridges in every driver (and getting it > wrong half of the time, as many drivers forget to set the drm_bridge > encoder pointer), do so in core code. The drm_bridge_attach() function > needs the encoder and

[PATCH v3 1/2] dt-bindings: drm/bridge: adv7511: Add regulator bindings

2016-11-29 Thread Archit Taneja
On 11/29/2016 12:03 PM, Laurent Pinchart wrote: > Hi Archit, > > Thank you for the patch. > > On Tuesday 29 Nov 2016 11:37:41 Archit Taneja wrote: >> Add the regulator supply properties needed by ADV7511 and ADV7533. >> >> The regulators are specified as op

[PATCH v3 2/2] drm/bridge: adv7511: Initialize regulators

2016-11-29 Thread Archit Taneja
Maintain a table of regulator names expect by ADV7511 and ADV7533. Use regulator_bulk_* api to configure these. Initialize and enable the regulators during probe itself. Controlling these dynamically is left for later. Signed-off-by: Archit Taneja --- v3: - Drop the additional 1.8V supply names

[PATCH v3 1/2] dt-bindings: drm/bridge: adv7511: Add regulator bindings

2016-11-29 Thread Archit Taneja
-by: Rob Herring Signed-off-by: Archit Taneja --- v3: - Revert back to having a common avdd-supply property for the 1.8V supplies Documentation/devicetree/bindings/display/bridge/adi,adv7511.txt | 9 + 1 file changed, 9 insertions(+) diff --git a/Documentation/devicetree/bindings/display

[PATCH v3 0/2] drm/bridge: adv7511: Enable regulators

2016-11-29 Thread Archit Taneja
supply for ADV7511 too in the binding docs. - Update the driver to manage regulators for both ADV7511 and ADV7533. - Have separate supply entries for AVDD, DVDD, PVDD, A2VDD pins. - Use regulator_bulk_* API to configure regulators. Archit Taneja (2): dt-bindings: drm/bridge: adv7511: Add regulator

[PATCH] drm: bridge: dw-hdmi: add ASoC dependency

2016-11-28 Thread Archit Taneja
On 11/28/2016 06:37 AM, Kuninori Morimoto wrote: > > Hi > >> The newly added sound driver depends on SND_SOC_HDMI_CODEC, which in >> turn only makes sense when ASoC is enabled, as shown by this warning: >> >> warning: (DRM_MSM && DRM_STI && DRM_MEDIATEK_HDMI && DRM_I2C_NXP_TDA998X && >>

[PATCH v4] drm: bridge: add DesignWare HDMI I2S audio support

2016-11-24 Thread Archit Taneja
Hi, On 11/24/2016 10:43 AM, Kuninori Morimoto wrote: > > Hi Archit, David, and DRM ML > > I had heared that Archit is the maintainer of dw-hdmi driver, but am I wrong > ?? > I'm posting this patch series since half year ago, but no response > from him, and nothing happen (I got review from

[RFC][PATCH 1/3] drm/bridge: adv7511: Rework adv7511_power_on/off() so they can be reused internally

2016-11-23 Thread Archit Taneja
On 11/23/2016 01:16 AM, John Stultz wrote: > On Tue, Nov 22, 2016 at 9:38 AM, Laurent Pinchart > wrote: >> Hi John, >> >> On Tuesday 22 Nov 2016 09:25:22 John Stultz wrote: >>> On Tue, Nov 22, 2016 at 12:14 AM, Laurent Pinchart wrote: On Monday 21 Nov 2016 16:37:30 John Stultz wrote: >>>

[PATCH] drm: document standard connector properties

2016-11-17 Thread Archit Taneja
typed this to give Manasi a place to add her new link status > property documentation. Reviewed-by: Archit Taneja > > v2: forgot to git add all the bits (Manasi). > > v3: Be more epxlicit about integrated tiled panels (Archit) > > Cc: Manasi Navare > Cc: Arc

[PATCH v4 1/2] drm/bridge: dumb-vga-dac: Support a VDD regulator supply

2016-11-17 Thread Archit Taneja
On 11/17/2016 01:25 PM, Chen-Yu Tsai wrote: > On Thu, Nov 17, 2016 at 3:48 PM, Archit Taneja > wrote: >> Hi, >> >> Thanks for the patch. >> >> >> On 11/16/2016 09:12 PM, Chen-Yu Tsai wrote: >>> >>> Some dumb VGA DACs are act

[PATCH] drm: document standard connector properties

2016-11-17 Thread Archit Taneja
On 11/17/2016 01:08 PM, Daniel Vetter wrote: > There's a really big pile of additional connector properties, a lot of > them standardized. But they're all for specific outputs (panels, TV, > scaling, ...) so I left them out for now since this is enough for a > start. > > I typed this to give

[PATCH v4 1/2] drm/bridge: dumb-vga-dac: Support a VDD regulator supply

2016-11-17 Thread Archit Taneja
Hi, Thanks for the patch. On 11/16/2016 09:12 PM, Chen-Yu Tsai wrote: > Some dumb VGA DACs are active components which require external power. > Add support for specifying a regulator as its power supply. > > Signed-off-by: Chen-Yu Tsai > Acked-by: Rob Herring > --- >

[PATCH] drm/msm: Remove bad calls to of_node_put()

2016-11-17 Thread Archit Taneja
. Remove the extra of_node_put calls. This fixes warnings seen when we try to insert the driver as a module on IFC6410. Reported-by: Ilia Mirkin Signed-off-by: Archit Taneja --- drivers/gpu/drm/msm/msm_drv.c | 9 ++--- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/drivers/gpu/drm

[PATCH 3/3] MAINTAINERS: Add Archit as drm bridge maintainer

2016-11-17 Thread Archit Taneja
On 11/16/2016 07:38 PM, Daniel Vetter wrote: > Again something that's in the drm-misc fold. Acked-by: Archit Taneja > > Cc: Archit Taneja > Signed-off-by: Daniel Vetter > --- > MAINTAINERS | 6 ++ > 1 file changed, 6 insertions(+) > > diff --git a/MAINTA

[linux-sunxi] Re: [PATCH v3 1/2] drm/bridge: dumb-vga-dac: Support a VDD regulator supply

2016-11-16 Thread Archit Taneja
Hi, On 11/15/2016 08:29 AM, Chen-Yu Tsai wrote: > Hi, > > On Wed, Nov 2, 2016 at 9:33 AM, Chen-Yu Tsai wrote: >> On Mon, Oct 31, 2016 at 2:28 PM, Rob Herring wrote: >>> On Sat, Oct 29, 2016 at 07:06:10PM +0800, Chen-Yu Tsai wrote: Some dumb VGA DACs are active components which require

[PATCH] drm/bridge: analogix_dp: return error if transfer none byte

2016-11-16 Thread Archit Taneja
On 11/15/2016 10:39 PM, Sean Paul wrote: > On Thu, Nov 3, 2016 at 3:17 AM, Jianqun Xu wrote: >> Reference from drm_dp_aux description (about transfer): >> Upon success, the implementation should return the number of payload bytes >> that were transferred, or a negative error-code on failure.

[PATCH v18 2/2] drm/bridge: Add I2C based driver for ps8640 bridge

2016-11-15 Thread Archit Taneja
Hi, On 11/14/2016 07:11 PM, Jitao Shi wrote: > This patch adds drm_bridge driver for parade DSI to eDP bridge chip. Thanks for the incorporating the fixes. I have commented on one issue below. The only thing that seems to be left now is the firmware update bits, right? Can we get the firmware

[v17 2/2] drm/bridge: Add I2C based driver for ps8640 bridge

2016-11-11 Thread Archit Taneja
Hi Jitao, I couldn't locate the original mail, so posting on this thread instead. Some comments below. On 11/10/2016 10:09 PM, Enric Balletbo Serra wrote: > Hi Jitao, > > 2016-08-27 8:44 GMT+02:00 Jitao Shi : >> This patch adds drm_bridge driver for parade DSI to eDP bridge chip. >> >>

[PATCH 1/5] drm/msm/mdp5: introduce mdp5_hw_pipe

2016-11-07 Thread Archit Taneja
On 11/7/2016 8:18 PM, Rob Clark wrote: > On Mon, Nov 7, 2016 at 5:38 AM, Archit Taneja > wrote: >> >> >> On 11/05/2016 09:55 PM, Rob Clark wrote: >>> >>> Split out the hardware pipe specifics from mdp5_plane. To start, the hw >>> pip

[PATCH 1/5] drm/msm/mdp5: introduce mdp5_hw_pipe

2016-11-07 Thread Archit Taneja
he N layer-mixers (LM). The first N planes become primary > + * planes for the CRTCs, with the remainder as overlay planes: > + */ Jfyi, we might need to change this a bit in the future. It'll be better to get the max number of displays connected on our platform via parsing DT, et

[PATCH 4/5] drm/msm/mdp5: dynamically assign hw pipes to planes

2016-11-07 Thread Archit Taneja
Hi, Minor comments below. LGTM otherwise. On 11/05/2016 09:56 PM, Rob Clark wrote: > (re)assign the hw pipes to planes based on required caps, and to handle > situations where we could not modify an in-use plane (ie. SMP block > reallocation). > > This means all planes advertise the superset of

[PATCH 3/5] drm/msm/mdp5: add skeletal mdp5_state

2016-11-07 Thread Archit Taneja
drm_modeset_lock_init(_kms->state_lock); > + mdp5_kms->state = kzalloc(sizeof(*mdp5_kms->state), GFP_KERNEL); > + if (!mdp5_kms->state) { > + ret = -ENOMEM; > + goto fail; > + } > + This would probably be better in mdp5_kms_init(

[PATCH] drm/msm: Fix error handling crashes seen when VRAM allocation fails

2016-11-03 Thread Archit Taneja
>funcs is also non-NULL. This is needed for MDP5, since during msm_drm_int(), priv->kms becomes non-NULL early, but msm_kms_init() is called on it only later in mdp5_kms_init(). Signed-off-by: Archit Taneja --- drivers/gpu/drm/msm/msm_drv.c | 2 +- drivers/gpu/drm/msm/msm_gem_sh

[PATCH v2 1/8] drm/bridge: rgb-to-vga: Support an enable GPIO

2016-10-27 Thread Archit Taneja
On 10/25/2016 02:29 PM, Chen-Yu Tsai wrote: > On Tue, Oct 25, 2016 at 4:09 PM, Archit Taneja > wrote: >> Hi, >> >> On 10/20/2016 09:13 AM, Chen-Yu Tsai wrote: >>> >>> Some rgb-to-vga bridges have an enable GPIO, either directly tied to >>

[PATCH v2 2/2] drm/msm: Set CLK_IGNORE_UNUSED flag for PLL clocks

2016-10-27 Thread Archit Taneja
domains beforehand. We set the CLK_IGNORE_UNUSED flag for PLL clocks for now. This needs to be revisited, since bootloaders can enable display, and we would want to disable the PLL clocks if there isn't a display driver using them. Cc: Stephen Boyd Signed-off-by: Archit Taneja --- drivers/gpu/drm/msm

[PATCH v5 3/3] drm/bridge: add Silicon Image SiI8620 driver

2016-10-26 Thread Archit Taneja
On 10/10/2016 01:09 PM, Andrzej Hajda wrote: > SiI8620 transmitter converts eTMDS/HDMI signal to MHL 3.0. > It is controlled via I2C bus. Its interaction with other > devices in video pipeline is performed mainly on HW level. > The only interaction it does on device driver level is >

[PATCH v4 2/3] dt-bindings: add Silicon Image SiI8620 bridge bindings

2016-10-26 Thread Archit Taneja
On 10/07/2016 12:32 PM, Andrzej Hajda wrote: > SiI8620 transmitter converts eTMDS/HDMI signal to MHL 3.0. It is controlled > via I2C bus. queued to drm-misc. Thanks, Archit > > Signed-off-by: Andrzej Hajda > Acked-by: Rob Herring > --- > .../bindings/video/bridge/sil-sii8620.txt |

[PATCH v4 1/3] video: add header file for Mobile High-Definition Link (MHL) interface

2016-10-26 Thread Archit Taneja
On 10/07/2016 12:32 PM, Andrzej Hajda wrote: > This header adds definitions specific to MHL protocol. queued to drm-misc. Thanks, Archit -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project

[PATCH 2/2] drm/msm: Don't provide 'is_enabled' PLL clk_ops

2016-10-26 Thread Archit Taneja
On 10/26/2016 01:58 AM, Stephen Boyd wrote: > On 10/25, Archit Taneja wrote: >> The DSI/HDMI PLLs in MSM require resources like interface clocks, power >> domains to be enabled before we can access their registers. >> >> The clock framework doesn't have a mechanism

[PATCH v5 3/3] drm/bridge: add Silicon Image SiI8620 driver

2016-10-26 Thread Archit Taneja
Hi Andrzej, On 10/10/2016 01:09 PM, Andrzej Hajda wrote: > SiI8620 transmitter converts eTMDS/HDMI signal to MHL 3.0. > It is controlled via I2C bus. Its interaction with other > devices in video pipeline is performed mainly on HW level. > The only interaction it does on device driver level is >

[PATCH v2 8/8] ARM: dts: sun6i: hummingbird-a31: Enable display output through VGA bridge

2016-10-25 Thread Archit Taneja
On 10/20/2016 09:13 AM, Chen-Yu Tsai wrote: > The Hummingbird A31 board has a RGB-to-VGA bridge which converts RGB > output from the LCD interface to VGA signals. > > Enable this part of the display pipeline. I couldn't find the enable-gpios binding for the bridge that you introduced in the

[PATCH v2 1/8] drm/bridge: rgb-to-vga: Support an enable GPIO

2016-10-25 Thread Archit Taneja
Hi, On 10/20/2016 09:13 AM, Chen-Yu Tsai wrote: > Some rgb-to-vga bridges have an enable GPIO, either directly tied to > an enable pin on the bridge IC, or indirectly controlling a power > switch. > > Add support for it. Does the bridge on your platform have an active/passive DAC, or is it a

[PATCH 2/2] drm/msm: Don't provide 'is_enabled' PLL clk_ops

2016-10-25 Thread Archit Taneja
domains beforehand. We remove the is_enabled clk_ops from the PLL clocks for now since they aren't mandatory. This needs to be revisited, since bootloaders can enable display, the enable count maintained by clock framework wouldn't work in such cases. Cc: Stephen Boyd Signed-off-by: Archit Taneja

[PATCH 1/2] drm/msm/dsi: Queue HPD helper work in attach/detach callbacks

2016-10-25 Thread Archit Taneja
instead. Signed-off-by: Archit Taneja --- drivers/gpu/drm/msm/dsi/dsi_host.c | 14 -- 1 file changed, 12 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/msm/dsi/dsi_host.c b/drivers/gpu/drm/msm/dsi/dsi_host.c index f05ed0e..6f24002 100644 --- a/drivers/gpu/drm/msm/dsi

[PATCH 0/2] drm/msm: Fixes for issues seen on DB410c

2016-10-25 Thread Archit Taneja
clocks enabled. The other reason is a deadlock caused by calling drm_hpd_helper_irq_event in a context that might already hold drm_device's mode_config.mutex. Archit Taneja (2): drm/msm/dsi: Queue HPD helper work in attach/detach callbacks drm/msm: Don't provide 'is_enabled' PLL clk_ops

<    1   2   3   4   5   6   7   8   9   10   >