Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1524204

Re: [PATCH v10 01/11] remoteproc: st_slim_rproc: add a slimcore rproc driver

From Vinod Koul <vinod.koul@intel.com>
Newsgroups linux.kernel
Subject Re: [PATCH v10 01/11] remoteproc: st_slim_rproc: add a slimcore rproc driver
Date 2016-11-17 10:50 +0100
Message-ID <sErup-7iB-1@gated-at.bofh.it> (permalink)
References (1 earlier) <stzbX-8gZ-23@gated-at.bofh.it> <szxTR-UG-85@gated-at.bofh.it> <sAUbE-4Wh-29@gated-at.bofh.it> <sDhGN-2pH-5@gated-at.bofh.it> <sEowx-5vR-7@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


On Wed, Nov 16, 2016 at 10:36:46PM -0800, Bjorn Andersson wrote:
> On Sun 13 Nov 21:18 PST 2016, Vinod Koul wrote:
> 
> > On Mon, Nov 07, 2016 at 01:57:35PM +0000, Peter Griffin wrote:
> > > > 
> > > > As you now make changes to the entire remoteproc Kconfig file, rather
> > > > than simply add a Kconfig symbol we can't bring this in via Vinod's tree
> > > > without providing Linus with a messy merge conflict.
> > > > 
> > > > So the remoteproc parts now has to go through my tree.
> > > 
> > > OK, I think the best approach is for Vinod to create an immutable
> > > branch with the entire fdma series on, and then both of you merge that branch into
> > > your respective trees.
> > 
> > my topic/st_fdma is immutable branch. You cna merge it, if you need a signed
> > tag, please do let me know
> > 
> 
> Hi Vinod,
> 
> It looks like you reverted the wrong Kconfig fix, the one I objected to
> was the change in drivers/remoteproc, not the one in drivers/dma.
> 
> The ST_FMDA depends on functions exposed by REMOTEPROC and
> ST_SLIM_REMOTEPROC, the latter in turn depends on REMOTEPROC, which you
> guys made user selectable - and as such should not be selected - but I
> think we should move forward and get everything merged and then we can
> go back and figure out how this should be addressed (or left alone?).
> 
> I have merged "topic/st_fdma" into rproc-next, so that I can fix up the
> now broken drivers/remoteproc/Kconfig.
> 
> We do however both need to revert the revert or there will be link
> errors if you build the dma driver with remoteproc=n. If you do this I
> can merge the topic once more and we'll keep the set of changes in sync.

Oops my bad, thanks for letting me know. I have reverted this now and
pushing out. Please do let me know if this was fine

Thanks
-- 
~Vinod

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

Re: [PATCH v10 01/11] remoteproc: st_slim_rproc: add a slimcore  rproc driver Vinod Koul <vinod.koul@intel.com> - 2016-11-14 06:10 +0100
  Re: [PATCH v10 01/11] remoteproc: st_slim_rproc: add a slimcore  rproc driver Peter Griffin <peter.griffin@linaro.org> - 2016-11-14 12:50 +0100
    Re: [PATCH v10 01/11] remoteproc: st_slim_rproc: add a slimcore  rproc driver Vinod Koul <vinod.koul@intel.com> - 2016-11-16 04:30 +0100
  Re: [PATCH v10 01/11] remoteproc: st_slim_rproc: add a slimcore  rproc driver Bjorn Andersson <bjorn.andersson@linaro.org> - 2016-11-17 07:40 +0100
    Re: [PATCH v10 01/11] remoteproc: st_slim_rproc: add a slimcore  rproc driver Vinod Koul <vinod.koul@intel.com> - 2016-11-17 10:50 +0100
      Re: [PATCH v10 01/11] remoteproc: st_slim_rproc: add a slimcore  rproc driver Bjorn Andersson <bjorn.andersson@linaro.org> - 2016-11-18 17:50 +0100

csiph-web