linux.git/drivers/gpu/drm/msm/edp, branch v4.9

drm: msm: Rely on the default ->best_encoder() behavior where appropriate

2016-06-09T06:43:26+00:00

For all outputs except DSI we have a 1:1 relationship between connectors
and encoders and the driver is relying on the atomic helpers: we can
drop the custom ->best_encoder() and let the core call
drm_atomic_helper_best_encoder() for us.

Signed-off-by: Boris Brezillon 
Tested-by: Archit Taneja 
Signed-off-by: Daniel Vetter 
Link: http://patchwork.freedesktop.org/patch/msgid/1465300095-16971-9-git-send-email-boris.brezillon@free-electrons.com

drm/msm: Centralize connector registration/unregistration

2016-05-08T14:22:19+00:00

Move the drm_connector registration from the encoder(HDMI/DSI etc) drivers
to the msm platform driver. This will simplify the task of ensuring that
the connectors are registered only after the drm_device itself is
registered.

The connectors' destroy ops are made to use kzalloc instead of
devm_kzalloc to ensure that that the connectors can be successfully
unregistered when the msm driver module is removed. The memory for the
connectors is unallocated when drm_mode_config_cleanup() is called
during either during an error or during driver remove.

Signed-off-by: Archit Taneja 
Signed-off-by: Rob Clark

drm/msm/edp: Drop regulator_set_voltage call

2016-05-08T14:22:17+00:00

The eDP driver tries to set a fixed voltage for one of its regulators(vdda)
before enabling it. This shouldn't be done by the driver, the voltage
constraints should be specified on the regulator via DT and managed by
the regulator core. A driver should call regulator_set_voltage only if
it needs to change the voltage during runtime. Drop the
regulator_set_voltage call. Mention in a comment the voltage that the
regulator expects.

Signed-off-by: Archit Taneja 
Signed-off-by: Rob Clark

drm/msm: Move call to PTR_ERR_OR_ZERO after reassignment

2016-05-08T14:22:16+00:00

Here, a location is reset to NULL before being passed to PTR_ERR.
So, PTR_ERR should be called before its argument is reassigned
to NULL. Further to simplify things use PTR_ERR_OR_ZERO instead
of PTR_ERR and IS_ERR.

Problem found using Coccinelle.

Signed-off-by: Vaishali Thakkar 
Reviewed-by: Eric Engestrom 
[fixed fmt string warning (s/%ld/%d/)]
Signed-off-by: Rob Clark

drm/msm: update generated headers

2016-03-03T16:55:27+00:00

Pull in additional regs needed for a430, etc.

Signed-off-by: Rob Clark

drm/msm: update generated headers

2015-10-22T19:39:44+00:00

Signed-off-by: Rob Clark

drm/msm/dp: use flags argument of devm_gpiod_get to set direction

2015-08-15T22:27:18+00:00

Since 39b2bbe3d715 (gpio: add flags argument to gpiod_get*() functions)
which appeared in v3.17-rc1, the gpiod_get* functions take an additional
parameter that allows to specify direction and initial value for output.

Use this to simplify the driver. Furthermore this is one caller less
that stops us making the flags argument to gpiod_get*() mandatory.

Acked-by: Alexandre Courbot 
Acked-by: Linus Walleij 
Signed-off-by: Uwe Kleine-König 
Signed-off-by: Rob Clark

drm/msm: update generated headers

2015-08-15T22:27:10+00:00

Signed-off-by: Rob Clark

drm/msm: drop redundant debug output

2015-06-11T17:11:03+00:00

wait_for_completion_timeout returns 0 in case of timeout and never
return < 0 so there is no additional information in printing the
value of time_left here as it will always be 0, thus it can be dropped.

Signed-off-by: Nicholas Mc Guire 
Signed-off-by: Rob Clark

drm/msm: wait_for_completion_timeout return is never negative

2015-06-11T17:11:03+00:00

wait_for_completion_timeout returns >= 0 but never
negative  - so the error check should be against equality
to 0 not <= 0.

Signed-off-by: Nicholas Mc Guire 
Signed-off-by: Rob Clark