From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1034952AbcIZO2u (ORCPT ); Mon, 26 Sep 2016 10:28:50 -0400 Received: from mail-lf0-f52.google.com ([209.85.215.52]:34605 "EHLO mail-lf0-f52.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1034879AbcIZO2t (ORCPT ); Mon, 26 Sep 2016 10:28:49 -0400 MIME-Version: 1.0 In-Reply-To: References: <1473618641-10159-1-git-send-email-jagan@amarulasolutions.com> <1473618641-10159-3-git-send-email-jagan@amarulasolutions.com> <119ad2fc-8e74-1193-8ee2-417cf1e6fd49@engicam.com> From: Jagan Teki Date: Mon, 26 Sep 2016 19:58:39 +0530 Message-ID: Subject: Re: [PATCH v4 2/5] ARM: dts: imx6q: Add Engicam i.CoreM6 Quad/Dual initial support To: Michael Trimarchi Cc: Fabio Estevam , Matteo Lisi , Jagan Teki , "linux-arm-kernel@lists.infradead.org" , "devicetree@vger.kernel.org" , linux-kernel , Sascha Hauer , Fabio Estevam , Shawn Guo Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Fabio, On Mon, Sep 26, 2016 at 7:26 PM, Michael Trimarchi wrote: > Hi > > On Mon, Sep 26, 2016 at 3:55 PM, Fabio Estevam wrote: >> On Mon, Sep 26, 2016 at 10:54 AM, Michael Trimarchi >> wrote: >> >>>> Why don't you simply do like this? >>>> >>>> reg_3p3v: regulator-3p3v { >>>> compatible = "regulator-fixed"; >>>> regulator-name = "3P3V"; >>>> regulator-min-microvolt = <3300000>; >>>> regulator-max-microvolt = <3300000>; >>>> }; >>> >>> Should dts rispect the schematic connection? >> >> Yes > > So boot on and always on are used to declare their status and their > behavionr. I don't > see the point to not specify them in this way Please try to understand that due to the case where we used fixed voltage regulators on SOM as mentioned By Metteo and Michael dt nodes should have regulator-always-on and regulator-boot-on properties. And even we added one of our module which is already in ML[1] Hope this is helpful conclude this thread. [1] http://git.kernel.org/cgit/linux/kernel/git/next/linux-next.git/commit/?id=a58e4e608bc88735581fdd9861676e522fa3c20d thanks! -- Jagan Teki Free Software Engineer | Amarula Solutions U-Boot, Linux | Upstream Maintainer Hyderabad, India. From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jagan Teki Subject: Re: [PATCH v4 2/5] ARM: dts: imx6q: Add Engicam i.CoreM6 Quad/Dual initial support Date: Mon, 26 Sep 2016 19:58:39 +0530 Message-ID: References: <1473618641-10159-1-git-send-email-jagan@amarulasolutions.com> <1473618641-10159-3-git-send-email-jagan@amarulasolutions.com> <119ad2fc-8e74-1193-8ee2-417cf1e6fd49@engicam.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Return-path: In-Reply-To: Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Michael Trimarchi Cc: Fabio Estevam , Matteo Lisi , Jagan Teki , "linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org" , "devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , linux-kernel , Sascha Hauer , Fabio Estevam , Shawn Guo List-Id: devicetree@vger.kernel.org Hi Fabio, On Mon, Sep 26, 2016 at 7:26 PM, Michael Trimarchi wrote: > Hi > > On Mon, Sep 26, 2016 at 3:55 PM, Fabio Estevam wrote: >> On Mon, Sep 26, 2016 at 10:54 AM, Michael Trimarchi >> wrote: >> >>>> Why don't you simply do like this? >>>> >>>> reg_3p3v: regulator-3p3v { >>>> compatible = "regulator-fixed"; >>>> regulator-name = "3P3V"; >>>> regulator-min-microvolt = <3300000>; >>>> regulator-max-microvolt = <3300000>; >>>> }; >>> >>> Should dts rispect the schematic connection? >> >> Yes > > So boot on and always on are used to declare their status and their > behavionr. I don't > see the point to not specify them in this way Please try to understand that due to the case where we used fixed voltage regulators on SOM as mentioned By Metteo and Michael dt nodes should have regulator-always-on and regulator-boot-on properties. And even we added one of our module which is already in ML[1] Hope this is helpful conclude this thread. [1] http://git.kernel.org/cgit/linux/kernel/git/next/linux-next.git/commit/?id=a58e4e608bc88735581fdd9861676e522fa3c20d thanks! -- Jagan Teki Free Software Engineer | Amarula Solutions U-Boot, Linux | Upstream Maintainer Hyderabad, India. -- 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 From: jagan@amarulasolutions.com (Jagan Teki) Date: Mon, 26 Sep 2016 19:58:39 +0530 Subject: [PATCH v4 2/5] ARM: dts: imx6q: Add Engicam i.CoreM6 Quad/Dual initial support In-Reply-To: References: <1473618641-10159-1-git-send-email-jagan@amarulasolutions.com> <1473618641-10159-3-git-send-email-jagan@amarulasolutions.com> <119ad2fc-8e74-1193-8ee2-417cf1e6fd49@engicam.com> Message-ID: To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi Fabio, On Mon, Sep 26, 2016 at 7:26 PM, Michael Trimarchi wrote: > Hi > > On Mon, Sep 26, 2016 at 3:55 PM, Fabio Estevam wrote: >> On Mon, Sep 26, 2016 at 10:54 AM, Michael Trimarchi >> wrote: >> >>>> Why don't you simply do like this? >>>> >>>> reg_3p3v: regulator-3p3v { >>>> compatible = "regulator-fixed"; >>>> regulator-name = "3P3V"; >>>> regulator-min-microvolt = <3300000>; >>>> regulator-max-microvolt = <3300000>; >>>> }; >>> >>> Should dts rispect the schematic connection? >> >> Yes > > So boot on and always on are used to declare their status and their > behavionr. I don't > see the point to not specify them in this way Please try to understand that due to the case where we used fixed voltage regulators on SOM as mentioned By Metteo and Michael dt nodes should have regulator-always-on and regulator-boot-on properties. And even we added one of our module which is already in ML[1] Hope this is helpful conclude this thread. [1] http://git.kernel.org/cgit/linux/kernel/git/next/linux-next.git/commit/?id=a58e4e608bc88735581fdd9861676e522fa3c20d thanks! -- Jagan Teki Free Software Engineer | Amarula Solutions U-Boot, Linux | Upstream Maintainer Hyderabad, India.