[PATCH 3/7] Basic Freescale MPC512x support
jrigby at freescale.com
Wed Jan 9 05:01:18 EST 2008
Olof Johansson wrote:
>> diff --git a/arch/powerpc/Kconfig b/arch/powerpc/Kconfig
>> index 66a3d8c..81c3f05 100644
>> --- a/arch/powerpc/Kconfig
>> +++ b/arch/powerpc/Kconfig
>> @@ -470,7 +470,7 @@ config PCI
>> bool "PCI support" if 40x || CPM2 || PPC_83xx || PPC_85xx || PPC_86xx \
>> || PPC_MPC52xx || (EMBEDDED && (PPC_PSERIES || PPC_ISERIES)) \
>> || PPC_PS3
>> - default y if !40x && !CPM2 && !8xx && !PPC_83xx \
>> + default y if !40x && !CPM2 && !8xx && !PPC_512x && !PPC_83xx \
>> && !PPC_85xx && !PPC_86xx
> This is getting out of control. Not a comment to this specific patch,
> but it's getting silly.
> Btw, why no PCI by default on this platform when it seemingly is default
> on 5200? I thought they were fairly similar.
PCI has issues in first silicon so I left that out.
Also contrary to popular rumor, 5121 is more like a 83xx that 52xx.
The PCI and USB are very similar to 83xx.
>> diff --git a/arch/powerpc/platforms/512x/Kconfig b/arch/powerpc/platforms/512x/Kconfig
>> new file mode 100644
>> index 0000000..399d279
>> --- /dev/null
>> +++ b/arch/powerpc/platforms/512x/Kconfig
>> @@ -0,0 +1,20 @@
>> +menu "Platform support"
>> + depends on PPC_512x
>> + prompt "Machine Type"
>> + default MPC5121_ADS
>> +config MPC5121_ADS
>> + bool "Freescale MPC5121E ADS"
>> + select DEFAULT_UIMAGE
>> + help
>> + This option enables support for the MPC5121E ADS board.
> For new platforms it makes more sense to make it possible to build
> multiplatform kernels instead of using 'choice'.
> I know some embedded board guys prefer to build a custom kernel for just
> their board (since it means extra revenue for them to spin a kernel
> for every customer), but it's still convenient to be able to build
> multiplatform kernels. Especially since there's some companies out there
> with intent to make end-user products with this cpu.
Grant made the same comment about multiplatform support.
I'll change this.
>> @@ -0,0 +1,90 @@
>> +#include <asm/system.h>
>> +#include <asm/atomic.h>
>> +#include <asm/machdep.h>
>> +#include <asm/ipic.h>
>> +#include <asm/prom.h>
>> +#include <asm/time.h>
>> +#include <asm/of_platform.h>
> Stephen will tell you to include linux/of_platform.h instead. :-)
Oops, this patch was originally done before the open firmware
unification. I guess checkpatch.pl needs an update too.
>> @@ -229,7 +229,7 @@ config NR_CPUS
>> config NOT_COHERENT_CACHE
>> - depends on 4xx || 8xx || E200
>> + depends on 4xx || 8xx || E200 || PPC_512x
>> default y
> Why do you need this, when 5200 doesn't?
More information about the Linuxppc-dev