Re: [M68KNOMMU 06/06]: modify Makefiles to support common coldfire directory

[Greg Ungerer <gerg@snapgear.com>]

Hi Sam,

Sam Ravnborg wrote:
> On Fri, Feb 01, 2008 at 05:37:56PM +1000, Greg Ungerer wrote:
>> Modify Makefiles to support separate coldfire platform directory.
>>
>> Currently the common ColdFire CPU family code sits in the
>> arch/m68knommu/platform/5307 directory. This is confusing, the files
>> containing this common code are in no way specific to the 5307 ColdFire.
>> Create an arch/m68knommu/platform/coldfire directory to contain this
>> common code. Other m68knommu CPU varients do not need use this code
>> though, so it doesn't make sense to move it to arch/m68knommu/kernel.
>>
>> Signed-off-by: Greg Ungerer <gerg@uclinux.org
>>
>>
>> diff -Naurp linux-2.6.24/arch/m68knommu/Makefile linux-2.6.24.1/arch/m68knommu/Makefile
>> --- linux-2.6.24/arch/m68knommu/Makefile	2008-01-25 08:58:37.000000000 +1000
>> +++ linux-2.6.24.1/arch/m68knommu/Makefile	2008-01-31 17:04:07.000000000 +1000
>> @@ -61,17 +61,17 @@ MODEL := $(model-y)
>>  # for the selected cpu. ONLY need to define this for the non-base member
>>  # of the family.
>>  #
>> -cpuclass-$(CONFIG_M5206)	:= 5307
>> -cpuclass-$(CONFIG_M5206e)	:= 5307
>> -cpuclass-$(CONFIG_M520x)	:= 5307
>> -cpuclass-$(CONFIG_M523x)	:= 5307
>> -cpuclass-$(CONFIG_M5249)	:= 5307
>> -cpuclass-$(CONFIG_M527x)	:= 5307
>> -cpuclass-$(CONFIG_M5272)	:= 5307
>> -cpuclass-$(CONFIG_M528x)	:= 5307
>> -cpuclass-$(CONFIG_M5307)	:= 5307
>> -cpuclass-$(CONFIG_M532x)	:= 5307
>> -cpuclass-$(CONFIG_M5407)	:= 5307
>> +cpuclass-$(CONFIG_M5206)	:= coldfire
>> +cpuclass-$(CONFIG_M5206e)	:= coldfire
>> +cpuclass-$(CONFIG_M520x)	:= coldfire
>> +cpuclass-$(CONFIG_M523x)	:= coldfire
>> +cpuclass-$(CONFIG_M5249)	:= coldfire
>> +cpuclass-$(CONFIG_M527x)	:= coldfire
>> +cpuclass-$(CONFIG_M5272)	:= coldfire
>> +cpuclass-$(CONFIG_M528x)	:= coldfire
>> +cpuclass-$(CONFIG_M5307)	:= coldfire
>> +cpuclass-$(CONFIG_M532x)	:= coldfire
>> +cpuclass-$(CONFIG_M5407)	:= coldfire
> 
> I would suggest to alwyas descend into coldfire/ and
> in the coldfire/Makefile decide what to build.
> For some models thats nothing but thats OK.

I was really trying to separate the CPU class types here.
I didn't want all types pointlessly traversing all
sub-directories.


>>  cpuclass-$(CONFIG_M68328)	:= 68328
>>  cpuclass-$(CONFIG_M68EZ328)	:= 68328
>>  cpuclass-$(CONFIG_M68VZ328)	:= 68328
>> diff -Naurp linux-2.6.24/arch/m68knommu/platform/5307/Makefile linux-2.6.24.1/arch/m68knommu/platform/5307/Makefile
>> --- linux-2.6.24/arch/m68knommu/platform/5307/Makefile	2008-01-25 08:58:37.000000000 +1000
>> +++ linux-2.6.24.1/arch/m68knommu/platform/5307/Makefile	2008-01-31 16:35:54.000000000 +1000
>> @@ -16,17 +16,5 @@ ifdef CONFIG_FULLDEBUG
>>  EXTRA_AFLAGS += -DDEBUGGER_COMPATIBLE_CACHE=1
>>  endif
> 
> Please use:
> asflags-y := -DDEBUGGER_COMPATIBLE_CACHE=1
> 
> I know you did not change that line - but wanted to comment it anyway.

Ok, I'll generate a separate patch for that.

Thanks
Greg



>> diff -Naurp linux-2.6.24/arch/m68knommu/platform/coldfire/Makefile linux-2.6.24.1/arch/m68knommu/platform/coldfire/Makefile
>> --- linux-2.6.24/arch/m68knommu/platform/coldfire/Makefile	1970-01-01 10:00:00.000000000 +1000
>> +++ linux-2.6.24.1/arch/m68knommu/platform/coldfire/Makefile	2008-01-31 16:35:13.000000000 +1000
>> @@ -0,0 +1,32 @@
>> +#
>> +# Makefile for the m68knommu kernel.
>> +#
>> +
>> +#
>> +# If you want to play with the HW breakpoints then you will
>> +# need to add define this,  which will give you a stack backtrace
>> +# on the console port whenever a DBG interrupt occurs. You have to
>> +# set up you HW breakpoints to trigger a DBG interrupt:
>> +#
>> +# EXTRA_CFLAGS += -DTRAP_DBG_INTERRUPT
>> +# EXTRA_AFLAGS += -DTRAP_DBG_INTERRUPT
>> +#
>> +
>> +ifdef CONFIG_FULLDEBUG
>> +AFLAGS += -DDEBUGGER_COMPATIBLE_CACHE=1
>> +endif
> 
> Please use:
> asflags-$(CONFIG_FULL_DEBUG) := -DDEBUGGER_COMPATIBLE_CACHE=1
> 
>> +
>> +obj-$(CONFIG_COLDFIRE)	+= dma.o entry.o vectors.o
>> +obj-$(CONFIG_M5206)	+= timers.o
>> +obj-$(CONFIG_M5206e)	+= timers.o
>> +obj-$(CONFIG_M520x)	+= pit.o
>> +obj-$(CONFIG_M523x)	+= pit.o
>> +obj-$(CONFIG_M5249)	+= timers.o
>> +obj-$(CONFIG_M527x)	+= pit.o
>> +obj-$(CONFIG_M5272)	+= timers.o
>> +obj-$(CONFIG_M528x)	+= pit.o
>> +obj-$(CONFIG_M5307)	+= timers.o
>> +obj-$(CONFIG_M532x)	+= timers.o
>> +obj-$(CONFIG_M5407)	+= timers.o
>> +
>> +extra-y := head.o
> 
> 	Sam
> 


-- 
------------------------------------------------------------------------
Greg Ungerer  --  Chief Software Dude       EMAIL:     gerg@snapgear.com
SnapGear -- a Secure Computing Company      PHONE:       +61 7 3435 2888
825 Stanley St,                             FAX:         +61 7 3891 3630
Woolloongabba, QLD, 4102, Australia         WEB: http://www.SnapGear.com