Re: [PATCH 1/1] media: dvb_dummy_tuner: implement driver skeleton

Daniel W. S. Almeida
2020-01-09T20:43:21Z

Hi Greg! Thanks for chiming in.

> As you are a driver, you should never need to call any pr_* calls,
> instead use dev_*(). For this, you can use dev_dbg(), but really, why
> is that even needed except for your debugging bringup. And for that,
> you can use ftrace, right? So no need for any printing of anything
> here.
> Again, dev_err() would be proper, but there's no need for any error
> message here.

Let's take these out in v2 then.

> Don't you need to register the tuner ops with something in this
> function?
> Don't you need to unregister the tuner ops in here?

It is my understanding that bridge drivers are the ones responsible for
this. For instance, I don't see this with either xc4000.c, xc5000.c or
mt2060.c. I could be wrong, though. Maybe Mauro could clarify this?

What I did miss in this patch was an attach function. Let's also add
this in v2.

Thanks again,

- Daniel.