From mboxrd@z Thu Jan 1 00:00:00 1970 From: shawnguo@kernel.org (Shawn Guo) Date: Tue, 25 Jul 2017 21:17:16 +0800 Subject: [PATCH v4 2/2] ARM: dts: imx: add CX9020 Embedded PC device tree In-Reply-To: <3BB206AB2B1BD448954845CE6FF69A8E0182B4CFE8@nt-mail04> References: <20170712090408.12212-1-linux-kernel-dev@beckhoff.com> <20170721040640.31424-1-linux-kernel-dev@beckhoff.com> <20170721040640.31424-3-linux-kernel-dev@beckhoff.com> <20170725075934.GA20064@dragon> <3BB206AB2B1BD448954845CE6FF69A8E0182B4CFE8@nt-mail04> Message-ID: <20170725131715.GA29754@dragon> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Tue, Jul 25, 2017 at 11:41:56AM +0000, Patrick Br?nn wrote: > >From: Shawn Guo [mailto:shawnguo at kernel.org] > >On Fri, Jul 21, 2017 at 06:06:40AM +0200, linux-kernel-dev at beckhoff.com > >wrote: > >> From: Patrick Bruenn > >> > >> The CX9020 differs from i.MX53 Quick Start Board by: > >> - use uart2 instead of uart1 > >> - DVI-D connector instead of VGA > >> - no audio > >> - no SATA connector > >> - CCAT FPGA connected to emi > >> - enable rtc > >> > >> Signed-off-by: Patrick Bruenn > > > >Where is the patch 1/2? > > > You can find a copy here [1], or wait for the next revision, I will try to improve my git send-mail skills until then. > I used git send-email with "--to-cmd ./scripts/get_maintainer.pl" to send the series. I just > discovered it would have been better to use To: and Cc: in the cover letter and send with > "--to-cover --cc-cover". > I will drop "--in-reply-to" for the next revision, too... > Is there something like patman [2] from the u-boot project for the kernel, that I am missing? I do not use anything like that, but always manually edit recipients. It's a bit silly, but can make the recipients more precise and limited. > >> > >> --- > >> > >> v4: > >> - move alternative UART2 pinmux settings to imx53-pinfunc.h > >> - fix copyright notice and model name to clearify cx9020 is a > >> Beckhoff board and not from Freescale/NXP/Qualcomm > >> - add "bhf,cx9020" compatible > >> - remove ccat node and pin configuration as long as the ccat > >> driver is not mainlined > >> - use dvi-connector + ti,tfp410 instead of panel-simple > >> - add newlines between property list and child nodes > >> - replace underscores in node names with hypens > >> - replace magic number 0 with polarity defines from > >> include/dt-bindings/gpio/gpio.h > >> - move rtc node into imx53.dtsi, change it's name into 'srtc', > >> to avoid a conflict with 'rtc' node in imx53-m53.dtsi > >> - rename regulator-3p2v > >> - drop imx53-qsb container node > >> - make iomux configuration explicit > >> - remove unused audmux > >> - remove unused led_pin_gpio3_23 configuration > >> - use blue gpio-leds as disk-activity indicators for mmc0 and mmc1 > >> - add mmc indicator leds to sdhc pingroups > >> - keep node names in alphabetical order > >> - remove unused sata and ssi2 > >> - remove unused pin configs from hoggrp > >> - add entry for imx53-cx9020.dts to MAINTAINERS > >> > >> v3: add missig changelog > >> v2: > >> - keep alphabetic order of dts/Makefile > >> - configure uart2 with 'fsl,dte-mode' > >> - use display-0 and panel-0 as node names > >> - remove unnecessary "simple-bus" for fixed regulators > >> > >> Cc: Andrew Lunn > >> --- > >> MAINTAINERS | 1 + > > > >I do not take the changes on this file. > > > Do you mean you keep maintainership for imx53-cx9020.dts, or do you > mean I should group it in the patch which adds imx53-cx9020.dts? > I am fine with both and would be glad to have your guidance. I just > didn't understand clearly what was meant. I meant that I do not have the authority to take patches that update MAINTAINERS file. But now I think we do not even need to update MAINTAINERS just for imx53-cx9020.dts, since we already have IMX maintainer-ship covering all IMX dts files. ARM/FREESCALE IMX / MXC ARM ARCHITECTURE ... F: arch/arm/boot/dts/imx* Shawn From mboxrd@z Thu Jan 1 00:00:00 1970 From: Shawn Guo Subject: Re: [PATCH v4 2/2] ARM: dts: imx: add CX9020 Embedded PC device tree Date: Tue, 25 Jul 2017 21:17:16 +0800 Message-ID: <20170725131715.GA29754@dragon> References: <20170712090408.12212-1-linux-kernel-dev@beckhoff.com> <20170721040640.31424-1-linux-kernel-dev@beckhoff.com> <20170721040640.31424-3-linux-kernel-dev@beckhoff.com> <20170725075934.GA20064@dragon> <3BB206AB2B1BD448954845CE6FF69A8E0182B4CFE8@nt-mail04> Mime-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Transfer-Encoding: 8bit Return-path: Content-Disposition: inline In-Reply-To: <3BB206AB2B1BD448954845CE6FF69A8E0182B4CFE8@nt-mail04> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Patrick =?iso-8859-1?Q?Br=FCnn?= Cc: linux-kernel-dev , Rob Herring , Mark Rutland , Russell King , Sascha Hauer , Fabio Estevam , Greg Kroah-Hartman , "David S. Miller" , Mauro Carvalho Chehab , open list , "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , "moderated list:ARM PORT" , Andrew Lunn List-Id: devicetree@vger.kernel.org On Tue, Jul 25, 2017 at 11:41:56AM +0000, Patrick Brünn wrote: > >From: Shawn Guo [mailto:shawnguo-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org] > >On Fri, Jul 21, 2017 at 06:06:40AM +0200, linux-kernel-dev-QonKdJ6Bx35Wk0Htik3J/w@public.gmane.org > >wrote: > >> From: Patrick Bruenn > >> > >> The CX9020 differs from i.MX53 Quick Start Board by: > >> - use uart2 instead of uart1 > >> - DVI-D connector instead of VGA > >> - no audio > >> - no SATA connector > >> - CCAT FPGA connected to emi > >> - enable rtc > >> > >> Signed-off-by: Patrick Bruenn > > > >Where is the patch 1/2? > > > You can find a copy here [1], or wait for the next revision, I will try to improve my git send-mail skills until then. > I used git send-email with "--to-cmd ./scripts/get_maintainer.pl" to send the series. I just > discovered it would have been better to use To: and Cc: in the cover letter and send with > "--to-cover --cc-cover". > I will drop "--in-reply-to" for the next revision, too... > Is there something like patman [2] from the u-boot project for the kernel, that I am missing? I do not use anything like that, but always manually edit recipients. It's a bit silly, but can make the recipients more precise and limited. > >> > >> --- > >> > >> v4: > >> - move alternative UART2 pinmux settings to imx53-pinfunc.h > >> - fix copyright notice and model name to clearify cx9020 is a > >> Beckhoff board and not from Freescale/NXP/Qualcomm > >> - add "bhf,cx9020" compatible > >> - remove ccat node and pin configuration as long as the ccat > >> driver is not mainlined > >> - use dvi-connector + ti,tfp410 instead of panel-simple > >> - add newlines between property list and child nodes > >> - replace underscores in node names with hypens > >> - replace magic number 0 with polarity defines from > >> include/dt-bindings/gpio/gpio.h > >> - move rtc node into imx53.dtsi, change it's name into 'srtc', > >> to avoid a conflict with 'rtc' node in imx53-m53.dtsi > >> - rename regulator-3p2v > >> - drop imx53-qsb container node > >> - make iomux configuration explicit > >> - remove unused audmux > >> - remove unused led_pin_gpio3_23 configuration > >> - use blue gpio-leds as disk-activity indicators for mmc0 and mmc1 > >> - add mmc indicator leds to sdhc pingroups > >> - keep node names in alphabetical order > >> - remove unused sata and ssi2 > >> - remove unused pin configs from hoggrp > >> - add entry for imx53-cx9020.dts to MAINTAINERS > >> > >> v3: add missig changelog > >> v2: > >> - keep alphabetic order of dts/Makefile > >> - configure uart2 with 'fsl,dte-mode' > >> - use display-0 and panel-0 as node names > >> - remove unnecessary "simple-bus" for fixed regulators > >> > >> Cc: Andrew Lunn > >> --- > >> MAINTAINERS | 1 + > > > >I do not take the changes on this file. > > > Do you mean you keep maintainership for imx53-cx9020.dts, or do you > mean I should group it in the patch which adds imx53-cx9020.dts? > I am fine with both and would be glad to have your guidance. I just > didn't understand clearly what was meant. I meant that I do not have the authority to take patches that update MAINTAINERS file. But now I think we do not even need to update MAINTAINERS just for imx53-cx9020.dts, since we already have IMX maintainer-ship covering all IMX dts files. ARM/FREESCALE IMX / MXC ARM ARCHITECTURE ... F: arch/arm/boot/dts/imx* Shawn -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752037AbdGYNSE (ORCPT ); Tue, 25 Jul 2017 09:18:04 -0400 Received: from mail.kernel.org ([198.145.29.99]:44276 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751097AbdGYNSC (ORCPT ); Tue, 25 Jul 2017 09:18:02 -0400 DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org E38EF219AA Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=shawnguo@kernel.org Date: Tue, 25 Jul 2017 21:17:16 +0800 From: Shawn Guo To: Patrick =?iso-8859-1?Q?Br=FCnn?= Cc: linux-kernel-dev , Rob Herring , Mark Rutland , Russell King , Sascha Hauer , Fabio Estevam , Greg Kroah-Hartman , "David S. Miller" , Mauro Carvalho Chehab , open list , "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , "moderated list:ARM PORT" , Andrew Lunn Subject: Re: [PATCH v4 2/2] ARM: dts: imx: add CX9020 Embedded PC device tree Message-ID: <20170725131715.GA29754@dragon> References: <20170712090408.12212-1-linux-kernel-dev@beckhoff.com> <20170721040640.31424-1-linux-kernel-dev@beckhoff.com> <20170721040640.31424-3-linux-kernel-dev@beckhoff.com> <20170725075934.GA20064@dragon> <3BB206AB2B1BD448954845CE6FF69A8E0182B4CFE8@nt-mail04> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <3BB206AB2B1BD448954845CE6FF69A8E0182B4CFE8@nt-mail04> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Jul 25, 2017 at 11:41:56AM +0000, Patrick Brünn wrote: > >From: Shawn Guo [mailto:shawnguo@kernel.org] > >On Fri, Jul 21, 2017 at 06:06:40AM +0200, linux-kernel-dev@beckhoff.com > >wrote: > >> From: Patrick Bruenn > >> > >> The CX9020 differs from i.MX53 Quick Start Board by: > >> - use uart2 instead of uart1 > >> - DVI-D connector instead of VGA > >> - no audio > >> - no SATA connector > >> - CCAT FPGA connected to emi > >> - enable rtc > >> > >> Signed-off-by: Patrick Bruenn > > > >Where is the patch 1/2? > > > You can find a copy here [1], or wait for the next revision, I will try to improve my git send-mail skills until then. > I used git send-email with "--to-cmd ./scripts/get_maintainer.pl" to send the series. I just > discovered it would have been better to use To: and Cc: in the cover letter and send with > "--to-cover --cc-cover". > I will drop "--in-reply-to" for the next revision, too... > Is there something like patman [2] from the u-boot project for the kernel, that I am missing? I do not use anything like that, but always manually edit recipients. It's a bit silly, but can make the recipients more precise and limited. > >> > >> --- > >> > >> v4: > >> - move alternative UART2 pinmux settings to imx53-pinfunc.h > >> - fix copyright notice and model name to clearify cx9020 is a > >> Beckhoff board and not from Freescale/NXP/Qualcomm > >> - add "bhf,cx9020" compatible > >> - remove ccat node and pin configuration as long as the ccat > >> driver is not mainlined > >> - use dvi-connector + ti,tfp410 instead of panel-simple > >> - add newlines between property list and child nodes > >> - replace underscores in node names with hypens > >> - replace magic number 0 with polarity defines from > >> include/dt-bindings/gpio/gpio.h > >> - move rtc node into imx53.dtsi, change it's name into 'srtc', > >> to avoid a conflict with 'rtc' node in imx53-m53.dtsi > >> - rename regulator-3p2v > >> - drop imx53-qsb container node > >> - make iomux configuration explicit > >> - remove unused audmux > >> - remove unused led_pin_gpio3_23 configuration > >> - use blue gpio-leds as disk-activity indicators for mmc0 and mmc1 > >> - add mmc indicator leds to sdhc pingroups > >> - keep node names in alphabetical order > >> - remove unused sata and ssi2 > >> - remove unused pin configs from hoggrp > >> - add entry for imx53-cx9020.dts to MAINTAINERS > >> > >> v3: add missig changelog > >> v2: > >> - keep alphabetic order of dts/Makefile > >> - configure uart2 with 'fsl,dte-mode' > >> - use display-0 and panel-0 as node names > >> - remove unnecessary "simple-bus" for fixed regulators > >> > >> Cc: Andrew Lunn > >> --- > >> MAINTAINERS | 1 + > > > >I do not take the changes on this file. > > > Do you mean you keep maintainership for imx53-cx9020.dts, or do you > mean I should group it in the patch which adds imx53-cx9020.dts? > I am fine with both and would be glad to have your guidance. I just > didn't understand clearly what was meant. I meant that I do not have the authority to take patches that update MAINTAINERS file. But now I think we do not even need to update MAINTAINERS just for imx53-cx9020.dts, since we already have IMX maintainer-ship covering all IMX dts files. ARM/FREESCALE IMX / MXC ARM ARCHITECTURE ... F: arch/arm/boot/dts/imx* Shawn