Received: by 2002:a05:6a10:1a4d:0:0:0:0 with SMTP id nk13csp1555506pxb; Wed, 2 Feb 2022 07:33:44 -0800 (PST) X-Google-Smtp-Source: ABdhPJxEP79G0rKEh30XyLzNDzvsSE07MSJPsE0hr8BH80IeKIiIIdscEKiSHO06b6UEXw3nTaz0 X-Received: by 2002:a17:903:22cb:: with SMTP id y11mr31193497plg.138.1643816024080; Wed, 02 Feb 2022 07:33:44 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1643816024; cv=none; d=google.com; s=arc-20160816; b=gJS80rwXNkd7xPFgdjcfhipCs+PfquHg/NwgCbzICrEjYXE9a6jxdZMvkcmXyW0Q2f /vwWX6JxkmVHtti/cqJxpZ5rp0+PFrH3z9jLEJNJMmSsQBc9YrD9IuqB3fSAk1Lpo5KT a7Ypv99+hP+qCTvXMnFXfi6fvZ3k3egb7A+p0ZigmILIVNqdAfpFbKo3uXFut6sOP1x/ seRpgICt6kkL6Uy6wM+/KrpPmPtacc4DGa3mMnrgiwUbhXWoUyaSGZ5erp3rRdooAm0l RZgvn8RVcKrXcV1tACsYvY3NscG1BnYX4B8ESeajvqy066TGLADDuczyJ+DZi14xyqP2 k+7Q== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:in-reply-to:from :references:cc:to:content-language:subject:user-agent:mime-version :date:message-id:dkim-signature; bh=1yBAb+LzLjx2wSEzMHtSKhPuT2tYtGfr9jHBhoMyyMs=; b=zlOdrs3hIlW0R+i5V3NqxwfJYK7zigoQmsdyGS7FUVLACPA4QYntpOxcJakofyG4xq WsnumyPgqOhGwo5OAhOhV/hsQz9VdhD+SgQe3n/h6ES4q1iLjraJ8325ria3CFpCpWCE qUySYtSlZnsILsLvfYPmL97PrJIRBAahCB+A5d+s16oGafPVuEk5S246471EhcATBeeI VZsL9hys4JhnVz+j2u/8kfi4K/RtQvs9Hm54dlZCkiEhcazbntHsmmVpoLnw+ZGGB2fa 7m8vA8rQqvfvCI3fziyrf6u3cHHTXlXE7pdGMaCAPOjf09lWiiGjHTvA6VLKu6HFi4UW coDg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@collabora.com header.s=mail header.b=GJGaCITC; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=collabora.com Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id l63si13647344pgd.579.2022.02.02.07.33.31; Wed, 02 Feb 2022 07:33:44 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@collabora.com header.s=mail header.b=GJGaCITC; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=collabora.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S235175AbiBAIyz (ORCPT + 99 others); Tue, 1 Feb 2022 03:54:55 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:33630 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231912AbiBAIyx (ORCPT ); Tue, 1 Feb 2022 03:54:53 -0500 Received: from bhuna.collabora.co.uk (bhuna.collabora.co.uk [IPv6:2a00:1098:0:82:1000:25:2eeb:e3e3]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id BB35BC061714 for ; Tue, 1 Feb 2022 00:54:52 -0800 (PST) Received: from [127.0.0.1] (localhost [127.0.0.1]) (Authenticated sender: kholk11) with ESMTPSA id E38851F438FB DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=collabora.com; s=mail; t=1643705690; bh=cX6n8y9xqw3fyjvPkbflYeNt2MpCla3htWHQ0GpzOts=; h=Date:Subject:To:Cc:References:From:In-Reply-To:From; b=GJGaCITCXqQz2ChA2swrnHl7qatLEV8wEXtJMTwWsdv+EbNZdVsXrQaOBpKyzB/gG XFGiuUG15jZovcZOSwY+YpHwTzCfGZEqqii2d/XJ+mVOwXqhdpYQv5MKYL2VBcGZjt xWEUJXgYXAvHdUaRo0FzkyREoCsgX6ylFDKkTqOFN0Phw3Y5LFbxVXZJcZb6RRNhNP EuQD844FmkVOzuMuVbMNeZdKfhaEY+1mFpi1cx5jbZT+Ni+8xiuzFB+U9Y6J1EbwJL /mbkmWn7Ymg2Avlze52JoA1NcGLX04LY1pV3LQGSrCTWCghynYBVvCjlTnf1dU/dJF HSqOCN2bhAeVw== Message-ID: <90ec652d-6df7-3e7b-dd81-7ced053e1dcf@collabora.com> Date: Tue, 1 Feb 2022 09:54:47 +0100 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.5.1 Subject: Re: [PATCH v11] drm/bridge: add it6505 driver Content-Language: en-US To: Robert Foss , Hsin-Yi Wang Cc: 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 References: <20220114091502.333083-1-allen.chen@ite.com.tw> From: AngeloGioacchino Del Regno In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Il 31/01/22 19:36, Robert Foss ha scritto: > 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. > You have mine... the intention was to give a Reviewed-by, not a Acked-by - I'm sorry for that, I was sending more than one email and the wrong tag slipped through. So, please change my Acked-by tag to Reviewed-by: AngeloGioacchino Del Regno >> >>>> >>>> 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"); >>>>> >>>>