From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dan.rpsys.net (dan.rpsys.net [93.97.175.187]) by mail.openembedded.org (Postfix) with ESMTP id E20A0615F5 for ; Thu, 5 Sep 2013 08:51:10 +0000 (UTC) Received: from localhost (dan.rpsys.net [127.0.0.1]) by dan.rpsys.net (8.14.4/8.14.4/Debian-2.1ubuntu1) with ESMTP id r85949dW032017; Thu, 5 Sep 2013 10:04:09 +0100 X-Virus-Scanned: Debian amavisd-new at dan.rpsys.net Received: from dan.rpsys.net ([127.0.0.1]) by localhost (dan.rpsys.net [127.0.0.1]) (amavisd-new, port 10024) with LMTP id AqwvkfFZl2xY; Thu, 5 Sep 2013 10:04:09 +0100 (BST) Received: from [192.168.3.10] (rpvlan0 [192.168.3.10]) (authenticated bits=0) by dan.rpsys.net (8.14.4/8.14.4/Debian-2.1ubuntu1) with ESMTP id r85941hF032013 (version=TLSv1/SSLv3 cipher=DHE-RSA-CAMELLIA256-SHA bits=256 verify=NOT); Thu, 5 Sep 2013 10:04:02 +0100 Message-ID: <1378371047.32427.29.camel@ted> From: Richard Purdie To: Saul Wold Date: Thu, 05 Sep 2013 09:50:47 +0100 In-Reply-To: <52282B4E.9050204@linux.intel.com> References: <1378352047.14029.51.camel@dvhart-mobl4.amr.corp.intel.com> <52282B4E.9050204@linux.intel.com> X-Mailer: Evolution 3.6.4-0ubuntu1 Mime-Version: 1.0 Cc: yunguo.wei@windriver.com, Darren Hart , Poky , Otavio Salvador , Patches and discussions about the oe-core layer Subject: Re: [poky] [PATCH 1/4] linux-firmware: Update SRCREV, pull in iwlwifi-7260 support X-BeenThere: openembedded-core@lists.openembedded.org X-Mailman-Version: 2.1.12 Precedence: list List-Id: Patches and discussions about the oe-core layer List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 05 Sep 2013 08:51:11 -0000 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit On Wed, 2013-09-04 at 23:57 -0700, Saul Wold wrote: > On 09/04/2013 08:34 PM, Darren Hart wrote: > > On Wed, 2013-09-04 at 22:04 -0300, Otavio Salvador wrote: > >> On Wed, Sep 4, 2013 at 9:18 PM, Darren Hart wrote: > >>> Fixes [YOCTO #5110] > >>> > >>> Add support for the iwlwifi 7260 adapters. This creates a new package > >>> and includes support in the default linux-firmware (everything) package. > >>> > >>> Update the iwlwifi and radeon license checksums. Extensions to the > >>> copyright date ranges were the only change to the LICENSE files. > >>> > >>> Signed-off-by: Darren Hart > >>> Cc: yunguo.wei@windriver.com > >>> --- > >>> .../linux-firmware/linux-firmware_git.bb | 10 ++++++---- > >>> 1 file changed, 6 insertions(+), 4 deletions(-) > >>> > >>> diff --git a/meta/recipes-kernel/linux-firmware/linux-firmware_git.bb b/meta/recipes-kernel/linux-firmware/linux-firmware_git.bb > >>> index 3825889..1b7db08 100644 > >>> --- a/meta/recipes-kernel/linux-firmware/linux-firmware_git.bb > >>> +++ b/meta/recipes-kernel/linux-firmware/linux-firmware_git.bb > >>> @@ -7,12 +7,12 @@ SECTION = "kernel" > >>> > >>> LICENSE = "Proprietary" > >>> > >>> -LIC_FILES_CHKSUM = "file://LICENSE.radeon;md5=e56b405656593a0c97e478513051ea0e \ > >>> +LIC_FILES_CHKSUM = "file://LICENSE.radeon;md5=9c2faab1bfca55e1510d6bde67206f9c \ > >>> file://LICENSE.dib0700;md5=f7411825c8a555a1a3e5eab9ca773431 \ > >>> file://LICENCE.xc5000;md5=1e170c13175323c32c7f4d0998d53f66 \ > >>> file://LICENCE.ralink-firmware.txt;md5=ab2c269277c45476fb449673911a2dfd \ > >>> file://LICENCE.qla2xxx;md5=4005328a134054f0fa077bdc37aa64f2 \ > >>> - file://LICENCE.iwlwifi_firmware;md5=11545778abf78c43d7644d4f171ea1c7 \ > >>> + file://LICENCE.iwlwifi_firmware;md5=8b938534f77ffd453690eb34ed84ae8b \ > >>> file://LICENCE.i2400m;md5=14b901969e23c41881327c0d9e4b7d36 \ > >>> file://LICENCE.atheros_firmware;md5=30a14c7823beedac9fa39c64fdd01a13 \ > >>> file://LICENCE.agere;md5=af0133de6b4a9b2522defd5f188afd31 \ > >>> @@ -23,10 +23,10 @@ LIC_FILES_CHKSUM = "file://LICENSE.radeon;md5=e56b405656593a0c97e478513051ea0e \ > >>> file://LICENCE.via_vt6656;md5=e4159694cba42d4377a912e78a6e850f \ > >>> " > >>> > >>> -SRCREV = "c530a75c1e6a472b0eb9558310b518f0dfcd8860" > >>> +SRCREV = "600caefd83a406540b2a789be6415e44c9b87add" > >>> PE = "1" > >>> PV = "0.0+git${SRCPV}" > >>> -PR = "r1" > >>> +PR = "r0" > >> > >> I know this is OK but I'd prefer to omit the PR when it is r0; > >> specially now we use PRServer. > > > > Well, what is the official policy here? I'm happy to respin if > > necessary, but I'd rather not burn cycles if the policy is hazy on the > > subject. > > > Darren, sorry to say it is removal when possible. Policy was hazy, however to me at least its now clear. PR values are to be handled by the PR server and therefore we're clearing them out of recipes as and when we can (same applies to INC_PR). Ultimately they should disappear, the sooner the better as far as I'm concerned. Cheers, Richard