Vadim Catana wrote:
> Johannes Stezenbach wrote:
>
>> Patch looks good to me except two very minor nits which we can
>> fix ourselves before commit:
>>
>>> + switch (state->config->use_isl6421) {
>>
>> ...
>>
>>> + case 1:
>>
>> ...
>>
>>> + case 0:
>>
>> odd; why not if () ... else ... ?
>
>
> There could be more than 2 ways to implement LNB control via CX24123.
> For example, the KWorld DVB-S 100 card uses the default values for
> LNBDCPol and LNBDCODEn, but some other manufacturers may use a
> diferent design. I thought there could be case 2: ... , case 3 ... :,
> etc.
>
> But if you think if() ... else ... is better, please fix it,
> I have no objections.
>
> From CX24123 Software Description:
>
> LNBDCPol LNB_DC Pin Polarity. (Default: 0)
> 0 = Normal LNB_DC pin polarity.
> 1 = Inverted LNB_DC pin polarity.
>
> LNBDCODEn LNB_DC Pin Open Drain Enable. (Default: 0)
> The state of this register bit controls the configuration of the
> LNB_DC pin.
> 0 = Open drain (requires external pull-up).
> 1 = CMOS output
>
> Regards,
> Vadim Catana
I think Vadim makes a good point here. I will leave the case statement
intact.
--
Michael Krufky
