From patchwork Sat Jul 29 19:17:46 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: David Lechner X-Patchwork-Id: 9870097 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id DAC866035E for ; Sat, 29 Jul 2017 19:20:51 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id CA874286A8 for ; Sat, 29 Jul 2017 19:20:51 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id BE938287A9; Sat, 29 Jul 2017 19:20:51 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-1.9 required=2.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,RCVD_IN_DNSWL_NONE autolearn=unavailable version=3.3.1 Received: from bombadil.infradead.org (bombadil.infradead.org [65.50.211.133]) (using TLSv1.2 with cipher AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id E3625286A8 for ; Sat, 29 Jul 2017 19:20:50 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:MIME-Version:Cc:List-Subscribe: List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id:References: In-Reply-To:Message-Id:Date:Subject:To:From:Reply-To:Content-ID: Content-Description:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc :Resent-Message-ID:List-Owner; bh=hQnvLMu/PVnP6p5XUWMnL7tN+Gx+DdJdAXDr8Y5dhDE=; b=MhcngTlxMGGQvtEN8UvK5yrYb0 4sLidwnnnxGLQZrLxD725uiq4FWEauHETgupYVp4y6jF/8cFJPzYWF2tiCe0XuhEFjA8RWBbOcRee 7K78UihXVc8QSFaaCKc99zkyCyjDcELix3tsH35Ya//pQ1VZ9eBPyA77wDFFAySmS7bUHwqpR9JQl csq0E/cmaewlVXOfHg6whQZ2GAGeRizPMztgyMHlzGS1grpfXzHQZXHBdSvJOMN72hqlbBaX343JU nBGGHkrI3FbEnp8Bdoz1flG3W7WPmlCpWCOGynvmOvPosItz2HdPW/a2yeX7JuWnNOA4UPH5WqheP x2bWbsrA==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux)) id 1dbXI6-0007JC-JR; Sat, 29 Jul 2017 19:20:46 +0000 Received: from vern.gendns.com ([206.190.152.46]) by bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux)) id 1dbXGo-0005Tb-Vw for linux-arm-kernel@lists.infradead.org; Sat, 29 Jul 2017 19:19:32 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lechnology.com; s=default; h=References:In-Reply-To:Message-Id:Date:Subject :Cc:To:From:Sender:Reply-To:MIME-Version:Content-Type: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Id: List-Help:List-Unsubscribe:List-Subscribe:List-Post:List-Owner:List-Archive; bh=6BdOauNN8ldmi5Ug4izzpqX/En0kdTQ3mdYQUx0/ZDI=; b=VsZ6hf0gqfsS7kpw2/JTpPiwb K1m4k/VuXVHMkGAKUuQJip4uZR2F9sRbNrMCEJPfsJ8hA3etp2JREdY03zAu3ZcwCY1akFht0Bcwt zO5cInV1+Kbg1LHPHbxR1C8yYE8gS8Yd4NN7L4OVZMX+g+s8Ltwr+gmXW43tHDnnGnBwyoAhEDg4C zJFsqKYDtqmphtB8wI9hf0WidrOUgAuuQuqjIdn/j4TvovYqU2C1S5FBRiA6cTw3lCz930NLtrRtC tezq94KxAvZBIjphI+vbJWyoQmyDO3JGlI4pITDGlwk8fATtbhnyO2dWknSMSJVk5lIr8wJ8RGaX/ rIFYPV/Wg==; Received: from 108-198-5-147.lightspeed.okcbok.sbcglobal.net ([108.198.5.147]:58148 helo=freyr.lechnology.com) by vern.gendns.com with esmtpsa (TLSv1.2:ECDHE-RSA-AES128-SHA256:128) (Exim 4.89) (envelope-from ) id 1dbXCv-000DrP-00; Sat, 29 Jul 2017 15:15:25 -0400 From: David Lechner To: dri-devel@lists.freedesktop.org, devicetree@vger.kernel.org Subject: [PATCH 2/6] drm/tinydrm: add helpers for ST7586 controllers Date: Sat, 29 Jul 2017 14:17:46 -0500 Message-Id: <1501355870-13960-3-git-send-email-david@lechnology.com> X-Mailer: git-send-email 2.7.4 In-Reply-To: <1501355870-13960-1-git-send-email-david@lechnology.com> References: <1501355870-13960-1-git-send-email-david@lechnology.com> X-AntiAbuse: This header was added to track abuse, please include it with any abuse report X-AntiAbuse: Primary Hostname - vern.gendns.com X-AntiAbuse: Original Domain - lists.infradead.org X-AntiAbuse: Originator/Caller UID/GID - [47 12] / [47 12] X-AntiAbuse: Sender Address Domain - lechnology.com X-Get-Message-Sender-Via: vern.gendns.com: authenticated_id: davidmain+lechnology.com/only user confirmed/virtual account not confirmed X-Authenticated-Sender: vern.gendns.com: davidmain@lechnology.com X-Source: X-Source-Args: X-Source-Dir: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20170729_121927_296265_8409602E X-CRM114-Status: GOOD ( 16.85 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Mark Rutland , linux-fbdev@vger.kernel.org, David Lechner , Kevin Hilman , David Airlie , Sekhar Nori , linux-kernel@vger.kernel.org, Rob Herring , =?UTF-8?q?Noralf=20Tr=C3=B8nnes?= , linux-arm-kernel@lists.infradead.org MIME-Version: 1.0 Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+patchwork-linux-arm=patchwork.kernel.org@lists.infradead.org X-Virus-Scanned: ClamAV using ClamSMTP This adds helper functions and support for ST7586 controllers. These controllers have an unusual memory layout where 3 pixels are packed into 1 byte. +-------+-----------------+ | bit | 7 6 5 4 3 2 1 0 | +-------+-----------------+ | pixel | 0 0 0 1 1 1 2 2 | +-------+-----------------+ So, there are a nuber of places in the tinydrm pipline where this format needs to be taken into consideration. Signed-off-by: David Lechner --- drivers/gpu/drm/tinydrm/core/tinydrm-helpers.c | 148 +++++++++++++++++++++++++ drivers/gpu/drm/tinydrm/mipi-dbi.c | 96 ++++++++++++---- include/drm/tinydrm/tinydrm-helpers.h | 6 + include/video/mipi_display.h | 2 + 4 files changed, 232 insertions(+), 20 deletions(-) diff --git a/drivers/gpu/drm/tinydrm/core/tinydrm-helpers.c b/drivers/gpu/drm/tinydrm/core/tinydrm-helpers.c index d4cda33..4a36441 100644 --- a/drivers/gpu/drm/tinydrm/core/tinydrm-helpers.c +++ b/drivers/gpu/drm/tinydrm/core/tinydrm-helpers.c @@ -181,6 +181,154 @@ void tinydrm_xrgb8888_to_rgb565(u16 *dst, void *vaddr, EXPORT_SYMBOL(tinydrm_xrgb8888_to_rgb565); /** + * tinydrm_rgb565_to_st7586 - Convert RGB565 to ST7586 clip buffer + * @dst: ST7586 destination buffer + * @vaddr: RGB565 source buffer + * @fb: DRM framebuffer + * @clip: Clip rectangle area to copy + * @swap: Swap bytes + * + * The ST7586 controller has a non-standard pixel layout. It is 2-bit grayscale + * (or 1-bit monochrome) packed into 3 pixels per byte. + * + * Note: the @clip x values are modified! They are aligned to the 3 pixel + * boundtry and divied by 3 to get the starting and ending byte (+ 1) in the + * @dst buffer. + */ +void tinydrm_rgb565_to_st7586(u8 *dst, void *vaddr, + struct drm_framebuffer *fb, + struct drm_clip_rect *clip) +{ + size_t len; + unsigned int x, y; + u16 *src, *buf; + u8 val; + + /* 3 pixels per byte, so align to dst byte */ + clip->x1 -= clip->x1 % 3; + clip->x2 = (clip->x2 + 2) / 3 * 3; + len = (clip->x2 - clip->x1) * sizeof(u16); + + buf = kmalloc(len, GFP_KERNEL); + if (!buf) + return; + + for (y = clip->y1; y < clip->y2; y++) { + src = vaddr + (y * fb->pitches[0]); + src += clip->x1; + memcpy(buf, src, len); + src = buf; + for (x = clip->x1; x < clip->x2; x += 3) { + /* + * TODO: There is probably a better algorithm to get + * a better downsampling. + * If red or green is > 50%, set the high bit, which is + * bright gray. If blue is greater than 50%, set the + * low bit, which is dark gray. + */ + val = ((*src & 0x8000) >> 8) | + ((*src & 0x0400) >> 3) | + ((*src & 0x0010) << 2); + if (val & 0xC0) + val |= 0x20; + src++; + val |= ((*src & 0x8000) >> 11) | + ((*src & 0x0400) >> 6) | + ((*src & 0x0010) >> 1); + if (val & 0x18) + val |= 0x04; + src++; + val |= ((*src & 0x8000) >> 14) | + ((*src & 0x0400) >> 9) | + ((*src & 0x0010) >> 4); + src++; + *dst++ = ~val; + } + } + + /* now adjust the clip so it applies to dst */ + clip->x1 /= 3; + clip->x2 /= 3; + + kfree(buf); +} +EXPORT_SYMBOL(tinydrm_rgb565_to_st7586); + +/** + * tinydrm_xrgb8888_to_st7586 - Convert XRGB8888 to ST7586 clip buffer + * @dst: ST7586 destination buffer + * @vaddr: XRGB8888 source buffer + * @fb: DRM framebuffer + * @clip: Clip rectangle area to copy + * @swap: Swap bytes + * + * The ST7586 controller has a non-standard pixel layout. It is 2-bit grayscale + * (or 1-bit monochrome) packed into 3 pixels per byte. + * + * Note: the @clip x values are modified! They are aligned to the 3 pixel + * boundtry and divied by 3 to get the starting and ending byte (+ 1) in the + * @dst buffer. + */ +void tinydrm_xrgb8888_to_st7586(u8 *dst, void *vaddr, + struct drm_framebuffer *fb, + struct drm_clip_rect *clip) +{ + size_t len; + unsigned int x, y; + u32 *src, *buf; + u8 val; + + /* 3 pixels per byte, so align to dst byte */ + clip->x1 -= clip->x1 % 3; + clip->x2 = (clip->x2 + 2) / 3 * 3; + len = (clip->x2 - clip->x1) * sizeof(u32); + + buf = kmalloc(len, GFP_KERNEL); + if (!buf) + return; + + for (y = clip->y1; y < clip->y2; y++) { + src = vaddr + (y * fb->pitches[0]); + src += clip->x1; + memcpy(buf, src, len); + src = buf; + for (x = clip->x1; x < clip->x2; x += 3) { + /* + * TODO: There is probably a better algorithm to get + * a better downsampling. + * If red or green is > 50%, set the high bit, which is + * bright gray. If blue is greater than 50%, set the + * low bit, which is dark gray. + */ + val = ((*src & 0x00800000) >> 16) | + ((*src & 0x00008000) >> 8) | + ((*src & 0x00000080) >> 1); + if (val & 0xC0) + val |= 0x20; + src++; + val |= ((*src & 0x00800000) >> 19) | + ((*src & 0x00008000) >> 11) | + ((*src & 0x00000080) >> 4); + if (val & 0x18) + val |= 0x04; + src++; + val |= ((*src & 0x00800000) >> 22) | + ((*src & 0x00008000) >> 14) | + ((*src & 0x00000080) >> 7); + src++; + *dst++ = ~val; + } + } + + /* now adjust the clip so it applies to dst */ + clip->x1 /= 3; + clip->x2 /= 3; + + kfree(buf); +} +EXPORT_SYMBOL(tinydrm_xrgb8888_to_st7586); + +/** * tinydrm_of_find_backlight - Find backlight device in device-tree * @dev: Device * diff --git a/drivers/gpu/drm/tinydrm/mipi-dbi.c b/drivers/gpu/drm/tinydrm/mipi-dbi.c index 7d49366..c8fb622 100644 --- a/drivers/gpu/drm/tinydrm/mipi-dbi.c +++ b/drivers/gpu/drm/tinydrm/mipi-dbi.c @@ -154,7 +154,8 @@ int mipi_dbi_command_buf(struct mipi_dbi *mipi, u8 cmd, u8 *data, size_t len) EXPORT_SYMBOL(mipi_dbi_command_buf); static int mipi_dbi_buf_copy(void *dst, struct drm_framebuffer *fb, - struct drm_clip_rect *clip, bool swap) + struct drm_clip_rect *clip, + enum mipi_dcs_pixel_format pixel_fmt, bool swap) { struct drm_gem_cma_object *cma_obj = drm_fb_cma_get_gem_obj(fb, 0); struct dma_buf_attachment *import_attach = cma_obj->base.import_attach; @@ -169,21 +170,45 @@ static int mipi_dbi_buf_copy(void *dst, struct drm_framebuffer *fb, return ret; } - switch (fb->format->format) { - case DRM_FORMAT_RGB565: - if (swap) - tinydrm_swab16(dst, src, fb, clip); - else - tinydrm_memcpy(dst, src, fb, clip); + switch (pixel_fmt) { + case MIPI_DCS_PIXEL_FMT_16BIT: + switch (fb->format->format) { + case DRM_FORMAT_RGB565: + if (swap) + tinydrm_swab16(dst, src, fb, clip); + else + tinydrm_memcpy(dst, src, fb, clip); + break; + case DRM_FORMAT_XRGB8888: + tinydrm_xrgb8888_to_rgb565(dst, src, fb, clip, swap); + break; + default: + ret = -EINVAL; + break; + } break; - case DRM_FORMAT_XRGB8888: - tinydrm_xrgb8888_to_rgb565(dst, src, fb, clip, swap); + case MIPI_DCS_PIXEL_FMT_ST7586_332: + switch (fb->format->format) { + case DRM_FORMAT_RGB565: + tinydrm_rgb565_to_st7586(dst, src, fb, clip); + break; + case DRM_FORMAT_XRGB8888: + tinydrm_xrgb8888_to_st7586(dst, src, fb, clip); + break; + default: + ret = -EINVAL; + break; + } break; default: + ret = -EINVAL; + break; + } + if (ret) { dev_err_once(fb->dev->dev, "Format is not supported: %s\n", - drm_get_format_name(fb->format->format, - &format_name)); - return -EINVAL; + drm_get_format_name(fb->format->format, + &format_name)); + return ret; } if (import_attach) @@ -204,8 +229,9 @@ static int mipi_dbi_fb_dirty(struct drm_framebuffer *fb, bool swap = mipi->swap_bytes; struct drm_clip_rect clip; int ret = 0; - bool full; + bool full, pixel_fmt_match; void *tr; + size_t len; mutex_lock(&tdev->dirty_lock); @@ -218,20 +244,24 @@ static int mipi_dbi_fb_dirty(struct drm_framebuffer *fb, full = tinydrm_merge_clips(&clip, clips, num_clips, flags, fb->width, fb->height); + pixel_fmt_match = !swap && fb->format->format == DRM_FORMAT_RGB565 && + mipi->pixel_fmt == MIPI_DCS_PIXEL_FMT_16BIT; DRM_DEBUG("Flushing [FB:%d] x1=%u, x2=%u, y1=%u, y2=%u\n", fb->base.id, clip.x1, clip.x2, clip.y1, clip.y2); - if (!mipi->dc || !full || swap || - fb->format->format == DRM_FORMAT_XRGB8888) { + if (!mipi->dc || !full || !pixel_fmt_match) { tr = mipi->tx_buf; - ret = mipi_dbi_buf_copy(mipi->tx_buf, fb, &clip, swap); + ret = mipi_dbi_buf_copy(mipi->tx_buf, fb, &clip, + mipi->pixel_fmt, swap); if (ret) goto out_unlock; } else { tr = cma_obj->vaddr; } + /* NB: mipi_dbi_buf_copy() may modify clip! */ + mipi_dbi_command(mipi, MIPI_DCS_SET_COLUMN_ADDRESS, (clip.x1 >> 8) & 0xFF, clip.x1 & 0xFF, (clip.x2 >> 8) & 0xFF, (clip.x2 - 1) & 0xFF); @@ -239,8 +269,16 @@ static int mipi_dbi_fb_dirty(struct drm_framebuffer *fb, (clip.y1 >> 8) & 0xFF, clip.y1 & 0xFF, (clip.y2 >> 8) & 0xFF, (clip.y2 - 1) & 0xFF); - ret = mipi_dbi_command_buf(mipi, MIPI_DCS_WRITE_MEMORY_START, tr, - (clip.x2 - clip.x1) * (clip.y2 - clip.y1) * 2); + len = (clip.x2 - clip.x1) * (clip.y2 - clip.y1); + switch (mipi->pixel_fmt) { + case MIPI_DCS_PIXEL_FMT_16BIT: + len *= sizeof(u16); + break; + default: + break; + } + + ret = mipi_dbi_command_buf(mipi, MIPI_DCS_WRITE_MEMORY_START, tr, len); out_unlock: mutex_unlock(&tdev->dirty_lock); @@ -288,7 +326,20 @@ static void mipi_dbi_blank(struct mipi_dbi *mipi) struct drm_device *drm = mipi->tinydrm.drm; u16 height = drm->mode_config.min_height; u16 width = drm->mode_config.min_width; - size_t len = width * height * 2; + size_t len; + + switch (mipi->pixel_fmt) { + case MIPI_DCS_PIXEL_FMT_16BIT: + len = width * height * sizeof(u16); + break; + case MIPI_DCS_PIXEL_FMT_ST7586_332: + width = (width + 2) / 3; + len = width * height; + break; + default: + /* unsupported pixel format */ + return; + } memset(mipi->tx_buf, 0, len); @@ -367,6 +418,10 @@ int mipi_dbi_init(struct device *dev, struct mipi_dbi *mipi, case MIPI_DCS_PIXEL_FMT_16BIT: bufsize = mode->vdisplay * mode->hdisplay * sizeof(u16); break; + case MIPI_DCS_PIXEL_FMT_ST7586_332: + /* 3 pixels per byte */ + bufsize = (mode->vdisplay + 2) / 3 * mode->hdisplay; + break; default: DRM_ERROR("Pixel format is not supported\n"); return -EINVAL; @@ -776,7 +831,8 @@ static int mipi_dbi_typec3_command(struct mipi_dbi *mipi, u8 cmd, if (ret || !num) return ret; - if (cmd == MIPI_DCS_WRITE_MEMORY_START && !mipi->swap_bytes) + if (cmd == MIPI_DCS_WRITE_MEMORY_START && !mipi->swap_bytes && + mipi->pixel_fmt != MIPI_DCS_PIXEL_FMT_ST7586_332) bpw = 16; gpiod_set_value_cansleep(mipi->dc, 1); diff --git a/include/drm/tinydrm/tinydrm-helpers.h b/include/drm/tinydrm/tinydrm-helpers.h index 9b9b6cf..94792fb 100644 --- a/include/drm/tinydrm/tinydrm-helpers.h +++ b/include/drm/tinydrm/tinydrm-helpers.h @@ -43,6 +43,12 @@ void tinydrm_swab16(u16 *dst, void *vaddr, struct drm_framebuffer *fb, void tinydrm_xrgb8888_to_rgb565(u16 *dst, void *vaddr, struct drm_framebuffer *fb, struct drm_clip_rect *clip, bool swap); +void tinydrm_rgb565_to_st7586(u8 *dst, void *vaddr, + struct drm_framebuffer *fb, + struct drm_clip_rect *clip); +void tinydrm_xrgb8888_to_st7586(u8 *dst, void *vaddr, + struct drm_framebuffer *fb, + struct drm_clip_rect *clip); struct backlight_device *tinydrm_of_find_backlight(struct device *dev); int tinydrm_enable_backlight(struct backlight_device *backlight); diff --git a/include/video/mipi_display.h b/include/video/mipi_display.h index 84b70cd..1c134c4 100644 --- a/include/video/mipi_display.h +++ b/include/video/mipi_display.h @@ -135,6 +135,8 @@ enum mipi_dcs_pixel_format { MIPI_DCS_PIXEL_FMT_12BIT = 3, MIPI_DCS_PIXEL_FMT_8BIT = 2, MIPI_DCS_PIXEL_FMT_3BIT = 1, + /* non-standard format packing 1 or 2bpp in 3:3:2 bits */ + MIPI_DCS_PIXEL_FMT_ST7586_332 = -1, }; #endif