Re: [PATCH 1/2] ddr: imx8m: write fsp_table entry automatically

[Sascha Hauer <sha@pengutronix.de>]

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 |