Bug#593340: linux-2.6: please add new Debian arch: powerpcspe
* Bastian Blank | 2010-08-17 13:25:05 [+0200]:
Thanks for the quick response :)
>On Tue, Aug 17, 2010 at 01:04:13PM +0200, Sebastian Andrzej Siewior wrote:
>
>This have to be a patch against trunk.
will redo.
>> --- /dev/null
>> +++ b/debian/bin/initramfs-uimage-format
>
>No, this belongs to the bootloader.
Huh? The bootloader expects an uImage header around the initrd. So you
want me to come up with a bootloader package which puts the uImage
header around i.e. provides this script?
>> @@ -74,6 +75,7 @@ initramfs-tools: update-initramfs
>>
>> [relations]
>> gcc-4.3: gcc-4.3
>> +gcc-4.4: gcc-4.4
>
>Already done.
>
>> new file mode 100644
>> index 0000000..015c6d9
>> --- /dev/null
>> +++ b/debian/config/powerpcspe/config
>
>Why can't you reuse powerpc/config?
How does reuse work? A copy of the file or something like
powerpc/config powerpcspe/config
in the defines file?
I think it could work. CONFIG_PPC_PMAC is for different
platform/architecture. So I guess once it sees CONFIG_BOOKE it will drop
the remaining thigs like RTAS, XMON or PMAC drivers.
I would prefer not to have any FB devices built-in. Besides that, it
looks only driver specific.
>
>> @@ -0,0 +1,51 @@
>> +# CONFIG_CC_OPTIMIZE_FOR_SIZE is not set
>
>Why?
I haven't done benchmarks on Os vs O2 I just prefer the latter. Giving
the fact that I now have more modules as usual it would be probably
better to go for the size.
>
>> --- /dev/null
>> +++ b/debian/config/powerpcspe/defines
>> @@ -0,0 +1,32 @@
>> +[base]
>> +flavours:
>> + powerpcspe
>> + powerpcspe-smp
>
>Why two?
One with SMP support, the other without. SMP requires additional memory
coherents and spinlocks are not removed in an UP boot as it is the case
on x86 for instance.
>> +$(STAMPS_DIR)/build-devicetree_$(ARCH): SOURCE_DIR=$(BUILD_DIR)/source
>> +$(STAMPS_DIR)/build-devicetree_$(ARCH): DIR=$(BUILD_DIR)/build-devicetree_$(ARCH)
>> +$(STAMPS_DIR)/build-devicetree_$(ARCH): $(STAMPS_DIR)/source
>> + rm -rf $(DIR)
>> + mkdir -p $(DIR)
>> + for name in mpc8536ds mpc8572ds; do \
>> + dtc -O dtb -b 0 -p 1024 \
>> + -o $(DIR)/$$name-$(UPSTREAMVERSION).dtb \
>> + $(SOURCE_DIR)/arch/$(KERNEL_ARCH)/boot/dts/$$name.dts; \
>> + done
>> + @$(stamp)
>
>No hardcoding. Why does this need to be seperate from the kernel?
I wanted to avoid to build them all so picked two for now. Would you
prefer a list or "don't care build them all"? I seperated them from the
kernel image because they are the same in both kernal packages.
>Bastian
>
Sebastian
Reply to: