Received: by 2002:a05:6a10:1a4d:0:0:0:0 with SMTP id nk13csp901301pxb; Tue, 1 Feb 2022 12:47:21 -0800 (PST) X-Google-Smtp-Source: ABdhPJy5aEeMJdkJkXtAM+XIET0gLeQkGQRxctuqI/tv6Jf51DjkOhj1pkyBlCPGVVBPaK3t0/FI X-Received: by 2002:a17:90b:3e8e:: with SMTP id rj14mr4298066pjb.19.1643748441116; Tue, 01 Feb 2022 12:47:21 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1643748441; cv=none; d=google.com; s=arc-20160816; b=UWY+OXI62vcl7YttzKhpB8n3r3RE2l9GVg3rq83CudihWYDYcz4sG31Pe/sU+wb5aH A0Zl0KYQxQ/k/9LO/wzCguIpNjywm/NSVwDjsxaCgS05gTwqaR9ZsdOQlEztn67SgEVe Vgj74MrAgx15rWusYNGB6tJqKrIh+/XxM/YEu1oi95Rd2W7oMSd/QbSrwIJoQnS4Qahk wkFFnunSnJ2gL5rmfIx8S+z/XoFM446FgpLyYrAYoimyxM78iP8S/xxuWZnKX/jC4Dkd gbem5i0Sh4XBp3/NXcu/duJGq/zfbJ+PhjgQ6S+QDKGElxJhN9DtcadT3vI+KiuV3avf Aiww== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:cc:to:subject:message-id:date:from:in-reply-to :references:mime-version:dkim-signature; bh=jVP1HoXA+fkjiP7dYZEx1JX7fEwo0mBNJ8T54jUWwpw=; b=TDcSS1Y976tznI7VuYfEMVyhGoUDCiLUnH/kGlts2RJdWAr1/2Aai1GQ0p+ADt6qpD VWk+UiFNu35rBBkKhKvVa/VjVwthj0XAc9lJKY5WSbAH+xghF7wklspLTG01agd5qhSk s+JeEE4ciCmC3TAoFi6HoeEK5WYahY6oPOZkUbHXsyUifKa0eiR9v6bcrQHwBMe+4Pzz 3unH0qL2ponPWJhBD6Ujak+gb2fjwTwcDP3S5VF1QMIYHV+aozAVBnQ3Xqe2imFZ5tnY Vmb3X0IFgTMB4kyyemlg3+hypPRHcgmL5vNpx5pw5v+kVsJxLAYf47cE5vRR6TV68dqM rv6w== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=XfhmwR2k; 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; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id b5si3054729pjw.61.2022.02.01.12.47.09; Tue, 01 Feb 2022 12:47:21 -0800 (PST) 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; dkim=pass header.i=@linaro.org header.s=google header.b=XfhmwR2k; 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; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1356747AbiAaSg5 (ORCPT + 99 others); Mon, 31 Jan 2022 13:36:57 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:38012 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1355341AbiAaSgx (ORCPT ); Mon, 31 Jan 2022 13:36:53 -0500 Received: from mail-pj1-x1029.google.com (mail-pj1-x1029.google.com [IPv6:2607:f8b0:4864:20::1029]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id AC684C061714 for ; Mon, 31 Jan 2022 10:36:53 -0800 (PST) Received: by mail-pj1-x1029.google.com with SMTP id r59so14805708pjg.4 for ; Mon, 31 Jan 2022 10:36:53 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=jVP1HoXA+fkjiP7dYZEx1JX7fEwo0mBNJ8T54jUWwpw=; b=XfhmwR2k8wqx6HUMq6ZupVxiE0Wdd7dCjnqYGD7rZZuDiyRZFhEpyT3IU+nyUz+Tge qPQCJxrP3ecoE6CXTV5vRrCVepB8RHbRptQtQf4d/GUg9z/3fpFD25W6GclQxbGIpkmf P4kYt8Xw+mhsiezNivRxgw1rhC63mmCkzWeRLJgKciEaPpsWmQNpJ9Bj/Z0es+p9rUfI 5FZ7kFPCUwATfhF6oKx8iM7CM1Hn9Qo7776Vnjtil3VXt80RvtUziCWWkwHa1uonkDkc B8NioUeunqVoGvNGpm8YeeroaXqZ5ijGO5nHrcqIIs/Yf6RL/DOUlREz30cV1arb7fLl +T0g== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=jVP1HoXA+fkjiP7dYZEx1JX7fEwo0mBNJ8T54jUWwpw=; b=PpO6edCdOqmIfqJETfoB/zQKlmv2NgNT1DLrbOFI5Sxi2Ur2Su5goOnaXpvn31EZHP mQDdHt9VA7uNWB6JRS2ow361DryurAIIZj6VxQrJQe5TWbOlHZ3RN7b0lx/FXb7vDCz+ vFY31uPK15vYG5Jh6fQBGLZNAV2YbGly/Cjz9gYTEkXTOotS1q98/c/DFfbc8mh8vgID q6cdFjzDwa2arw3/WrjTIHKQZ4Ec1kvrFrIrhwvFB7N1iCWIxGEqfsQO55/LuhN/GCdk JHs8o6+R+oOiAqFTvH9Q06q1G94v8Pb/JjojqyDiwqh2k1/LvACORxlOaAFtTU/+2e6n PV2w== X-Gm-Message-State: AOAM530qzC8H+0GHuZyzr/9HddBHYC0F34CSkFHiwigVPd5FW84Bd8mI duSHyp1buhroBrTTWbzXX4dCr9769Dx2tMrMrpkWRQ== X-Received: by 2002:a17:90b:1802:: with SMTP id lw2mr26223360pjb.232.1643654213164; Mon, 31 Jan 2022 10:36:53 -0800 (PST) MIME-Version: 1.0 References: <20220114091502.333083-1-allen.chen@ite.com.tw> In-Reply-To: From: Robert Foss Date: Mon, 31 Jan 2022 19:36:41 +0100 Message-ID: Subject: Re: [PATCH v11] drm/bridge: add it6505 driver To: Hsin-Yi Wang Cc: AngeloGioacchino Del Regno , allen , Kenneth Hung , Jernej Skrabec , Jau-Chih Tseng , David Airlie , "open list:DRM DRIVERS" , Neil Armstrong , open list , Jonas Karlman , Hermes Wu , Laurent Pinchart , Andrzej Hajda , Hsin-Yi Wang Content-Type: text/plain; charset="UTF-8" Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, 31 Jan 2022 at 17:55, Hsin-Yi Wang wrote: > > On Tue, Feb 1, 2022 at 12:37 AM Robert Foss wrote: > > > > On Thu, 20 Jan 2022 at 16:25, AngeloGioacchino Del Regno > > wrote: > > > > > > Il 14/01/22 10:14, allen ha scritto: > > > > This adds support for the iTE IT6505. > > > > This device can convert DPI signal to DP output. > > > > > > > > From: Allen Chen > > > > Tested-by: Hsin-yi Wang > > > > Signed-off-by: Hermes Wu > > > > Signed-off-by: Allen Chen > > > > --- > > > > v10 -> v11 : remove drm_bridge_new_crtc_state > > > > --- > > > > drivers/gpu/drm/bridge/Kconfig | 8 + > > > > drivers/gpu/drm/bridge/Makefile | 1 + > > > > drivers/gpu/drm/bridge/ite-it6505.c | 3352 +++++++++++++++++++++++++++ > > > > 3 files changed, 3361 insertions(+) > > > > create mode 100644 drivers/gpu/drm/bridge/ite-it6505.c > > > > > > > > > > ...snip... > > > > > > > +static const struct of_device_id it6505_of_match[] = { > > > > + { .compatible = "ite,it6505" }, > > > > + { } > > > > +}; > > > > > > If you want to have a DT compatible and DT properties, you have to also add > > > dt-bindings (yaml) for this driver, otherwise, any SoC/device DT will fail > > > the dt binding check.... So, please, add that. > > > > Let me second this. A dt-binding is needed for this driver to be > > complete, it functions as both documentation and a way to test the DTS > > that use this device, so it is really important. > > > The binding seems to be accepted before the driver: > https://elixir.bootlin.com/linux/v5.16.4/source/Documentation/devicetree/bindings/display/bridge/ite,it6505.yaml I completely missed that. In that case we're only missing the reviewed-by tag from someone. > > > > > > > For the driver by itself, though: > > > > > > Acked-by: AngeloGioacchino Del Regno > > > > > > > + > > > > +static struct i2c_driver it6505_i2c_driver = { > > > > + .driver = { > > > > + .name = "it6505", > > > > + .of_match_table = it6505_of_match, > > > > + .pm = &it6505_bridge_pm_ops, > > > > + }, > > > > + .probe = it6505_i2c_probe, > > > > + .remove = it6505_i2c_remove, > > > > + .shutdown = it6505_shutdown, > > > > + .id_table = it6505_id, > > > > +}; > > > > + > > > > +module_i2c_driver(it6505_i2c_driver); > > > > + > > > > +MODULE_AUTHOR("Allen Chen "); > > > > +MODULE_DESCRIPTION("IT6505 DisplayPort Transmitter driver"); > > > > +MODULE_LICENSE("GPL v2"); > > > > > > >