Re: [U-Boot] [PATCH 7/9] x86: ivybridge: Fix car_uninit() to correctly set run state
On 18 October 2015 at 20:27, Simon Glass wrote: > Hi Bin, > > On 18 October 2015 at 20:23, Bin Meng wrote: >> Hi Simon, >> >> On Mon, Oct 19, 2015 at 5:55 AM, Simon Glass wrote: >>> At present a missing $ causes this code to hang when using the MRC cache/ >>> Fix it. >>> >>> Signed-off-by: Simon Glass >> >> Reviewed-by: Bin Meng >> >>> --- >>> >>> arch/x86/cpu/ivybridge/car.S | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >>> >>> diff --git a/arch/x86/cpu/ivybridge/car.S b/arch/x86/cpu/ivybridge/car.S >>> index 770ef17..b75c2a5 100644 >>> --- a/arch/x86/cpu/ivybridge/car.S >>> +++ b/arch/x86/cpu/ivybridge/car.S >>> @@ -188,7 +188,7 @@ car_uninit: >>> wrmsr >>> >>> /* Disable the no-eviction run state */ >>> - movlNOEVICTMOD_MSR, %ecx >>> + movl$NOEVICTMOD_MSR, %ecx >> >> I am wondering why compiler does not complain this. >> > > I suppose it just uses the value at that address. > >>> rdmsr >>> andl$~2, %eax >>> wrmsr >>> -- > > Regards, > Simon Applied to u-boot-x86. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 7/9] x86: ivybridge: Fix car_uninit() to correctly set run state
Hi Bin, On 18 October 2015 at 20:23, Bin Meng wrote: > Hi Simon, > > On Mon, Oct 19, 2015 at 5:55 AM, Simon Glass wrote: >> At present a missing $ causes this code to hang when using the MRC cache/ >> Fix it. >> >> Signed-off-by: Simon Glass > > Reviewed-by: Bin Meng > >> --- >> >> arch/x86/cpu/ivybridge/car.S | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/arch/x86/cpu/ivybridge/car.S b/arch/x86/cpu/ivybridge/car.S >> index 770ef17..b75c2a5 100644 >> --- a/arch/x86/cpu/ivybridge/car.S >> +++ b/arch/x86/cpu/ivybridge/car.S >> @@ -188,7 +188,7 @@ car_uninit: >> wrmsr >> >> /* Disable the no-eviction run state */ >> - movlNOEVICTMOD_MSR, %ecx >> + movl$NOEVICTMOD_MSR, %ecx > > I am wondering why compiler does not complain this. > I suppose it just uses the value at that address. >> rdmsr >> andl$~2, %eax >> wrmsr >> -- Regards, Simon ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 7/9] x86: ivybridge: Fix car_uninit() to correctly set run state
Hi Simon, On Mon, Oct 19, 2015 at 5:55 AM, Simon Glass wrote: > At present a missing $ causes this code to hang when using the MRC cache/ > Fix it. > > Signed-off-by: Simon Glass Reviewed-by: Bin Meng > --- > > arch/x86/cpu/ivybridge/car.S | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/x86/cpu/ivybridge/car.S b/arch/x86/cpu/ivybridge/car.S > index 770ef17..b75c2a5 100644 > --- a/arch/x86/cpu/ivybridge/car.S > +++ b/arch/x86/cpu/ivybridge/car.S > @@ -188,7 +188,7 @@ car_uninit: > wrmsr > > /* Disable the no-eviction run state */ > - movlNOEVICTMOD_MSR, %ecx > + movl$NOEVICTMOD_MSR, %ecx I am wondering why compiler does not complain this. > rdmsr > andl$~2, %eax > wrmsr > -- Regards, Bin ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
[U-Boot] [PATCH 7/9] x86: ivybridge: Fix car_uninit() to correctly set run state
At present a missing $ causes this code to hang when using the MRC cache/ Fix it. Signed-off-by: Simon Glass --- arch/x86/cpu/ivybridge/car.S | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/x86/cpu/ivybridge/car.S b/arch/x86/cpu/ivybridge/car.S index 770ef17..b75c2a5 100644 --- a/arch/x86/cpu/ivybridge/car.S +++ b/arch/x86/cpu/ivybridge/car.S @@ -188,7 +188,7 @@ car_uninit: wrmsr /* Disable the no-eviction run state */ - movlNOEVICTMOD_MSR, %ecx + movl$NOEVICTMOD_MSR, %ecx rdmsr andl$~2, %eax wrmsr -- 2.6.0.rc2.230.g3dd15c0 ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot