Re: [PATCH v4 00/14] cx231xx: Use muxed i2c adapters instead of custom switching

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Em Thu, 30 Oct 2014 21:12:21 +0100
Matthias Schwarzott <zzam@xxxxxxxxxx> escreveu:

> Hi!
> 
> This time the series got only small updates from Antti's review.

Oh, no! I just merged the v3 patch series. I manually fixed the
points that Antti has made.

I'm also testing it already.

Could you, instead, rebase it on the top of my merge?

I added it at:
	ssh://linuxtv.org/git/mchehab/experimental.git cx231xx

While I'm still at the test phases.

> 
> Additionally I added a patch to no longer directly modify
> the content of the port3 switch bit in PWR_CTL_EN (from function cx231xx_set_power_mode).
> 
> Now there are two places where I wonder what happens:
> 1. cx231xx_set_Colibri_For_LowIF writes a fixed number into all 8bit parts of PWR_CTL_EN
>    What is this for?
> 2. I guess, cx231xx_demod_reset should reset the digital demod. For this it should toggle the bits 0 and 1 of PWR_CTL_EN. 
>    But instead it touches but 8 and 9.
>    Does someone know what this is?
> 
> 3. Is remembering the last status of the port3 bit working good enough?
>    Currently it is only used for the is_tuner hack function.
> 
> Regards
> Matthias
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux