Jean Delvare wrote:
> Hi Andrey,
>
>
>>Possible too late to include in 2.6.15,
>>but better later then never :).
>
>
> You must be kidding. It might be too late for 2.6._16_. Reviewing takes
> time, reworking afterwards takes time, then you get some testing in -mm
> and it takes time again.
>
:((.
>
>>Comments?
>
>
> Sure, although I don't really have the time right now for a complete
> review. And I'd rather not review the code if it finally isn't used.
(see my prev. reply to Andrew and you)
>
> First, a question. Can't you merge the M41T85 support into the m41t00
> driver?
It was first thing what I try, but this chips are very similar only at
first glance. m41t85 have _really_ extended sets of regs and result was
very littered by #if/#else file.
>
> Mark, care to comment on that possibility, and/or on the code itself?
>
And, please, remove unnecessary PPC dependence from Kconfig.
>>+config SENSORS_M41T85_SQW_FRQ_ENABLE
>>+ depends on SENSORS_M41T85
>>+ bool "Square Wave Output"
>
>
> What a mess. Please just have a sysfs file for that, it's more flexible
> and less intrusive.
I agree, it's look messed, but if sombody use SQW,
then must exist some startup constant for some custom board.
Changing this frq may exist only as option.
--
Regards
Andrey Volkov
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
