Re: [PATCH] powerpc/85xx: allow CONFIG_PHYS_64BIT to be selectable

2012-03-16 Thread Kumar Gala
On Feb 21, 2012, at 1:53 PM, Timur Tabi wrote: > Remove the "select PHYS_64BIT" from the Kconfig entry for the P1022DS, > so that large physical address support is a selectable option for non-CoreNet > reference boards. > > The option is enabled in mpc85xx_[smp_]defconfig so that the default is

Re: [PATCH] powerpc/85xx: allow CONFIG_PHYS_64BIT to be selectable

2012-02-23 Thread Tabi Timur-B04825
Li Yang-R58472 wrote: > It's a good point. Why can't we decide to use 32-bit/36-bit TLB at runtime > even for e500v2? That's not what PHYS_64BIT does. PHYS_64BIT determines whether phys_addr_t is a u64 or a u32. This is something that must be determined at compilation time. >> Please remem

RE: [PATCH] powerpc/85xx: allow CONFIG_PHYS_64BIT to be selectable

2012-02-23 Thread Li Yang-R58472
> Subject: Re: [PATCH] powerpc/85xx: allow CONFIG_PHYS_64BIT to be > selectable > > Li Yang-R58472 wrote: > > > The mpc85xx_defconfig does include silicons with e500v1 core which > > doesn't have the 36-bit support. Won't enabling 36-bit support by &

Re: [PATCH] powerpc/85xx: allow CONFIG_PHYS_64BIT to be selectable

2012-02-23 Thread Tabi Timur-B04825
Li Yang-R58472 wrote: > The mpc85xx_defconfig does include silicons with e500v1 core which > doesn't have the 36-bit support. Won't enabling 36-bit support by > default break the support for them? No. The kernel will detect at runtime that that it's an e500v1 core and it won't try to create 36

RE: [PATCH] powerpc/85xx: allow CONFIG_PHYS_64BIT to be selectable

2012-02-23 Thread Li Yang-R58472
> Subject: Re: [PATCH] powerpc/85xx: allow CONFIG_PHYS_64BIT to be > selectable > > Li Yang-R58472 wrote: > > > Even though the user still need to know the addressing mode that > > u-boot is using. It won't work if the addressing mode of u-boot and > >

Re: [PATCH] powerpc/85xx: allow CONFIG_PHYS_64BIT to be selectable

2012-02-23 Thread Tabi Timur-B04825
Huang Changming-R66093 wrote: > I want to know if you have the other codes for different address? > > The current U-boot just detect the base address of DTS and the CCSR address. > If they are different, u-boot will print the warning and return 0, > so the kernel can't been booted. I had a patch t

RE: [PATCH] powerpc/85xx: allow CONFIG_PHYS_64BIT to be selectable

2012-02-23 Thread Huang Changming-R66093
> > Li Yang-R58472 wrote: > > > Even though the user still need to know the addressing mode that > > u-boot is using. It won't work if the addressing mode of u-boot and > > device tree are different. > > U-Boot will tell the user if the DT does not match. I added code to U- > Boot to do that.

Re: [PATCH] powerpc/85xx: allow CONFIG_PHYS_64BIT to be selectable

2012-02-23 Thread Tabi Timur-B04825
Li Yang-R58472 wrote: > Even though the user still need to know the addressing mode that u-boot > is using. It won't work if the addressing mode of u-boot and device > tree are different. U-Boot will tell the user if the DT does not match. I added code to U-Boot to do that. So if you have a 3

RE: [PATCH] powerpc/85xx: allow CONFIG_PHYS_64BIT to be selectable

2012-02-23 Thread Huang Changming-R66093
gt; > Subject: Re: [PATCH] powerpc/85xx: allow CONFIG_PHYS_64BIT to be > > selectable > > > > Li Yang-R58472 wrote: > > > > > I agree with Changming that we shouldn't setting PHYS_64BIT by > default. > > > > The default kernel should always

RE: [PATCH] powerpc/85xx: allow CONFIG_PHYS_64BIT to be selectable

2012-02-23 Thread Li Yang-R58472
> -Original Message- > From: Tabi Timur-B04825 > Sent: Friday, February 24, 2012 10:46 AM > To: Li Yang-R58472 > Cc: Huang Changming-R66093; ga...@kernel.crashing.org; > b...@kernel.crashing.org; Wood Scott-B07421; linuxppc-...@ozlabs.org > Subject: Re: [PATCH

Re: [PATCH] powerpc/85xx: allow CONFIG_PHYS_64BIT to be selectable

2012-02-23 Thread Tabi Timur-B04825
Li Yang-R58472 wrote: > I agree with Changming that we shouldn't setting PHYS_64BIT by default. The default kernel should always be the compatible with as much as possible. Disabling PHYS_64BIT by default means that the default kernel will not work with a 36-bit DTS. If you attempt to boot su

RE: [PATCH] powerpc/85xx: allow CONFIG_PHYS_64BIT to be selectable

2012-02-23 Thread Li Yang-R58472
> > Subject: Re: [PATCH] powerpc/85xx: allow CONFIG_PHYS_64BIT to be > > selectable > > > > Huang Changming-R66093 wrote: > > > I have one similar patch to remove the "select PHYS_64BIT". > > > http://patchwork.ozlabs.org/patch/132351/ > &g

RE: [PATCH] powerpc/85xx: allow CONFIG_PHYS_64BIT to be selectable

2012-02-23 Thread Huang Changming-R66093
> -Original Message- > From: Tabi Timur-B04825 > Sent: Thursday, February 23, 2012 8:25 PM > To: Huang Changming-R66093 > Cc: ga...@kernel.crashing.org; b...@kernel.crashing.org; Wood Scott- > B07421; Li Yang-R58472; linuxppc-...@ozlabs.org > Subject: Re: [PATCH

Re: [PATCH] powerpc/85xx: allow CONFIG_PHYS_64BIT to be selectable

2012-02-23 Thread Tabi Timur-B04825
Huang Changming-R66093 wrote: > I have one similar patch to remove the "select PHYS_64BIT". > http://patchwork.ozlabs.org/patch/132351/ That one doesn't update the defconfigs, which means that the default kernel will not have PHYS_64BIT enabled. -- Timur Tabi Linux kernel developer at Freescale

RE: [PATCH] powerpc/85xx: allow CONFIG_PHYS_64BIT to be selectable

2012-02-22 Thread Huang Changming-R66093
I have one similar patch to remove the "select PHYS_64BIT". http://patchwork.ozlabs.org/patch/132351/ Thanks Jerry Huang > -Original Message- > From: linuxppc-dev-bounces+r66093=freescale@lists.ozlabs.org > [mailto:linuxppc-dev-bounces+r66093=freescale@lists.ozlabs.org] On > Beh