Re: [PATCH v2 0/7] Cleanup Tegra210 EMC frequency scaling

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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




[Index of Archives]     [ARM Kernel]     [Linux ARM]     [Linux ARM MSM]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux