]> rtime.felk.cvut.cz Git - vajnamar/linux-xlnx.git/commitdiff
drm: xilinx: dsi: Add static and remove unused variable
authorHyun Kwon <hyun.kwon@xilinx.com>
Mon, 27 Mar 2017 17:15:16 +0000 (10:15 -0700)
committerMichal Simek <michal.simek@xilinx.com>
Mon, 29 May 2017 10:25:36 +0000 (12:25 +0200)
Fix sparse warnings:
drivers/gpu/drm/xilinx/xilinx_drm_dsi.c:217:27: warning: dubious: !x & y
drivers/gpu/drm/xilinx/xilinx_drm_dsi.c:791:24: warning: symbol 'dsi_driver' was not declared. Should it be static?
CC      drivers/gpu/drm/xilinx/xilinx_drm_dsi.o
drivers/gpu/drm/xilinx/xilinx_drm_dsi.c: In function âlinx_dsi_connector_dpmsâdrivers/gpu/drm/xilinx/xilinx_drm_dsi.c:400:7: warning: unused variable ânel_onâ-Wunused-variable]
  bool panel_on = 0;
       ^~~~~~~~

Signed-off-by: Hyun Kwon <hyun.kwon@xilinx.com>
CC: Siva Rajesh <siva.rajesh.jarugula@xilinx.com>
Signed-off-by: Michal Simek <michal.simek@xilinx.com>
drivers/gpu/drm/xilinx/xilinx_drm_dsi.c

index 9aec8261cdc550c86fd0fcd0fef19409c33604f1..d3f9d9799bdd9707a8093606a0bf0f61b2af48da 100644 (file)
@@ -214,8 +214,8 @@ static void xilinx_dsi_set_display_mode(struct xilinx_dsi *dsi)
                                XDSI_PCR_VIDEOMODE_SHIFT);
 
        /* configure the HSA value only if non_burst_sync_pluse video mode */
-       if ((!video_mode) &
-               (dsi->mode_flags & MIPI_DSI_MODE_VIDEO_SYNC_PULSE)) {
+       if ((!video_mode) &&
+           (dsi->mode_flags & MIPI_DSI_MODE_VIDEO_SYNC_PULSE)) {
                reg = XDSI_TIME1_HSA(vm->hsync_len);
                xilinx_dsi_writel(dsi->iomem, XDSI_TIME1, reg);
        }
@@ -397,7 +397,6 @@ static int xilinx_dsi_connector_dpms(struct drm_connector *connector,
 {
        struct xilinx_dsi *dsi = connector_to_dsi(connector);
        int ret;
-       bool panel_on = 0;
 
        dev_dbg(dsi->dev, "connector dpms state: %d\n", mode);
 
@@ -788,7 +787,7 @@ static const struct of_device_id xilinx_dsi_of_match[] = {
 };
 MODULE_DEVICE_TABLE(of, xilinx_dsi_of_match);
 
-struct platform_driver dsi_driver = {
+static struct platform_driver dsi_driver = {
        .probe = xilinx_dsi_probe,
        .remove = xilinx_dsi_remove,
        .driver = {