Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1250873 > unrolled thread
| Started by | Tony Lindgren <tony@atomide.com> |
|---|---|
| First post | 2015-10-19 18:10 +0200 |
| Last post | 2015-10-21 10:10 +0200 |
| Articles | 2 — 2 participants |
Back to article view | Back to linux.kernel
This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by
below is the oldest one visible, not the original post.
Re: [PATCH 2/2] ARM: dts: omap3: keep ssi ports by default Tony Lindgren <tony@atomide.com> - 2015-10-19 18:10 +0200
Re: [PATCH 2/2] ARM: dts: omap3: keep ssi ports by default Roger Quadros <rogerq@ti.com> - 2015-10-21 10:10 +0200
| From | Tony Lindgren <tony@atomide.com> |
|---|---|
| Date | 2015-10-19 18:10 +0200 |
| Subject | Re: [PATCH 2/2] ARM: dts: omap3: keep ssi ports by default |
| Message-ID | <qllay-Yc-5@gated-at.bofh.it> |
* Roger Quadros <rogerq@ti.com> [151014 03:48]:
> Let's keep the SSI ports disabled in the omap3.dtsi to avoid
> getting the following noise on the console for boards that don't
> use the SSI ports.
>
> "omap_ssi_port 4805a000.ssi-port: DT data is missing cawake gpio (err=-2)"
>
> As omap3-n900 uses one SSI port, mark it enabled there.
This is the wrong way around.. The SSI ports are there on all omap3 SoCs,
we do want to have the struct device created for them. It's best to have
the driver probe and then idle SSI and bail out. Marking something with
status = "disabled" should be only used if you really want to avoid kernel
touching the device at all, for example, if something is managed from
the start by a copropcessor.
Regards,
Tony
> Signed-off-by: Roger Quadros <rogerq@ti.com>
> ---
> arch/arm/boot/dts/omap3-n900.dts | 1 +
> arch/arm/boot/dts/omap3.dtsi | 2 ++
> 2 files changed, 3 insertions(+)
>
> diff --git a/arch/arm/boot/dts/omap3-n900.dts b/arch/arm/boot/dts/omap3-n900.dts
> index 5f5e0f3..bdb72fb 100644
> --- a/arch/arm/boot/dts/omap3-n900.dts
> +++ b/arch/arm/boot/dts/omap3-n900.dts
> @@ -949,6 +949,7 @@
> };
>
> &ssi_port1 {
> + status = "ok";
> pinctrl-names = "default";
> pinctrl-0 = <&ssi_pins>;
>
> diff --git a/arch/arm/boot/dts/omap3.dtsi b/arch/arm/boot/dts/omap3.dtsi
> index 8a2b253..25b0b37 100644
> --- a/arch/arm/boot/dts/omap3.dtsi
> +++ b/arch/arm/boot/dts/omap3.dtsi
> @@ -815,6 +815,7 @@
> interrupt-parent = <&intc>;
> interrupts = <67>,
> <68>;
> + status = "disabled";
> };
>
> ssi_port2: ssi-port@4805b000 {
> @@ -828,6 +829,7 @@
> interrupt-parent = <&intc>;
> interrupts = <69>,
> <70>;
> + status = "disabled";
> };
> };
> };
> --
> 2.1.4
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[toc] | [next] | [standalone]
| From | Roger Quadros <rogerq@ti.com> |
|---|---|
| Date | 2015-10-21 10:10 +0200 |
| Message-ID | <qlWD8-5sb-11@gated-at.bofh.it> |
| In reply to | #1250873 |
On 19/10/15 19:01, Tony Lindgren wrote:
> * Roger Quadros <rogerq@ti.com> [151014 03:48]:
>> Let's keep the SSI ports disabled in the omap3.dtsi to avoid
>> getting the following noise on the console for boards that don't
>> use the SSI ports.
>>
>> "omap_ssi_port 4805a000.ssi-port: DT data is missing cawake gpio (err=-2)"
>>
>> As omap3-n900 uses one SSI port, mark it enabled there.
>
> This is the wrong way around.. The SSI ports are there on all omap3 SoCs,
> we do want to have the struct device created for them. It's best to have
> the driver probe and then idle SSI and bail out. Marking something with
> status = "disabled" should be only used if you really want to avoid kernel
> touching the device at all, for example, if something is managed from
> the start by a copropcessor.
Agreed. Please ignore this patch.
cheers,
-roger
>
>
>> Signed-off-by: Roger Quadros <rogerq@ti.com>
>> ---
>> arch/arm/boot/dts/omap3-n900.dts | 1 +
>> arch/arm/boot/dts/omap3.dtsi | 2 ++
>> 2 files changed, 3 insertions(+)
>>
>> diff --git a/arch/arm/boot/dts/omap3-n900.dts b/arch/arm/boot/dts/omap3-n900.dts
>> index 5f5e0f3..bdb72fb 100644
>> --- a/arch/arm/boot/dts/omap3-n900.dts
>> +++ b/arch/arm/boot/dts/omap3-n900.dts
>> @@ -949,6 +949,7 @@
>> };
>>
>> &ssi_port1 {
>> + status = "ok";
>> pinctrl-names = "default";
>> pinctrl-0 = <&ssi_pins>;
>>
>> diff --git a/arch/arm/boot/dts/omap3.dtsi b/arch/arm/boot/dts/omap3.dtsi
>> index 8a2b253..25b0b37 100644
>> --- a/arch/arm/boot/dts/omap3.dtsi
>> +++ b/arch/arm/boot/dts/omap3.dtsi
>> @@ -815,6 +815,7 @@
>> interrupt-parent = <&intc>;
>> interrupts = <67>,
>> <68>;
>> + status = "disabled";
>> };
>>
>> ssi_port2: ssi-port@4805b000 {
>> @@ -828,6 +829,7 @@
>> interrupt-parent = <&intc>;
>> interrupts = <69>,
>> <70>;
>> + status = "disabled";
>> };
>> };
>> };
>> --
>> 2.1.4
>>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web