OMAP: DSS2: DSI: configure all DSI VCs
authorTomi Valkeinen <tomi.valkeinen@nokia.com>
Wed, 16 Dec 2009 14:49:03 +0000 (16:49 +0200)
committerTomi Valkeinen <tomi.valkeinen@nokia.com>
Mon, 15 Feb 2010 13:14:42 +0000 (15:14 +0200)
Instead of configuring only VC0 to be usable, configure all four VCs
similarly. This is needed to utilize the other VCs.

Setting the FIFO sizes evenly for all VCs, regardless of how many VCs are
actually used, is not optimal. However, this affects only cases when
larger amounts of data are written or read via L4, meaning that normal use
cases are not affected.

At some point this could be optimized better to suit different use cases.

Signed-off-by: Tomi Valkeinen <tomi.valkeinen@nokia.com>
drivers/video/omap2/dss/dsi.c

index e003fe9ee9c41bb29e73025caf3dc2e590512105..131bd53b6b6d287e19f589f76632f67ba1692722 100644 (file)
@@ -224,7 +224,6 @@ static struct
                enum dsi_vc_mode mode;
                struct omap_dss_device *dssdev;
                enum fifo_size fifo_size;
-               int dest_per;   /* destination peripheral 0-3 */
        } vc[4];
 
        struct mutex lock;
@@ -2020,8 +2019,7 @@ static inline void dsi_vc_write_long_header(int channel, u8 data_type,
 
        WARN_ON(!dsi_bus_is_locked());
 
-       /*data_id = data_type | channel << 6; */
-       data_id = data_type | dsi.vc[channel].dest_per << 6;
+       data_id = data_type | channel << 6;
 
        val = FLD_VAL(data_id, 7, 0) | FLD_VAL(len, 23, 8) |
                FLD_VAL(ecc, 31, 24);
@@ -2127,7 +2125,7 @@ static int dsi_vc_send_short(int channel, u8 data_type, u16 data, u8 ecc)
                return -EINVAL;
        }
 
-       data_id = data_type | dsi.vc[channel].dest_per << 6;
+       data_id = data_type | channel << 6;
 
        r = (data_id << 0) | (data << 8) | (ecc << 24);
 
@@ -2529,15 +2527,15 @@ static int dsi_proto_config(struct omap_dss_device *dssdev)
        u32 r;
        int buswidth = 0;
 
-       dsi_config_tx_fifo(DSI_FIFO_SIZE_128,
-                       DSI_FIFO_SIZE_0,
-                       DSI_FIFO_SIZE_0,
-                       DSI_FIFO_SIZE_0);
+       dsi_config_tx_fifo(DSI_FIFO_SIZE_32,
+                       DSI_FIFO_SIZE_32,
+                       DSI_FIFO_SIZE_32,
+                       DSI_FIFO_SIZE_32);
 
-       dsi_config_rx_fifo(DSI_FIFO_SIZE_128,
-                       DSI_FIFO_SIZE_0,
-                       DSI_FIFO_SIZE_0,
-                       DSI_FIFO_SIZE_0);
+       dsi_config_rx_fifo(DSI_FIFO_SIZE_32,
+                       DSI_FIFO_SIZE_32,
+                       DSI_FIFO_SIZE_32,
+                       DSI_FIFO_SIZE_32);
 
        /* XXX what values for the timeouts? */
        dsi_set_stop_state_counter(1000);
@@ -2575,12 +2573,9 @@ static int dsi_proto_config(struct omap_dss_device *dssdev)
        dsi_write_reg(DSI_CTRL, r);
 
        dsi_vc_initial_config(0);
-
-       /* set all vc targets to peripheral 0 */
-       dsi.vc[0].dest_per = 0;
-       dsi.vc[1].dest_per = 0;
-       dsi.vc[2].dest_per = 0;
-       dsi.vc[3].dest_per = 0;
+       dsi_vc_initial_config(1);
+       dsi_vc_initial_config(2);
+       dsi_vc_initial_config(3);
 
        return 0;
 }
@@ -2846,9 +2841,6 @@ static void dsi_update_screen_dispc(struct omap_dss_device *dssdev,
        if (bytespf % packet_payload)
                total_len += (bytespf % packet_payload) + 1;
 
-       if (0)
-               dsi_vc_print_status(1);
-
        l = FLD_VAL(total_len, 23, 0); /* TE_SIZE */
        dsi_write_reg(DSI_VC_TE(channel), l);
 
@@ -3014,7 +3006,7 @@ static void dsi_handle_framedone(void)
        /* RX_FIFO_NOT_EMPTY */
        if (REG_GET(DSI_VC_CTRL(channel), 20, 20)) {
                DSSERR("Received error during frame transfer:\n");
-               dsi_vc_flush_receive_data(0);
+               dsi_vc_flush_receive_data(channel);
        }
 
 #ifdef CONFIG_OMAP2_DSS_FAKE_VSYNC
@@ -3268,6 +3260,9 @@ static int dsi_display_init_dsi(struct omap_dss_device *dssdev)
 
        /* enable interface */
        dsi_vc_enable(0, 1);
+       dsi_vc_enable(1, 1);
+       dsi_vc_enable(2, 1);
+       dsi_vc_enable(3, 1);
        dsi_if_enable(1);
        dsi_force_tx_stop_mode_io();