On Thu, Apr 25, 2024 at 08:11:20AM GMT, Krzysztof Kozlowski wrote: > On 24/04/2024 20:16, Krzysztof Kozlowski wrote: > > > > On Fri, 19 Apr 2024 11:45:08 +0100, Diogo Ivo wrote: > >> This patch series consists of a general cleanup of the Tegra210 EMC > >> frequency scaling code for revision 7. > >> > >> Currently the code is relying heavily on a function, update_clock_tree_delay(), > >> that is responsible for too many things, making it long and confusing. > >> The general idea with these patches is to simplify this function and its > >> surrounding code, making it more modular. > >> > >> [...] > > > > Applied, thanks! > > > > [1/7] memory: tegra: Remove periodic compensation duplicate calls > > https://git.kernel.org/krzk/linux-mem-ctrl/c/f8037b3d9a75f4963c1fd3eaf39b968f206d37df > > [2/7] memory: tegra: Move DQSOSC measurement to common place > > https://git.kernel.org/krzk/linux-mem-ctrl/c/09dd3744d7c02a3844e8a6c96ab2fb3c1e94de52 > > [3/7] memory: tegra: Reword and correct comments > > https://git.kernel.org/krzk/linux-mem-ctrl/c/d48076e5bb9be787b769efb3e6439902dc8e99f7 > > [4/7] memory: tegra: Change macros to interpret parameter as integer > > https://git.kernel.org/krzk/linux-mem-ctrl/c/56c95667042ba12a66157477282ca73605a25481 > > [5/7] memory: tegra: Loop update_clock_tree_delay() > > https://git.kernel.org/krzk/linux-mem-ctrl/c/0d647b9031fe4f6b20d2f18b3d087dea740eecf1 > > [6/7] memory: tegra: Move compare/update current delay values to a function > > https://git.kernel.org/krzk/linux-mem-ctrl/c/c1a70060c654327be4aed6e617f53bae5af8625f > > [7/7] memory: tegra: Rework update_clock_tree_delay() > > https://git.kernel.org/krzk/linux-mem-ctrl/c/edbf2db5ae95c4d8f189d05e827d9dd3ea75cfbc > > And entire patchset dropped because it fails to compile on several > architectures. > > Don't send code which does not compile. > > Best regards, > Krzysztof Sorry for that! I was testing this on an ARM64 platform where the code does compile, so I did not catch this error. Thanks for pointing it out, I will send a fixed version. Best regards, Diogo