Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1190903

Re: [PATCH v2] Staging: fbtft: Add support for the Ultrachip UC1611 LCD controller

From Noralf Trønnes <noralf@tronnes.org>
Newsgroups linux.kernel
Subject Re: [PATCH v2] Staging: fbtft: Add support for the Ultrachip UC1611 LCD controller
Date 2015-07-23 15:10 +0200
Message-ID <pPoq7-7Bj-41@gated-at.bofh.it> (permalink)
References <pM7Yt-2gQ-5@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


Den 14.07.2015 14:59, skrev Henri Chain:
> This is a driver chip for 240x160 4-bit greyscale LCDs.
> It is capable of 4-wire (8 bit) or 3-wire (9 bit) SPI that have both been
> tested. (It also has a 6800 or 8080-style parallel interface, but I have
> not included support for it.)
>
> Signed-off-by: Henri Chain <henri.chain@eleves.ec-nantes.fr>
> ---

> diff --git a/drivers/staging/fbtft/fb_uc1611.c b/drivers/staging/fbtft/fb_uc1611.c

> +static int init_display(struct fbtft_par *par)
> +{
> +	int ret;
> +
> +	fbtft_par_dbg(DEBUG_INIT_DISPLAY, par, "%s()\n", __func__);
> +
> +	/* Set CS active high */
> +	par->spi->mode |= SPI_CS_HIGH;

Why is this set here and not in fbtft_device along side SPI_MODE_3?

> +static int set_var(struct fbtft_par *par)
> +{
> +	fbtft_par_dbg(DEBUG_INIT_DISPLAY, par, "%s()\n", __func__);
> +
> +	/* par->info->fix.visual = FB_VISUAL_PSEUDOCOLOR; */

This comment can be removed I guess.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Back to linux.kernel | Previous | Next | Find similar | Unroll thread


Thread

Re: [PATCH v2] Staging: fbtft: Add support for the Ultrachip UC1611  LCD controller Noralf Trønnes <noralf@tronnes.org> - 2015-07-23 15:10 +0200

csiph-web