Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id F243CC6FD1F for ; Sat, 11 Mar 2023 10:05:55 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229703AbjCKKFy (ORCPT ); Sat, 11 Mar 2023 05:05:54 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:36660 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229639AbjCKKFt (ORCPT ); Sat, 11 Mar 2023 05:05:49 -0500 Received: from mail-wm1-x331.google.com (mail-wm1-x331.google.com [IPv6:2a00:1450:4864:20::331]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 71D4CE6FFD for ; Sat, 11 Mar 2023 02:05:47 -0800 (PST) Received: by mail-wm1-x331.google.com with SMTP id o38-20020a05600c512600b003e8320d1c11so5586127wms.1 for ; Sat, 11 Mar 2023 02:05:47 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1678529146; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=ciFroiEJcPCf1IcNM4Oy4MInn1VtQ6qbpV3yMFcCzXc=; b=bXoj40nIICUGmqpLuoYZkYyZ3eI1jvBtOnrUizE4q5FSvmvB5V6XKjaQIjIn/K4RAD YG/Yy9xDyLtzuREY+xUphpPJTOWG1eFoji9vDNCAniLttDeONIj9tIRDgWLZ1Cf40zpR b/RKl1dVBshh/xoXbru8cKano9n5bnp3QKU0PV53SFT1gsQ2twK2XvBLdIKj6LE6xvJM 6A9h4v7nU2K+0MxrCf+MFDwjq5i8q1WjzlmsLzG4G8bXR4aMyl0mUagsw5m+hvfN1mIH /gSuilOgJ6Gifhmro4r8vIoSoQF0DiQKjO9e+GaFN8lgreefLqfQkjhwpYdbTamSAqK3 cEhA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; t=1678529146; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=ciFroiEJcPCf1IcNM4Oy4MInn1VtQ6qbpV3yMFcCzXc=; b=wlJxSilz1HHUUGl3lZRBL81+1MTtv7b6s/c5s+WG6xDfGu6/2huyZempUFWe4/TiH+ w4d3N3Njd3bnthCFNGCOjcHE1IHnFLXl+PuEMwCshLjJFVbeei5NgSbsBKKf1ddqhIWL H3Ixac4my38m4OzJCwuFWb9WLrq/0K8o+bDcgJotmO2Yul1x6Di78f7nW1G6L+yV+VW+ QMuqqm3QuhZlRo2rfwUUKO6K3aJnYeU5yzwQoaHxxBRWjvyYfUC/F1fmtlI79wKgXVQz iMniaPTV3OVv+bG5Zas0VaI5Dou8broLElLEkmO4qp7MPbdOXrh4JH8zRnOJB+N7x38T VyfA== X-Gm-Message-State: AO0yUKVsHYE5+YE434M0+ShCjUAmpZj/Lwi2Bf9W2eRn52HmS/q5THoX DqXmN5uGHohvliFiGZtMBwKL7g== X-Google-Smtp-Source: AK7set8zoDvWWEIFn13DW0itfJK1PANtvoXCqkiW5qigJfdhRgGaFsfi9mqjxqBQBIxpQJm1HRWahQ== X-Received: by 2002:a05:600c:3491:b0:3eb:2e19:ff3a with SMTP id a17-20020a05600c349100b003eb2e19ff3amr5293547wmq.7.1678529145902; Sat, 11 Mar 2023 02:05:45 -0800 (PST) Received: from ?IPV6:2a01:e0a:982:cbb0:fc3e:aa04:e32d:cc69? ([2a01:e0a:982:cbb0:fc3e:aa04:e32d:cc69]) by smtp.gmail.com with ESMTPSA id p23-20020a1c7417000000b003e11f280b8bsm2334725wmc.44.2023.03.11.02.05.44 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Sat, 11 Mar 2023 02:05:45 -0800 (PST) Message-ID: <7765365e-2bde-4ccc-3251-d4bb38c34ecb@linaro.org> Date: Sat, 11 Mar 2023 11:05:43 +0100 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:102.0) Gecko/20100101 Thunderbird/102.7.2 Subject: Re: [PATCH v4 2/2] drm/panel: Add driver for Novatek NT36523 To: Linus Walleij , Jianhua Lu Cc: Konrad Dybcio , Thierry Reding , Sam Ravnborg , David Airlie , Daniel Vetter , Rob Herring , Krzysztof Kozlowski , dri-devel@lists.freedesktop.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, ~postmarketos/upstreaming@lists.sr.ht, phone-devel@vger.kernel.org References: <20230310132144.2241-1-lujianhua000@gmail.com> <20230310132144.2241-2-lujianhua000@gmail.com> <6539f132-c740-b9b6-8810-0affd6ab91ba@linaro.org> Content-Language: en-GB From: Neil Armstrong In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Le 10/03/2023 à 23:50, Linus Walleij a écrit : > On Fri, Mar 10, 2023 at 7:35 PM Jianhua Lu wrote: >> On Fri, Mar 10, 2023 at 07:10:21PM +0100, Konrad Dybcio wrote: >>> >>>> +#define mipi_dsi_dual_dcs_write_seq(dsi0, dsi1, cmd, seq...) \ >>>> + do { \ >>>> + mipi_dsi_dcs_write_seq(dsi0, cmd, seq); \ >>>> + mipi_dsi_dcs_write_seq(dsi1, cmd, seq); \ >>>> + } while (0) >>> This should be in the same file as mipi_dsi_dcs_write_seq, imo >> I have sent a patch to do it, upstream don't think this wrapper is a proper >> approach to deal with dual dsi and wrap all of mipi_dsi_* function is >> useless. >> https://lore.kernel.org/lkml/20230310110542.6649-1-lujianhua000@gmail.com/ > > We can keep it locally if the fight isn't worthwhile, but I will try to enter > the discussion. It's ok to keep it in the driver until we find a way to move it in the generic includes. Neil > > Yours, > Linus Walleij