From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from list by lists.gnu.org with archive (Exim 4.71) id 1UMhrL-0004Vt-3i for mharc-grub-devel@gnu.org; Mon, 01 Apr 2013 12:45:27 -0400 Received: from eggs.gnu.org ([208.118.235.92]:42563) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UMhrC-0004Pk-0A for grub-devel@gnu.org; Mon, 01 Apr 2013 12:45:26 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1UMhr4-0002NX-5b for grub-devel@gnu.org; Mon, 01 Apr 2013 12:45:17 -0400 Received: from mail-ee0-f44.google.com ([74.125.83.44]:56957) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UMhr3-0002My-VK for grub-devel@gnu.org; Mon, 01 Apr 2013 12:45:10 -0400 Received: by mail-ee0-f44.google.com with SMTP id l10so1140994eei.31 for ; Mon, 01 Apr 2013 09:45:09 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=x-received:message-id:date:from:user-agent:mime-version:to:subject :references:in-reply-to:content-type:content-transfer-encoding; bh=X3UG+j77htm3E8T5Vn9CKXZvS2zc2YEPH3Ug21j44v8=; b=byJNrFWcACNVFpRrETWjx1WXyq83sKNdJREfbvyRkQewWC7eTFUYL5lcJ3N5dd9IUE gAZqx4bIHVxfCSFypra0+aGcXWECuHzlmVEB7t02Gofme9aKVbHPOACbiVnxlzSOXE9V coplXQkwW/lEEL6D3n0KSVWLNPx6ziHSmQAaY0Kq2O5P2Z7wihcDRZy8VYe3r8Lsu8xi SBePLHxyW7NPNPtqe71lbWMwvPxS77R0/PYns/MWUPVInhVbs0R6iVr17aoJYyLoLvf+ HdeKeInOOTHHdvkWrwNbRAw+LN2F9vnRq4nPjybhQls75Nzgo/7mRIPnwdcmnl6HVAb9 lqJA== X-Received: by 10.15.61.8 with SMTP id h8mr272112eex.33.1364834709105; Mon, 01 Apr 2013 09:45:09 -0700 (PDT) Received: from [192.168.56.2] ([151.36.245.148]) by mx.google.com with ESMTPS id u44sm22067104eel.7.2013.04.01.09.45.07 (version=TLSv1 cipher=ECDHE-RSA-RC4-SHA bits=128/128); Mon, 01 Apr 2013 09:45:08 -0700 (PDT) Message-ID: <5159B99C.9030304@gmail.com> Date: Mon, 01 Apr 2013 18:45:16 +0200 From: Francesco Lavra User-Agent: Mozilla/5.0 (X11; Linux i686; rv:7.0.1) Gecko/20110929 Thunderbird/7.0.1 MIME-Version: 1.0 To: The development of GNU GRUB Subject: Re: [PATCH] Add missing ARM relocation codes and fix existing ones References: <51462936.1020106@gmail.com> In-Reply-To: <51462936.1020106@gmail.com> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit X-detected-operating-system: by eggs.gnu.org: GNU/Linux 3.x [fuzzy] X-Received-From: 74.125.83.44 X-BeenThere: grub-devel@gnu.org X-Mailman-Version: 2.1.14 Precedence: list Reply-To: The development of GNU GRUB List-Id: The development of GNU GRUB List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 01 Apr 2013 16:45:26 -0000 On 03/17/2013 09:36 PM, Francesco Lavra wrote: > I'm working on the GRUB port to ARM using the EFI platform. When > generating a relocatable ARM image, some relocation entries use a > relocation code (0x33) which is not defined in include/grub/elf.h. So I > went through the latest edition of the ELF specification for the ARM > architecture (ARM IHI 0044E) and added the missing definitions to the > header file. I also noticed some differences between the existing > definitions and the spec: some relocation names are different from the > spec, some definitions aren't in the spec at all, and one > (R_ARM_IRELATIVE) has a different value than that from the spec. So > while at it I fixed those differences as well. Hence this patch. Ping? Do you see any merit in this patch? Thanks, Francesco