Re: [WILINK R8.5 2/2] wl18xx, bt-firmware: Update to R8.5

[Denys Dmytriyenko <denys@ti.com>]

On Wed, Jan 07, 2015 at 06:14:52PM +0530, Karthik Ramanan wrote:
> On 06-Jan-15 11:33 PM, Denys Dmytriyenko wrote:
> >Karthik,
> >
> >You've dropped version from the filenames of all the WiLink components, but
> >updated PV only in ti-compat-wireless. All other packages will go back to 0.0
> >version due to missing PV. And since that version sorts lower, it won't be
> >installed on top of the old version.
> >
> 
> Denys,
> 
> Yes. This was intended, the decision was taken to drop the version
> names so that the recipes can be maintained in a easier manner.
> Note that PV is not defined in many of these recipes.
> 
> I agree with your comment about the issue arising out of this
> missing PV update but since this issue will not arise from a clean
> build and that we don't have any customer on the dra7xx-evm at the
> moment who is on daisy. Can this be patch be taken in the current
> state?

I'm fine dropping the version portion from the filename, but it has to have it 
in some way. Having recipe_X.Y.Z.bb is equivalent to having PV = "X.Y.Z" 
inside the recipe. So, this patch has to be updated to add PV to all the 
recipes, since the version no longer comes from the filename. Yes, many WiLink 
recipes don't have PV set explicitly, only because it was coming from the 
filename. So, either filename should provide the version or set PV explicitly.

And requiring a clean build to avoid any versioning issues is not a good 
strategy - we do interative builds on a nightly basis. Moreover, upgrade path 
at runtime will become an important matter soon with binary feeds provided to 
developers...

-- 
Denys