> ifdef CONFIG_VISWS
> -MACHINE := mach-visws
> +MACHINE_C := mach-visws
> +MACHINE_H := mach-visws
> else
> -MACHINE := mach-generic
> +MACHINE_C := mach-generic
> +MACHINE_H := mach-generic
No reason to have two different variables assigned the same value.
If you are modifying this anyway consider something like:
machine-y := mach-generic
machine-$(CONFIG_VISWS) := mach-visws
And then replace $(MACHINE) with $(machine-y).
This makes it much cleaner to add summit for example.
> -CFLAGS += -Iarch/i386/$(MACHINE)
> -AFLAGS += -Iarch/i386/$(MACHINE)
> +CFLAGS += -Iinclude/asm-i386/$(MACHINE_H) -Iinclude/asm-i386/mach-generic
> +AFLAGS += -Iinclude/asm-i386/$(MACHINE_H) -Iinclude/asm-i386/mach-generic
What's wrong with:
CFLAGS += -Iarch/i386/$(MACHINE_H) -Iarch/i386/mach-generic
That should achieve the same effect?
Sam
-
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/