From mboxrd@z Thu Jan 1 00:00:00 1970 From: gregkh@suse.de (Greg KH) Date: Tue, 5 Jul 2011 19:00:49 -0700 Subject: [PATCH v5 1/1] drivers:staging:pruss: add pruss staging mfd driver. In-Reply-To: <131E5DFBE7373E4C8D813795A6AA7F08034B5C4C48@dlee06.ent.ti.com> References: <1306827939-4133-1-git-send-email-subhasish@mistralsolutions.com> <1306827939-4133-2-git-send-email-subhasish@mistralsolutions.com> <20110628210112.GA31081@kroah.com> <20110705161558.GA26309@kroah.com> <131E5DFBE7373E4C8D813795A6AA7F08034B5C4C48@dlee06.ent.ti.com> Message-ID: <20110706020049.GB15988@suse.de> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Tue, Jul 05, 2011 at 01:43:03PM -0500, Watkins, Melissa wrote: > On Tue, Jun 28, 2011 at 02:01:12PM -0700, Greg KH wrote: > > On Tue, May 31, 2011 at 01:15:39PM +0530, Subhasish Ghosh wrote: > > > > This patch adds the pruss MFD driver and associated include files. > > > > For details regarding the PRUSS please refer the folowing link: > > > > http://processors.wiki.ti.com/index.php/Programmable_Realtime_Unit_Subsystem > > > > > > > > The rational behind the MFD driver being the fact that multiple devices can > > > > be implemented on the cores independently. This is determined by the nature > > > > of the program which is loaded into the PRU's instruction memory. > > > > A device may be de-initialized and another loaded or two different devices > > > > can be run simultaneously on the two cores. > > > > It's also possible, as in our case, to implement a single device on both > > > > the PRU's resulting in improved load sharing. > > > > > > > > Signed-off-by: Subhasish Ghosh > > > > > [MW] Greg, thanks for reviewing Mistral's patch submission. I have > been following the traffic on their patches and will provide an > initial response to address some of your questions. > > > > Please refresh my memory as to why this can't get merged into the > > > "normal" part of the kernel? > > [MW] Arnd had recommended moving the driver to the staging area until > all interfaces have stabilized. One of the main reasons it cannot > currently be merged in the "normal" part of the kernel is the question > of how firmware should be configured and loaded. > (https://lkml.org/lkml/2011/5/24/230, > https://lkml.org/lkml/2011/5/10/463) > > > Dropped from my queue due to lack of response :( > > > Please resend when you get the chance, and address the above comment in > > your changelog entry (and in your TODO file as well.) > > [MW] Please let us know if Mistral still needs to resubmit their patch > or if we can continue discussions on this submission. Please resend it, with the above information in the changelog and in the TODO file. greg k-h From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755889Ab1GFCeP (ORCPT ); Tue, 5 Jul 2011 22:34:15 -0400 Received: from cantor2.suse.de ([195.135.220.15]:44827 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755773Ab1GFCeM (ORCPT ); Tue, 5 Jul 2011 22:34:12 -0400 Date: Tue, 5 Jul 2011 19:00:49 -0700 From: Greg KH To: "Watkins, Melissa" Cc: Greg KH , Subhasish Ghosh , "open list:STAGING SUBSYSTEM" , "sachi@mistralsolutions.com" , "davinci-linux-open-source@linux.davincidsp.com" , "stalin.s@mistralsolutions.com" , open list , "linux-arm-kernel@lists.infradead.org" Subject: Re: [PATCH v5 1/1] drivers:staging:pruss: add pruss staging mfd driver. Message-ID: <20110706020049.GB15988@suse.de> References: <1306827939-4133-1-git-send-email-subhasish@mistralsolutions.com> <1306827939-4133-2-git-send-email-subhasish@mistralsolutions.com> <20110628210112.GA31081@kroah.com> <20110705161558.GA26309@kroah.com> <131E5DFBE7373E4C8D813795A6AA7F08034B5C4C48@dlee06.ent.ti.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <131E5DFBE7373E4C8D813795A6AA7F08034B5C4C48@dlee06.ent.ti.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Jul 05, 2011 at 01:43:03PM -0500, Watkins, Melissa wrote: > On Tue, Jun 28, 2011 at 02:01:12PM -0700, Greg KH wrote: > > On Tue, May 31, 2011 at 01:15:39PM +0530, Subhasish Ghosh wrote: > > > > This patch adds the pruss MFD driver and associated include files. > > > > For details regarding the PRUSS please refer the folowing link: > > > > http://processors.wiki.ti.com/index.php/Programmable_Realtime_Unit_Subsystem > > > > > > > > The rational behind the MFD driver being the fact that multiple devices can > > > > be implemented on the cores independently. This is determined by the nature > > > > of the program which is loaded into the PRU's instruction memory. > > > > A device may be de-initialized and another loaded or two different devices > > > > can be run simultaneously on the two cores. > > > > It's also possible, as in our case, to implement a single device on both > > > > the PRU's resulting in improved load sharing. > > > > > > > > Signed-off-by: Subhasish Ghosh > > > > > [MW] Greg, thanks for reviewing Mistral's patch submission. I have > been following the traffic on their patches and will provide an > initial response to address some of your questions. > > > > Please refresh my memory as to why this can't get merged into the > > > "normal" part of the kernel? > > [MW] Arnd had recommended moving the driver to the staging area until > all interfaces have stabilized. One of the main reasons it cannot > currently be merged in the "normal" part of the kernel is the question > of how firmware should be configured and loaded. > (https://lkml.org/lkml/2011/5/24/230, > https://lkml.org/lkml/2011/5/10/463) > > > Dropped from my queue due to lack of response :( > > > Please resend when you get the chance, and address the above comment in > > your changelog entry (and in your TODO file as well.) > > [MW] Please let us know if Mistral still needs to resubmit their patch > or if we can continue discussions on this submission. Please resend it, with the above information in the changelog and in the TODO file. greg k-h