From: Paul Kocialkowski <paul.kocialkowski@bootlin.com>
To: Bin Liu <b-liu@ti.com>, Maxime Ripard <maxime.ripard@bootlin.com>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
linux-kernel@vger.kernel.org, linux-usb@vger.kernel.org,
Chen-Yu Tsai <wens@csie.org>
Subject: Re: [PATCH] usb: musb: Support gadget mode when the port is set to dual role
Date: Fri, 22 Mar 2019 14:34:41 +0100 [thread overview]
Message-ID: <bf2e01b54009e074140aedd0159503f14fdeb450.camel@bootlin.com> (raw)
In-Reply-To: <20190322132846.GD25852@uda0271908>
Le vendredi 22 mars 2019 à 08:28 -0500, Bin Liu a écrit :
> Again, think about an embedded product, if dr_mode is 'otg' which
> indicates the peripheral mode will be used at some point, when and how
> to load the gadget driver if it is not loaded automatically when Linux
> boots up? the end user doesn't have access to the console.
Why should we think of an embedded product where the end user doesn't
have access to the console? Unless I'm mistaken, the Linux kernel
doesn't target commercial products where users are powerless in
particular, and leaves out all other use cases (which may or may not be
commercial).
I don't think this assumption makes any sense in Linux as a project (or
that it's sane in any context of software development for that matter,
but that's beside the point).
> > Because no other controller requires it and therefore it's not
> > standard and violates the principle of least surprise?
>
> I know no other controller does this, but this doesn't mean it is not
> standard.
>
> > And even without taking this into account, there's also the fact that
> > while the *hardware* can do dual role, the software might decide
> > otherwise. If I don't want to have support for any gadget (at all) in
> > the end system, then why should I be forced to compile and load
> > something I don't even want to use in the first place?
>
> then dr_mode should be set to 'host' instead, you don't have to load a
> gadget if peripheral mode will never be used.
I disagree: dr_mode describes the hardware capabilities, not what the
software does with it.
--
Paul Kocialkowski, Bootlin
Embedded Linux and kernel engineering
https://bootlin.com
next prev parent reply other threads:[~2019-03-22 13:34 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-03-28 21:52 Paul Kocialkowski
2018-03-29 9:23 ` Maxime Ripard
2018-03-29 11:57 ` Paul Kocialkowski
2018-04-03 9:29 ` Maxime Ripard
2018-04-21 10:51 ` Paul Kocialkowski
2018-04-20 14:25 ` Bin Liu
2018-04-21 10:59 ` Paul Kocialkowski
2018-04-21 14:34 ` Bin Liu
2018-04-30 21:08 ` Paul Kocialkowski
2018-05-01 12:25 ` Bin Liu
2018-05-01 13:26 ` Paul Kocialkowski
2018-05-01 16:22 ` Bin Liu
2019-03-21 10:02 ` Bin Liu
2019-03-21 13:01 ` Maxime Ripard
2019-03-21 16:41 ` Greg Kroah-Hartman
2019-03-22 12:46 ` Bin Liu
2019-03-22 13:09 ` Maxime Ripard
2019-03-22 13:28 ` Bin Liu
2019-03-22 13:34 ` Paul Kocialkowski [this message]
2019-03-22 13:44 ` Bin Liu
2019-03-22 13:46 ` Maxime Ripard
2019-03-22 14:04 ` Bin Liu
2019-03-22 13:10 ` Paul Kocialkowski
2019-03-22 13:36 ` Bin Liu
2019-03-22 13:37 ` Paul Kocialkowski
2019-03-22 13:46 ` 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=bf2e01b54009e074140aedd0159503f14fdeb450.camel@bootlin.com \
--to=paul.kocialkowski@bootlin.com \
--cc=b-liu@ti.com \
--cc=gregkh@linuxfoundation.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-usb@vger.kernel.org \
--cc=maxime.ripard@bootlin.com \
--cc=wens@csie.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Powered by JetHome