Re: [U-Boot] [PATCH 1/3] arm: socfpga: Remove unused FPGA feature from gen5 SPL

2017-07-31 Thread Chee, Tien Fong
On Jum, 2017-07-28 at 13:40 +0200, Marek Vasut wrote: > On 07/28/2017 01:35 PM, Chee, Tien Fong wrote: > > > > On Jum, 2017-07-28 at 09:52 +0200, Marek Vasut wrote: > > > > > > On 07/28/2017 07:16 AM, Chee, Tien Fong wrote: > > > > > > > > > > > > On Kha, 2017-07-27 at 11:37 +0200, Marek Vasut

Re: [U-Boot] [PATCH 1/3] arm: socfpga: Remove unused FPGA feature from gen5 SPL

2017-07-28 Thread Marek Vasut
On 07/28/2017 01:35 PM, Chee, Tien Fong wrote: > On Jum, 2017-07-28 at 09:52 +0200, Marek Vasut wrote: >> On 07/28/2017 07:16 AM, Chee, Tien Fong wrote: >>> >>> On Kha, 2017-07-27 at 11:37 +0200, Marek Vasut wrote: On 07/27/2017 11:21 AM, Chee, Tien Fong wrote: > > > On Kha, 2

Re: [U-Boot] [PATCH 1/3] arm: socfpga: Remove unused FPGA feature from gen5 SPL

2017-07-28 Thread Chee, Tien Fong
On Jum, 2017-07-28 at 09:52 +0200, Marek Vasut wrote: > On 07/28/2017 07:16 AM, Chee, Tien Fong wrote: > > > > On Kha, 2017-07-27 at 11:37 +0200, Marek Vasut wrote: > > > > > > On 07/27/2017 11:21 AM, Chee, Tien Fong wrote: > > > > > > > > > > > > On Kha, 2017-07-27 at 10:21 +0200, Marek Vasut

Re: [U-Boot] [PATCH 1/3] arm: socfpga: Remove unused FPGA feature from gen5 SPL

2017-07-28 Thread Marek Vasut
On 07/28/2017 07:16 AM, Chee, Tien Fong wrote: > On Kha, 2017-07-27 at 11:37 +0200, Marek Vasut wrote: >> On 07/27/2017 11:21 AM, Chee, Tien Fong wrote: >>> >>> On Kha, 2017-07-27 at 10:21 +0200, Marek Vasut wrote: On 07/27/2017 06:36 AM, tien.fong.c...@intel.com wrote: > > >

Re: [U-Boot] [PATCH 1/3] arm: socfpga: Remove unused FPGA feature from gen5 SPL

2017-07-27 Thread Chee, Tien Fong
On Kha, 2017-07-27 at 11:37 +0200, Marek Vasut wrote: > On 07/27/2017 11:21 AM, Chee, Tien Fong wrote: > > > > On Kha, 2017-07-27 at 10:21 +0200, Marek Vasut wrote: > > > > > > On 07/27/2017 06:36 AM, tien.fong.c...@intel.com wrote: > > > > > > > > > > > > From: Tien Fong Chee > > > > > > > >

Re: [U-Boot] [PATCH 1/3] arm: socfpga: Remove unused FPGA feature from gen5 SPL

2017-07-27 Thread Marek Vasut
On 07/27/2017 11:21 AM, Chee, Tien Fong wrote: > On Kha, 2017-07-27 at 10:21 +0200, Marek Vasut wrote: >> On 07/27/2017 06:36 AM, tien.fong.c...@intel.com wrote: >>> >>> From: Tien Fong Chee >>> >>> Remove unused FPGA feature for saving some space in gen5 SPL. >> I really dislike the ifdefery. Why

Re: [U-Boot] [PATCH 1/3] arm: socfpga: Remove unused FPGA feature from gen5 SPL

2017-07-27 Thread Chee, Tien Fong
On Kha, 2017-07-27 at 10:21 +0200, Marek Vasut wrote: > On 07/27/2017 06:36 AM, tien.fong.c...@intel.com wrote: > > > > From: Tien Fong Chee > > > > Remove unused FPGA feature for saving some space in gen5 SPL. > I really dislike the ifdefery. Why is this patch even needed? > I thought we had no

Re: [U-Boot] [PATCH 1/3] arm: socfpga: Remove unused FPGA feature from gen5 SPL

2017-07-27 Thread Marek Vasut
On 07/27/2017 06:36 AM, tien.fong.c...@intel.com wrote: > From: Tien Fong Chee > > Remove unused FPGA feature for saving some space in gen5 SPL. I really dislike the ifdefery. Why is this patch even needed? I thought we had no space problems in the Gen5 SPL? > Signed-off-by: Tien Fong Chee > -

[U-Boot] [PATCH 1/3] arm: socfpga: Remove unused FPGA feature from gen5 SPL

2017-07-26 Thread tien . fong . chee
From: Tien Fong Chee Remove unused FPGA feature for saving some space in gen5 SPL. Signed-off-by: Tien Fong Chee --- arch/arm/mach-socfpga/reset_manager_gen5.c |9 + arch/arm/mach-socfpga/system_manager_gen5.c |6 -- drivers/ddr/altera/sdram.c |8 +