From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755935AbeEAQb3 (ORCPT ); Tue, 1 May 2018 12:31:29 -0400 Received: from mail-wm0-f68.google.com ([74.125.82.68]:53228 "EHLO mail-wm0-f68.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751071AbeEAQb0 (ORCPT ); Tue, 1 May 2018 12:31:26 -0400 X-Google-Smtp-Source: AB8JxZozQ1zwjHIaWGejbaLr2XkKkOsraTg+BjXa89SLHA6SAEB+SAp//wF1UrVedhL/P9YWafa5mA== MIME-Version: 1.0 In-Reply-To: <20180501161723.GA376@rob-hp-laptop> References: <20180430114058.5061-1-jagan@amarulasolutions.com> <20180430114058.5061-5-jagan@amarulasolutions.com> <20180501161723.GA376@rob-hp-laptop> From: Chen-Yu Tsai Date: Wed, 2 May 2018 00:31:04 +0800 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH 04/21] bindings: display: Add compatible for A64 DE2 pipeline To: Rob Herring Cc: Jagan Teki , Maxime Ripard , Icenowy Zheng , Jernej Skrabec , Mark Rutland , Catalin Marinas , Will Deacon , David Airlie , dri-devel , Michael Turquette , Stephen Boyd , linux-clk , Michael Trimarchi , linux-arm-kernel , devicetree , linux-kernel , linux-sunxi Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, May 2, 2018 at 12:17 AM, Rob Herring wrote: > On Mon, Apr 30, 2018 at 05:10:41PM +0530, Jagan Teki wrote: >> Allwinner A64 has DE2 pipeline similar to other Allwinner >> SOC's like A83T, H3/H5. > > 'dt-bindings: ' for the subject prefix. > >> >> Signed-off-by: Jagan Teki >> --- >> Documentation/devicetree/bindings/display/sunxi/sun4i-drm.txt | 1 + >> 1 file changed, 1 insertion(+) > > Otherwise, > > Reviewed-by: Rob Herring Would it make sense to put all the display component related compatible string additions in the same patch? i.e. "Add compatible strings for all documented display component blocks". MIPI-DSI is missing, but it's not documented. ChenYu From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chen-Yu Tsai Subject: Re: [PATCH 04/21] bindings: display: Add compatible for A64 DE2 pipeline Date: Wed, 2 May 2018 00:31:04 +0800 Message-ID: References: <20180430114058.5061-1-jagan@amarulasolutions.com> <20180430114058.5061-5-jagan@amarulasolutions.com> <20180501161723.GA376@rob-hp-laptop> Reply-To: wens-jdAy2FN1RRM@public.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Return-path: Sender: linux-sunxi-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org In-Reply-To: <20180501161723.GA376@rob-hp-laptop> List-Post: , List-Help: , List-Archive: , List-Unsubscribe: , To: Rob Herring Cc: Jagan Teki , Maxime Ripard , Icenowy Zheng , Jernej Skrabec , Mark Rutland , Catalin Marinas , Will Deacon , David Airlie , dri-devel , Michael Turquette , Stephen Boyd , linux-clk , Michael Trimarchi , linux-arm-kernel , devicetree , linux-kernel , linux-sunxi List-Id: devicetree@vger.kernel.org On Wed, May 2, 2018 at 12:17 AM, Rob Herring wrote: > On Mon, Apr 30, 2018 at 05:10:41PM +0530, Jagan Teki wrote: >> Allwinner A64 has DE2 pipeline similar to other Allwinner >> SOC's like A83T, H3/H5. > > 'dt-bindings: ' for the subject prefix. > >> >> Signed-off-by: Jagan Teki >> --- >> Documentation/devicetree/bindings/display/sunxi/sun4i-drm.txt | 1 + >> 1 file changed, 1 insertion(+) > > Otherwise, > > Reviewed-by: Rob Herring Would it make sense to put all the display component related compatible string additions in the same patch? i.e. "Add compatible strings for all documented display component blocks". MIPI-DSI is missing, but it's not documented. ChenYu From mboxrd@z Thu Jan 1 00:00:00 1970 From: wens@csie.org (Chen-Yu Tsai) Date: Wed, 2 May 2018 00:31:04 +0800 Subject: [PATCH 04/21] bindings: display: Add compatible for A64 DE2 pipeline In-Reply-To: <20180501161723.GA376@rob-hp-laptop> References: <20180430114058.5061-1-jagan@amarulasolutions.com> <20180430114058.5061-5-jagan@amarulasolutions.com> <20180501161723.GA376@rob-hp-laptop> Message-ID: To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Wed, May 2, 2018 at 12:17 AM, Rob Herring wrote: > On Mon, Apr 30, 2018 at 05:10:41PM +0530, Jagan Teki wrote: >> Allwinner A64 has DE2 pipeline similar to other Allwinner >> SOC's like A83T, H3/H5. > > 'dt-bindings: ' for the subject prefix. > >> >> Signed-off-by: Jagan Teki >> --- >> Documentation/devicetree/bindings/display/sunxi/sun4i-drm.txt | 1 + >> 1 file changed, 1 insertion(+) > > Otherwise, > > Reviewed-by: Rob Herring Would it make sense to put all the display component related compatible string additions in the same patch? i.e. "Add compatible strings for all documented display component blocks". MIPI-DSI is missing, but it's not documented. ChenYu