From mboxrd@z Thu Jan 1 00:00:00 1970 From: Guenter Roeck Subject: Re: [PATCH] I2C: SiByte: Convert the driver to make use of interrupts Date: Mon, 22 Aug 2011 13:02:56 -0700 Message-ID: <1314043376.3235.105.camel@groeck-laptop> References: <1313710991-3596-1-git-send-email-mattst88@gmail.com> Reply-To: guenter.roeck-IzeFyvvaP7pWk0Htik3J/w@public.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1313710991-3596-1-git-send-email-mattst88-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> Sender: linux-i2c-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Matt Turner Cc: Jean Delvare , "linux-i2c-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , "linux-mips-6z/3iImG2C8G8FEW9MqTrA@public.gmane.org" , Ralf Baechle , "Maciej W. Rozycki" List-Id: linux-i2c@vger.kernel.org On Thu, 2011-08-18 at 19:43 -0400, Matt Turner wrote: > From: Maciej W. Rozycki > > This is a rewrite of large parts of the driver mainly so that it uses > SMBus interrupts to offload the CPU from busy-waiting on status inputs. > As a part of the overhaul of the init and exit calls, all accesses to the > hardware got converted to use accessory functions via an ioremap() cookie. > > [mattst88] Added BCM1480 interrupts and rebased minimally. > > Signed-off-by: Matt Turner > Signed-off-by: Maciej W. Rozycki Patch works fine on our target, and shows significant speed improvements for i2c accesses. Linux version 3.0.3-428-g17c1f3f (groeck@rbos-pc-13) (gcc version 4.4.1 (Debian 4.4.1-1) ) #2 SMP Mon Aug 22 12:56:41 PDT 2011 bootconsole [early0] enabled CPU revision is: 01041100 (SiByte SB1A) FPU revision is: 000f0103 Checking for the multiply/shift bug... no. Checking for the daddiu bug... no. Broadcom SiByte BCM1480 B1 (pass2) @ 900 MHz (SB-1A rev 0) Tested-by: Guenter Roeck Guenter