We will port w83792d.c to linux-2.6

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

 



Dear Rudolf, Patrick

Quoting CLHuang2:
>     The bit7(BYTE_DONE_STS) - The SMBus host controller sets this bit
> after receiving a byte for Block Read or after transmitting a byte for
> Block Write.This bit will be set even when transferring the last byte of the
> block transfer. It will be cleared by writing a '1' to this bit.
>     And we set BYTE Read/Write transaction in our driver , so that this bit
> is useless.

According to our 792 Windows driver author CLHuang2 and the information from
ALI M1563 data sheet, it seems that we should move the HST_STS_DONE check from
ali1563_transaction() into the function ali1563_access(), only when the
transaction is Block Read/Write, and ignore the Byte Read/Write transaction.
Patrick, would you please confirm it?

I want to supply more information here:
When I debug the i2c-ali1563 and my w83792d driver, I find that when I load
these two modules, the function ali1563_access()/ali1563_transaction()
will be called for 81 times, but the error message "SMBus Error: Timeout Device
Error Transaction Never Finished" only appears at the first 4 times, In the next
77 times, there is no "SMBus Error" message. And in these 81
ali1563_access()/ali1563_transaction() calls, I2C_SMBUS_BYTE_DATA byte
transaction appears for 79 times, and I2C_SMBUS_QUICK transaction appears for
2 times, there is no block transaction appeared.
Could you tell me the function of I2C_SMBUS_QUICK transaction? And why the first
4 data transaction lead to "SMBus Error"?

I attached the debug message and the modified i2c-ali1563.c in this mail, please
check it.


Thanks
Best Regards
Chunhao


---------------------------------------------------------
From: PI13 CLHuang2 
Sent: 2005??3??24?? 18:23
To: Rudolf Marek; PI14 HUANG0
Cc: mochel at digitalimplant.org; sensors at Stimpy.netroedge.com; PI10 LHHsu; PI14 DZSHEN
Subject: RE: We will port w83792d.c to linux-2.6

Hi , Rudolf:
    The bit7(BYTE_DONE_STS) - The SMBus host controller sets this bit after
receiving a byte for Block Read or after transmitting a byte for Block Write.This
bit will be set even when transferring the last byte of the block transfer. It
will be cleared by writing a ??1?? to this bit.
    And we set BYTE Read/Write transaction in our driver , so that this bit is
useless.

-----Original Message-----
From: Rudolf Marek [mailto:R.Marek at sh.cvut.cz] 
Sent: Thursday, March 24, 2005 4:44 PM
To: PI14 HUANG0
Cc: mochel at digitalimplant.org; sensors at Stimpy.netroedge.com; PI10 LHHsu; PI14 DZSHEN; PI13 CLHuang2
Subject: RE: We will port w83792d.c to linux-2.6

Hello

> Since our 792 Windows driver has been proved/test work well,
> I suggest that the i2c-ali1563 driver remove the HST_STS_DONE related check
> so that our 792 linux driver can work.

I'm sorry I cant give you authoritative answer. But from my point of
view it is not good strategy. I simply do not like "lets do something and
hope it worked)

Jean? MDS? Patrick? Philip?

I would like really to know if SMM code is talking to the bus same time.
If so, then it would explain a lot. And if we now how it is working, we
might find some more clean solution.

Regards

Rudolf



===========================================================================================The privileged confidential information contained in this email is intended for use only by the addressees as indicated by the original author of this email. If you are not the addressee indicated in this email or are not responsible for delivery of the email to such person, please kindly reply the sender indicating accordingly and delete all copies of it from your computer and network server immediately. We thank you for your cooperation. It is advisable that any unauthorized use of confidential information of Winbond is strictly prohibited; and any information in this email that does not relate to the official business of Winbond shall be deemed as neither given nor endorsed by Winbond.===========================================================================================If your computer is unable to decode Chinese font, please ignore the following message. They essentially repeat the  English statement above.???H???????t?????q?l???]???????K?????T, ?????v???o?H?H???w?????H?H???\????. ?????z???D?Q???w?????H?H???]???????]?b???g???v?????????U???????H??, ???z?i?????o?H?H?????Y?N?H???q?q???P???????A???????H????. ?????z???X?@, ?????????P??. ?S??????, ???????g???v?????????????q?l?????K???T???????O?Q?Y???T????. ?H???P?????q?l???~?L???????e,???o?????????q?l?????????N??.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: messages
Type: application/octet-stream
Size: 68815 bytes
Desc: messages
Url : http://lists.lm-sensors.org/pipermail/lm-sensors/attachments/20050324/80369c47/attachment.obj 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: i2c-ali1563.c
Type: application/octet-stream
Size: 11698 bytes
Desc: i2c-ali1563.c
Url : http://lists.lm-sensors.org/pipermail/lm-sensors/attachments/20050324/80369c47/attachment-0001.obj 


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

  Powered by Linux