From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
	id S1754505AbbA0Kh5 (ORCPT <rfc822;w@1wt.eu>);
	Tue, 27 Jan 2015 05:37:57 -0500
Received: from cantor2.suse.de ([195.135.220.15]:44770 "EHLO mx2.suse.de"
	rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP
	id S1751425AbbA0Khz (ORCPT <rfc822;linux-kernel@vger.kernel.org>);
	Tue, 27 Jan 2015 05:37:55 -0500
Subject: Re: [lm-sensors] [PATCH] hwmon: (ads7828) Make sample interval
 configurable
From: Jean Delvare <jdelvare@suse.de>
To: Robert Rosengren <robert.rosengren@axis.com>
Cc: Guenter Roeck <linux@roeck-us.net>, lm-sensors <lm-sensors@lm-sensors.org>,
        Johan Adolfsson <johana@axis.com>,
        linux-kernel <linux-kernel@vger.kernel.org>
In-Reply-To: <54C74567.9020201@axis.com>
References: <1421409922-28299-1-git-send-email-robert.rosengren@axis.com>
	 <54B92596.2040206@roeck-us.net> <20150116183020.GA6262@roeck-us.net>
	 <54C74567.9020201@axis.com>
Content-Type: text/plain; charset="UTF-8"
Organization: Suse Linux
Date: Tue, 27 Jan 2015 11:37:50 +0100
Message-ID: <1422355070.4499.136.camel@chaos.site>
Mime-Version: 1.0
X-Mailer: Evolution 2.28.2 
Content-Transfer-Encoding: 8bit
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Hi Robert,

Le Tuesday 27 January 2015 à 08:59 +0100, Robert Rosengren a écrit :
> On 01/16/2015 07:30 PM, Guenter Roeck wrote:
> > I sent a couple of patches a minute ago which should take care of the issue.
> > Would be great if you can test it.
> I applied the v2 versions of the patches for a quick test, but it was 
> not successful. Haven't done any further investigation on what the 
> problem might be, but when trying to read the sysfs driver I get "read 
> error: No such device or address".
> 
> The sysfs path have moved with the REGMAP settings, from 
> /sys/bus/i2c/devices/<x>/in0_input to 
> /sys/bus/i2c/devices/<x>/hwmon/hwmon0/in0_input.

This is expected, shouldn't be an issue if you are using a
libsensors-based application with a recent enough version of libsensors.

> Any ideas? Haven't had REGMAP configured in my kernel earlier, might it 
> be some specific configuration I miss?

How are you testing/reading exactly?

-- 
Jean Delvare
SUSE L3 Support


From mboxrd@z Thu Jan  1 00:00:00 1970
From: Jean Delvare <jdelvare@suse.de>
Date: Tue, 27 Jan 2015 10:37:50 +0000
Subject: Re: [lm-sensors] [PATCH] hwmon: (ads7828) Make sample interval configurable
Message-Id: <1422355070.4499.136.camel@chaos.site>
List-Id: <lm-sensors.vger.kernel.org>
References: <1421409922-28299-1-git-send-email-robert.rosengren@axis.com>
	 <54B92596.2040206@roeck-us.net> <20150116183020.GA6262@roeck-us.net>
	 <54C74567.9020201@axis.com>
In-Reply-To: <54C74567.9020201@axis.com>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: Robert Rosengren <robert.rosengren@axis.com>
Cc: Guenter Roeck <linux@roeck-us.net>, lm-sensors <lm-sensors@lm-sensors.org>, Johan Adolfsson <johana@axis.com>, linux-kernel <linux-kernel@vger.kernel.org>

SGkgUm9iZXJ0LAoKTGUgVHVlc2RheSAyNyBKYW51YXJ5IDIwMTUgw6AgMDg6NTkgKzAxMDAsIFJv
YmVydCBSb3NlbmdyZW4gYSDDqWNyaXQgOgo+IE9uIDAxLzE2LzIwMTUgMDc6MzAgUE0sIEd1ZW50
ZXIgUm9lY2sgd3JvdGU6Cj4gPiBJIHNlbnQgYSBjb3VwbGUgb2YgcGF0Y2hlcyBhIG1pbnV0ZSBh
Z28gd2hpY2ggc2hvdWxkIHRha2UgY2FyZSBvZiB0aGUgaXNzdWUuCj4gPiBXb3VsZCBiZSBncmVh
dCBpZiB5b3UgY2FuIHRlc3QgaXQuCj4gSSBhcHBsaWVkIHRoZSB2MiB2ZXJzaW9ucyBvZiB0aGUg
cGF0Y2hlcyBmb3IgYSBxdWljayB0ZXN0LCBidXQgaXQgd2FzIAo+IG5vdCBzdWNjZXNzZnVsLiBI
YXZlbid0IGRvbmUgYW55IGZ1cnRoZXIgaW52ZXN0aWdhdGlvbiBvbiB3aGF0IHRoZSAKPiBwcm9i
bGVtIG1pZ2h0IGJlLCBidXQgd2hlbiB0cnlpbmcgdG8gcmVhZCB0aGUgc3lzZnMgZHJpdmVyIEkg
Z2V0ICJyZWFkIAo+IGVycm9yOiBObyBzdWNoIGRldmljZSBvciBhZGRyZXNzIi4KPiAKPiBUaGUg
c3lzZnMgcGF0aCBoYXZlIG1vdmVkIHdpdGggdGhlIFJFR01BUCBzZXR0aW5ncywgZnJvbSAKPiAv
c3lzL2J1cy9pMmMvZGV2aWNlcy88eD4vaW4wX2lucHV0IHRvIAo+IC9zeXMvYnVzL2kyYy9kZXZp
Y2VzLzx4Pi9od21vbi9od21vbjAvaW4wX2lucHV0LgoKVGhpcyBpcyBleHBlY3RlZCwgc2hvdWxk
bid0IGJlIGFuIGlzc3VlIGlmIHlvdSBhcmUgdXNpbmcgYQpsaWJzZW5zb3JzLWJhc2VkIGFwcGxp
Y2F0aW9uIHdpdGggYSByZWNlbnQgZW5vdWdoIHZlcnNpb24gb2YgbGlic2Vuc29ycy4KCj4gQW55
IGlkZWFzPyBIYXZlbid0IGhhZCBSRUdNQVAgY29uZmlndXJlZCBpbiBteSBrZXJuZWwgZWFybGll
ciwgbWlnaHQgaXQgCj4gYmUgc29tZSBzcGVjaWZpYyBjb25maWd1cmF0aW9uIEkgbWlzcz8KCkhv
dyBhcmUgeW91IHRlc3RpbmcvcmVhZGluZyBleGFjdGx5PwoKLS0gCkplYW4gRGVsdmFyZQpTVVNF
IEwzIFN1cHBvcnQKCgpfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f
X19fXwpsbS1zZW5zb3JzIG1haWxpbmcgbGlzdApsbS1zZW5zb3JzQGxtLXNlbnNvcnMub3JnCmh0
dHA6Ly9saXN0cy5sbS1zZW5zb3JzLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2xtLXNlbnNvcnM