[PATCH 2.6.12-rc5-mm2] w1 search cleanup

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

 



On Thu, 2 Jun 2005 13:09:52 -0400
<BGardner at Wabtec.com> wrote:

> While writing a driver for the DS2482 (an i2c to w1 bridge), 
> I made a few enhancements to the w1 subsystem.
> 
> 1.
> I added a new I/O function: triplet
> A triplet reads two bits and writes a direction bit.  The DS2482
> implements this in hardware.
> I modified w1_search() to use w1_triplet() at its core.
> 
> 2.
> I cleaned up the I/O functions to separate emulated vs native w1
> support.
> A w1 bus master must be able to do one of:
>   a. Set and sample the line via write_bit() and read_bit()
>   b. Support reset_bus() and touch_bit()
> Function set (a) is only needed for emulated devices (ie, a parallel
> port).
> I hid w1_read_bit() and w1_write_bit() behind w1_touch_bit(), and
> changed functions to call touch_bit() instead or read/write_bit().
>  
> 3. 
> Searching is fairly slow - it requires about 200 w1 bit cycles per
> device, multiplied by the number of devices on the bus.
> I modified the w1_process() to NOT periodically search the bus.
> A sysfs entry was added to request a search. (w1_master_search).
> To request a search, echo anything into w1_master_search and it'll run
> another search.
> 
> 4.
> I added a default family so that a slave device will get reported even
> if there isn't a driver for that family.
> 
> Signed-off-by: Ben Gardner <bgardner at wabtec.com>

I've tested your patches and all looks ok.
But patches 3 and 4 were applied with several hunks.

Greg, did you applied my previous 4 cleanups for w1?
Current patches are made against pure rc5-mm2 patches
and thus can fuzz and even fail with your tree with previous w1 cleanups.
Can your try to apply this 5 patches [inlined in the next e-mails] 
and one I will send in a next e-mail with reconnect feature
and confirm that it can not be applied or not.

Thanks.

	Evgeniy Polyakov

Only failure makes us experts. -- Theo de Raadt




[Index of Archives]     [Linux Kernel]     [Linux Hardware Monitoring]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux