On 22-09-22, Sascha Hauer wrote: > On Wed, Sep 21, 2022 at 07:20:51PM +0200, Marco Felsch wrote: > > Currently the fsp_table must be set manually within the 'struct > > dram_timing_info'. Since the 'struct fsp_msg' already has all > > information needed for the fsp_table we can use it to set it > > automatically. This approach is less error-prone and avoids information > > duplication. > > Is this really worth it? The whole dram timing code comes directly from > the NXP DRAM tools, so the fsp_table comes back with each new board or > with a DRAM timing update. I had a problem with the missing fsp_table for the MX8MM-EVK and a buggy TF-A. Therefore I would say yes, since it is not guaranteed that the fsp_table is existing. Regards, Marco
