On Tue, Jul 04, 2017 at 10:48:36PM +0300, Andy Shevchenko wrote: > On Tue, 2017-04-18 at 16:49 +0200, Lukasz Majewski wrote: > > > > The code seems OK, but recently patches to add wdt-uclass has been > > posted: > > > > http://patchwork.ozlabs.org/patch/751448/ > > http://patchwork.ozlabs.org/patch/751451/ > > > > Maybe it would be better to port this driver to the uclass from the > > very beginning? > > I started looking into this direction and have a question: > what the point to move to that class if it's broken from the beginning? > > I really do not understand who on the earth would like to have > wdt_stop() at ->probe() (the only two users do exactly that!).
It's quite possible I'm just missing something, but I don't see it. If the uclass is totally broken, can you make a suggestion on fixing it? I can see a common case being "lets turn off the watchdog now" and not want a wdt until the full OS, but we must also fully support "U-Boot has and pets the Watchdog". > So, it looks to me now as bikeshedding, otherwise where is the > documentation which describes how this all stuff should work? > > Can we go with the initial patch? If it looks like some nightmare to fix the DM uclass, I suppose, but, is it really that bad off atm? -- Tom
signature.asc
Description: Digital signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot