gma500: medfield: Fix possible NULL pointer dereference
The use of pointer sender should be after the NULL check. Signed-off-by: Syam Sidhardhan <s.syam@samsung.com> Signed-off-by: Patrik Jakobsson <patrik.r.jakobsson@gmail.com>
This commit is contained in:
parent
bc6a541941
commit
22ccb2a146
|
@ -92,8 +92,8 @@ void mdfld_dsi_brightness_init(struct mdfld_dsi_config *dsi_config, int pipe)
|
||||||
{
|
{
|
||||||
struct mdfld_dsi_pkg_sender *sender =
|
struct mdfld_dsi_pkg_sender *sender =
|
||||||
mdfld_dsi_get_pkg_sender(dsi_config);
|
mdfld_dsi_get_pkg_sender(dsi_config);
|
||||||
struct drm_device *dev = sender->dev;
|
struct drm_device *dev;
|
||||||
struct drm_psb_private *dev_priv = dev->dev_private;
|
struct drm_psb_private *dev_priv;
|
||||||
u32 gen_ctrl_val;
|
u32 gen_ctrl_val;
|
||||||
|
|
||||||
if (!sender) {
|
if (!sender) {
|
||||||
|
@ -101,6 +101,9 @@ void mdfld_dsi_brightness_init(struct mdfld_dsi_config *dsi_config, int pipe)
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
dev = sender->dev;
|
||||||
|
dev_priv = dev->dev_private;
|
||||||
|
|
||||||
/* Set default display backlight value to 85% (0xd8)*/
|
/* Set default display backlight value to 85% (0xd8)*/
|
||||||
mdfld_dsi_send_mcs_short(sender, write_display_brightness, 0xd8, 1,
|
mdfld_dsi_send_mcs_short(sender, write_display_brightness, 0xd8, 1,
|
||||||
true);
|
true);
|
||||||
|
|
Loading…
Reference in a new issue