From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-7.7 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 10E85C433E0 for ; Thu, 28 Jan 2021 06:53:20 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id A7C2664DA1 for ; Thu, 28 Jan 2021 06:53:19 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229551AbhA1GxT (ORCPT ); Thu, 28 Jan 2021 01:53:19 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:58054 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229695AbhA1GxS (ORCPT ); Thu, 28 Jan 2021 01:53:18 -0500 Received: from mail-lj1-x22a.google.com (mail-lj1-x22a.google.com [IPv6:2a00:1450:4864:20::22a]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id A4152C061573; Wed, 27 Jan 2021 22:52:37 -0800 (PST) Received: by mail-lj1-x22a.google.com with SMTP id c18so5030340ljd.9; Wed, 27 Jan 2021 22:52:37 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to; bh=0f8ZmrzuRXVxsk0SyBLitP6T9N64hUKSEphOtPuOnnU=; b=s+g+9RsgEvfyHBAfUcVSsvVK/MPPAhyVtK61GUGybbJo3USYD2l22Xsq5KyeCpb0f/ l1xM12+qQnzcDbEEwE5NLLvrD1GkzD3o6ODrZXwCpwUiuHBGvcW4/gcxN7/QesOCbvRw qZzz213l28pva/a4EMbmsfli5alFUpJDcK/5ctFYL7S4mKgjcuLe+HBeNwfv31f6o592 eefP7qqqkvXrSvxcezl8+8YrbiZi7tbRSVFNpUxApCFR8x48GWr/a6UXw8+gm5GA7cXU PldFlVfnAt6ruHdg4Fzwg0D6SryNeBQPFysHDqBV57Tk611P+KNV7o5H2hSK6tRoFJHJ a9Ow== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to; bh=0f8ZmrzuRXVxsk0SyBLitP6T9N64hUKSEphOtPuOnnU=; b=EhO4h6f9ufAsPNPlPNgKh5/legr302nsrIU1SYQPba/v3R+56tWEdC5zt/SZ0rpv8G 6Rd86UtRTMSfppeBBT/7ZPH87aMaAi1twLJ2qFLbIMvNe+p//ZSoyP5wOym6IMHl5Pjl E6GVSx9kS92mSFNOaEZ1GQyUE7gpKHePPhsLgh8k1IUg2nn3635L2VIcx7k2zW/LETAc 2UY2NwihCTg3+IheRMAi3q1G8BbXdYQDW2OinNIbrmm92xL2wID2nbb12wpRU10S5wVZ Y/Y0Y8hplQmbZ5coM6L8+jOpBI6m6nEtVTj0kfQay5lAJohRtnhaGeG2rTGIEH/iT+Pi 2dUw== X-Gm-Message-State: AOAM531NEpaeLY3LIPZ9RgMrAxGlnvsElnjVBr4Q/q2rAsSDzxirQyOo xXSpC9r+lpJfYZ5/pjFZ1fY= X-Google-Smtp-Source: ABdhPJzrGUlABaISZ4hc6GbDuExEK/6vRiKAKwuM77hmE1jRQLaxmTiugZ80sCFhjv+vgSxmjuzXCw== X-Received: by 2002:a2e:8106:: with SMTP id d6mr7678908ljg.217.1611816755777; Wed, 27 Jan 2021 22:52:35 -0800 (PST) Received: from kari-VirtualBox (87-95-193-210.bb.dnainternet.fi. [87.95.193.210]) by smtp.gmail.com with ESMTPSA id o4sm1313687lfo.231.2021.01.27.22.52.34 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 27 Jan 2021 22:52:35 -0800 (PST) Date: Thu, 28 Jan 2021 08:52:33 +0200 From: Kari Argillander To: carlis Cc: gregkh@linuxfoundation.org, colin.king@canonical.com, oliver.graute@kococonnector.com, zhangxuezhi1@yulong.com, mh12gx2825@gmail.com, sbrivio@redhat.com, dri-devel@lists.freedesktop.org, linux-fbdev@vger.kernel.org, devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v10] staging: fbtft: add tearing signal detect Message-ID: <20210128065233.ji4b7ea54ihyu2l5@kari-VirtualBox> References: <1611754972-151016-1-git-send-email-zhangxuezhi3@gmail.com> <20210127223222.3lavtl3roc4cabso@kari-VirtualBox> <20210128094258.000012c3@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20210128094258.000012c3@gmail.com> Precedence: bulk List-ID: X-Mailing-List: linux-fbdev@vger.kernel.org On Thu, Jan 28, 2021 at 09:42:58AM +0800, carlis wrote: > On Thu, 28 Jan 2021 00:32:22 +0200 > Kari Argillander wrote: > > > #include "fbtft.h" > > > > > > #define DRVNAME "fb_st7789v" > > > @@ -66,6 +69,32 @@ enum st7789v_command { > > > #define MADCTL_MX BIT(6) /* bitmask for column address order */ > > > #define MADCTL_MY BIT(7) /* bitmask for page address order */ > > > > > > +#define SPI_PANEL_TE_TIMEOUT 400 /* msecs */ > > > +static struct mutex te_mutex;/* mutex for set te gpio irq status > > > */ > > > > Space after ; > hi, i have fix it in the patch v11 > > Yeah sorry. I accidentally review wrong patch. But mostly stuff are still relevant. > > > @@ -82,6 +111,33 @@ enum st7789v_command { > > > */ > > > static int init_display(struct fbtft_par *par) > > > { > > > + int rc; > > > + struct device *dev = par->info->device; > > > + > > > + par->gpio.te = devm_gpiod_get_index_optional(dev, "te", 0, > > > GPIOD_IN); > > > + if (IS_ERR(par->gpio.te)) { > > > + rc = PTR_ERR(par->gpio.te); > > > + dev_err(par->info->device, "Failed to request te > > > gpio: %d\n", rc); > > > + return rc; > > > + } > > > > You request with optinal and you still want to error out? We could > > just continue and not care about that error. User will be happier if > > device still works somehow. > You mean i just delete this dev_err print ?! > like this: > par->gpio.te = devm_gpiod_get_index_optional(dev, "te", > 0,GPIOD_IN); > if (IS_ERR(par->gpio.te)) > return PTR_ERR(par->gpio.te); Not exactly. I'm suggesting something like this. if (IS_ERR(par->gpio.te) == -EPROBE_DEFER) { return -EPROBE_DEFER; if (IS_ERR(par->gpio.te)) par-gpio.te = NULL; This like beginning of your patch series but the difference is that if EPROBE_DEFER then we will try again later. Any other error and we will just ignore TE gpio. But this is up to you what you want to do. To me this just seems place where this kind of logic can work. > > > + if (par->gpio.te) { > > > + set_spi_panel_te_irq_status(par, true); > > > + reinit_completion(&spi_panel_te); > > > + ret = > > > wait_for_completion_timeout(&spi_panel_te, > > > + > > > msecs_to_jiffies(SPI_PANEL_TE_TIMEOUT)); > > > + if (ret == 0) > > > > !ret > > > > > + dev_err(par->info->device, "wait > > > panel TE time out\n"); > > > + } > > > + ret = par->fbtftops.write(par, par->txbuf.buf, > > > + startbyte_size + to_copy > > > * 2); > > > + if (par->gpio.te) > > > + set_spi_panel_te_irq_status(par, false); > > > + if (ret < 0) > > > + return ret; > > > + remain -= to_copy; > > > + } > > > + > > > + return ret; > > > > Do we want to return something over 0? If not then this can be return > > 0. And then you do not need to even init ret value at the beginning. > > > > Also wait little bit like Greg sayd before sending new version. > > Someone might nack about what I say or say something more. > > > hi, i copy fbtft_write_vmem16_bus8 from file fbtft_bus.c and modify it > ,just add te wait logic, i will take more time to check this original > function. It might be ok or not. You should still check.