On Tue, 2011-03-01 at 06:32 -0600, Taneja, Archit wrote:
> Introduce functions which request and release VC's. This will be used in panel
> drivers in their probes.
>
> omap_dsi_request_vc() takes in the pointer to the omap_dss_device, the VC_ID
> parameter which goes into the header of the DSI packets, and returns a Virtual
> channel number (or virtual channel register set) which it can use.
>
> omap_dsi_set_vc_id() takes the omap_dss_device pointer, the Virtual Channel
> number and the VC_ID that needs to be set for the specifed Virtual Channel.
>
> omap_dsi_releae_vc() takes the omap_dss_device pointer and the Virtual Channel
> number that needs to be made free.
>
> Initialisation of VC parameters is done in dsi_init().
>
> Signed-off-by: Archit Taneja <arc...@ti.com>
> ---
> arch/arm/plat-omap/include/plat/display.h | 3 +
> drivers/video/omap2/dss/dsi.c | 58 ++++++++++++++++++++++++++--
> 2 files changed, 56 insertions(+), 5 deletions(-)
>
> diff --git a/arch/arm/plat-omap/include/plat/display.h
> b/arch/arm/plat-omap/include/plat/display.h
> index d45f107..18c103d 100644
> --- a/arch/arm/plat-omap/include/plat/display.h
> +++ b/arch/arm/plat-omap/include/plat/display.h
> @@ -560,6 +560,9 @@ int omap_dsi_update(struct omap_dss_device *dssdev,
> int channel,
> u16 x, u16 y, u16 w, u16 h,
> void (*callback)(int, void *), void *data);
> +int omap_dsi_request_vc(struct omap_dss_device *dssdev, int *channel);
> +int omap_dsi_set_vc_id(struct omap_dss_device *dssdev, int channel, int
> vc_id);
> +void omap_dsi_release_vc(struct omap_dss_device *dssdev, int channel);
>
> int omapdss_dsi_display_enable(struct omap_dss_device *dssdev);
> void omapdss_dsi_display_disable(struct omap_dss_device *dssdev);
> diff --git a/drivers/video/omap2/dss/dsi.c b/drivers/video/omap2/dss/dsi.c
> index 6779785..7758c80 100644
> --- a/drivers/video/omap2/dss/dsi.c
> +++ b/drivers/video/omap2/dss/dsi.c
> @@ -233,6 +233,7 @@ static struct
> enum dsi_vc_mode mode;
> struct omap_dss_device *dssdev;
> enum fifo_size fifo_size;
> + int vc_id;
> } vc[4];
>
> struct mutex lock;
> @@ -1778,8 +1779,6 @@ static void dsi_vc_initial_config(int channel)
> r = FLD_MOD(r, 4, 23, 21); /* DMA_TX_REQ_NB = no dma */
>
> dsi_write_reg(DSI_VC_CTRL(channel), r);
> -
> - dsi.vc[channel].mode = DSI_VC_MODE_L4;
> }
>
> static int dsi_vc_config_l4(int channel)
> @@ -1986,7 +1985,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].vc_id << 6;
>
> val = FLD_VAL(data_id, 7, 0) | FLD_VAL(len, 23, 8) |
> FLD_VAL(ecc, 31, 24);
> @@ -2089,7 +2088,7 @@ static int dsi_vc_send_short(int channel, u8 data_type,
> u16 data, u8 ecc)
> return -EINVAL;
> }
>
> - data_id = data_type | channel << 6;
> + data_id = data_type | dsi.vc[channel].vc_id << 6;
>
> r = (data_id << 0) | (data << 8) | (ecc << 24);
>
> @@ -3264,6 +3263,48 @@ int dsi_init_display(struct omap_dss_device *dssdev)
> return 0;
> }
>
> +int omap_dsi_request_vc(struct omap_dss_device *dssdev, int *channel)
> +{
> + int i;
> +
> + for (i = 0; i < 4; i++) {
I think this could use ARRAY_SIZE().
> + if (!dsi.vc[i].dssdev) {
> + dsi.vc[i].dssdev = dssdev;
> + *channel = i;
> + return 0;
> + }
> + }
> +
> + DSSERR("cannot get VC for display %s", dssdev->name);
> + return -ENOSPC;
> +}
> +EXPORT_SYMBOL(omap_dsi_request_vc);
> +
> +int omap_dsi_set_vc_id(struct omap_dss_device *dssdev, int channel, int
> vc_id)
> +{
> + if (vc_id < 0 || vc_id > 4) {
> + DSSERR("VC ID out of range\n");
> + return -EINVAL;
> + }
> +
> + if (dsi.vc[channel].dssdev == dssdev) {
> + dsi.vc[channel].vc_id = vc_id;
> + return 0;
> + }
> +
> + return -EINVAL;
Perhaps a matter of taste, but I like to check the arguments, and return
if they are wrong, and do the work in the end. So:
if (vc_id ...)
return -EINVAL;
if (dsi.vc[channel]...)
return -EINVAL;
dsi.vc[channel].vc_id = vc_id;
return 0;
> +}
> +EXPORT_SYMBOL(omap_dsi_set_vc_id);
> +
> +void omap_dsi_release_vc(struct omap_dss_device *dssdev, int channel)
> +{
This and the one above are exported functions, so I think it would be
good to check the channel parameter.
> + if (dsi.vc[channel].dssdev == dssdev) {
> + dsi.vc[channel].dssdev = NULL;
> + dsi.vc[channel].vc_id = 0;
> + }
> +}
> +EXPORT_SYMBOL(omap_dsi_release_vc);
> +
> void dsi_wait_pll_hsdiv_dispc_active(void)
> {
> if (wait_for_bit_change(DSI_PLL_STATUS, 7, 1) != 1)
> @@ -3283,7 +3324,7 @@ void dsi_wait_pll_hsdiv_dsi_active(void)
> static int dsi_init(struct platform_device *pdev)
> {
> u32 rev;
> - int r;
> + int r, i;
> struct resource *dsi_mem;
>
> spin_lock_init(&dsi.errors_lock);
> @@ -3337,6 +3378,13 @@ static int dsi_init(struct platform_device *pdev)
> goto err2;
> }
>
> + /* DSI VCs initialization */
> + for (i = 0; i < 4; i++) {
This could also use ARRAY_SIZE()
Tomi
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
