Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754187AbaDOUfE (ORCPT ); Tue, 15 Apr 2014 16:35:04 -0400 Received: from mail-vc0-f182.google.com ([209.85.220.182]:50689 "EHLO mail-vc0-f182.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753367AbaDOUe7 (ORCPT ); Tue, 15 Apr 2014 16:34:59 -0400 MIME-Version: 1.0 In-Reply-To: <1395949317-8738-1-git-send-email-robherring2@gmail.com> References: <1395949317-8738-1-git-send-email-robherring2@gmail.com> Date: Tue, 15 Apr 2014 15:34:58 -0500 Message-ID: Subject: Re: [RFC PATCH 0/4] vmlinux.lds.h clean-ups From: Rob Herring To: Arnd Bergmann Cc: "linux-arch@vger.kernel.org" , "linux-kernel@vger.kernel.org" , Russell King , Rob Herring Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Mar 27, 2014 at 2:41 PM, Rob Herring wrote: > From: Rob Herring > > Several DeviceTree related sections have been added recently to enable > early DT initialization calls. I have the need to add yet another one > in order to do a DT based earlycon. Since each section follows the same > pattern, this seemed like a good opportunity for some clean-up. So > this series creates a common macro to define these sections. with this > adding any new OF table section is a 2 line change. > > Each section ends with a zeroed struct of_device_id (except cpu_method). > Right now there is an end section and a dummy variable to create the end > marker. Really, we just need padding here, but I haven't come up with a > way for the linker script to get the struct size. > > The last patch extends the same concept to another common pattern for > section definitons. There's still others which could be easily converted > also, but I'm looking for feedback on this approach before proceeding > any further. > > Rob > > Rob Herring (4): > irqchip: align irqchip OF match table section naming > ARM: align cpu_method_of_table naming > vmlinuz.lds: define OF table sections with macros > vmlinux.lds: define conditional sections with macros > > arch/arm/kernel/devtree.c | 11 +-- > drivers/irqchip/irqchip.c | 6 +- > include/asm-generic/vmlinux.lds.h | 142 +++++++++++--------------------------- Arnd, Any comments on this series? Rob -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/