Re: [PATCH] media: em28xx-dvb unregister i2c tuner and demod after fe detach

From: Antti Palosaari
Date: Tue Jul 15 2014 - 13:37:04 EST


Moikka!

On 07/15/2014 08:29 PM, Shuah Khan wrote:
On 07/12/2014 02:14 PM, Antti Palosaari wrote:
Moikka Shuah!
I suspect that patch makes no sense. On DVB there is runtime PM
controlled by DVB frontend. It wakes up all FE sub-devices when frontend
device is opened and sleeps when closed.

FE release() is not relevant at all for those sub-devices which are
implemented as a proper I2C client. I2C client has own remove() for that.

em28xx_dvb_init and em28xx_dvb_fini are counterparts. Those I2C drivers
are load on em28xx_dvb_init so logical place for unload is
em28xx_dvb_fini.

Is there some real use case you need that change?

regards
Antti


Hi Antti,

The reason I made this change is because dvb_frontend_detach()
calls release interfaces for fe as well as tuner. So it made
sense to move the remove after that is all done. Are you saying
fe and tuner release calls aren't relevant when sub-devices
implement a proper i2c client? If that is the case then, and
there is no chance for these release calls to be invoked when a
proper i2c is present, then my patch isn't needed.

Yes, that is just case. Proprietary DVB binding model uses attach / release, but I2C binding model has probe / remove. I see no reason use DVB proprietary model, instead drivers should be converted to kernel I2C model.

regards
Antti

--
http://palosaari.fi/
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/