26 Feb
2024
26 Feb
'24
11:43 a.m.
On Mon, 26 Feb 2024 at 14:24, Marek Vasut marex@denx.de wrote:
On 2/26/24 9:04 AM, Sumit Garg wrote:
[...]
+static int imx7_reset_assert_imx8mp(struct reset_ctl *rst)
Linux calls those imx8mp_reset_set() can co. which is less confusing than imx7...imx8mp() , use it.
IMO, it would be more confusing if we choose two different naming patterns for the same driver, see existing function names imx7_reset_{deassert/assert}_imx* pattern.
In fact, why not copy the code from Linux outright ?
I suppose the question here is to not convert the driver to use regmap, right? If it is then that should be done as a separate cleanup patch.
However, I can extract common bits as imx8mp_reset_set() for better code reuse as the Linux kernel driver does.
-Sumit