All of lore.kernel.org
 help / color / mirror / Atom feed
From: b-liu@ti.com (Bin Liu)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v4 4/4] ARM: dts: da850: Add the usb otg device nodeg
Date: Wed, 16 Nov 2016 11:24:24 -0600	[thread overview]
Message-ID: <20161116172424.GA9487@uda0271908> (raw)
In-Reply-To: <9a73c416-fe4c-57c3-af53-87146dae2206@ti.com>

On Wed, Nov 16, 2016 at 12:06:51PM +0530, Sekhar Nori wrote:
> On Wednesday 16 November 2016 02:49 AM, Bin Liu wrote:
> > On Tue, Nov 15, 2016 at 04:16:02PM +0530, Sekhar Nori wrote:
> >> On Thursday 03 November 2016 09:29 PM, Alexandre Bailon wrote:
> >>> This adds the device tree node for the usb otg
> >>> controller present in the da850 family of SoC's.
> >>> This also enables the otg usb controller for the lcdk board.
> >>>
> >>> Signed-off-by: Alexandre Bailon <abailon@baylibre.com>
> >>> ---
> >>>  arch/arm/boot/dts/da850-lcdk.dts |  8 ++++++++
> >>>  arch/arm/boot/dts/da850.dtsi     | 15 +++++++++++++++
> >>>  2 files changed, 23 insertions(+)
> >>>
> >>> diff --git a/arch/arm/boot/dts/da850-lcdk.dts b/arch/arm/boot/dts/da850-lcdk.dts
> >>> index 7b8ab21..9f5040c 100644
> >>> --- a/arch/arm/boot/dts/da850-lcdk.dts
> >>> +++ b/arch/arm/boot/dts/da850-lcdk.dts
> >>> @@ -158,6 +158,14 @@
> >>>  	rx-num-evt = <32>;
> >>>  };
> >>>  
> >>> +&usb_phy {
> >>> +	status = "okay";
> >>> +	};
> >>
> >> As mentioned by David already, this node needs to be removed. Please
> >> rebase this on top of latest linux-davinci/master when ready for merging
> >> (driver changes accepted).
> > 
> > Dropped this patch due to this comment.
> 
> Bin, Please do not apply dts or arch/arm/mach-davinci patches. I have a
> bunch queued through my tree and more in pipeline and it will cause
> unnecessary merge conflicts in linux-next or at Linus.

Sure, I will drop this whole set, and only apply the musb patches in the
new v5.

> 
> For future, I have asked Alexandre to send driver and dts patches as
> separate series so there is no confusion on who should apply.

I will keep in mind to ping other domain maintainers before applying
non-musb patches in future.

> 
> Thanks,
> Sekhar

Regards,
-Bin.

WARNING: multiple messages have this Message-ID (diff)
From: Bin Liu <b-liu-l0cyMroinI0@public.gmane.org>
To: Sekhar Nori <nsekhar-l0cyMroinI0@public.gmane.org>
Cc: Alexandre Bailon
	<abailon-rdvid1DuHRBWk0Htik3J/w@public.gmane.org>,
	khilman-rdvid1DuHRBWk0Htik3J/w@public.gmane.org,
	robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linux-usb-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org
Subject: Re: [PATCH v4 4/4] ARM: dts: da850: Add the usb otg device nodeg
Date: Wed, 16 Nov 2016 11:24:24 -0600	[thread overview]
Message-ID: <20161116172424.GA9487@uda0271908> (raw)
In-Reply-To: <9a73c416-fe4c-57c3-af53-87146dae2206-l0cyMroinI0@public.gmane.org>

On Wed, Nov 16, 2016 at 12:06:51PM +0530, Sekhar Nori wrote:
> On Wednesday 16 November 2016 02:49 AM, Bin Liu wrote:
> > On Tue, Nov 15, 2016 at 04:16:02PM +0530, Sekhar Nori wrote:
> >> On Thursday 03 November 2016 09:29 PM, Alexandre Bailon wrote:
> >>> This adds the device tree node for the usb otg
> >>> controller present in the da850 family of SoC's.
> >>> This also enables the otg usb controller for the lcdk board.
> >>>
> >>> Signed-off-by: Alexandre Bailon <abailon-rdvid1DuHRBWk0Htik3J/w@public.gmane.org>
> >>> ---
> >>>  arch/arm/boot/dts/da850-lcdk.dts |  8 ++++++++
> >>>  arch/arm/boot/dts/da850.dtsi     | 15 +++++++++++++++
> >>>  2 files changed, 23 insertions(+)
> >>>
> >>> diff --git a/arch/arm/boot/dts/da850-lcdk.dts b/arch/arm/boot/dts/da850-lcdk.dts
> >>> index 7b8ab21..9f5040c 100644
> >>> --- a/arch/arm/boot/dts/da850-lcdk.dts
> >>> +++ b/arch/arm/boot/dts/da850-lcdk.dts
> >>> @@ -158,6 +158,14 @@
> >>>  	rx-num-evt = <32>;
> >>>  };
> >>>  
> >>> +&usb_phy {
> >>> +	status = "okay";
> >>> +	};
> >>
> >> As mentioned by David already, this node needs to be removed. Please
> >> rebase this on top of latest linux-davinci/master when ready for merging
> >> (driver changes accepted).
> > 
> > Dropped this patch due to this comment.
> 
> Bin, Please do not apply dts or arch/arm/mach-davinci patches. I have a
> bunch queued through my tree and more in pipeline and it will cause
> unnecessary merge conflicts in linux-next or at Linus.

Sure, I will drop this whole set, and only apply the musb patches in the
new v5.

> 
> For future, I have asked Alexandre to send driver and dts patches as
> separate series so there is no confusion on who should apply.

I will keep in mind to ping other domain maintainers before applying
non-musb patches in future.

> 
> Thanks,
> Sekhar

Regards,
-Bin.
--
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

WARNING: multiple messages have this Message-ID (diff)
From: Bin Liu <b-liu@ti.com>
To: Sekhar Nori <nsekhar@ti.com>
Cc: Alexandre Bailon <abailon@baylibre.com>, <khilman@baylibre.com>,
	<robh+dt@kernel.org>, <linux-kernel@vger.kernel.org>,
	<linux-usb@vger.kernel.org>, <devicetree@vger.kernel.org>,
	<linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH v4 4/4] ARM: dts: da850: Add the usb otg device nodeg
Date: Wed, 16 Nov 2016 11:24:24 -0600	[thread overview]
Message-ID: <20161116172424.GA9487@uda0271908> (raw)
In-Reply-To: <9a73c416-fe4c-57c3-af53-87146dae2206@ti.com>

On Wed, Nov 16, 2016 at 12:06:51PM +0530, Sekhar Nori wrote:
> On Wednesday 16 November 2016 02:49 AM, Bin Liu wrote:
> > On Tue, Nov 15, 2016 at 04:16:02PM +0530, Sekhar Nori wrote:
> >> On Thursday 03 November 2016 09:29 PM, Alexandre Bailon wrote:
> >>> This adds the device tree node for the usb otg
> >>> controller present in the da850 family of SoC's.
> >>> This also enables the otg usb controller for the lcdk board.
> >>>
> >>> Signed-off-by: Alexandre Bailon <abailon@baylibre.com>
> >>> ---
> >>>  arch/arm/boot/dts/da850-lcdk.dts |  8 ++++++++
> >>>  arch/arm/boot/dts/da850.dtsi     | 15 +++++++++++++++
> >>>  2 files changed, 23 insertions(+)
> >>>
> >>> diff --git a/arch/arm/boot/dts/da850-lcdk.dts b/arch/arm/boot/dts/da850-lcdk.dts
> >>> index 7b8ab21..9f5040c 100644
> >>> --- a/arch/arm/boot/dts/da850-lcdk.dts
> >>> +++ b/arch/arm/boot/dts/da850-lcdk.dts
> >>> @@ -158,6 +158,14 @@
> >>>  	rx-num-evt = <32>;
> >>>  };
> >>>  
> >>> +&usb_phy {
> >>> +	status = "okay";
> >>> +	};
> >>
> >> As mentioned by David already, this node needs to be removed. Please
> >> rebase this on top of latest linux-davinci/master when ready for merging
> >> (driver changes accepted).
> > 
> > Dropped this patch due to this comment.
> 
> Bin, Please do not apply dts or arch/arm/mach-davinci patches. I have a
> bunch queued through my tree and more in pipeline and it will cause
> unnecessary merge conflicts in linux-next or at Linus.

Sure, I will drop this whole set, and only apply the musb patches in the
new v5.

> 
> For future, I have asked Alexandre to send driver and dts patches as
> separate series so there is no confusion on who should apply.

I will keep in mind to ping other domain maintainers before applying
non-musb patches in future.

> 
> Thanks,
> Sekhar

Regards,
-Bin.

  reply	other threads:[~2016-11-16 17:24 UTC|newest]

Thread overview: 34+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-11-03 15:59 [PATCH v4 0/4] Add DT support for DA8xx Alexandre Bailon
2016-11-03 15:59 ` Alexandre Bailon
2016-11-03 15:59 ` [PATCH v4 1/4] dt/bindings: Add binding for the DA8xx MUSB driver Alexandre Bailon
2016-11-03 15:59   ` Alexandre Bailon
2016-11-03 15:59 ` [PATCH v4 2/4] usb: musb: core: added helper function for parsing DT Alexandre Bailon
2016-11-03 15:59   ` Alexandre Bailon
2016-11-03 15:59 ` [PATCH v4 3/4] usb: musb: da8xx: Add DT support for the DA8xx driver Alexandre Bailon
2016-11-03 15:59   ` Alexandre Bailon
2016-11-03 15:59 ` [PATCH v4 4/4] ARM: dts: da850: Add the usb otg device node Alexandre Bailon
2016-11-03 15:59   ` Alexandre Bailon
2016-11-04  3:40   ` David Lechner
2016-11-15 10:46     ` Sekhar Nori
2016-11-15 10:46   ` Sekhar Nori
2016-11-15 10:46     ` Sekhar Nori
2016-11-15 10:46     ` Sekhar Nori
2016-11-15 21:19     ` [PATCH v4 4/4] ARM: dts: da850: Add the usb otg device nodeg Bin Liu
2016-11-15 21:19       ` Bin Liu
2016-11-15 21:19       ` Bin Liu
2016-11-16  6:36       ` Sekhar Nori
2016-11-16  6:36         ` Sekhar Nori
2016-11-16  6:36         ` Sekhar Nori
2016-11-16 17:24         ` Bin Liu [this message]
2016-11-16 17:24           ` Bin Liu
2016-11-16 17:24           ` Bin Liu
2016-11-16 10:35     ` [PATCH v4 4/4] ARM: dts: da850: Add the usb otg device node Alexandre Bailon
2016-11-16 10:35       ` Alexandre Bailon
2016-11-16 10:41       ` Sekhar Nori
2016-11-16 10:41         ` Sekhar Nori
2016-11-16 10:41         ` Sekhar Nori
2016-11-16 10:47         ` Alexandre Bailon
2016-11-16 10:47           ` Alexandre Bailon
2016-11-15 21:12 ` [PATCH v4 0/4] Add DT support for DA8xx Bin Liu
2016-11-15 21:12   ` Bin Liu
2016-11-15 21:12   ` Bin Liu

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=20161116172424.GA9487@uda0271908 \
    --to=b-liu@ti.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    /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.