Re:Re: [PATCH] hwmon: (ads7828) Check return value of regmap_read() in probe
"Qingshuang Fu" <[email protected]> Tue, 4 Aug 2026 18:14:41 +0800 (CST)
| Newsgroups | org.kernel.vger.linux-hwmon,dev.linux.lists.sashiko-reviews |
|---|---|
| Message-ID | <[email protected]> |
CkF0IDIwMjYtMDgtMDQgMTc6MjQ6MTEsIHNhc2hpa28tYm90QGtlcm5lbC5vcmcgd3JvdGU6Cj5U aGFuayB5b3UgZm9yIHlvdXIgY29udHJpYnV0aW9uISBTYXNoaWtvIEFJIHJldmlldyBmb3VuZCAx IHBvdGVudGlhbCBpc3N1ZShzKSB0byBjb25zaWRlcjoKPgo+UHJlLWV4aXN0aW5nIGlzc3VlczoK Pi0gW0hpZ2hdIFRoZSBkcml2ZXIgaWdub3JlcyBgLUVQUk9CRV9ERUZFUmAgYW5kIG90aGVyIGVy cm9ycyBmcm9tIGBkZXZtX3JlZ3VsYXRvcl9nZXRfb3B0aW9uYWwoKWAsIGltcHJvcGVybHkgZmFs bGluZyBiYWNrIHRvIHRoZSBpbnRlcm5hbCBWUkVGLgo+LS0KPgo+LS0tIFBhdGNoIFsxXTogW1BB VENIXSBod21vbjogKGFkczc4MjgpIENoZWNrIHJldHVybiB2YWx1ZSBvZiByZWdtYXBfcmVhZCgp IGluIHByb2JlIC0tLQo+Y29tbWl0IDJhNzM3Nzk5MmFmNzE3YmI2ZDI1MTEwOWQyOGJhZjAyNmJj NGE2NjUKPkF1dGhvcjogUWluZ3NodWFuZyBGdSA8ZnVxaW5nc2h1YW5nQGt5bGlub3MuY24+Cj5T dWJqZWN0OiBod21vbjogKGFkczc4MjgpIENoZWNrIHJldHVybiB2YWx1ZSBvZiByZWdtYXBfcmVh ZCgpIGluIHByb2JlCj4KPlRoaXMgcGF0Y2ggY2hlY2tzIHRoZSByZXR1cm4gdmFsdWUgb2YgYSBk dW1teSByZWdtYXBfcmVhZCgpIHVzZWQgdG8gZW5hYmxlCj50aGUgaW50ZXJuYWwgcmVmZXJlbmNl IHZvbHRhZ2UgaW4gYWRzNzgyOF9wcm9iZSgpLiBJZiB0aGUgcmVhZCBmYWlscywgaXQKPmxvZ3Mg YW4gZXJyb3IgYW5kIGFib3J0cyB0aGUgcHJvYmUgaW5zdGVhZCBvZiBzaWxlbnRseSBmYWlsaW5n IGFuZAo+cmVnaXN0ZXJpbmcgYSBub24tZnVuY3Rpb25hbCBkZXZpY2UuCj4KPj4gZGlmZiAtLWdp dCBhL2RyaXZlcnMvaHdtb24vYWRzNzgyOC5jIGIvZHJpdmVycy9od21vbi9hZHM3ODI4LmMKPj4g aW5kZXggMTQ5Y2ZjZWM3OGRjYy4uOTFlZjBjNGEwOGM0ZSAxMDA2NDQKPj4gLS0tIGEvZHJpdmVy cy9od21vbi9hZHM3ODI4LmMKPj4gKysrIGIvZHJpdmVycy9od21vbi9hZHM3ODI4LmMKPj4gQEAg LTExMiw2ICsxMTIsNyBAQCBzdGF0aWMgaW50IGFkczc4MjhfcHJvYmUoc3RydWN0IGkyY19jbGll bnQgKmNsaWVudCkKPj4gIAl1bnNpZ25lZCBpbnQgcmVndmFsOwo+PiAgCWVudW0gYWRzNzgyOF9j aGlwcyBjaGlwOwo+PiAgCXN0cnVjdCByZWd1bGF0b3IgKnJlZzsKPj4gKwlpbnQgcmV0Owo+PiAg Cj4+ICAJZGF0YSA9IGRldm1fa3phbGxvYyhkZXYsIHNpemVvZihzdHJ1Y3QgYWRzNzgyOF9kYXRh KSwgR0ZQX0tFUk5FTCk7Cj4+ICAJaWYgKCFkYXRhKQo+Cj5bU2V2ZXJpdHk6IEhpZ2hdCj5UaGlz IGlzIGEgcHJlLWV4aXN0aW5nIGlzc3VlLCBidXQgZG9lcyB0aGUgZHJpdmVyIG1pc2hhbmRsZSBl cnJvcnMgbGlrZQo+LUVQUk9CRV9ERUZFUiBmcm9tIGRldm1fcmVndWxhdG9yX2dldF9vcHRpb25h bCgpPwo+Cj5Mb29raW5nIHNsaWdodGx5IGZ1cnRoZXIgZG93biBpbiBhZHM3ODI4X3Byb2JlKCkg aW4gZHJpdmVycy9od21vbi9hZHM3ODI4LmM6Cj4KPiAgICB9IGVsc2UgaWYgKGRldi0+b2Zfbm9k ZSkgewo+ICAgICAgICBkaWZmX2lucHV0ID0gb2ZfcHJvcGVydHlfcmVhZF9ib29sKGRldi0+b2Zf bm9kZSwKPiAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAidGksZGlm ZmVyZW50aWFsLWlucHV0Iik7Cj4gICAgICAgIHJlZyA9IGRldm1fcmVndWxhdG9yX2dldF9vcHRp b25hbChkZXYsICJ2cmVmIik7Cj4gICAgICAgIGlmICghSVNfRVJSKHJlZykpIHsKPiAgICAgICAg ICAgIHZyZWZfdXYgPSByZWd1bGF0b3JfZ2V0X3ZvbHRhZ2UocmVnKTsKPgo+SWYgZGV2bV9yZWd1 bGF0b3JfZ2V0X29wdGlvbmFsKCkgcmV0dXJucyAtRVBST0JFX0RFRkVSIGJlY2F1c2UgdGhlCj5y ZWd1bGF0b3IgZHJpdmVyIGhhcyBub3QgeWV0IGNvbXBsZXRlZCBwcm9iaW5nLCB0aGUgZHJpdmVy IGlnbm9yZXMgdGhlCj5lcnJvci4gSXQgcHJvY2VlZHMgYXMgaWYgdGhlcmUgaXMgbm8gZXh0ZXJu YWwgcmVmZXJlbmNlIGFuZCBpbmNvcnJlY3RseQo+ZmFsbHMgYmFjayB0byBlbmFibGluZyB0aGUg aW50ZXJuYWwgcmVmZXJlbmNlLgo+Cj5JZiBhbiBleHRlcm5hbCB2b2x0YWdlIHJlZmVyZW5jZSBp cyBwaHlzaWNhbGx5IGNvbm5lY3RlZCBhbmQgc3BlY2lmaWVkIGluCj50aGUgZGV2aWNlIHRyZWUs IGNvdWxkIGRyaXZpbmcgdGhlIGludGVybmFsIHJlZmVyZW5jZSBvbnRvIHRoZSBzYW1lIHBpbgo+ cmVzdWx0IGluIGluY29ycmVjdCBBREMgcmVhZGluZ3Mgb3IgcG90ZW50aWFsbHkgaGFyZHdhcmUg ZGFtYWdlIGR1ZSB0byBhbgo+ZWxlY3RyaWNhbCBjb25mbGljdD8KPgo+WyAuLi4gXQo+Cj4+IEBA IC0xNjYsOCArMTY3LDEzIEBAIHN0YXRpYyBpbnQgYWRzNzgyOF9wcm9iZShzdHJ1Y3QgaTJjX2Ns aWVudCAqY2xpZW50KQo+PiAgCSAqIHZvbHRhZ2UgbmVlZHMgdG8gc2V0dGxlIGJlZm9yZSBnZXR0 aW5nIHZhbGlkIEFEQyBkYXRhLiBTbyBwZXJmb3JtIGEKPj4gIAkgKiBkdW1teSByZWFkIHRvIGVu YWJsZSB0aGUgaW50ZXJuYWwgcmVmZXJlbmNlIHZvbHRhZ2UuCj4+ICAJICovCj4+IC0JaWYgKCFl eHRfdnJlZikKPj4gLQkJcmVnbWFwX3JlYWQoZGF0YS0+cmVnbWFwLCBkYXRhLT5jbWRfYnl0ZSwg JnJlZ3ZhbCk7Cj4+ICsJaWYgKCFleHRfdnJlZikgewo+PiArCQlyZXQgPSByZWdtYXBfcmVhZChk YXRhLT5yZWdtYXAsIGRhdGEtPmNtZF9ieXRlLCAmcmVndmFsKTsKPj4gKwkJaWYgKHJldCkgewo+ PiArCQkJZGV2X2VycihkZXYsICJkdW1teSByZWFkIGZhaWxlZCB0byBlbmFibGUgaW50ZXJuYWwg VlJFRjogJWRcbiIsIHJldCk7Cj4+ICsJCQlyZXR1cm4gcmV0Owo+PiArCQl9Cj4+ICsJfQo+PiAg Cj4+ICAJaHdtb25fZGV2ID0gZGV2bV9od21vbl9kZXZpY2VfcmVnaXN0ZXJfd2l0aF9ncm91cHMo ZGV2LCBjbGllbnQtPm5hbWUsCj4+ICAJCQkJCQkJICAgZGF0YSwKPgo+LS0gCj5TYXNoaWtvIEFJ IHJldmlldyChpCBodHRwczovL3Nhc2hpa28uZGV2LyMvcGF0Y2hzZXQvMjAyNjA4MDQwOTE3MTku NjUxNDc5LTEtZmZmc3FpYW5AMTYzLmNvbT9wYXJ0PTEKCgpUaGFua3MgZm9yIHRoZSByZXZpZXcu CgpUaGUgbWlzaGFuZGxpbmcgb2YgLUVQUk9CRV9ERUZFUiBmcm9tIGRldm1fcmVndWxhdG9yX2dl dF9vcHRpb25hbCgpIGlzIGEgcHJlLWV4aXN0aW5nIGlzc3VlIGFuZCBpcyBub3QgYWRkcmVzc2Vk IGJ5IHRoaXMgcGF0Y2guIFRoaXMgY2hhbmdlIG9ubHkgYWRkcyBlcnJvciBjaGVja2luZyBmb3Ig dGhlIGR1bW15IHJlZ21hcF9yZWFkKCkgY2FsbC4KClFpbmdzaHVhbmcgRnUKCgo=