[ltp] Re: [Hdaps-devel] Patch for X40 (was: [ANN] tp_smapi 0.19 adds dates, improves stability)

Shem Multinymous linux-thinkpad@linux-thinkpad.org
Tue, 11 Apr 2006 23:06:25 +0300


------=_Part_13272_31558900.1144785985539
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: quoted-printable
Content-Disposition: inline

On 4/11/06, Shem Multinymous <multinymous@gmail.com> wrote:
> What happens if you replace that part of the code with
> tp_controller_read(0x13,0x01,buf), verify that it returns 0 and that
> buf[0xf] is 0, and set status=3Dbuf[1]? That's logically equivalent, but
> has some error checking and retries along the way.

More explicitly, how well does the attached path work?
If it fails, please add debugging code to find out where...

  Shem

------=_Part_13272_31558900.1144785985539
Content-Type: text/x-patch; name="hdaps-init-using-tp_base.patch"
Content-Transfer-Encoding: base64
Content-Disposition: attachment; filename="hdaps-init-using-tp_base.patch"
X-Attachment-Id: f_elwnzcd3

LS0tIHRwX3NtYXBpLTAuMTkvaGRhcHMuYwkyMDA2LTA0LTExIDIzOjAwOjU4LjAwMDAwMDAwMCAr
MDAwMAorKysgdHBfc21hcGktdGVzdC9oZGFwcy5jCTIwMDYtMDQtMTEgMjI6NTk6MjYuMDAwMDAw
MDAwICswMDAwCkBAIC0xODcsMTYgKzE4NywxNyBAQCBzdGF0aWMgaW50IGhkYXBzX3VwZGF0ZSh2
b2lkKQogc3RhdGljIGludCBoZGFwc19kZXZpY2VfaW5pdCh2b2lkKQogewogCWludCByZXQgPSAt
RU5YSU87CisJdTggcm93W1RQX0NPTlRST0xMRVJfUk9XX0xFTl07CiAJdTggc3RhdHVzOwogCiAJ
dHBfY29udHJvbGxlcl9sb2NrKCk7CiAKLQlvdXRiKDB4MTMsIDB4MTYxMCk7Ci0Jb3V0YigweDAx
LCAweDE2MWYpOwotCWlmIChfX3dhaXRfbGF0Y2goMHgxNjFmLCAweDAwKSkKKwlpZiAodHBfY29u
dHJvbGxlcl9yZWFkX3JvdygweDEzLCAweDAxLCByb3cpKQogCQlnb3RvIG91dDsKLQotCXN0YXR1
cyA9IGluYigweDE2MTEpOworCWlmIChyb3dbMHhmXSE9MHgwMCkKKwkJZ290byBvdXQ7CisJc3Rh
dHVzID0gcm93WzFdOwkKKwkJCiAJaWYgKHN0YXR1cyAhPSAweDAzICYmIC8qIEludmVydGV4IGF4
ZXMgKFRoaW5rUGFkIFI1MHAsIFQ0MXAsIFI0MnApICovCiAJICAgIHN0YXR1cyAhPSAweDAyICYm
IC8qIENoaXAgYWxyZWFkeSBpbml0aWFsaXplZCAqLwogCSAgICBzdGF0dXMgIT0gMHgwMSApICAv
KiBOb3JtYWwgYXhlcyAqLwo=
------=_Part_13272_31558900.1144785985539--