mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: "Kuppuswamy, Sathyanarayanan" <sathyaosid@gmail.com>
To: Andy Shevchenko <andy.shevchenko@gmail.com>,
	Peter Rosin <peda@axentia.se>,
	"Krogerus, Heikki" <heikki.krogerus@linux.intel.com>
Cc: Kuppuswamy Sathyanarayanan 
	<sathyanarayanan.kuppuswamy@linux.intel.com>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH v1 1/1] mux: Add new API to get mux_control ref by device name.
Date: Sat, 8 Jul 2017 16:28:10 -0700	[thread overview]
Message-ID: <c7ec471a-2b82-68c5-b7bb-d2837c4025e4@gmail.com> (raw)
In-Reply-To: <CAHp75Ve-x6Vfv039JNzAcONiMw0QFAJSzXua7bcJG8Rxu1b95g@mail.gmail.com>

Hi Andy,


On 7/8/2017 2:26 PM, Andy Shevchenko wrote:
> On Sun, Jul 9, 2017 at 12:12 AM, Peter Rosin <peda@axentia.se> wrote:
>> On 2017-07-08 00:03, sathyanarayanan.kuppuswamy@linux.intel.com wrote:
>>> From: Kuppuswamy Sathyanarayanan <sathyanarayanan.kuppuswamy@linux.intel.com>
>>>
>>> Currently this driver only provides a single API, mux_control_get() to
>>> get mux_control reference based on mux_name, and also this API has tight
>>> dependency on device tree node. For devices, that does not use device
>>> tree, it makes it difficult to use this API. This patch adds new
>>> API to access mux_control reference based on device name, chip index and
>>> controller index value.
>> I assume this is for the Intel USB Multiplexer that you sent a driver for
>> a month or so ago? If so, you still have not answered these questions:
>>
>>     Is any other consumer in the charts at all? Can this existing consumer
>>     ever make use of some other mux? If the answer to both those questions
>>     are 'no', then I do not see much point in involving the mux subsystem at
>>     all. The Broxton USB PHY driver could just as well write to the register
>>     all by itself, no?
>>
>> that I asked in https://lkml.org/lkml/2017/5/31/58
>>
>> What is the point of that driver?
> Without Heikki's blessing, NAK for this activity.
I dropped the idea of Intel USB MUX driver after my last discussion with 
Hans. He is currently working on a solution for this issue. Once its 
merged, May be I can try improving it handle my use cases.

But I submitted these changes because it can be useful if any one wants 
to use MUX framework for non-dt case.
>

  reply	other threads:[~2017-07-08 23:28 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-07-08 19:52 kbuild test robot
2017-07-07 22:03 ` sathyanarayanan.kuppuswamy
2017-07-08 19:52   ` [PATCH] mux: fix err_cast.cocci warnings kbuild test robot
2017-07-08 21:12   ` [PATCH v1 1/1] mux: Add new API to get mux_control ref by device name Peter Rosin
2017-07-08 21:26     ` Andy Shevchenko
2017-07-08 23:28       ` Kuppuswamy, Sathyanarayanan [this message]
2017-07-08 23:24     ` Kuppuswamy, Sathyanarayanan
2017-07-10 10:07       ` Peter Rosin
2017-07-10 21:37         ` sathyanarayanan kuppuswamy

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=c7ec471a-2b82-68c5-b7bb-d2837c4025e4@gmail.com \
    --to=sathyaosid@gmail.com \
    --cc=andy.shevchenko@gmail.com \
    --cc=heikki.krogerus@linux.intel.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=peda@axentia.se \
    --cc=sathyanarayanan.kuppuswamy@linux.intel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®