On Mon, Dec 20, 2010 at 04:10:47PM +0100, Michal Marek wrote:
> On Mon, Dec 06, 2010 at 02:27:49PM -0500, Arnaud Lacombe wrote:
> > Hi,
> >
> > This is an updated version of the patch I sent in mid-September to simplify the
> > unmet dependency warnings.
> >
> > - Arnaud
> >
> > ---
> >
> > This is an attempt to simplify the expressing printed by kconfig when a
> > symbol is selected but still has direct unmet dependency.
> >
> > First, the symbol reverse dependency is split in sub-expression. Then,
> > each sub-expression is checked to ensure that it does not contains the
> > unmet dependency. This removes all the false-positive symbols which
> > already have the correct dependency. Finally, only the symbol doing the
> > "select" is printed, instead of the full dependency tree.
> >
> > CC: Catalin Marinas <catalin.marinas@xxxxxxx>
> > Signed-off-by: Arnaud Lacombe <lacombar@xxxxxxxxx>
> > ---
> > scripts/kconfig/expr.c | 35 ++++++++++++++++++++++++++++++++++-
> > scripts/kconfig/lkc_proto.h | 2 +-
> > scripts/kconfig/symbol.c | 6 +++++-
> > 3 files changed, 40 insertions(+), 3 deletions(-)
>
> I applied this to kbuild-2.6.git#kconfig now. If someone encounters a
> warning that hides the actual culprit due to this patch, we will have to
> solve it somehow, but otherwise these simplified expressions are wanted
> IMO.
BTW, this called for a minor warning fix:
From: Michal Marek <mmarek@xxxxxxx>
Subject: [PATCH] kconfig: Make expr_copy() take a const argument
Fixes
scripts/kconfig/expr.c: In function âexpr_get_leftmost_symbolâ:
scripts/kconfig/expr.c:1026:2: warning: passing argument 1 of âexpr_copyâ discards qualifiers from pointer target type
scripts/kconfig/expr.c:67:14: note: expected âstruct expr *â but argument is of type âconst struct expr *â
Signed-off-by: Michal Marek <mmarek@xxxxxxx>
diff --git a/scripts/kconfig/expr.c b/scripts/kconfig/expr.c
index 65531a7..0010034 100644
--- a/scripts/kconfig/expr.c
+++ b/scripts/kconfig/expr.c
@@ -64,7 +64,7 @@ struct expr *expr_alloc_or(struct expr *e1, struct expr *e2)
return e2 ? expr_alloc_two(E_OR, e1, e2) : e1;
}
-struct expr *expr_copy(struct expr *org)
+struct expr *expr_copy(const struct expr *org)
{
struct expr *e;
diff --git a/scripts/kconfig/expr.h b/scripts/kconfig/expr.h
index b267933..76ee319 100644
--- a/scripts/kconfig/expr.h
+++ b/scripts/kconfig/expr.h
@@ -193,7 +193,7 @@ struct expr *expr_alloc_two(enum expr_type type, struct expr *e1, struct expr *e
struct expr *expr_alloc_comp(enum expr_type type, struct symbol *s1, struct symbol *s2);
struct expr *expr_alloc_and(struct expr *e1, struct expr *e2);
struct expr *expr_alloc_or(struct expr *e1, struct expr *e2);
-struct expr *expr_copy(struct expr *org);
+struct expr *expr_copy(const struct expr *org);
void expr_free(struct expr *e);
int expr_eq(struct expr *e1, struct expr *e2);
void expr_eliminate_eq(struct expr **ep1, struct expr **ep2);
--
To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
