Re: [U-Boot] [PATCH] ARM: Disable "DISCARD" for secure section if CONFIG_ARMV7_SECURE_BASE isn't defined

Hi Tom, Thanks for your review. I will update this patch. Regards, -Dongsheng > On Mon, Jan 11, 2016 at 02:51:39AM +, Dongsheng Wang wrote: > > Hi Tom, > > > > Sorry for my late reply, and thanks for your reply. > > > > How about the following comments, following your suggestion I remove >

Re: [U-Boot] [PATCH] ARM: Disable "DISCARD" for secure section if CONFIG_ARMV7_SECURE_BASE isn't defined

On Mon, Jan 11, 2016 at 02:51:39AM +, Dongsheng Wang wrote: > Hi Tom, > > Sorry for my late reply, and thanks for your reply. > > How about the following comments, following your suggestion I remove some > redundant comments? > > If my understanding is wrong, please correct me, thanks: >

Re: [U-Boot] [PATCH] ARM: Disable "DISCARD" for secure section if CONFIG_ARMV7_SECURE_BASE isn't defined

Hi Tom, Sorry for my late reply, and thanks for your reply. How about the following comments, following your suggestion I remove some redundant comments? If my understanding is wrong, please correct me, thanks: #if defined(CONFIG_ARMV7_SECURE_BASE) && defined(CONFIG_ARMV7_NONSEC) /*

Re: [U-Boot] [PATCH] ARM: Disable "DISCARD" for secure section if CONFIG_ARMV7_SECURE_BASE isn't defined

Hi Tom, On Mon, Jan 04, 2016 at 10:50:16AM -0500, Tom Rini wrote: >On Fri, Dec 11, 2015 at 03:30:24PM +, Dongsheng Wang wrote: >> Hi Tom, >> >> > On Fri, Dec 11, 2015 at 10:15:03AM +, Dongsheng Wang wrote: >> > > Hi Tom, >> > > >> > > Thanks for your review. >> > > >> > > > On Thu, Dec

Re: [U-Boot] [PATCH] ARM: Disable "DISCARD" for secure section if CONFIG_ARMV7_SECURE_BASE isn't defined

On Fri, Dec 11, 2015 at 03:30:24PM +, Dongsheng Wang wrote: > Hi Tom, > > > On Fri, Dec 11, 2015 at 10:15:03AM +, Dongsheng Wang wrote: > > > Hi Tom, > > > > > > Thanks for your review. > > > > > > > On Thu, Dec 10, 2015 at 10:49:01AM +0800, Dongsheng Wang wrote: > > > > > > > > > From:

Re: [U-Boot] [PATCH] ARM: Disable "DISCARD" for secure section if CONFIG_ARMV7_SECURE_BASE isn't defined

On Fri, Dec 11, 2015 at 10:15:03AM +, Dongsheng Wang wrote: > Hi Tom, > > Thanks for your review. > > > On Thu, Dec 10, 2015 at 10:49:01AM +0800, Dongsheng Wang wrote: > > > > > From: Wang Dongsheng > > > > > > Fix PSCI hang up without CONFIG_ARMV7_SECURE_BASE

Re: [U-Boot] [PATCH] ARM: Disable "DISCARD" for secure section if CONFIG_ARMV7_SECURE_BASE isn't defined

Hi Tom, Thanks for your review. > On Thu, Dec 10, 2015 at 10:49:01AM +0800, Dongsheng Wang wrote: > > > From: Wang Dongsheng > > > > Fix PSCI hang up without CONFIG_ARMV7_SECURE_BASE define. > > "DISCARD" will remove ._secure.text relocate, but PSCI framework has > >

Re: [U-Boot] [PATCH] ARM: Disable "DISCARD" for secure section if CONFIG_ARMV7_SECURE_BASE isn't defined

Hi Tom, > On Fri, Dec 11, 2015 at 10:15:03AM +, Dongsheng Wang wrote: > > Hi Tom, > > > > Thanks for your review. > > > > > On Thu, Dec 10, 2015 at 10:49:01AM +0800, Dongsheng Wang wrote: > > > > > > > From: Wang Dongsheng > > > > > > > > Fix PSCI hang up without

Re: [U-Boot] [PATCH] ARM: Disable "DISCARD" for secure section if CONFIG_ARMV7_SECURE_BASE isn't defined

On Thu, Dec 10, 2015 at 10:49:01AM +0800, Dongsheng Wang wrote: > From: Wang Dongsheng > > Fix PSCI hang up without CONFIG_ARMV7_SECURE_BASE define. > "DISCARD" will remove ._secure.text relocate, but PSCI framework > has already used some absolute address those need to

[U-Boot] [PATCH] ARM: Disable "DISCARD" for secure section if CONFIG_ARMV7_SECURE_BASE isn't defined

From: Wang Dongsheng Fix PSCI hang up without CONFIG_ARMV7_SECURE_BASE define. "DISCARD" will remove ._secure.text relocate, but PSCI framework has already used some absolute address those need to relocate. Use readelf -t -r u-boot show us: .__secure_start addr: