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. Sascha -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |