From: Chris Wilson <chris-Y6uKTt2uX1cEflXRtASbqLVCufUGDwFn@public.gmane.org>
To: Jean Delvare <khali-PUYAD+kWke1g9hUCZPvPmw@public.gmane.org>,
Marek Otahal <markotahal-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
Cc: Andrew Morton
<akpm-de/tnXTf+JLsfHDXvbKv3WD2FQJk+8+b@public.gmane.org>,
bugzilla-daemon-590EEB7GvNiWaY/ihj7yzEB+6BGkLq7r@public.gmane.org,
linux-i2c-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
Keith Packard <keithp-aN4HjG94KOLQT0dZR+AlfA@public.gmane.org>
Subject: Re: [Bug 35572] eeprom module: modprobing hangs
Date: Thu, 02 Jun 2011 21:30:05 +0100 [thread overview]
Message-ID: <d08817$5rsvf@azsmga001.ch.intel.com> (raw)
In-Reply-To: <20110602161742.1b22051a-R0o5gVi9kd7kN2dkZ6Wm7A@public.gmane.org>
On Thu, 2 Jun 2011 16:17:42 +0200, Jean Delvare <khali-PUYAD+kWke1g9hUCZPvPmw@public.gmane.org> wrote:
> For the long term, I think that the i2c implementation of i915 should
> be reworked. Having two i2c adapters per actual bus, one for hardware
> controlled and one for software controlled, is confusing and dangerous.
> And creating a bus on the fly in the middle of a transaction is just as
> bad. If you really can't instantiate only the one you need, then you
> should instantiate the software controlled one (i2c_bit_add_adapter),
> then update the algorithm to switch to hardware controlled where
> possible, and ultimately undo that update if the hardware controller
> turns out to be non-working. I can't write the code as I don't have any
> supported piece of hardware and I'm not familiar with the code, but I
> can help review any patch if you want.
Reading email, in reverse order...
Thanks Jean, I will try to do you've outlined.
-Chris
--
Chris Wilson, Intel Open Source Technology Centre
prev parent reply other threads:[~2011-06-02 20:30 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <bug-35572-10286@https.bugzilla.kernel.org/>
[not found] ` <bug-35572-10286-3bo0kxnWaOQUvHkbgXJLS5sdmw4N0Rt+2LY78lusg7I@public.gmane.org/>
2011-06-01 21:42 ` [Bugme-new] [Bug 35572] New: eeprom module: modprobing hangs Andrew Morton
[not found] ` <20110601144240.9525825b.akpm-de/tnXTf+JLsfHDXvbKv3WD2FQJk+8+b@public.gmane.org>
2011-06-01 21:45 ` Andrew Morton
[not found] ` <20110601144552.32dc519e.akpm-de/tnXTf+JLsfHDXvbKv3WD2FQJk+8+b@public.gmane.org>
2011-06-02 8:40 ` [Bug 35572] " Jean Delvare
2011-06-02 15:53 ` [Bugme-new] [Bug 35572] New: " Marek Otahal
[not found] ` <201106021753.50961.markotahal-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2011-06-02 14:17 ` [Bug 35572] " Jean Delvare
[not found] ` <20110602161742.1b22051a-R0o5gVi9kd7kN2dkZ6Wm7A@public.gmane.org>
2011-06-02 20:30 ` Chris Wilson [this message]
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='d08817$5rsvf@azsmga001.ch.intel.com' \
--to=chris-y6uktt2ux1ceflxrtasbqlvcufugdwfn@public.gmane.org \
--cc=akpm-de/tnXTf+JLsfHDXvbKv3WD2FQJk+8+b@public.gmane.org \
--cc=bugzilla-daemon-590EEB7GvNiWaY/ihj7yzEB+6BGkLq7r@public.gmane.org \
--cc=keithp-aN4HjG94KOLQT0dZR+AlfA@public.gmane.org \
--cc=khali-PUYAD+kWke1g9hUCZPvPmw@public.gmane.org \
--cc=linux-i2c-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=markotahal-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.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;
as well as URLs for NNTP newsgroup(s).