From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kenneth Heitke Subject: Re: [RFC PATCH] slimbus: Linux driver framework for SLIMbus. Date: Thu, 11 Aug 2011 14:51:19 -0600 Message-ID: <4E4440C7.70208@codeaurora.org> References: <1313019091-15354-1-git-send-email-kheitke@codeaurora.org> <201108111455.56319.arnd@arndb.de> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-15; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <201108111455.56319.arnd@arndb.de> Sender: linux-doc-owner@vger.kernel.org To: Arnd Bergmann Cc: davidb@codeaurora.org, bryanh@codeaurora.org, linux-arm-msm@vger.kernel.org, Sagar Dharia , rdunlap@xenotime.net, rmk+kernel@arm.linux.org.uk, john.stultz@linaro.org, akpm@linux-foundation.org, ohad@wizery.com, gregkh@suse.de, stefanr@s5r6.in-berlin.de, lethal@linux-sh.org, linville@tuxdriver.com, zajec5@gmail.com, linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org List-Id: linux-arm-msm@vger.kernel.org On 08/11/2011 06:55 AM, Arnd Bergmann wrote: > On Thursday 11 August 2011, Kenneth Heitke wrote: >> From: Sagar Dharia >> >> SLIMbus (Serial Low Power Interchip Media Bus) is a specification >> developed by MIPI (Mobile Industry Processor Interface) alliance. > > Hi Kenneth and Sagar, > > On a very high level, I think the driver is in the right place here, > and it's appropriate to have a new bus type along with similar > existing bus_types like i2c. It's also good to see that you have > a good documentation file for this bus. > > My main issue with the driver is the device registration method > that now looks a bit aged. More about this below. > Arnd, Thank you for taking the time to review the driver. I appreciate your comments and will roll them in to the next revision of the driver. I'll also try to follow up with any questions and concerns that you raised. thanks, Ken -- Sent by an employee of the Qualcomm Innovation Center, Inc. The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum.