Am Mon, 26 Jun 2017 06:19:20 -0300
schrieb Mauro Carvalho Chehab <mche...@s-opensource.com>:

> Yes, config flags work, but please don't use a config flag like this:
> 
>       if (initflags & TDA18212_INIT_DDSTV) 
> 
> The flags should identify the required functionality, not if the
> caller is ddbridge driver. On a quick look at your patch, I suspect
> that it would need two flags, like:
> 
>       TDA18212_FLAG_SLEEP - with would enable sleep/wakeup/standby
>       TDA18212_FLAG_CALIBRATION - with would enable calibration

Yes, of course. Actually, this was just some (early) random attempt at
fixing something which turned out unrelated and even doesn't apply as is
anymore. But that patch might be a start to get this done.

Best regards,
Daniel Scheller
-- 
https://github.com/herrnst

Reply via email to