2 Oct
2015
2 Oct
'15
5 p.m.
On Fri, Oct 2, 2015 at 11:42 AM, Wolfgang Denk wd@denx.de wrote:
So this fixes the reset problem for now - but what happens when someone wants to use the watchdog for real? Will we create a copy of drivers/watchdog/imx_watchdog.c using big-endian accessors? This cannot be right?
If someone wants to use the watchdog for real on LS102x then this is a new feature that needs to be properly implemented.
In the kernel we have a common watchdog driver for i.MX, Vybrid and LS, which uses regmap and take the endianness into consideration. We should make drivers/watchdog/imx_watchdog.c endianness-aware so that it can work for all these different SoCs. I was not suggesting to create a new copy of drivers/watchdog/imx_watchdog.c.
Regards,
Fabio Estevam