Hi Marcin, Could you please test the attached updated driver and let me know if the 2 issues you found are fixed? Thanks ...juerg On 7/16/07, Marcin Dawidowicz <marcin.dawidowicz at kontron.pl> wrote: > Hi Juerg, > > Thank you. > > I've done short test of your driver and at first sight it seems to work > quite well with SCH3112. It was tested on CP6012 CPU board from Kontron. > > I found some issues which should be improved. One of those is resetting > of alarms. Now, after alarm appeared it remains set - it is not cleared > by sensor reading. (maybe not perfect but simplest would be resetting > all alarms after sensors are read - look into my driver) > > I think that temperature fault reading return improper values (see > printout below). > > Below is short snapshot of test: > ..... > marcda at marcda-laptop:/lib/modules/2.6.22/kernel/drivers/hwmon$ sudo > modprobe dme1737 > Password: > marcda at marcda-laptop:/lib/modules/2.6.22/kernel/drivers/hwmon$ > ..... > > ..... > marcda at marcda-laptop:/lib/modules/2.6.22/kernel/drivers/hwmon$ dmesg |tail > [ 26.916000] Bluetooth: HCI socket layer initialized > [ 27.016000] Bluetooth: L2CAP ver 2.8 > [ 27.016000] Bluetooth: L2CAP socket layer initialized > [ 27.052000] Bluetooth: RFCOMM socket layer initialized > [ 27.052000] Bluetooth: RFCOMM TTY layer initialized > [ 27.052000] Bluetooth: RFCOMM ver 1.8 > [ 33.604000] eth5: no IPv6 routers present > [ 3879.928000] PM: Adding info for platform:dme1737.2672 > [ 3879.928000] dme1737 dme1737.2672: Found a SCH311X chip at 0x0a70 > [ 3879.928000] dme1737 dme1737.2672: Optional features: pwm3=yes, > pwm5=no, pwm6=no, fan3=yes, fan4=no, fan5=no, fan6=no. > marcda at marcda-laptop:/lib/modules/2.6.22/kernel/drivers/hwmon$ > ..... > > ..... > root at marcda-laptop:/sys/class/hwmon/hwmon0/device/driver/dme1737.2672# > clear;for file in *_*; do echo "$file = `cat $file`"; done cpu0_vid = 0 > fan1_alarm = 0 > fan1_input = 0 > fan1_min = 0 > fan1_type = 2 > fan2_alarm = 0 > fan2_input = 0 > fan2_min = 0 > fan2_type = 2 > fan3_alarm = 0 > fan3_input = 0 > fan3_min = 0 > fan3_type = 2 > in0_alarm = 0 > in0_input = 3610 > in0_max = 6641 > in0_min = 0 > in1_alarm = 0 > in1_input = 1625 > in1_max = 2988 > in1_min = 0 > in2_alarm = 1 > in2_input = 3303 > in2_max = 3334 > in2_min = 0 > in3_alarm = 0 > in3_input = 4993 > in3_max = 6641 > in3_min = 0 > in4_alarm = 0 > in4_input = 11965 > in4_max = 15938 > in4_min = 0 > in5_alarm = 0 > in5_input = 3309 > in5_max = 4383 > in5_min = 0 > in6_alarm = 0 > in6_input = 2983 > in6_max = 4383 > in6_min = 0 > pwm1_auto_channels_zone = 1 > pwm1_auto_point1_pwm = 128 > pwm1_auto_point2_pwm = 255 > pwm1_auto_pwm_min = 128 > pwm1_enable = 0 > pwm1_freq = 25000 > pwm1_ramp_rate = 206 > pwm2_auto_channels_zone = 2 > pwm2_auto_point1_pwm = 128 > pwm2_auto_point2_pwm = 255 > pwm2_auto_pwm_min = 128 > pwm2_enable = 0 > pwm2_freq = 25000 > pwm2_ramp_rate = 206 > pwm3_auto_channels_zone = 4 > pwm3_auto_point1_pwm = 128 > pwm3_auto_point2_pwm = 255 > pwm3_auto_pwm_min = 128 > pwm3_enable = 0 > pwm3_freq = 25000 > pwm3_ramp_rate = 0 > temp1_alarm = 0 > temp1_fault = 0 > temp1_input = 35562 > temp1_max = 127000 > temp1_min = -127000 > temp1_offset = 0 > temp2_alarm = 0 > temp2_fault = 0 > temp2_input = 33750 > temp2_max = 127000 > temp2_min = -127000 > temp2_offset = 0 > temp3_alarm = 1 > temp3_fault = 0 > temp3_input = -128000 > temp3_max = 127000 > temp3_min = -127000 > temp3_offset = 0 > zone1_auto_channels_temp = 1 > zone1_auto_point1_temp = 90000 > zone1_auto_point1_temp_hyst = 90000 > zone1_auto_point2_temp = 122000 > zone1_auto_point3_temp = 100000 > zone2_auto_channels_temp = 2 > zone2_auto_point1_temp = 90000 > zone2_auto_point1_temp_hyst = 90000 > zone2_auto_point2_temp = 122000 > zone2_auto_point3_temp = 100000 > zone3_auto_channels_temp = 4 > zone3_auto_point1_temp = 90000 > zone3_auto_point1_temp_hyst = 90000 > zone3_auto_point2_temp = 122000 > zone3_auto_point3_temp = 100000 > root at marcda-laptop:/sys/class/hwmon/hwmon0/device/driver/dme1737.2672# > ..... > > I will let you know if I find something more. > > Regards, > Marcin > > Juerg Haefliger wrote: > > Hi Marcin, > > > > The attached updated dme1737 driver includes support for the sch311x > > chips. Could you please test it and report success/failure? > > > > Thanks > > ...juerg > > -------------- next part -------------- A non-text attachment was scrubbed... Name: dme1737.c Type: application/octet-stream Size: 67953 bytes Desc: not available Url : http://lists.lm-sensors.org/pipermail/lm-sensors/attachments/20070716/3ef202d6/attachment.obj
