Re: [for dora] [PATCH v2 01/12] wl18xx-firmware: specify the branch being used for this version

[Denys Dmytriyenko <denys@ti.com>]

On Wed, Sep 03, 2014 at 03:13:24PM -0400, Cooper Jr., Franklin wrote:
> 
> 
> > -----Original Message-----
> > From: Dmytriyenko, Denys
> > Sent: Wednesday, September 03, 2014 2:04 PM
> > To: Cooper Jr., Franklin
> > Cc: R, Karthik; meta-arago@arago-project.org
> > Subject: Re: [meta-arago] [for dora] [PATCH v2 01/12] wl18xx-firmware:
> > specify the branch being used for this version
> > 
> > On Wed, Sep 03, 2014 at 02:23:43PM -0400, Cooper Jr., Franklin wrote:
> > >
> > >
> > > > -----Original Message-----
> > > > From: Dmytriyenko, Denys
> > > > Sent: Wednesday, September 03, 2014 9:45 AM
> > > > To: Cooper Jr., Franklin
> > > > Cc: R, Karthik; meta-arago@arago-project.org
> > > > Subject: Re: [meta-arago] [for dora] [PATCH v2 01/12] wl18xx-firmware:
> > > > specify the branch being used for this version
> > > >
> > > > On Wed, Sep 03, 2014 at 02:42:45PM +0000, Cooper Jr., Franklin wrote:
> > > > > Acked-by: Franklin Cooper Jr. <fcooper@ti.com>
> > > >
> > > > Do you use Dora?
> > 
> > > [Franklin] I'm not sure if I understand your question. I try to review
> > > patches no matter what branch it is for if I have time.
> > 
> > I was just curious if you had any specific interest in these patches for the old
> > branch, that's all.
> > 
> > Don't get me wrong. I do appreciate your help with the review process! And
> > I'm very grateful for your support!
> > 
> > But these patches are just backports from master and were previously
> > reviewed and Acked already, so I was just wondering why would you spend
> > your time on them and skip all my recent patches I had submitted:

> [Franklin] Just making sure that backports were done right since there are 
> always room for error. For AMSDK SDK 7.0 users I still recommend that they 
> use dora if they can't use the SDK config. So I am interested in making sure 
> things that are pulled in to dora isn't breaking things for am335x and 
> am437x.

I thought AMSDK 7.0 was Dylan based... I don't think we ever productized Dora, 
so GLSDK will be the first one, while we already switched to Daisy.


> > http://thread.gmane.org/gmane.linux.embedded.yocto.meta-arago/2958
> > http://thread.gmane.org/gmane.linux.embedded.yocto.meta-arago/2960
> > http://thread.gmane.org/gmane.linux.embedded.yocto.meta-arago/3005
> > http://thread.gmane.org/gmane.linux.embedded.yocto.meta-arago/3022
> > 
> > I was just wondering about the selection process... If it's just a matter of
> > time availability, that is fine with me too - not an issue at all.

> [Franklin] Honest answer is time and an equally bad answer is sometimes I 
> miss things when scanning my inbox.

No big deal, thanks.


> > > > > > -----Original Message-----
> > > > > > From: meta-arago-bounces@arago-project.org [mailto:meta-arago-
> > > > > > bounces@arago-project.org] On Behalf Of R, Karthik
> > > > > > Sent: Wednesday, September 03, 2014 7:41 AM
> > > > > > To: meta-arago@arago-project.org
> > > > > > Subject: [meta-arago] [for dora] [PATCH v2 01/12] wl18xx-firmware:
> > > > > > specify the branch being used for this version
> > > > > >
> > > > > > From: Denys Dmytriyenko <denys@ti.com>
> > > > > >
> > > > > > Bitbake fetcher now requires the branch to be specified when SRCREV
> > > > > > is not on master
> > > > > >
> > > > > > Signed-off-by: Denys Dmytriyenko <denys@ti.com>
> > > > > > Acked-by: Chase Maupin <chase.maupin@ti.com>
> > > > > > Acked-by: Franklin Cooper Jr. <fcooper@ti.com>
> > > > > > Tested-by: Karthik Ramanan <a0393906@ti.com>
> > > > > > ---
> > > > > >  .../recipes-bsp/wl18xx-firmware/wl18xx-firmware_r8.a8.10.bb
> > | 2
> > > > +-
> > > > > >  1 file changed, 1 insertion(+), 1 deletion(-)
> > > > > >
> > > > > > diff --git a/meta-arago-extras/recipes-bsp/wl18xx-firmware/wl18xx-
> > > > > > firmware_r8.a8.10.bb b/meta-arago-extras/recipes-bsp/wl18xx-
> > > > > > firmware/wl18xx-firmware_r8.a8.10.bb
> > > > > > index 5db1c99..b59f1c4 100644
> > > > > > --- a/meta-arago-extras/recipes-bsp/wl18xx-firmware/wl18xx-
> > > > > > firmware_r8.a8.10.bb
> > > > > > +++ b/meta-arago-extras/recipes-bsp/wl18xx-firmware/wl18xx-
> > > > > > firmware_r8.a8.10.bb
> > > > > > @@ -13,7 +13,7 @@ RCONFLICTS_${PN} += "wl12xx-firmware"
> > > > > >
> > > > > >  # Tag: ol_r8.a8.10
> > > > > >  SRCREV = "89dc93a5c23fd69ebacd64c936ba5aabf173c29b"
> > > > > > -SRC_URI = "git://github.com/TI-
> > OpenLink/wl18xx_fw.git;protocol=git
> > > > > > \
> > > > > > +SRC_URI = "git://github.com/TI-
> > > > > > OpenLink/wl18xx_fw.git;protocol=git;branch=mbss \
> > > > > >             file://0001-Add-Makefile-for-SDK.patch \
> > > > > >            "
> > > > > >
> > > > > > --
> > > > > > 2.0.1
> > > > > >
> > > > > > _______________________________________________
> > > > > > meta-arago mailing list
> > > > > > meta-arago@arago-project.org
> > > > > > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> > > > > _______________________________________________
> > > > > meta-arago mailing list
> > > > > meta-arago@arago-project.org
> > > > > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago