From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chee, Tien Fong Date: Tue, 6 Jun 2017 09:46:32 +0000 Subject: [U-Boot] [PATCH v8 4/7] arm: socfpga: Enable FPGA driver on SPL In-Reply-To: <032b0da7-4818-f330-000a-85d6b9154375@denx.de> References: <1496730959-13353-1-git-send-email-tien.fong.chee@intel.com> <1496730959-13353-5-git-send-email-tien.fong.chee@intel.com> <1496737192.2428.52.camel@intel.com> <38c8b2ad-935c-0e82-b7c6-4f1b793e023d@denx.de> <1496741766.2428.60.camel@intel.com> <032b0da7-4818-f330-000a-85d6b9154375@denx.de> Message-ID: <1496742391.2428.64.camel@intel.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Sel, 2017-06-06 at 11:41 +0200, Marek Vasut wrote: > On 06/06/2017 11:36 AM, Chee, Tien Fong wrote: > > > > On Sel, 2017-06-06 at 10:35 +0200, Marek Vasut wrote: > > > > > > On 06/06/2017 10:19 AM, Chee, Tien Fong wrote: > > > > > > > > > > > > On Sel, 2017-06-06 at 10:03 +0200, Marek Vasut wrote: > > > > > > > > > > > > > > > On 06/06/2017 08:35 AM, tien.fong.chee at intel.com wrote: > > > > > > > > > > > > > > > > > > > > > > > > From: Tien Fong Chee > > > > > > > > > > > > This patch is for enabling FPGA driver support on SPL > > > > > Why would we want that on Gen5 ? I believe this is only > > > > > needed on > > > > > Gen10. > > > > > > > > > I already moved the fpga_manager driver into drivers/fpga/ on > > > > patch > > > > 6, > > > > and fpga_manager drivers are required on SPL. Actually > > > > fpga_manager > > > > driver should be part of the drivers/fpga. > > > I think I miss some fundamental piece of information . Why would > > > I > > > need > > > anything from the FPGA framework in SPL on Gen5 ? It is not > > > needed > > > thus > > > far. Is it because you shuffled some of the code around or what ? > > > > > Because we need to know some status and mode type from FPGA even we > > did > > not program FPGA in SPL. > But we didn't have this option enabled before and everything worked > on > gen5, why do we need it now ? > Because i already move them into fpga driver, because those functions should be part of fpga driver. So, this moving happen in patch 6.