From: =?UTF-8?Q?Horia_Geant=c4=83?= Subject: Re: [PATCH] caam: desc: Remove unused JUMP_TYPE_MASK definition Date: Tue, 15 Sep 2015 19:42:37 +0300 Message-ID: <55F84A7D.1070002@freescale.com> References: <1441304489-21738-1-git-send-email-festevam@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Cc: , , "Fabio Estevam" To: Fabio Estevam , Return-path: Received: from mail-bn1bon0142.outbound.protection.outlook.com ([157.56.111.142]:56009 "EHLO na01-bn1-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1752771AbbIOQmv (ORCPT ); Tue, 15 Sep 2015 12:42:51 -0400 In-Reply-To: <1441304489-21738-1-git-send-email-festevam@gmail.com> Sender: linux-crypto-owner@vger.kernel.org List-ID: On 9/3/2015 9:21 PM, Fabio Estevam wrote: > From: Fabio Estevam >=20 > JUMP_TYPE_MASK is defined in desc.h and it is never used, so we can > safely remove it to avoid the following build warning: >=20 > In file included from drivers/crypto/caam/desc_constr.h:7:0, > from drivers/crypto/caam/ctrl.c:15: > drivers/crypto/caam/desc.h:1495:0: warning: "JUMP_TYPE_MASK" redefine= d > #define JUMP_TYPE_MASK (0x03 << JUMP_TYPE_SHIFT) > ^ > In file included from include/linux/module.h:19:0, > from drivers/crypto/caam/compat.h:9, > from drivers/crypto/caam/ctrl.c:11: > include/linux/jump_label.h:131:0: note: this is the location of the p= revious definition > #define JUMP_TYPE_MASK 1UL >=20 > Reported-by: Olof's autobuilder > Signed-off-by: Fabio Estevam The build warning is introduced by commit a1efb01feca5 ("jump_label, locking/static_keys: Rename JUMP_LABEL_TYPE_= * and related helpers to the static_key* pattern") which currently is not in cryptodev-2.6 tree, i.e. can only be found upstream. Other than this, Reviewed-by: Horia Geant=C4=83 Horia