public inbox for linux-arm-kernel@lists.infradead.org 
 help / color / mirror / Atom feed
* [PATCH v3] ARM: dts: da850-evm: drop VPIF endpoints
@ 2017-06-15 21:57 Kevin Hilman
  2017-07-17 11:00 ` Sekhar Nori
  0 siblings, 1 reply; 3+ messages in thread
From: Kevin Hilman @ 2017-06-15 21:57 UTC (permalink / raw)
  To: linux-arm-kernel

Drop the unused endpoints.  They should only be used when there is
an actual remote-endpoint connected.

Signed-off-by: Kevin Hilman <khilman@baylibre•com>
---
Since v2: actually CC DT maintainers
Since v1: attempt to Cc DT maintainers :(

 arch/arm/boot/dts/da850-evm.dts | 21 ---------------------
 1 file changed, 21 deletions(-)

diff --git a/arch/arm/boot/dts/da850-evm.dts b/arch/arm/boot/dts/da850-evm.dts
index a423e8ebfb37..67e72bc72e80 100644
--- a/arch/arm/boot/dts/da850-evm.dts
+++ b/arch/arm/boot/dts/da850-evm.dts
@@ -301,25 +301,4 @@
 	pinctrl-names = "default";
 	pinctrl-0 = <&vpif_capture_pins>, <&vpif_display_pins>;
 	status = "okay";
-
-	/* VPIF capture port */
-	port at 0 {
-		vpif_input_ch0: endpoint at 0 {
-			reg = <0>;
-			bus-width = <8>;
-		};
-
-		vpif_input_ch1: endpoint at 1 {
-			reg = <1>;
-			bus-width = <8>;
-			data-shift = <8>;
-		};
-	};
-
-	/* VPIF display port */
-	port at 1 {
-		vpif_output_ch0: endpoint {
-			bus-width = <8>;
-		};
-	};
 };
-- 
2.9.3

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* [PATCH v3] ARM: dts: da850-evm: drop VPIF endpoints
  2017-06-15 21:57 [PATCH v3] ARM: dts: da850-evm: drop VPIF endpoints Kevin Hilman
@ 2017-07-17 11:00 ` Sekhar Nori
  2017-07-17 17:47   ` Kevin Hilman
  0 siblings, 1 reply; 3+ messages in thread
From: Sekhar Nori @ 2017-07-17 11:00 UTC (permalink / raw)
  To: linux-arm-kernel

Hi Kevin,

On Friday 16 June 2017 03:27 AM, Kevin Hilman wrote:
> Drop the unused endpoints.  They should only be used when there is
> an actual remote-endpoint connected.
> 
> Signed-off-by: Kevin Hilman <khilman@baylibre•com>

A similar patch is needed for da850-lcdk too. In fact with v4.13-rc1, 
without it, we get a warning splat that starts with:

WARNING: CPU: 0 PID: 129 at fs/sysfs/dir.c:31 sysfs_warn_dup+0x5c/0x7c
sysfs: cannot create duplicate filename '/bus/platform/devices/vpif_capture'

I am adding this patch to fixes and would like to send this along with 
a similar LCDK patch (hopefully both for -rc2). While committing this I 
will add "unused" to subject line too.

Thanks,
Sekhar

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [PATCH v3] ARM: dts: da850-evm: drop VPIF endpoints
  2017-07-17 11:00 ` Sekhar Nori
@ 2017-07-17 17:47   ` Kevin Hilman
  0 siblings, 0 replies; 3+ messages in thread
From: Kevin Hilman @ 2017-07-17 17:47 UTC (permalink / raw)
  To: linux-arm-kernel

Sekhar Nori <nsekhar@ti•com> writes:

> Hi Kevin,
>
> On Friday 16 June 2017 03:27 AM, Kevin Hilman wrote:
>> Drop the unused endpoints.  They should only be used when there is
>> an actual remote-endpoint connected.
>> 
>> Signed-off-by: Kevin Hilman <khilman@baylibre•com>
>
> A similar patch is needed for da850-lcdk too. In fact with v4.13-rc1, 
> without it, we get a warning splat that starts with:
>
> WARNING: CPU: 0 PID: 129 at fs/sysfs/dir.c:31 sysfs_warn_dup+0x5c/0x7c
> sysfs: cannot create duplicate filename '/bus/platform/devices/vpif_capture'
>
> I am adding this patch to fixes and would like to send this along with 
> a similar LCDK patch (hopefully both for -rc2). While committing this I 
> will add "unused" to subject line too.

Patch for LCDK sent, with "unused" added to the subject line.

Kevin

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2017-07-17 17:47 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-06-15 21:57 [PATCH v3] ARM: dts: da850-evm: drop VPIF endpoints Kevin Hilman
2017-07-17 11:00 ` Sekhar Nori
2017-07-17 17:47   ` Kevin Hilman

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox