All of lore.kernel.org
 help / color / mirror / Atom feed
From: Kishon Vijay Abraham I <kishon@ti.com>
To: Roger Quadros <rogerq@ti.com>
Cc: bcousson@baylibre.com, tony@atomide.com, balbi@ti.com,
	devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
	linux-omap@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
	linux-usb@vger.kernel.org, robh+dt@kernel.org,
	pawel.moll@arm.com, mark.rutland@arm.com,
	ijc+devicetree@hellion.org.uk, galak@codeaurora.org,
	linux@arm.linux.org.uk, gregkh@linuxfoundation.org,
	nsekhar@ti.com
Subject: Re: [PATCH 08/17] ARM: dts: dra7: Add dt node for the sycon pcie
Date: Wed, 24 Jun 2015 18:59:13 +0530	[thread overview]
Message-ID: <558AB0A9.5020400@ti.com> (raw)
In-Reply-To: <20150624145017.9b70a1fba477d8b99ce85702@ti.com>

Hi,

On Wednesday 24 June 2015 05:20 PM, Roger Quadros wrote:
> On Tue, 23 Jun 2015 17:28:53 +0530
> Kishon Vijay Abraham I <kishon@ti.com> wrote:
> 
>> Add new device tree node for the control module register space where
>> PCIe registers are present.
>>
>> Signed-off-by: Kishon Vijay Abraham I <kishon@ti.com>
>> ---
>>  arch/arm/boot/dts/dra7.dtsi |    5 +++++
>>  1 file changed, 5 insertions(+)
>>
>> diff --git a/arch/arm/boot/dts/dra7.dtsi b/arch/arm/boot/dts/dra7.dtsi
>> index 260f300..3f434f7 100644
>> --- a/arch/arm/boot/dts/dra7.dtsi
>> +++ b/arch/arm/boot/dts/dra7.dtsi
>> @@ -291,6 +291,11 @@
>>  			reg = <0x4a002e00 0x7c>;
>>  		};
>>  
>> +		dra7_ctrl_pcie: tisyscon@4a003c00 {
>> +			compatible = "syscon";
>> +			reg = <0x4a003c00 0x48>;
>> +		};
>> +
> 
> Why do you need to start from 0x4a003c00?
> CTRL_CORE_PCIESS1_PCS1 is at 0x4a003c24

Actually wanted to have minimum number of syscon dt nodes. So thought even
others can use dra7_ctrl_pcie phandle. But I looked only at dra7_ctrl_core and
dra7_ctrl_general and not scm_conf :-(

Thanks
Kishon

WARNING: multiple messages have this Message-ID (diff)
From: kishon@ti.com (Kishon Vijay Abraham I)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 08/17] ARM: dts: dra7: Add dt node for the sycon pcie
Date: Wed, 24 Jun 2015 18:59:13 +0530	[thread overview]
Message-ID: <558AB0A9.5020400@ti.com> (raw)
In-Reply-To: <20150624145017.9b70a1fba477d8b99ce85702@ti.com>

Hi,

On Wednesday 24 June 2015 05:20 PM, Roger Quadros wrote:
> On Tue, 23 Jun 2015 17:28:53 +0530
> Kishon Vijay Abraham I <kishon@ti.com> wrote:
> 
>> Add new device tree node for the control module register space where
>> PCIe registers are present.
>>
>> Signed-off-by: Kishon Vijay Abraham I <kishon@ti.com>
>> ---
>>  arch/arm/boot/dts/dra7.dtsi |    5 +++++
>>  1 file changed, 5 insertions(+)
>>
>> diff --git a/arch/arm/boot/dts/dra7.dtsi b/arch/arm/boot/dts/dra7.dtsi
>> index 260f300..3f434f7 100644
>> --- a/arch/arm/boot/dts/dra7.dtsi
>> +++ b/arch/arm/boot/dts/dra7.dtsi
>> @@ -291,6 +291,11 @@
>>  			reg = <0x4a002e00 0x7c>;
>>  		};
>>  
>> +		dra7_ctrl_pcie: tisyscon at 4a003c00 {
>> +			compatible = "syscon";
>> +			reg = <0x4a003c00 0x48>;
>> +		};
>> +
> 
> Why do you need to start from 0x4a003c00?
> CTRL_CORE_PCIESS1_PCS1 is at 0x4a003c24

Actually wanted to have minimum number of syscon dt nodes. So thought even
others can use dra7_ctrl_pcie phandle. But I looked only at dra7_ctrl_core and
dra7_ctrl_general and not scm_conf :-(

Thanks
Kishon

WARNING: multiple messages have this Message-ID (diff)
From: Kishon Vijay Abraham I <kishon@ti.com>
To: Roger Quadros <rogerq@ti.com>
Cc: <bcousson@baylibre.com>, <tony@atomide.com>, <balbi@ti.com>,
	<devicetree@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
	<linux-omap@vger.kernel.org>,
	<linux-arm-kernel@lists.infradead.org>,
	<linux-usb@vger.kernel.org>, <robh+dt@kernel.org>,
	<pawel.moll@arm.com>, <mark.rutland@arm.com>,
	<ijc+devicetree@hellion.org.uk>, <galak@codeaurora.org>,
	<linux@arm.linux.org.uk>, <gregkh@linuxfoundation.org>,
	<nsekhar@ti.com>
Subject: Re: [PATCH 08/17] ARM: dts: dra7: Add dt node for the sycon pcie
Date: Wed, 24 Jun 2015 18:59:13 +0530	[thread overview]
Message-ID: <558AB0A9.5020400@ti.com> (raw)
In-Reply-To: <20150624145017.9b70a1fba477d8b99ce85702@ti.com>

Hi,

On Wednesday 24 June 2015 05:20 PM, Roger Quadros wrote:
> On Tue, 23 Jun 2015 17:28:53 +0530
> Kishon Vijay Abraham I <kishon@ti.com> wrote:
> 
>> Add new device tree node for the control module register space where
>> PCIe registers are present.
>>
>> Signed-off-by: Kishon Vijay Abraham I <kishon@ti.com>
>> ---
>>  arch/arm/boot/dts/dra7.dtsi |    5 +++++
>>  1 file changed, 5 insertions(+)
>>
>> diff --git a/arch/arm/boot/dts/dra7.dtsi b/arch/arm/boot/dts/dra7.dtsi
>> index 260f300..3f434f7 100644
>> --- a/arch/arm/boot/dts/dra7.dtsi
>> +++ b/arch/arm/boot/dts/dra7.dtsi
>> @@ -291,6 +291,11 @@
>>  			reg = <0x4a002e00 0x7c>;
>>  		};
>>  
>> +		dra7_ctrl_pcie: tisyscon@4a003c00 {
>> +			compatible = "syscon";
>> +			reg = <0x4a003c00 0x48>;
>> +		};
>> +
> 
> Why do you need to start from 0x4a003c00?
> CTRL_CORE_PCIESS1_PCS1 is at 0x4a003c24

Actually wanted to have minimum number of syscon dt nodes. So thought even
others can use dra7_ctrl_pcie phandle. But I looked only at dra7_ctrl_core and
dra7_ctrl_general and not scm_conf :-(

Thanks
Kishon

  parent reply	other threads:[~2015-06-24 13:29 UTC|newest]

Thread overview: 150+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-06-23 11:58 [PATCH 00/17] Deprecate phy-omap-control and use SYSCON Kishon Vijay Abraham I
2015-06-23 11:58 ` Kishon Vijay Abraham I
2015-06-23 11:58 ` Kishon Vijay Abraham I
2015-06-23 11:58 ` [PATCH 01/17] phy: ti-pipe3: use ti_pipe3_power_off to power off the PHY during probe Kishon Vijay Abraham I
2015-06-23 11:58   ` Kishon Vijay Abraham I
2015-06-23 11:58   ` Kishon Vijay Abraham I
2015-06-23 14:47   ` Roger Quadros
2015-06-23 14:47     ` Roger Quadros
2015-06-23 14:47     ` Roger Quadros
2015-06-23 11:58 ` [PATCH 02/17] phy: ti-pipe3: use *syscon* framework to power on/off the PHY Kishon Vijay Abraham I
2015-06-23 11:58   ` Kishon Vijay Abraham I
2015-06-23 11:58   ` Kishon Vijay Abraham I
     [not found]   ` <1435060743-5511-3-git-send-email-kishon-l0cyMroinI0@public.gmane.org>
2015-06-23 14:53     ` Roger Quadros
2015-06-23 14:53       ` Roger Quadros
2015-06-23 14:53       ` Roger Quadros
     [not found]       ` <20150623175355.d80c1d6626439fbb6c26169c-l0cyMroinI0@public.gmane.org>
2015-06-24 11:26         ` Kishon Vijay Abraham I
2015-06-24 11:26           ` Kishon Vijay Abraham I
2015-06-24 11:26           ` Kishon Vijay Abraham I
2015-06-23 11:58 ` [PATCH 03/17] phy: ti-pipe3: use *syscon* framework to set PCS value of " Kishon Vijay Abraham I
2015-06-23 11:58   ` Kishon Vijay Abraham I
2015-06-23 11:58   ` Kishon Vijay Abraham I
     [not found]   ` <1435060743-5511-4-git-send-email-kishon-l0cyMroinI0@public.gmane.org>
2015-06-24 10:34     ` Roger Quadros
2015-06-24 10:34       ` Roger Quadros
2015-06-24 10:34       ` Roger Quadros
2015-06-24 13:16       ` Kishon Vijay Abraham I
2015-06-24 13:16         ` Kishon Vijay Abraham I
2015-06-24 13:16         ` Kishon Vijay Abraham I
     [not found]         ` <558AAD98.1090602-l0cyMroinI0@public.gmane.org>
2015-06-24 13:48           ` Roger Quadros
2015-06-24 13:48             ` Roger Quadros
2015-06-24 13:48             ` Roger Quadros
2015-06-23 11:58 ` [PATCH 04/17] phy: omap-usb2: use omap_usb_power_off to power off the PHY during probe Kishon Vijay Abraham I
2015-06-23 11:58   ` Kishon Vijay Abraham I
2015-06-23 11:58   ` Kishon Vijay Abraham I
     [not found]   ` <1435060743-5511-5-git-send-email-kishon-l0cyMroinI0@public.gmane.org>
2015-06-24 10:58     ` Roger Quadros
2015-06-24 10:58       ` Roger Quadros
2015-06-24 10:58       ` Roger Quadros
2015-06-23 11:58 ` [PATCH 07/17] usb: musb: omap2430: use *syscon* framework to write to mailbox register Kishon Vijay Abraham I
2015-06-23 11:58   ` Kishon Vijay Abraham I
2015-06-23 11:58   ` Kishon Vijay Abraham I
2015-06-24 11:39   ` Roger Quadros
2015-06-24 11:39     ` Roger Quadros
2015-06-24 11:39     ` Roger Quadros
2015-06-24 13:21     ` Kishon Vijay Abraham I
2015-06-24 13:21       ` Kishon Vijay Abraham I
2015-06-24 13:21       ` Kishon Vijay Abraham I
     [not found]   ` <1435060743-5511-8-git-send-email-kishon-l0cyMroinI0@public.gmane.org>
2015-06-24 11:49     ` Tony Lindgren
2015-06-24 11:49       ` Tony Lindgren
2015-06-24 11:49       ` Tony Lindgren
     [not found]       ` <20150624114907.GF4156-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org>
2015-06-24 13:06         ` Kishon Vijay Abraham I
2015-06-24 13:06           ` Kishon Vijay Abraham I
2015-06-24 13:06           ` Kishon Vijay Abraham I
2015-06-23 11:58 ` [PATCH 08/17] ARM: dts: dra7: Add dt node for the sycon pcie Kishon Vijay Abraham I
2015-06-23 11:58   ` Kishon Vijay Abraham I
2015-06-23 11:58   ` Kishon Vijay Abraham I
     [not found]   ` <1435060743-5511-9-git-send-email-kishon-l0cyMroinI0@public.gmane.org>
2015-06-24 11:50     ` Roger Quadros
2015-06-24 11:50       ` Roger Quadros
2015-06-24 11:50       ` Roger Quadros
2015-06-24 11:56       ` Roger Quadros
2015-06-24 11:56         ` Roger Quadros
2015-06-24 11:56         ` Roger Quadros
     [not found]         ` <20150624145617.e5248d466fe5b2e0eb1ca087-l0cyMroinI0@public.gmane.org>
2015-06-24 12:09           ` Tony Lindgren
2015-06-24 12:09             ` Tony Lindgren
2015-06-24 12:09             ` Tony Lindgren
     [not found]             ` <20150624120938.GG4156-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org>
2015-06-24 12:19               ` Roger Quadros
2015-06-24 12:19                 ` Roger Quadros
2015-06-24 12:19                 ` Roger Quadros
     [not found]                 ` <20150624151931.87252315d0e61ddfd55c4724-l0cyMroinI0@public.gmane.org>
2015-06-24 12:49                   ` Tony Lindgren
2015-06-24 12:49                     ` Tony Lindgren
2015-06-24 12:49                     ` Tony Lindgren
2015-06-24 13:30           ` Kishon Vijay Abraham I
2015-06-24 13:30             ` Kishon Vijay Abraham I
2015-06-24 13:30             ` Kishon Vijay Abraham I
2015-06-24 13:29       ` Kishon Vijay Abraham I [this message]
2015-06-24 13:29         ` Kishon Vijay Abraham I
2015-06-24 13:29         ` Kishon Vijay Abraham I
2015-06-23 11:58 ` [PATCH 12/17] ARM: dts: dra7: Use "syscon-phy-power" instead of "ctrl-module" in USB PHY node Kishon Vijay Abraham I
2015-06-23 11:58   ` Kishon Vijay Abraham I
2015-06-23 11:58   ` Kishon Vijay Abraham I
2015-06-23 11:59 ` [PATCH 15/17] ARM: dts: OMAP5: Use "syscon-phy-power" instead of "ctrl-module" in SATA " Kishon Vijay Abraham I
2015-06-23 11:59   ` Kishon Vijay Abraham I
2015-06-23 11:59   ` Kishon Vijay Abraham I
     [not found] ` <1435060743-5511-1-git-send-email-kishon-l0cyMroinI0@public.gmane.org>
2015-06-23 11:58   ` [PATCH 05/17] phy: omap-usb2: Add a new compatible string for USB2 PHY2 Kishon Vijay Abraham I
2015-06-23 11:58     ` Kishon Vijay Abraham I
2015-06-23 11:58     ` Kishon Vijay Abraham I
2015-06-24 11:03     ` Roger Quadros
2015-06-24 11:03       ` Roger Quadros
2015-06-24 11:03       ` Roger Quadros
     [not found]       ` <20150624140354.c3c1b0fc24dc37eae5097b4c-l0cyMroinI0@public.gmane.org>
2015-06-24 13:21         ` Kishon Vijay Abraham I
2015-06-24 13:21           ` Kishon Vijay Abraham I
2015-06-24 13:21           ` Kishon Vijay Abraham I
2015-06-23 11:58   ` [PATCH 06/17] phy: omap-usb2: use *syscon* framework to power on/off the PHY Kishon Vijay Abraham I
2015-06-23 11:58     ` Kishon Vijay Abraham I
2015-06-23 11:58     ` Kishon Vijay Abraham I
     [not found]     ` <1435060743-5511-7-git-send-email-kishon-l0cyMroinI0@public.gmane.org>
2015-06-24 11:33       ` Roger Quadros
2015-06-24 11:33         ` Roger Quadros
2015-06-24 11:33         ` Roger Quadros
2015-06-24 12:59         ` Kishon Vijay Abraham I
2015-06-24 12:59           ` Kishon Vijay Abraham I
2015-06-24 12:59           ` Kishon Vijay Abraham I
2015-06-23 11:58   ` [PATCH 09/17] ARM: dts: dra7: Use "syscon-phy-power" instead of "ctrl-module" in SATA PHY node Kishon Vijay Abraham I
2015-06-23 11:58     ` Kishon Vijay Abraham I
2015-06-23 11:58     ` Kishon Vijay Abraham I
2015-06-24 12:06     ` Roger Quadros
2015-06-24 12:06       ` Roger Quadros
2015-06-24 12:06       ` Roger Quadros
     [not found]       ` <20150624150602.0e73fc8f9dda1624f22059db-l0cyMroinI0@public.gmane.org>
2015-06-24 12:12         ` Roger Quadros
2015-06-24 12:12           ` Roger Quadros
2015-06-24 12:12           ` Roger Quadros
     [not found]           ` <20150624151257.fe5f1594db1afe4c1da8027d-l0cyMroinI0@public.gmane.org>
2015-06-24 13:31             ` Kishon Vijay Abraham I
2015-06-24 13:31               ` Kishon Vijay Abraham I
2015-06-24 13:31               ` Kishon Vijay Abraham I
2015-06-23 11:58   ` [PATCH 10/17] ARM: dts: dra7: Use "syscon-phy-power" and "syscon-pcs" in PCIe " Kishon Vijay Abraham I
2015-06-23 11:58     ` Kishon Vijay Abraham I
2015-06-23 11:58     ` Kishon Vijay Abraham I
2015-06-23 11:58   ` [PATCH 11/17] ARM: dts: dra7: Use "ti,dra7x-usb2-phy2" compatible string for USB2 PHY2 Kishon Vijay Abraham I
2015-06-23 11:58     ` Kishon Vijay Abraham I
2015-06-23 11:58     ` [PATCH 11/17] ARM: dts: dra7: Use "ti, dra7x-usb2-phy2" " Kishon Vijay Abraham I
     [not found]     ` <1435060743-5511-12-git-send-email-kishon-l0cyMroinI0@public.gmane.org>
2015-06-24 12:07       ` [PATCH 11/17] ARM: dts: dra7: Use "ti,dra7x-usb2-phy2" " Roger Quadros
2015-06-24 12:07         ` Roger Quadros
2015-06-24 12:07         ` Roger Quadros
     [not found]         ` <20150624150750.9435096c85253cfe528ebbc3-l0cyMroinI0@public.gmane.org>
2015-06-24 13:10           ` Kishon Vijay Abraham I
2015-06-24 13:10             ` Kishon Vijay Abraham I
2015-06-24 13:10             ` [PATCH 11/17] ARM: dts: dra7: Use "ti, dra7x-usb2-phy2" " Kishon Vijay Abraham I
     [not found]             ` <558AAC47.2070305-l0cyMroinI0@public.gmane.org>
2015-06-24 13:50               ` [PATCH 11/17] ARM: dts: dra7: Use "ti,dra7x-usb2-phy2" " Roger Quadros
2015-06-24 13:50                 ` Roger Quadros
2015-06-24 13:50                 ` Roger Quadros
2015-06-23 11:58   ` [PATCH 13/17] ARM: dts: am4372: Use "syscon-phy-power" instead of "ctrl-module" in USB PHY node Kishon Vijay Abraham I
2015-06-23 11:58     ` Kishon Vijay Abraham I
2015-06-23 11:58     ` Kishon Vijay Abraham I
2015-06-23 11:58   ` [PATCH 14/17] ARM: dts: OMAP5: " Kishon Vijay Abraham I
2015-06-23 11:58     ` Kishon Vijay Abraham I
2015-06-23 11:58     ` Kishon Vijay Abraham I
2015-06-23 11:59   ` [PATCH 16/17] ARM: dts: omap4: " Kishon Vijay Abraham I
2015-06-23 11:59     ` Kishon Vijay Abraham I
2015-06-23 11:59     ` Kishon Vijay Abraham I
2015-06-23 11:59   ` [PATCH 17/17] ARM: dts: omap4: Use "syscon-otghs" instead of "ctrl-module" in USB node Kishon Vijay Abraham I
2015-06-23 11:59     ` Kishon Vijay Abraham I
2015-06-23 11:59     ` Kishon Vijay Abraham I
     [not found]     ` <1435060743-5511-18-git-send-email-kishon-l0cyMroinI0@public.gmane.org>
2015-06-24 10:41       ` Tony Lindgren
2015-06-24 10:41         ` Tony Lindgren
2015-06-24 10:41         ` Tony Lindgren
2015-06-24 11:21         ` Kishon Vijay Abraham I
2015-06-24 11:21           ` Kishon Vijay Abraham I
2015-06-24 11:21           ` Kishon Vijay Abraham I
     [not found]           ` <558A92A5.5070506-l0cyMroinI0@public.gmane.org>
2015-06-24 11:46             ` Tony Lindgren
2015-06-24 11:46               ` Tony Lindgren
2015-06-24 11:46               ` Tony Lindgren
2015-06-24 12:02         ` Roger Quadros
2015-06-24 12:02           ` Roger Quadros
2015-06-24 12:02           ` Roger Quadros

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=558AB0A9.5020400@ti.com \
    --to=kishon@ti.com \
    --cc=balbi@ti.com \
    --cc=bcousson@baylibre.com \
    --cc=devicetree@vger.kernel.org \
    --cc=galak@codeaurora.org \
    --cc=gregkh@linuxfoundation.org \
    --cc=ijc+devicetree@hellion.org.uk \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-omap@vger.kernel.org \
    --cc=linux-usb@vger.kernel.org \
    --cc=linux@arm.linux.org.uk \
    --cc=mark.rutland@arm.com \
    --cc=nsekhar@ti.com \
    --cc=pawel.moll@arm.com \
    --cc=robh+dt@kernel.org \
    --cc=rogerq@ti.com \
    --cc=tony@atomide.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.