Received: by 2002:a05:6a10:22f:0:0:0:0 with SMTP id 15csp3207468pxk; Mon, 21 Sep 2020 07:54:06 -0700 (PDT) X-Google-Smtp-Source: ABdhPJxhia/v1TxMAW5dJeujKDhAIIL03rCqGTgikw4RM6saUVXVlK6b+UWfR2nOJUt+PGiTh/o5 X-Received: by 2002:a17:906:4755:: with SMTP id j21mr51161545ejs.228.1600700046205; Mon, 21 Sep 2020 07:54:06 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1600700046; cv=none; d=google.com; s=arc-20160816; b=SabGvagxxZYdESKuz/x4Fk/Y06LHWqieGFVIGBzpivplcuDXjwu8cBDMSTYcBR+kbe a6Ki7io+XkWrgWGJBKQQi6gvTiXT8GCDK3U5fi+cWQsy4e+Ii+Ki9aGmDmn7RXNmqoCq 80SnhL+plYivMhPnWiikfLU4mZfeVifFWKBGv/fJonl2UOCL29piaPwHBA8iGNfSpKri yhVbNp72Tru9PCB0HaDrcCWIiSsCQrjnAgGP5bXbjE8aMiEAeW2PslWuvSppxWouI4aC /bi30wKAnGRsbJhNE8/9RCRezDub+YAL+HLimg0+pdovJ+s8oHdjN3DvFrB55wohK9S8 IKRw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from; bh=YlbW/4DOBihJSevRSA5jlm0JpAqgHRQ9nzWTGVoFbjQ=; b=EGW7ScgyhHHBacjdHnx0LhXQeJkTnDSUeX7XCEz4XTbO9QkNz5b2L060govBHzNhzb ZhcT8Crry0KAI5RNyVXv80c8n7AqaUVdzgBaHgAhVhmgngUPbnxH3O1/Dr9xazeSy5hp Z4D4QpKxvcq+CC9LxLpAm+n4EY6BQQ2lCh+BEXQ48FQncugwOXkNT3DISRyG5UjUMNZV ZNrek3y49AoVjNdiQ5pOiKCaTPS6HKjCMEo9LqZB6hl74lK0cZvHDHjY2BE15x0UG7ES fFp9LCBqgSlwVSlWXOOm5A4Y0BFIK0c+U7oyq4xJx1pfyHwUcMD7qfCCgMueBnRPdOrZ E4lQ== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id s11si8318160eju.295.2020.09.21.07.53.41; Mon, 21 Sep 2020 07:54:06 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727759AbgIUOti (ORCPT + 99 others); Mon, 21 Sep 2020 10:49:38 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:44352 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727870AbgIUOsF (ORCPT ); Mon, 21 Sep 2020 10:48:05 -0400 Received: from smtp1.goneo.de (smtp1.goneo.de [IPv6:2001:1640:5::8:30]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 093ACC0613CF for ; Mon, 21 Sep 2020 07:48:05 -0700 (PDT) Received: from localhost (localhost [127.0.0.1]) by smtp1.goneo.de (Postfix) with ESMTP id E143923F160; Mon, 21 Sep 2020 16:48:03 +0200 (CEST) X-Virus-Scanned: by goneo X-Spam-Flag: NO X-Spam-Score: -3.002 X-Spam-Level: X-Spam-Status: No, score=-3.002 tagged_above=-999 tests=[ALL_TRUSTED=-1, AWL=-0.102, BAYES_00=-1.9] autolearn=ham Received: from smtp1.goneo.de ([127.0.0.1]) by localhost (smtp1.goneo.de [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id UWz6Yn5YK0mi; Mon, 21 Sep 2020 16:48:02 +0200 (CEST) Received: from lem-wkst-02.lemonage.de. (hq.lemonage.de [87.138.178.34]) by smtp1.goneo.de (Postfix) with ESMTPA id E9B8823F029; Mon, 21 Sep 2020 16:48:01 +0200 (CEST) From: poeschel@lemonage.de To: Miguel Ojeda Sandonis , Willy Tarreau , Ksenija Stanojevic , linux-kernel@vger.kernel.org (open list) Cc: Lars Poeschel Subject: [PATCH v2 16/32] auxdisplay: Implement hd44780_common_display_shift Date: Mon, 21 Sep 2020 16:46:28 +0200 Message-Id: <20200921144645.2061313-17-poeschel@lemonage.de> X-Mailer: git-send-email 2.28.0 In-Reply-To: <20200921144645.2061313-1-poeschel@lemonage.de> References: <20191016082430.5955-1-poeschel@lemonage.de> <20200921144645.2061313-1-poeschel@lemonage.de> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Lars Poeschel Implement a hd44780_common_display_shift function for hd44780 drivers to use. charlcd uses this through its ops function pointer. Signed-off-by: Lars Poeschel --- drivers/auxdisplay/charlcd.c | 10 ++-------- drivers/auxdisplay/charlcd.h | 2 ++ drivers/auxdisplay/hd44780.c | 2 ++ drivers/auxdisplay/hd44780_common.c | 16 ++++++++++++++++ drivers/auxdisplay/hd44780_common.h | 2 ++ drivers/auxdisplay/panel.c | 3 +++ 6 files changed, 27 insertions(+), 8 deletions(-) diff --git a/drivers/auxdisplay/charlcd.c b/drivers/auxdisplay/charlcd.c index 85747e13bf48..e48287197732 100644 --- a/drivers/auxdisplay/charlcd.c +++ b/drivers/auxdisplay/charlcd.c @@ -31,10 +31,6 @@ #define LCD_CMD_CURSOR_ON 0x02 /* Set cursor on */ #define LCD_CMD_BLINK_ON 0x01 /* Set blink on */ -#define LCD_CMD_SHIFT 0x10 /* Shift cursor/display */ -#define LCD_CMD_DISPLAY_SHIFT 0x08 /* Shift display instead of cursor */ -#define LCD_CMD_SHIFT_RIGHT 0x04 /* Shift display/cursor to the right */ - #define LCD_CMD_FUNCTION_SET 0x20 /* Set function */ #define LCD_CMD_DATA_LEN_8BITS 0x10 /* Set data length to 8 bits */ #define LCD_CMD_TWO_LINES 0x08 /* Set to two display lines */ @@ -303,13 +299,11 @@ static inline int handle_lcd_special_code(struct charlcd *lcd) processed = 1; break; case 'L': /* shift display left */ - hdc->write_cmd(hdc, LCD_CMD_SHIFT | LCD_CMD_DISPLAY_SHIFT); + lcd->ops->shift_display(lcd, CHARLCD_SHIFT_LEFT); processed = 1; break; case 'R': /* shift display right */ - hdc->write_cmd(hdc, - LCD_CMD_SHIFT | LCD_CMD_DISPLAY_SHIFT | - LCD_CMD_SHIFT_RIGHT); + lcd->ops->shift_display(lcd, CHARLCD_SHIFT_RIGHT); processed = 1; break; case 'k': { /* kill end of line */ diff --git a/drivers/auxdisplay/charlcd.h b/drivers/auxdisplay/charlcd.h index 0a340beacd3e..6fd4a77f5794 100644 --- a/drivers/auxdisplay/charlcd.h +++ b/drivers/auxdisplay/charlcd.h @@ -59,6 +59,7 @@ struct charlcd { * function. * @init_display: Initialize the display. * @shift_cursor: Shift cursor left or right one position. + * @shift_display: Shift whole display content left or right. */ struct charlcd_ops { void (*clear_fast)(struct charlcd *lcd); @@ -69,6 +70,7 @@ struct charlcd_ops { int (*clear_display)(struct charlcd *lcd); int (*init_display)(struct charlcd *lcd); int (*shift_cursor)(struct charlcd *lcd, enum charlcd_shift_dir dir); + int (*shift_display)(struct charlcd *lcd, enum charlcd_shift_dir dir); }; void charlcd_backlight(struct charlcd *lcd, enum charlcd_onoff on); diff --git a/drivers/auxdisplay/hd44780.c b/drivers/auxdisplay/hd44780.c index 6ffeeb3fb0d0..a3b18bbc2e4a 100644 --- a/drivers/auxdisplay/hd44780.c +++ b/drivers/auxdisplay/hd44780.c @@ -132,6 +132,7 @@ static const struct charlcd_ops hd44780_ops_gpio8 = { .clear_display = hd44780_common_clear_display, .init_display = hd44780_common_init_display, .shift_cursor = hd44780_common_shift_cursor, + .shift_display = hd44780_common_shift_display, }; /* Send a command to the LCD panel in 4 bit GPIO mode */ @@ -181,6 +182,7 @@ static const struct charlcd_ops hd44780_ops_gpio4 = { .clear_display = hd44780_common_clear_display, .init_display = hd44780_common_init_display, .shift_cursor = hd44780_common_shift_cursor, + .shift_display = hd44780_common_shift_display, }; static int hd44780_probe(struct platform_device *pdev) diff --git a/drivers/auxdisplay/hd44780_common.c b/drivers/auxdisplay/hd44780_common.c index b2ed00635766..318ad81a39ed 100644 --- a/drivers/auxdisplay/hd44780_common.c +++ b/drivers/auxdisplay/hd44780_common.c @@ -18,6 +18,7 @@ #define LCD_CMD_BLINK_ON 0x01 /* Set blink on */ #define LCD_CMD_SHIFT 0x10 /* Shift cursor/display */ +#define LCD_CMD_DISPLAY_SHIFT 0x08 /* Shift display instead of cursor */ #define LCD_CMD_SHIFT_RIGHT 0x04 /* Shift display/cursor to the right */ #define LCD_CMD_FUNCTION_SET 0x20 /* Set function */ @@ -179,6 +180,21 @@ int hd44780_common_shift_cursor(struct charlcd *lcd, enum charlcd_shift_dir dir) } EXPORT_SYMBOL_GPL(hd44780_common_shift_cursor); +int hd44780_common_shift_display(struct charlcd *lcd, + enum charlcd_shift_dir dir) +{ + struct hd44780_common *hdc = lcd->drvdata; + + if (dir == CHARLCD_SHIFT_LEFT) + hdc->write_cmd(hdc, LCD_CMD_SHIFT | LCD_CMD_DISPLAY_SHIFT); + else if (dir == CHARLCD_SHIFT_RIGHT) + hdc->write_cmd(hdc, LCD_CMD_SHIFT | LCD_CMD_DISPLAY_SHIFT | + LCD_CMD_SHIFT_RIGHT); + + return 0; +} +EXPORT_SYMBOL_GPL(hd44780_common_shift_display); + struct hd44780_common *hd44780_common_alloc(void) { struct hd44780_common *hd; diff --git a/drivers/auxdisplay/hd44780_common.h b/drivers/auxdisplay/hd44780_common.h index 432fc37213cf..7badb96cb8a9 100644 --- a/drivers/auxdisplay/hd44780_common.h +++ b/drivers/auxdisplay/hd44780_common.h @@ -22,5 +22,7 @@ int hd44780_common_clear_display(struct charlcd *lcd); int hd44780_common_init_display(struct charlcd *lcd); int hd44780_common_shift_cursor(struct charlcd *lcd, enum charlcd_shift_dir dir); +int hd44780_common_shift_display(struct charlcd *lcd, + enum charlcd_shift_dir dir); struct hd44780_common *hd44780_common_alloc(void); diff --git a/drivers/auxdisplay/panel.c b/drivers/auxdisplay/panel.c index 4275223080e8..7ebbfb338518 100644 --- a/drivers/auxdisplay/panel.c +++ b/drivers/auxdisplay/panel.c @@ -880,6 +880,7 @@ static const struct charlcd_ops charlcd_serial_ops = { .clear_display = hd44780_common_clear_display, .init_display = hd44780_common_init_display, .shift_cursor = hd44780_common_shift_cursor, + .shift_display = hd44780_common_shift_display, }; static const struct charlcd_ops charlcd_parallel_ops = { @@ -890,6 +891,7 @@ static const struct charlcd_ops charlcd_parallel_ops = { .clear_display = hd44780_common_clear_display, .init_display = hd44780_common_init_display, .shift_cursor = hd44780_common_shift_cursor, + .shift_display = hd44780_common_shift_display, }; static const struct charlcd_ops charlcd_tilcd_ops = { @@ -900,6 +902,7 @@ static const struct charlcd_ops charlcd_tilcd_ops = { .clear_display = hd44780_common_clear_display, .init_display = hd44780_common_init_display, .shift_cursor = hd44780_common_shift_cursor, + .shift_display = hd44780_common_shift_display, }; /* initialize the LCD driver */ -- 2.28.0