On 11/26/21 7:34 AM, Henning Schild wrote:
Hi all, in p3 not too much was left open so i hope this might be the last and might be quick. The two points that have been open where: 1 wish to split wdt driver into two 2 wish to use pinctrl for LEDs/WDT 1 was ignored for now. We can split later when we add more devices. It remains unclear if splitting is the way to go when more devices come.
The code is already quite messy, in part because memory regions are declared locally and not passed through the parent device as they should. I don't see how splitting the driver into multiple drivers would improve the situation. The platform code claims to be inspired by the lpc_ich driver. Maybe it should take a real example from that and pass version or variant specific details through platform data instead of maintaining it in the watchdog driver. Guenter
