All of lore.kernel.org
 help / color / mirror / Atom feed
From: david@lechnology.com (David Lechner)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 3/5] ARM: davinci: da8xx: add cfgchip2 to resources
Date: Wed, 16 Mar 2016 13:27:36 -0500	[thread overview]
Message-ID: <56E9A598.40308@lechnology.com> (raw)
In-Reply-To: <56E9A456.2000204@cogentembedded.com>

On 03/16/2016 01:22 PM, Sergei Shtylyov wrote:
>
>     The set_mode() method doesn't have anything to do with the
> predefined roles. What CFGCHIP2 setting do is to override the ID input
> (and also the VBUS level comparator). This is not required for the
> normal functioning of either host or peripheral AFAIR.
>

OK, so it sounds like I should just remove set_mode from the musb driver 
completely and make this configurable in the phy driver via platform 
data or device tree.

WARNING: multiple messages have this Message-ID (diff)
From: David Lechner <david@lechnology.com>
To: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>,
	Sekhar Nori <nsekhar@ti.com>, Kevin Hilman <khilman@kernel.org>,
	Alan Stern <stern@rowland.harvard.edu>, Bin Liu <b-liu@ti.com>,
	Petr Kulhavy <petr@barix.com>
Cc: Russell King <linux@arm.linux.org.uk>,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	Felipe Balbi <balbi@kernel.org>,
	linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org, linux-usb@vger.kernel.org
Subject: Re: [PATCH 3/5] ARM: davinci: da8xx: add cfgchip2 to resources
Date: Wed, 16 Mar 2016 13:27:36 -0500	[thread overview]
Message-ID: <56E9A598.40308@lechnology.com> (raw)
In-Reply-To: <56E9A456.2000204@cogentembedded.com>

On 03/16/2016 01:22 PM, Sergei Shtylyov wrote:
>
>     The set_mode() method doesn't have anything to do with the
> predefined roles. What CFGCHIP2 setting do is to override the ID input
> (and also the VBUS level comparator). This is not required for the
> normal functioning of either host or peripheral AFAIR.
>

OK, so it sounds like I should just remove set_mode from the musb driver 
completely and make this configurable in the phy driver via platform 
data or device tree.

  parent reply	other threads:[~2016-03-16 18:27 UTC|newest]

Thread overview: 41+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-03-15 22:37 [PATCH 1/5] ARM: davinci: defined missing CFGCHIP2_REFFREQ_* macros for MUSB PHY David Lechner
2016-03-15 22:37 ` David Lechner
2016-03-15 22:37 ` [PATCH 2/5] ARM: davinci: da8xx: add usb phy clocks David Lechner
2016-03-15 22:37   ` David Lechner
2016-03-16 12:27   ` Sergei Shtylyov
2016-03-16 12:27     ` Sergei Shtylyov
2016-03-16 17:58     ` David Lechner
2016-03-16 17:58       ` David Lechner
2016-03-16 18:04       ` Sergei Shtylyov
2016-03-16 18:04         ` Sergei Shtylyov
2016-03-16 18:21         ` David Lechner
2016-03-16 18:21           ` David Lechner
2016-03-15 22:37 ` [PATCH 3/5] ARM: davinci: da8xx: add cfgchip2 to resources David Lechner
2016-03-15 22:37   ` David Lechner
2016-03-15 22:45   ` Sergei Shtylyov
2016-03-15 22:45     ` Sergei Shtylyov
2016-03-16  3:46     ` David Lechner
2016-03-16  3:46       ` David Lechner
2016-03-16  4:57       ` David Lechner
2016-03-16  4:57         ` David Lechner
2016-03-16 17:38         ` Sergei Shtylyov
2016-03-16 17:38           ` Sergei Shtylyov
2016-03-16 18:14           ` David Lechner
2016-03-16 18:14             ` David Lechner
2016-03-16 18:22             ` Sergei Shtylyov
2016-03-16 18:22               ` Sergei Shtylyov
2016-03-16 18:27               ` Sergei Shtylyov
2016-03-16 18:27                 ` Sergei Shtylyov
2016-03-16 18:27               ` David Lechner [this message]
2016-03-16 18:27                 ` David Lechner
2016-03-16 11:34       ` Sergei Shtylyov
2016-03-16 11:34         ` Sergei Shtylyov
2016-03-15 22:37 ` [PATCH 4/5] usb: ohci-da8xx: Remove clock code that references mach David Lechner
2016-03-15 22:37   ` David Lechner
2016-03-16 14:50   ` Alan Stern
2016-03-16 18:00     ` David Lechner
2016-03-16 18:00       ` David Lechner
2016-03-15 22:37 ` [PATCH 5/5] usb: musb-da8xx: remove board-specific clock handling David Lechner
2016-03-15 22:37   ` David Lechner
2016-03-16 11:57   ` Sergei Shtylyov
2016-03-16 11:57     ` Sergei Shtylyov

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=56E9A598.40308@lechnology.com \
    --to=david@lechnology.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.