From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1750986AbaJVFVT (ORCPT ); Wed, 22 Oct 2014 01:21:19 -0400 Received: from devils.ext.ti.com ([198.47.26.153]:60851 "EHLO devils.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750813AbaJVFVR (ORCPT ); Wed, 22 Oct 2014 01:21:17 -0400 Message-ID: <54473E8C.20303@ti.com> Date: Wed, 22 Oct 2014 10:50:12 +0530 From: Kishon Vijay Abraham I User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.0 MIME-Version: 1.0 To: Gabriel Fernandez CC: Gabriel FERNANDEZ , Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Srinivas Kandagatla , Maxime Coquelin , Patrice Chotard , Russell King , Grant Likely , "devicetree@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "kernel@stlinux.com" , Lee Jones , alexandre torgue Subject: Re: [PATCH v3 2/8] phy: miphy28lp: Add MiPHY28lp header file for DT x Driver defines References: <1411721657-9924-1-git-send-email-gabriel.fernandez@linaro.org> <1411721657-9924-3-git-send-email-gabriel.fernandez@linaro.org> <544637A3.8010002@ti.com> In-Reply-To: Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tuesday 21 October 2014 09:19 PM, Gabriel Fernandez wrote: > Hi Kishon, > > To be sure, if i rename MIPHY_TYPE_SATA into PHY_TYPE_SATA, i have to > rename the filename also ? > > include/dt-bindings/phy/phy-miphy.h into include/dt-bindings/phy/phy.h > > am I right? That's right.. Cheers Kishon > > Best Regards > > Gabriel > > On 21 October 2014 12:38, Kishon Vijay Abraham I wrote: >> Hi Gabriel, >> >> On Friday 26 September 2014 02:24 PM, Gabriel FERNANDEZ wrote: >>> This provides the shared header file which will be reference from both >>> the MiPHY28lp driver and its associated Device Tree node(s). >>> >>> Signed-off-by: alexandre torgue >>> Signed-off-by: Gabriel Fernandez >>> --- >>> include/dt-bindings/phy/phy-miphy.h | 12 ++++++++++++ >>> 1 file changed, 12 insertions(+) >>> create mode 100644 include/dt-bindings/phy/phy-miphy.h >>> >>> diff --git a/include/dt-bindings/phy/phy-miphy.h b/include/dt-bindings/phy/phy-miphy.h >>> new file mode 100644 >>> index 0000000..9bc49b9 >>> --- /dev/null >>> +++ b/include/dt-bindings/phy/phy-miphy.h >>> @@ -0,0 +1,12 @@ >>> +/* >>> + * This header provides constants for the phy framework >>> + * based on the STMicroelectronics miphxxxx. >>> + */ >>> +#ifndef _DT_BINDINGS_PHY_MIPHY >>> +#define _DT_BINDINGS_PHY_MIPHY >>> + >>> +#define MIPHY_TYPE_SATA 1 >>> +#define MIPHY_TYPE_PCIE 2 >>> +#define MIPHY_TYPE_USB 3 >> >> Since these PHY types are common across SoCs, we can have something like >> PHY_TYPE_SATA, PHY_TYPE_PCIE so that each SoCs don't have to create a separate >> header file. >> >> Cheers >> Kishon