[PATCH] MIPS: lantiq: set physical_memsize

[PATCH] MIPS: lantiq: set physical_memsize

[Hauke Mehrtens]

physical_memsize is needed by the vpe loader code and the platform
specific code has to define it. This value will be given to the
firmware loaded with the VPE loader. I am not aware of any standard
interface or better value to provide here.

Reported-by: kbuild test robot <lkp@intel.com>
Signed-off-by: Hauke Mehrtens <hauke@hauke-m.de>
---
 arch/mips/lantiq/prom.c | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/arch/mips/lantiq/prom.c b/arch/mips/lantiq/prom.c
index 4cbb000e778e..96773bed8a8a 100644
--- a/arch/mips/lantiq/prom.c
+++ b/arch/mips/lantiq/prom.c
@@ -26,6 +26,12 @@ DEFINE_SPINLOCK(ebu_lock);
 EXPORT_SYMBOL_GPL(ebu_lock);
 
 /*
+ * This is needed by the VPE loader code, just set it to 0 and assume
+ * that the firmware hardcodes this value to something useful.
+ */
+unsigned long physical_memsize = 0L;
+
+/*
  * this struct is filled by the soc specific detection code and holds
  * information about the specific soc type, revision and name
  */
-- 
2.11.0

Re: [PATCH] MIPS: lantiq: set physical_memsize

[Hauke Mehrtens]

Hi Ralf,

Kernel-CI is complaining about this build error:
https://kernelci.org/build/id/5893cc9d59b5145e1b3daa9a/logs/
and I also get mails from kbuild test robot about this problem.

Can we get this patch in the mips tree somehow or revert the patch that
activated VPE support in the defconfig, see
https://patchwork.linux-mips.org/patch/14628/

Hauke

On 12/27/2016 04:31 PM, Hauke Mehrtens wrote:
> physical_memsize is needed by the vpe loader code and the platform
> specific code has to define it. This value will be given to the
> firmware loaded with the VPE loader. I am not aware of any standard
> interface or better value to provide here.
> 
> Reported-by: kbuild test robot <lkp@intel.com>
> Signed-off-by: Hauke Mehrtens <hauke@hauke-m.de>
> ---
>  arch/mips/lantiq/prom.c | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/arch/mips/lantiq/prom.c b/arch/mips/lantiq/prom.c
> index 4cbb000e778e..96773bed8a8a 100644
> --- a/arch/mips/lantiq/prom.c
> +++ b/arch/mips/lantiq/prom.c
> @@ -26,6 +26,12 @@ DEFINE_SPINLOCK(ebu_lock);
>  EXPORT_SYMBOL_GPL(ebu_lock);
>  
>  /*
> + * This is needed by the VPE loader code, just set it to 0 and assume
> + * that the firmware hardcodes this value to something useful.
> + */
> +unsigned long physical_memsize = 0L;
> +
> +/*
>   * this struct is filled by the soc specific detection code and holds
>   * information about the specific soc type, revision and name
>   */
> 

Re: [PATCH] MIPS: lantiq: set physical_memsize

[James Hogan]

[-- Attachment #1: Type: text/plain, Size: 1185 bytes --]

On Tue, Dec 27, 2016 at 04:31:43PM +0100, Hauke Mehrtens wrote:
> physical_memsize is needed by the vpe loader code and the platform
> specific code has to define it. This value will be given to the
> firmware loaded with the VPE loader. I am not aware of any standard
> interface or better value to provide here.
> 
> Reported-by: kbuild test robot <lkp@intel.com>
> Signed-off-by: Hauke Mehrtens <hauke@hauke-m.de>

Applied

Thanks
James

> ---
>  arch/mips/lantiq/prom.c | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/arch/mips/lantiq/prom.c b/arch/mips/lantiq/prom.c
> index 4cbb000e778e..96773bed8a8a 100644
> --- a/arch/mips/lantiq/prom.c
> +++ b/arch/mips/lantiq/prom.c
> @@ -26,6 +26,12 @@ DEFINE_SPINLOCK(ebu_lock);
>  EXPORT_SYMBOL_GPL(ebu_lock);
>  
>  /*
> + * This is needed by the VPE loader code, just set it to 0 and assume
> + * that the firmware hardcodes this value to something useful.
> + */
> +unsigned long physical_memsize = 0L;
> +
> +/*
>   * this struct is filled by the soc specific detection code and holds
>   * information about the specific soc type, revision and name
>   */
> -- 
> 2.11.0
> 
> 

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 801 bytes --]

Re: [PATCH] MIPS: lantiq: set physical_memsize

[James Hogan]

[-- Attachment #1: Type: text/plain, Size: 1185 bytes --]

On Tue, Dec 27, 2016 at 04:31:43PM +0100, Hauke Mehrtens wrote:
> physical_memsize is needed by the vpe loader code and the platform
> specific code has to define it. This value will be given to the
> firmware loaded with the VPE loader. I am not aware of any standard
> interface or better value to provide here.
> 
> Reported-by: kbuild test robot <lkp@intel.com>
> Signed-off-by: Hauke Mehrtens <hauke@hauke-m.de>

Applied

Thanks
James

> ---
>  arch/mips/lantiq/prom.c | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/arch/mips/lantiq/prom.c b/arch/mips/lantiq/prom.c
> index 4cbb000e778e..96773bed8a8a 100644
> --- a/arch/mips/lantiq/prom.c
> +++ b/arch/mips/lantiq/prom.c
> @@ -26,6 +26,12 @@ DEFINE_SPINLOCK(ebu_lock);
>  EXPORT_SYMBOL_GPL(ebu_lock);
>  
>  /*
> + * This is needed by the VPE loader code, just set it to 0 and assume
> + * that the firmware hardcodes this value to something useful.
> + */
> +unsigned long physical_memsize = 0L;
> +
> +/*
>   * this struct is filled by the soc specific detection code and holds
>   * information about the specific soc type, revision and name
>   */
> -- 
> 2.11.0
> 
> 

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 801 bytes --]