diff mbox

[U-Boot,v2,05/32] dm: part: Drop the common.h header

Message ID 1456784765-10788-6-git-send-email-sjg@chromium.org
State Accepted
Delegated to: Simon Glass
Headers show

Commit Message

Simon Glass Feb. 29, 2016, 10:25 p.m. UTC 
We should not include <common.h> in header files. Each C file should include
it if needed.

Signed-off-by: Simon Glass <sjg@chromium.org>
Reviewed-by: Bin Meng <bmeng.cn@gmail.com>
---

Changes in v2: None

 include/part.h | 1 -
 1 file changed, 1 deletion(-)

Comments

Simon Glass March 13, 2016, 1:52 a.m. UTC | #1 
On 29 February 2016 at 15:25, Simon Glass <sjg@chromium.org> wrote:
> We should not include <common.h> in header files. Each C file should include
> it if needed.
>
> Signed-off-by: Simon Glass <sjg@chromium.org>
> Reviewed-by: Bin Meng <bmeng.cn@gmail.com>
> ---
>
> Changes in v2: None
>
>  include/part.h | 1 -
>  1 file changed, 1 deletion(-)

Applied to u-boot-dm/next.
diff mbox

Patch

diff --git a/include/part.h b/include/part.h
index d1e9d0f..140c9b6 100644
--- a/include/part.h
+++ b/include/part.h
@@ -8,7 +8,6 @@ 
 #define _PART_H
 
 #include <ide.h>
-#include <common.h>
 
 struct blk_desc {
 	int		if_type;	/* type of the interface */