From: One Thousand Gnomes <gnomes@lxorguk.ukuu.org.uk>
To: Jiri Kosina <jikos@kernel.org>
Cc: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>,
jic23@kernel.org, benjamin.tissoires@redhat.com,
linux-input@vger.kernel.org, linux-iio@vger.kernel.org,
linux-kernel@vger.kernel.org, chaya.golan@intel.com,
daniel.drubin@intel.com, A.Bhattacharya@ulg.ac.be
Subject: Re: [PATCH 2/6] hid: intel_ish-hid: ISH Transport layer
Date: Mon, 20 Jun 2016 15:19:54 +0100 [thread overview]
Message-ID: <20160620151954.0c9094db@lxorguk.ukuu.org.uk> (raw)
In-Reply-To: <alpine.LNX.2.00.1606201129060.6874@cbobk.fhfr.pm>
On Mon, 20 Jun 2016 11:29:28 +0200 (CEST)
Jiri Kosina <jikos@kernel.org> wrote:
> On Fri, 17 Jun 2016, Srinivas Pandruvada wrote:
>
> > > > +config INTEL_ISH_HID_TRANSPORT
> > > > + bool
> > > > + default n
> > > > +
> > > > +config INTEL_ISH_HID
> > > > + bool "Intel Integrated Sensor Hub"
> > >
> > > Why can't the transport driver be built as a module?
> > In current use case for PM, we don't want anyone to unload and
> > complain.
>
> Sorry, I don't understand this explanation, could you please elaborate?
If the driver isn't loaded your machine doesn't do power management.
There are a small set of drivers that need to be loaded to get pm even if
not using that device. ISH is one of them, and unlike the others not
sucked into a standard configuration.
If it's going to get loaded due to the presence of the PCI identifiers on
any normal distribution then it's in the same category as graphics and
ADSP based audio, both of which can be modules and the only real world
impact is they get loaded a second or two later during boot.
Alan
next prev parent reply other threads:[~2016-06-20 14:20 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-06-11 12:13 [PATCH 0/6] Intel Integrated Sensor Hub Support (ISH) Srinivas Pandruvada
2016-06-11 12:13 ` [PATCH 1/6] Documentation: hid: Intel ISH HID document Srinivas Pandruvada
2016-06-17 20:43 ` Jiri Kosina
2016-06-11 12:13 ` [PATCH 2/6] hid: intel_ish-hid: ISH Transport layer Srinivas Pandruvada
2016-06-17 20:43 ` Jiri Kosina
2016-06-17 20:45 ` Jiri Kosina
2016-06-17 21:09 ` Srinivas Pandruvada
2016-06-17 21:12 ` Srinivas Pandruvada
2016-06-20 9:29 ` Jiri Kosina
2016-06-20 14:19 ` One Thousand Gnomes [this message]
2016-06-11 12:13 ` [PATCH 3/6] hid: intel-ish-hid: ipc layer Srinivas Pandruvada
2016-06-11 12:13 ` [PATCH 4/6] hid: intel-ish-hid: ISH HID client driver Srinivas Pandruvada
2016-06-11 12:13 ` [PATCH 5/6] iio: hid-sensors: use asynchronous resume Srinivas Pandruvada
2016-06-11 12:13 ` [PATCH 6/6] hid: hid-sensor-hub: Add ISH quirk Srinivas Pandruvada
2016-06-16 19:14 ` [PATCH 0/6] Intel Integrated Sensor Hub Support (ISH) Grant Likely
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=20160620151954.0c9094db@lxorguk.ukuu.org.uk \
--to=gnomes@lxorguk.ukuu.org.uk \
--cc=A.Bhattacharya@ulg.ac.be \
--cc=benjamin.tissoires@redhat.com \
--cc=chaya.golan@intel.com \
--cc=daniel.drubin@intel.com \
--cc=jic23@kernel.org \
--cc=jikos@kernel.org \
--cc=linux-iio@vger.kernel.org \
--cc=linux-input@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=srinivas.pandruvada@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®