From: Pavel Machek <pavel@ucw.cz>
To: Kenneth Heitke <kheitke@codeaurora.org>
Cc: khali@linux-fr.org, ben-linux@fluff.org,
linux-arm-msm@vger.kernel.org, sdharia@codeaurora.org,
Crane Cai <crane.cai@amd.com>,
Samuel Ortiz <sameo@linux.intel.com>,
Linus Walleij <linus.walleij@stericsson.com>,
Ralf Baechle <ralf@linux-mips.org>,
srinidhi kasagar <srinidhi.kasagar@stericsson.com>,
linux-i2c@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] i2c: Single-wire Serial Bus Interface for Qualcomm MSM chipsets
Date: Tue, 10 Aug 2010 23:41:11 +0200 [thread overview]
Message-ID: <20100810214111.GA28968@elf.ucw.cz> (raw)
In-Reply-To: <4C60B8BB.2050209@codeaurora.org>
Hi!
> >>This bus driver supports the Single-wire Serial Bus Interface (SSBI)
> >>controller in the Qualcomm MSM SOCs. SSBI is not an I2C but is
> >>functionally related enough such that it is able to leaverage the I2C
> >>framework.
> >>
> >>Unlike I2C, SSBI is a point-to-point connection, and therefore there is no
> >>need to specify a slave device address. The SSBI implementation
> >>overrides the slave device address to be a device register address
> >
> >Then perhaps it should not go to drivers/i2c?
>
> I'd like to make use of the i2c framework rather than implement
> something else. Are you suggesting that the driver reside elsewhere
> (i.e. mach-msm) or that the driver not use the i2c framework?
I believe it should go elsewere at the very least. drivers/ssbi ?
Pavel
--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
WARNING: multiple messages have this Message-ID (diff)
From: Pavel Machek <pavel-+ZI9xUNit7I@public.gmane.org>
To: Kenneth Heitke <kheitke-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>
Cc: khali-PUYAD+kWke1g9hUCZPvPmw@public.gmane.org,
ben-linux-elnMNo+KYs3YtjvyW6yDsg@public.gmane.org,
linux-arm-msm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
sdharia-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org,
Crane Cai <crane.cai-5C7GfCeVMHo@public.gmane.org>,
Samuel Ortiz <sameo-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>,
Linus Walleij
<linus.walleij-0IS4wlFg1OjSUeElwK9/Pw@public.gmane.org>,
Ralf Baechle <ralf-6z/3iImG2C8G8FEW9MqTrA@public.gmane.org>,
srinidhi kasagar
<srinidhi.kasagar-0IS4wlFg1OjSUeElwK9/Pw@public.gmane.org>,
linux-i2c-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Subject: Re: [PATCH] i2c: Single-wire Serial Bus Interface for Qualcomm MSM chipsets
Date: Tue, 10 Aug 2010 23:41:11 +0200 [thread overview]
Message-ID: <20100810214111.GA28968@elf.ucw.cz> (raw)
In-Reply-To: <4C60B8BB.2050209-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>
Hi!
> >>This bus driver supports the Single-wire Serial Bus Interface (SSBI)
> >>controller in the Qualcomm MSM SOCs. SSBI is not an I2C but is
> >>functionally related enough such that it is able to leaverage the I2C
> >>framework.
> >>
> >>Unlike I2C, SSBI is a point-to-point connection, and therefore there is no
> >>need to specify a slave device address. The SSBI implementation
> >>overrides the slave device address to be a device register address
> >
> >Then perhaps it should not go to drivers/i2c?
>
> I'd like to make use of the i2c framework rather than implement
> something else. Are you suggesting that the driver reside elsewhere
> (i.e. mach-msm) or that the driver not use the i2c framework?
I believe it should go elsewere at the very least. drivers/ssbi ?
Pavel
--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
next prev parent reply other threads:[~2010-08-10 21:41 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-07-21 17:52 [PATCH] i2c: Single-wire Serial Bus Interface for Qualcomm MSM chipsets Kenneth Heitke
2010-07-21 17:52 ` Kenneth Heitke
2010-07-23 10:18 ` Daniel Glöckner
2010-07-23 18:32 ` Kenneth Heitke
2010-07-23 18:32 ` Kenneth Heitke
2010-07-23 20:48 ` Kenneth Heitke
2010-07-23 20:48 ` Kenneth Heitke
2010-07-23 20:50 ` Kenneth Heitke
2010-07-28 13:26 ` Pavel Machek
2010-07-28 13:26 ` Pavel Machek
2010-08-10 2:26 ` Kenneth Heitke
2010-08-10 2:26 ` Kenneth Heitke
2010-08-10 21:41 ` Pavel Machek [this message]
2010-08-10 21:41 ` Pavel Machek
2010-09-24 22:48 ` Kenneth Heitke
2010-09-24 22:48 ` Kenneth Heitke
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=20100810214111.GA28968@elf.ucw.cz \
--to=pavel@ucw.cz \
--cc=ben-linux@fluff.org \
--cc=crane.cai@amd.com \
--cc=khali@linux-fr.org \
--cc=kheitke@codeaurora.org \
--cc=linus.walleij@stericsson.com \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-i2c@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=ralf@linux-mips.org \
--cc=sameo@linux.intel.com \
--cc=sdharia@codeaurora.org \
--cc=srinidhi.kasagar@stericsson.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 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.