All of lore.kernel.org
 help / color / mirror / Atom feed
From: Wang YanQing <udknight@gmail.com>
To: hpa@zytor.com
Cc: tglx@linutronix.de, mingo@redhat.com, x86@kernel.org,
	linux-kernel@vger.kernel.org
Subject: [PATCH v2]x86:boot: Remove unneccessary headers
Date: Wed, 6 Mar 2013 00:48:06 +0800	[thread overview]
Message-ID: <20130305164806.GA8515@udknight> (raw)

asm/segment.h come from setup.S:
we used to need GDT_ENTRY_BOOT_CS
to setup gdt, but now we don't need it,
boot_gdt had been moved to kernel/head_32.S

generated/utsrelease.h come from setup.S:
we used to define kernel_version in setup.S,
but we don't need it now, because the
definition of kernel_version had been move
to boot/version.c

asm/e820.h come from setup.S:
we used to do the memory detect in setup.S,
but we don't need it now, because
we do memory detect in boot/memory.c

I can't finger out the commits import the changes
above, because all the changes happened before git
epoch :)

asm/page_types.h come from setup.S:
we used to need the __PAGE_OFFSET define
in page_types.h to compute ramdisk_max,
but now we don't need it in header.S anymore,
cf8fa920c(i386: handle an initrd in highmem (version 2))
remove it.

This patch remove the unneeded head files included
in header.S.

Signed-off-by: Wang YanQing <udknight@gmail.com>
---
Changes:
V1-V2:
1:add more detail and long long log

 arch/x86/boot/header.S | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/arch/x86/boot/header.S b/arch/x86/boot/header.S
index 9ec06a1..43a3502 100644
--- a/arch/x86/boot/header.S
+++ b/arch/x86/boot/header.S
@@ -15,11 +15,7 @@
  *
  */
 
-#include <asm/segment.h>
-#include <generated/utsrelease.h>
 #include <asm/boot.h>
-#include <asm/e820.h>
-#include <asm/page_types.h>
 #include <asm/setup.h>
 #include <asm/bootparam.h>
 #include "boot.h"
-- 
1.7.12.4.dirty

                 reply	other threads:[~2013-03-05 16:47 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20130305164806.GA8515@udknight \
    --to=udknight@gmail.com \
    --cc=hpa@zytor.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@redhat.com \
    --cc=tglx@linutronix.de \
    --cc=x86@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.