Bug#593340: linux-2.6: please add new Debian arch: powerpcspe
On Tue, Aug 17, 2010 at 01:04:13PM +0200, Sebastian Andrzej Siewior wrote:
This have to be a patch against trunk.
> --- /dev/null
> +++ b/debian/bin/initramfs-uimage-format
No, this belongs to the bootloader.
> @@ -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?
> @@ -0,0 +1,51 @@
> +# CONFIG_CC_OPTIMIZE_FOR_SIZE is not set
Why?
> --- /dev/null
> +++ b/debian/config/powerpcspe/defines
> @@ -0,0 +1,32 @@
> +[base]
> +flavours:
> + powerpcspe
> + powerpcspe-smp
Why two?
> +$(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?
Bastian
--
Vulcans never bluff.
-- Spock, "The Doomsday Machine", stardate 4202.1
Reply to: