[{"id":1767229,"web_url":"http://patchwork.ozlabs.org/comment/1767229/","msgid":"<20170912163958.xr5qxt5doqlk44dp@rob-hp-laptop>","list_archive_url":null,"date":"2017-09-12T16:39:58","subject":"Re: [PATCH 1/2] drm: bridge: Add bindings for TI THS8134","submitter":{"id":62529,"url":"http://patchwork.ozlabs.org/api/people/62529/","name":"Rob Herring (Arm)","email":"robh@kernel.org"},"content":"On Fri, Sep 01, 2017 at 11:40:37AM +0200, Linus Walleij wrote:\n> This adds device tree bindings for the Texas Instruments\n> THS8134A and THS8134B VGA DACs by extending and renaming the\n> existing bindings for THS8135.\n> \n> These DACs are used for the VGA outputs on the ARM reference\n> designs such as Integrator, Versatile and RealView.\n> \n> Cc: devicetree@vger.kernel.org\n> Cc: Bartosz Golaszewski \n> Signed-off-by: Linus Walleij \n> ---\n> .../display/bridge/{ti,ths8135.txt => ti,ths813x.txt} | 12 ++++++++----\n> 1 file changed, 8 insertions(+), 4 deletions(-)\n> rename Documentation/devicetree/bindings/display/bridge/{ti,ths8135.txt => ti,ths813x.txt} (73%)\n\nAcked-by: Rob Herring \n--\nTo unsubscribe from this list: send the line \"unsubscribe devicetree\" in\nthe body of a message to majordomo@vger.kernel.org\nMore majordomo info at http://vger.kernel.org/majordomo-info.html","headers":{"Return-Path":"","X-Original-To":"incoming-dt@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-dt@bilbo.ozlabs.org","Authentication-Results":"ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=vger.kernel.org\n\t(client-ip=209.132.180.67; helo=vger.kernel.org;\n\tenvelope-from=devicetree-owner@vger.kernel.org; receiver=)","Received":["from vger.kernel.org (vger.kernel.org [209.132.180.67])\n\tby ozlabs.org (Postfix) with ESMTP id 3xs9WQ4TH3z9sNV\n\tfor ;\n\tWed, 13 Sep 2017 02:40:02 +1000 (AEST)","(majordomo@vger.kernel.org) by vger.kernel.org via listexpand\n\tid S1751423AbdILQkB (ORCPT\n\t);\n\tTue, 12 Sep 2017 12:40:01 -0400","from mail-io0-f193.google.com ([209.85.223.193]:34073 \"EHLO\n\tmail-io0-f193.google.com\" rhost-flags-OK-OK-OK-OK) by vger.kernel.org\n\twith ESMTP id S1751054AbdILQkB (ORCPT\n\t); Tue, 12 Sep 2017 12:40:01 -0400","by mail-io0-f193.google.com with SMTP id g32so7124192ioj.1\n\tfor ;\n\tTue, 12 Sep 2017 09:40:00 -0700 (PDT)","from localhost (216-188-254-6.dyn.grandenetworks.net.\n\t[216.188.254.6]) by smtp.gmail.com with ESMTPSA id\n\tm131sm11277565oif.18.2017.09.12.09.39.58\n\t(version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256);\n\tTue, 12 Sep 2017 09:39:59 -0700 (PDT)"],"X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:date:from:to:cc:subject:message-id:references\n\t:mime-version:content-disposition:in-reply-to:user-agent;\n\tbh=ik0kTMxfM3NZLYD3TDbTAfLWyAQdB+MD/5fCqucxxiA=;\n\tb=Oz2r1XlW3a/yu5uAFuwkM3sKnKAvOydaO0yq5+yGuicrmfUaWeQ6m3A71co4jxHEpF\n\tXsVRcM38AqkPpVvDiMyNqbky//FfjSrJ82aYXaJQXj9QISPVXiZMsAOXRiDglzZhdZTb\n\tcirP7m2NAWB8el+tTKRZPwXHJjtKutWlHc7lWc1m0eji6zym54e3XqXEkoI0EmGg9t+6\n\t1ubBXcVMpbhQ3Ydevkespr6wfnR2RtGyRXAczlcSpc7DJ5WAj/8enCtMiG03qIGPP+u+\n\tTW56ud71NcPQJqP3ldHADWJ8b5QI1D3ag/ci4QUcau2dqrT9wgTViitqwIiJ1P8iyqQ2\n\t7ohQ==","X-Gm-Message-State":"AHPjjUgbZrTedbl3LSc4x22Fw8iV7Ok5x+hlLQvX1z93VDUGkBD3b7sB\n\tCJS2/6p8TauVrw==","X-Google-Smtp-Source":"AOwi7QBcUqNGT9sEq+WelZ0RGP5R/LcWguqkyb7VLxa1iXCbHWuZ8cWQJxmoA0G6QdAlZW2IFe3EsQ==","X-Received":"by 10.202.82.4 with SMTP id g4mr16283638oib.154.1505234400220;\n\tTue, 12 Sep 2017 09:40:00 -0700 (PDT)","Date":"Tue, 12 Sep 2017 11:39:58 -0500","From":"Rob Herring ","To":"Linus Walleij ","Cc":"Archit Taneja ,\n\tAndrzej Hajda ,\n\tLaurent Pinchart ,\n\tdri-devel@lists.freedesktop.org,\n\tlinux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org,\n\tBartosz Golaszewski ","Subject":"Re: [PATCH 1/2] drm: bridge: Add bindings for TI THS8134","Message-ID":"<20170912163958.xr5qxt5doqlk44dp@rob-hp-laptop>","References":"<20170901094038.4312-1-linus.walleij@linaro.org>","MIME-Version":"1.0","Content-Type":"text/plain; charset=us-ascii","Content-Disposition":"inline","In-Reply-To":"<20170901094038.4312-1-linus.walleij@linaro.org>","User-Agent":"NeoMutt/20170113 (1.7.2)","Sender":"devicetree-owner@vger.kernel.org","Precedence":"bulk","List-ID":"","X-Mailing-List":"devicetree@vger.kernel.org"}},{"id":1767439,"web_url":"http://patchwork.ozlabs.org/comment/1767439/","msgid":"<2446705.uuxjYiEYNm@avalon>","list_archive_url":null,"date":"2017-09-13T00:08:14","subject":"Re: [PATCH 1/2] drm: bridge: Add bindings for TI THS8134","submitter":{"id":11034,"url":"http://patchwork.ozlabs.org/api/people/11034/","name":"Laurent Pinchart","email":"laurent.pinchart@ideasonboard.com"},"content":"Hi Linus,\n\nThank you for the patch.\n\nOn Friday, 1 September 2017 12:40:37 EEST Linus Walleij wrote:\n> This adds device tree bindings for the Texas Instruments\n> THS8134A and THS8134B VGA DACs by extending and renaming the\n> existing bindings for THS8135.\n> \n> These DACs are used for the VGA outputs on the ARM reference\n> designs such as Integrator, Versatile and RealView.\n> \n> Cc: devicetree@vger.kernel.org\n> Cc: Bartosz Golaszewski \n> Signed-off-by: Linus Walleij \n> ---\n> .../display/bridge/{ti,ths8135.txt => ti,ths813x.txt} | 12 ++++++---\n> 1 file changed, 8 insertions(+), 4 deletions(-)\n> rename Documentation/devicetree/bindings/display/bridge/{ti,ths8135.txt =>\n> ti,ths813x.txt} (73%)\n> \n> diff --git a/Documentation/devicetree/bindings/display/bridge/ti,ths8135.txt\n> b/Documentation/devicetree/bindings/display/bridge/ti,ths813x.txt\n> similarity index 73%\n> rename from Documentation/devicetree/bindings/display/bridge/ti,ths8135.txt\n> rename to Documentation/devicetree/bindings/display/bridge/ti,ths813x.txt\n> index 6ec1a880ac18..608435f04796 100644\n> --- a/Documentation/devicetree/bindings/display/bridge/ti,ths8135.txt\n> +++ b/Documentation/devicetree/bindings/display/bridge/ti,ths813x.txt\n> @@ -1,11 +1,15 @@\n> -THS8135 Video DAC\n> ------------------\n> +THS8134 and THS8135 Video DAC\n> +-----------------------------\n> \n> -This is the binding for Texas Instruments THS8135 Video DAC bridge.\n> +This is the binding for Texas Instruments THS8134A, THS8134B and THS8135\n> +Video DAC bridge.\n> \n> Required properties:\n> \n> -- compatible: Must be \"ti,ths8135\"\n> +- compatible: Must be one of\n> + \"ti,ths8134a\"\n> + \"ti,ths8134b\"\n> + \"ti,ths8135\"\n\nAs mentioned in the review of patch 2/2, would it make sense to also introduce \na ti,ths813x compatible string ? I haven't checked the datasheets so the \nanswer might very well be no.\n\n> Required nodes:","headers":{"Return-Path":"","X-Original-To":"incoming-dt@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-dt@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=vger.kernel.org\n\t(client-ip=209.132.180.67; helo=vger.kernel.org;\n\tenvelope-from=devicetree-owner@vger.kernel.org; receiver=)","ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=ideasonboard.com header.i=@ideasonboard.com\n\theader.b=\"GV/tv994\"; dkim-atps=neutral"],"Received":["from vger.kernel.org (vger.kernel.org [209.132.180.67])\n\tby ozlabs.org (Postfix) with ESMTP id 3xsMSj04xZz9sPt\n\tfor ;\n\tWed, 13 Sep 2017 10:08:21 +1000 (AEST)","(majordomo@vger.kernel.org) by vger.kernel.org via listexpand\n\tid S1751424AbdIMAIO (ORCPT\n\t);\n\tTue, 12 Sep 2017 20:08:14 -0400","from galahad.ideasonboard.com ([185.26.127.97]:50256 \"EHLO\n\tgalahad.ideasonboard.com\" rhost-flags-OK-OK-OK-OK) by vger.kernel.org\n\twith ESMTP id S1750944AbdIMAIO (ORCPT\n\t); Tue, 12 Sep 2017 20:08:14 -0400","from avalon.localnet (unknown [12.145.98.24])\n\tby galahad.ideasonboard.com (Postfix) with ESMTPSA id 1B39C201F5;\n\tWed, 13 Sep 2017 02:05:53 +0200 (CEST)"],"DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1505261153;\n\tbh=TzA+LoZt+dm/ZFnPEWyKs47HMC3yU/I6mGQldr8QT24=;\n\th=From:To:Cc:Subject:Date:In-Reply-To:References:From;\n\tb=GV/tv994RPakE3sPMyaUBxduMoQjwe89Iy14SOpX2pRyscXipTKrt1PveAIYgRDSZ\n\t0BjXv+RuYwsFxP7ZQLuMWxZse5Fmhnv3o/Uh1QVElPtcL5byIsEbM6muoS9e24mCAo\n\t10hfnsJGD/IZ5He4P23e53yg3e7+FMZPb2qjBIjs=","From":"Laurent Pinchart ","To":"Linus Walleij ","Cc":"Archit Taneja ,\n\tAndrzej Hajda , dri-devel@lists.freedesktop.org, \n\tlinux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org,\n\tBartosz Golaszewski ","Subject":"Re: [PATCH 1/2] drm: bridge: Add bindings for TI THS8134","Date":"Wed, 13 Sep 2017 03:08:14 +0300","Message-ID":"<2446705.uuxjYiEYNm@avalon>","In-Reply-To":"<20170901094038.4312-1-linus.walleij@linaro.org>","References":"<20170901094038.4312-1-linus.walleij@linaro.org>","MIME-Version":"1.0","Content-Transfer-Encoding":"7Bit","Content-Type":"text/plain; charset=\"us-ascii\"","Sender":"devicetree-owner@vger.kernel.org","Precedence":"bulk","List-ID":"","X-Mailing-List":"devicetree@vger.kernel.org"}},{"id":1787409,"web_url":"http://patchwork.ozlabs.org/comment/1787409/","msgid":"","list_archive_url":null,"date":"2017-10-16T12:52:23","subject":"Re: [PATCH 1/2] drm: bridge: Add bindings for TI THS8134","submitter":{"id":7055,"url":"http://patchwork.ozlabs.org/api/people/7055/","name":"Linus Walleij","email":"linus.walleij@linaro.org"},"content":"On Wed, Sep 13, 2017 at 2:08 AM, Laurent Pinchart\n wrote:\n\n>> -- compatible: Must be \"ti,ths8135\"\n>> +- compatible: Must be one of\n>> + \"ti,ths8134a\"\n>> + \"ti,ths8134b\"\n>> + \"ti,ths8135\"\n>\n> As mentioned in the review of patch 2/2, would it make sense to also introduce\n> a ti,ths813x compatible string ? I haven't checked the datasheets so the\n> answer might very well be no.\n\nThat should be fine, compatible-properties from the most specific to the\nmost generic should work just fine.\n\nI'll respin with a ths813x compatible.\n\nYours,\nLinus Walleij\n--\nTo unsubscribe from this list: send the line \"unsubscribe devicetree\" in\nthe body of a message to majordomo@vger.kernel.org\nMore majordomo info at http://vger.kernel.org/majordomo-info.html","headers":{"Return-Path":"","X-Original-To":"incoming-dt@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-dt@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=vger.kernel.org\n\t(client-ip=209.132.180.67; helo=vger.kernel.org;\n\tenvelope-from=devicetree-owner@vger.kernel.org; receiver=)","ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=linaro.org header.i=@linaro.org\n\theader.b=\"TLpscWfB\"; dkim-atps=neutral"],"Received":["from vger.kernel.org (vger.kernel.org [209.132.180.67])\n\tby ozlabs.org (Postfix) with ESMTP id 3yFys55F0xz9t2W\n\tfor ;\n\tMon, 16 Oct 2017 23:52:25 +1100 (AEDT)","(majordomo@vger.kernel.org) by vger.kernel.org via listexpand\n\tid S1752122AbdJPMwY (ORCPT\n\t);\n\tMon, 16 Oct 2017 08:52:24 -0400","from mail-io0-f175.google.com ([209.85.223.175]:51387 \"EHLO\n\tmail-io0-f175.google.com\" rhost-flags-OK-OK-OK-OK) by vger.kernel.org\n\twith ESMTP id S1752080AbdJPMwY (ORCPT\n\t); Mon, 16 Oct 2017 08:52:24 -0400","by mail-io0-f175.google.com with SMTP id b186so15982429iof.8\n\tfor ;\n\tMon, 16 Oct 2017 05:52:24 -0700 (PDT)","by 10.79.14.140 with HTTP; Mon, 16 Oct 2017 05:52:23 -0700 (PDT)"],"DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google;\n\th=mime-version:in-reply-to:references:from:date:message-id:subject:to\n\t:cc; bh=y8146/m3N+84NHdMsEuoygn9DGDkC6VaxhVQr5+LzGs=;\n\tb=TLpscWfBXhjyvopKfRla/DiZj+L01JhJfY9I3K+1WnpFYTMiD6hYJyTIBMs0HMs5ws\n\texpj0nAbTvzzs1L4ZWF+hFwWa5DYBXx1nCtC6t9jLQP1JAv0AI957WMqGnC0X5YfNubs\n\tM40jsnNknqxLeDYAJTjsBNcCTI/TJBrziS/Tc=","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:mime-version:in-reply-to:references:from:date\n\t:message-id:subject:to:cc;\n\tbh=y8146/m3N+84NHdMsEuoygn9DGDkC6VaxhVQr5+LzGs=;\n\tb=c6o28yFFRDO93cs43bwEmL2JiHEg/zrw/DbSL4S2v2MyuiiSnFcaX0C3C7pIl6DUQ8\n\tkBZMDqFadQytS2H/NqFW9fz6fhFa/XMww6r5tBhVUEfChcTlva2xJhsfjMgdeDLyQzby\n\tS1eoTpoxI+vMzZMAkNZ8SlY/a4GepBNDaInxTcQKR3MlfCZ0T2KmhKk8Z10mmreIs87o\n\tR0aBtbsU2sEV1OJx3k4BoIJtJNDshNjCOAjM9X+Y3prg/dRktvHYlyfY/1grnHumhO6n\n\tIxU2icp90NryjBYzZoiJqKWZXFXVJ7qyU2fPnTpd4vusWMc6xLvYh5E+3RwB4XN+P2Er\n\tksDw==","X-Gm-Message-State":"AMCzsaU592gQs+XGjH/Pt2P1Vf2pmTTl0Io7WagNxSExR1cWEFGAvxty\n\tqKZXzBkTPCh18XWouN1BKk+LAmPOtNrvJWL/ahhuvA==","X-Google-Smtp-Source":"ABhQp+Q0yxKAG6eL0dvBZBF1Xi8jEJJPp9oHv2DY8M8rd3nkNRLL7cR0EVEBWv7BRPDKyHjiXpNboSc1UxldlLV55kw=","X-Received":"by 10.107.140.143 with SMTP id\n\to137mr4406798iod.206.1508158343744; \n\tMon, 16 Oct 2017 05:52:23 -0700 (PDT)","MIME-Version":"1.0","In-Reply-To":"<2446705.uuxjYiEYNm@avalon>","References":"<20170901094038.4312-1-linus.walleij@linaro.org>\n\t<2446705.uuxjYiEYNm@avalon>","From":"Linus Walleij ","Date":"Mon, 16 Oct 2017 14:52:23 +0200","Message-ID":"","Subject":"Re: [PATCH 1/2] drm: bridge: Add bindings for TI THS8134","To":"Laurent Pinchart ","Cc":"Archit Taneja ,\n\tAndrzej Hajda ,\n\t\"open list:DRM PANEL DRIVERS\" ,\n\t\"linux-arm-kernel@lists.infradead.org\" \n\t,\n\t\"devicetree@vger.kernel.org\" ,\n\tBartosz Golaszewski ","Content-Type":"text/plain; charset=\"UTF-8\"","Sender":"devicetree-owner@vger.kernel.org","Precedence":"bulk","List-ID":"","X-Mailing-List":"devicetree@vger.kernel.org"}}]