lkml.org 
[lkml]   [2021]   [Nov]   [26]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH v4 0/4] add device drivers for Siemens Industrial PCs
From
Date
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

\
 
 \ /
  Last update: 2021-11-26 17:13    [W:0.528 / U:0.068 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site