From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754200AbcAHImy (ORCPT ); Fri, 8 Jan 2016 03:42:54 -0500 Received: from ibawizard.net ([82.208.49.253]:44163 "EHLO mengele.ibawizard.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751322AbcAHImw (ORCPT ); Fri, 8 Jan 2016 03:42:52 -0500 Date: Fri, 8 Jan 2016 09:42:47 +0100 From: Petr =?iso-8859-2?Q?=A9tetiar?= To: Marcel Ziswiler Cc: "ynezz@true.cz" , "linux-kernel@vger.kernel.org" , "robh+dt@kernel.org" , "pawel.moll@arm.com" , "stefan@agner.ch" , "shawn.guo@linaro.org" , "devicetree@vger.kernel.org" , "festevam@gmail.com" , "mark.rutland@arm.com" , "stillcompiling@gmail.com" , "ijc+devicetree@hellion.org.uk" , "galak@codeaurora.org" , "shawnguo@kernel.org" , "linux-arm-kernel@lists.infradead.org" , "l.stach@pengutronix.de" , "kernel@pengutronix.de" , "linux@arm.linux.org.uk" Subject: Re: [PATCH v2 1/2] ARM: dts: imx6: Add support for Toradex Apalis iMX6Q/D SoM Message-ID: <20160108084247.GB27359@ibawizard.net> Reply-To: Petr =?iso-8859-2?Q?=A9tetiar?= References: <1452011942-11940-1-git-send-email-marcel.ziswiler@toradex.com> <1452011942-11940-2-git-send-email-marcel.ziswiler@toradex.com> <20160106080358.GD9030@ibawizard.net> <1452071716.30372.61.camel@toradex.com> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-2 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <1452071716.30372.61.camel@toradex.com> User-Agent: Mutt/1.5.18 (2008-05-17) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Marcel Ziswiler [2016-01-06 09:15:18]: > > > <&pinctrl_regulator_usbotg_pwr>; > > > + regulator-name = "usb_otg_vbus"; > > > + regulator-min-microvolt = <5000000>; > > > + regulator-max-microvolt = <5000000>; > > > + gpio = <&gpio3 22 GPIO_ACTIVE_HIGH>; > > > + enable-active-high; > > > + status = "disabled"; > > > + }; > > > > I'm not sure, but it seems to me, that we should move this regulator into > > the carrier board DTS. On our custom carrier board we've this regulator > > always on and we use this GPIO for heartbeat LED, so I've this in our > > carrier board DTS[2]: > > Yes, you are absolutely right and e.g. on Apalis T30 [1] that is exactly how > we did it. And what about usdhc? It leads to the same overrides in the carrier board file: &iomuxc { usdhc { /* GPIO4_IO20 is LVDS */ pinctrl_mmc_cd: gpio_mmc_cd { }; /* Used as misc GPIOs */ pinctrl_usdhc1: usdhc1grp { }; }; }; > > > +/* PAD Ctrl values for common settings */ > > > +/* > > > + * (PAD_CTL_HYS | PAD_CTL_PUS_100K_UP | PAD_CTL_PUE | PAD_CTL_PKE > > > | > > > + *  PAD_CTL_SPEED_MED | PAD_CTL_DSE_40ohm) > > > + */ > > > +#define PAD_CTRL_HYS_PU 0x1b0b0 > > > > This was requested to be reworked. I've simply replaced all the > > macros with > > hex values. > > I don't think the reviewers really concluded on the action to be taken > on here and just using hex values is probably the most stupid solution. It's simple, if you just leave it as it is, then it will never get merged. That's how kernel development works :) We've at least two options here: 1. Follow the old path and just convert those macros to hex values 2. Rework it - move those macros to some header file etc. -- ynezz