From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mx1.redhat.com ([209.132.183.28]) by merlin.infradead.org with esmtp (Exim 4.80.1 #2 (Red Hat Linux)) id 1WQWMB-00044h-Sw for kexec@lists.infradead.org; Thu, 20 Mar 2014 06:21:37 +0000 Date: Thu, 20 Mar 2014 13:41:30 +0800 From: Dave Young Subject: Re: [PATCH 2/2] kexec: pass initrd position in dtb Message-ID: <20140320054130.GE4722@dhcp-16-126.nay.redhat.com> References: <1395282993-37549-1-git-send-email-wangnan0@huawei.com> <1395282993-37549-3-git-send-email-wangnan0@huawei.com> <20140320032645.GB4722@dhcp-16-126.nay.redhat.com> <20140320033459.GC4722@dhcp-16-126.nay.redhat.com> <532A6725.3010903@huawei.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <532A6725.3010903@huawei.com> List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "kexec" Errors-To: kexec-bounces+dwmw2=twosheds.infradead.org@lists.infradead.org To: Wang Nan Cc: Simon Horman , kexec@lists.infradead.org, Geng Hui On 03/20/14 at 11:57am, Wang Nan wrote: > On 2014/3/20 11:34, Dave Young wrote: > > On 03/20/14 at 11:26am, Dave Young wrote: > >> On 03/20/14 at 10:36am, Wang Nan wrote: > >>> This patch append the position of initrd to dtb when loading arm kernel > >>> and initrd without using atag. > >>> > >>> Signed-off-by: Wang Nan > >>> Cc: Simon Horman > >>> Cc: Dave Young > >>> Cc: Geng Hui > >>> --- > >>> kexec/arch/arm/kexec-zImage-arm.c | 84 +++++++++++++++++++++++++++------------ > >>> 1 file changed, 59 insertions(+), 25 deletions(-) > >>> > >>> diff --git a/kexec/arch/arm/kexec-zImage-arm.c b/kexec/arch/arm/kexec-zImage-arm.c > >>> index 8a35018..15d8829 100644 > >>> --- a/kexec/arch/arm/kexec-zImage-arm.c > >>> +++ b/kexec/arch/arm/kexec-zImage-arm.c > >>> @@ -20,6 +20,7 @@ > >>> #include "kexec-arm.h" > >>> #include "../../fs2dt.h" > >>> #include "crashdump-arm.h" > >>> +#include "libfdt_internal.h" > >> > >> I guess it's for the FDT_TAGALIGN and FDT_TAGSIZE? > >> libfdt_internal.h should be used as internal header for libfdt, a better way is > >> adding another patch which move these 2 macros to fdt.h > > > > Or moving your new function setup_dtb_prop to libfdt, I'm not sure if it works on > > other arch though. > > > >> Thanks > >> Dave > > > What about following midification (FDT_TAGSIZE is in fdt.h)? > I'll send it formally if you agree this one. Another problem is this patch is doing 2 things: * add a new function and use it for cmdline prop * append initrd prop So please split it to 2 patches. Thanks Dave _______________________________________________ kexec mailing list kexec@lists.infradead.org http://lists.infradead.org/mailman/listinfo/kexec