Re: [PATCH] Rename inode_cmp_iversion{+raw} to inode_eq_iversion{+raw}

2018-01-31 Thread Goffredo Baroncelli
On 01/31/2018 10:55 PM, Matthew Wilcox wrote: > On Wed, Jan 31, 2018 at 09:43:09PM +0100, Goffredo Baroncelli wrote: >> The function inode_cmp_iversion{+raw} is counter-intuitive, because it >> returns true when the counters are different and false when these are equa

Re: [PATCH] Rename inode_cmp_iversion{+raw} to inode_eq_iversion{+raw}

2018-01-31 Thread Goffredo Baroncelli
On 01/31/2018 10:55 PM, Matthew Wilcox wrote: > On Wed, Jan 31, 2018 at 09:43:09PM +0100, Goffredo Baroncelli wrote: >> The function inode_cmp_iversion{+raw} is counter-intuitive, because it >> returns true when the counters are different and false when these are equa

[PATCH] Rename make inode_cmp_iversion{+raw} to inode_eq_iversion{+raw}

2018-01-31 Thread Goffredo Baroncelli
From: Goffredo Baroncelli <kreij...@inwind.it> The function inode_cmp_iversion{+raw} is counter-intuitive, because it returns true when the counters are different and false when these are equal. Rename it to inode_eq_iversion{+raw}, which will returns true when the counters are equal and

[PATCH] Rename make inode_cmp_iversion{+raw} to inode_eq_iversion{+raw}

2018-01-31 Thread Goffredo Baroncelli
From: Goffredo Baroncelli The function inode_cmp_iversion{+raw} is counter-intuitive, because it returns true when the counters are different and false when these are equal. Rename it to inode_eq_iversion{+raw}, which will returns true when the counters are equal and false otherwise. Signed

[RESEND][PATCH 5/5] Export the temperatures via hwmon

2014-11-11 Thread Goffredo Baroncelli
U temp2_input: 36750 temp2_label: Case temp3_input: 37750 temp3_label: Case2 The Case2 temperature is the sensor temperature inside the adm1030. Signed-off-by: Goffredo Baroncelli --- drivers/macintosh/therm_windtunnel.c | 96 ++-- 1 file changed, 92 insertions(+), 4 dele

[RESEND][PATCH][v5] therm_windtunnel does not work properly on PowerMac G4

2014-11-11 Thread Goffredo Baroncelli
he debian bug #741663 which highlight the same problem. In the bug discussion there is a patch like the my ones. See also https://lists.ozlabs.org/pipermail/linuxppc-dev/2012-July/099561.html -- gpg @keyserver.linux.it: Goffredo Baroncelli Key fingerprint BBF5 1610 0B64 DAC6 5F7D 17B2 0EDA 9B37 8B82 E0

[RESEND][PATCH 4/5] Return the fan speed via sysfs

2014-11-11 Thread Goffredo Baroncelli
Return the fan speed via sysfs: /sys/devices/temperature/fan_level This patch is extracted from a Bryan Christianson's bigger one (see debian bug #741663) Signed-off-by: Goffredo Baroncelli --- drivers/macintosh/therm_windtunnel.c | 10 +- 1 file changed, 9 insertions(+), 1 deletion

[RESEND][PATCH 1/5] Update drivers names to the ones invoked by i2c-powermac

2014-11-11 Thread Goffredo Baroncelli
Update drivers names to the ones invoked by i2c-powermac: - therm_ds1775 -> MAC,ds1775 - therm_adm1030 -> MAC,adm1030 The background fan control loop is started from the devices probing methods. Signed-off-by: Goffredo Baroncelli --- drivers/macintosh/therm_windtunnel.

[RESEND][PATCH 2/5] Remove attach_method because un-used

2014-11-11 Thread Goffredo Baroncelli
Remove attach_method because i2c-powermac is in charge to instantiate the driver directly. Signed-off-by: Goffredo Baroncelli --- drivers/macintosh/therm_windtunnel.c | 35 --- 1 file changed, 35 deletions(-) diff --git a/drivers/macintosh/therm_windtunnel.c b

[RESEND][PATCH 3/5] Add the "verbose" module option.

2014-11-11 Thread Goffredo Baroncelli
Add a "verbose" option to control the message in the kernel log verbose = 0 no message verbose = 1 log only the fan speed changes verbose = 2 log the fan speed changes and the temperature changes Signed-off-by: Goffredo Baroncelli --- drivers/macintosh/therm_windtun

[RESEND][PATCH 3/5] Add the verbose module option.

2014-11-11 Thread Goffredo Baroncelli
Add a verbose option to control the message in the kernel log verbose = 0 no message verbose = 1 log only the fan speed changes verbose = 2 log the fan speed changes and the temperature changes Signed-off-by: Goffredo Baroncelli kreij...@inwind.it --- drivers/macintosh/therm_windtunnel.c

[RESEND][PATCH 1/5] Update drivers names to the ones invoked by i2c-powermac

2014-11-11 Thread Goffredo Baroncelli
Update drivers names to the ones invoked by i2c-powermac: - therm_ds1775 - MAC,ds1775 - therm_adm1030 - MAC,adm1030 The background fan control loop is started from the devices probing methods. Signed-off-by: Goffredo Baroncelli kreij...@inwind.it --- drivers/macintosh/therm_windtunnel.c | 23

[RESEND][PATCH 2/5] Remove attach_method because un-used

2014-11-11 Thread Goffredo Baroncelli
Remove attach_method because i2c-powermac is in charge to instantiate the driver directly. Signed-off-by: Goffredo Baroncelli kreij...@inwind.it --- drivers/macintosh/therm_windtunnel.c | 35 --- 1 file changed, 35 deletions(-) diff --git a/drivers/macintosh

[RESEND][PATCH 5/5] Export the temperatures via hwmon

2014-11-11 Thread Goffredo Baroncelli
: Case temp3_input: 37750 temp3_label: Case2 The Case2 temperature is the sensor temperature inside the adm1030. Signed-off-by: Goffredo Baroncelli kreij...@inwind.it --- drivers/macintosh/therm_windtunnel.c | 96 ++-- 1 file changed, 92 insertions(+), 4 deletions

[RESEND][PATCH][v5] therm_windtunnel does not work properly on PowerMac G4

2014-11-11 Thread Goffredo Baroncelli
the same problem. In the bug discussion there is a patch like the my ones. See also https://lists.ozlabs.org/pipermail/linuxppc-dev/2012-July/099561.html -- gpg @keyserver.linux.it: Goffredo Baroncelli kreijackATinwind.it Key fingerprint BBF5 1610 0B64 DAC6 5F7D 17B2 0EDA 9B37 8B82 E0B5

[RESEND][PATCH 4/5] Return the fan speed via sysfs

2014-11-11 Thread Goffredo Baroncelli
Return the fan speed via sysfs: /sys/devices/temperature/fan_level This patch is extracted from a Bryan Christianson's bigger one (see debian bug #741663) Signed-off-by: Goffredo Baroncelli kreij...@inwind.it --- drivers/macintosh/therm_windtunnel.c | 10 +- 1 file changed, 9 insertions

Re: [PATCH][v5] therm_windtunnel does not work properly on PowerMac G4

2014-08-27 Thread Goffredo Baroncelli
Hi Benjamin, do you have any feedback about this ? Do you think that it would be possible to include these patches in a next pull-for-linus ? Let me know, if you want other changes. BR G.Baroncelli On 08/09/2014 08:49 AM, Goffredo Baroncelli wrote: > Hi All, > > On a PowerMac G4

Re: [PATCH][v5] therm_windtunnel does not work properly on PowerMac G4

2014-08-27 Thread Goffredo Baroncelli
Hi Benjamin, do you have any feedback about this ? Do you think that it would be possible to include these patches in a next pull-for-linus ? Let me know, if you want other changes. BR G.Baroncelli On 08/09/2014 08:49 AM, Goffredo Baroncelli wrote: Hi All, On a PowerMac G4 I noticed

[PATCH 5/5] Export the temperatures via hwmon

2014-08-09 Thread Goffredo Baroncelli
U temp2_input: 36750 temp2_label: Case temp3_input: 37750 temp3_label: Case2 The Case2 temperature is the sensor temperature inside the adm1030. Signed-off-by: Goffredo Baroncelli --- drivers/macintosh/therm_windtunnel.c | 96 ++-- 1 file changed, 92 insertions(+), 4 dele

[PATCH 2/5] Remove attach_method because un-used

2014-08-09 Thread Goffredo Baroncelli
Remove attach_method because i2c-powermac is in charge to instantiate the driver directly. Signed-off-by: Goffredo Baroncelli --- drivers/macintosh/therm_windtunnel.c | 35 --- 1 file changed, 35 deletions(-) diff --git a/drivers/macintosh/therm_windtunnel.c b

[PATCH 1/5] Update drivers names to the ones invoked by i2c-powermac

2014-08-09 Thread Goffredo Baroncelli
Update drivers names to the ones invoked by i2c-powermac: - therm_ds1775 -> MAC,ds1775 - therm_adm1030 -> MAC,adm1030 The background fan control loop is started from the devices probing methods. Signed-off-by: Goffredo Baroncelli --- drivers/macintosh/therm_windtunnel.

[PATCH 4/5] Return the fan speed via sysfs

2014-08-09 Thread Goffredo Baroncelli
Return the fan speed via sysfs: /sys/devices/temperature/fan_level This patch is extracted from a Bryan Christianson's bigger one (see debian bug #741663) Signed-off-by: Goffredo Baroncelli --- drivers/macintosh/therm_windtunnel.c | 10 +- 1 file changed, 9 insertions(+), 1 deletion

[PATCH 3/5] Add the "verbose" module option.

2014-08-09 Thread Goffredo Baroncelli
Add a "verbose" option to control the message in the kernel log verbose = 0 no message verbose = 1 log only the fan speed changes verbose = 2 log the fan speed changes and the temperature changes Signed-off-by: Goffredo Baroncelli --- drivers/macintosh/therm_windtun

[PATCH][v5] therm_windtunnel does not work properly on PowerMac G4

2014-08-09 Thread Goffredo Baroncelli
il/linuxppc-dev/2012-July/099561.html -- gpg @keyserver.linux.it: Goffredo Baroncelli Key fingerprint BBF5 1610 0B64 DAC6 5F7D 17B2 0EDA 9B37 8B82 E0B5 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More major

[PATCH][v5] therm_windtunnel does not work properly on PowerMac G4

2014-08-09 Thread Goffredo Baroncelli
-- gpg @keyserver.linux.it: Goffredo Baroncelli kreijackATinwind.it Key fingerprint BBF5 1610 0B64 DAC6 5F7D 17B2 0EDA 9B37 8B82 E0B5 -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http

[PATCH 3/5] Add the verbose module option.

2014-08-09 Thread Goffredo Baroncelli
Add a verbose option to control the message in the kernel log verbose = 0 no message verbose = 1 log only the fan speed changes verbose = 2 log the fan speed changes and the temperature changes Signed-off-by: Goffredo Baroncelli kreij...@inwind.it --- drivers/macintosh/therm_windtunnel.c

[PATCH 1/5] Update drivers names to the ones invoked by i2c-powermac

2014-08-09 Thread Goffredo Baroncelli
Update drivers names to the ones invoked by i2c-powermac: - therm_ds1775 - MAC,ds1775 - therm_adm1030 - MAC,adm1030 The background fan control loop is started from the devices probing methods. Signed-off-by: Goffredo Baroncelli kreij...@inwind.it --- drivers/macintosh/therm_windtunnel.c | 23

[PATCH 4/5] Return the fan speed via sysfs

2014-08-09 Thread Goffredo Baroncelli
Return the fan speed via sysfs: /sys/devices/temperature/fan_level This patch is extracted from a Bryan Christianson's bigger one (see debian bug #741663) Signed-off-by: Goffredo Baroncelli kreij...@inwind.it --- drivers/macintosh/therm_windtunnel.c | 10 +- 1 file changed, 9 insertions

[PATCH 2/5] Remove attach_method because un-used

2014-08-09 Thread Goffredo Baroncelli
Remove attach_method because i2c-powermac is in charge to instantiate the driver directly. Signed-off-by: Goffredo Baroncelli kreij...@inwind.it --- drivers/macintosh/therm_windtunnel.c | 35 --- 1 file changed, 35 deletions(-) diff --git a/drivers/macintosh

[PATCH 5/5] Export the temperatures via hwmon

2014-08-09 Thread Goffredo Baroncelli
: Case temp3_input: 37750 temp3_label: Case2 The Case2 temperature is the sensor temperature inside the adm1030. Signed-off-by: Goffredo Baroncelli kreij...@inwind.it --- drivers/macintosh/therm_windtunnel.c | 96 ++-- 1 file changed, 92 insertions(+), 4 deletions

Re: [PATCH 5/5] Export the temperatures via hwmon

2014-08-08 Thread Goffredo Baroncelli
On 08/08/2014 06:30 PM, Guenter Roeck wrote: > On 08/08/2014 07:54 AM, Goffredo Baroncelli wrote: [...] > I just noticed - hwmon_device_register_with_groups() returns ERR_PTR > on errors, not NULL, so checking for a NULL return value won't help > much if there is an error.

Re: [PATCH 5/5] Export the temperatures via hwmon

2014-08-08 Thread Goffredo Baroncelli
On 08/07/2014 11:19 PM, Matt Helsley wrote: > On Thu, Aug 07, 2014 at 07:50:00PM +0200, Goffredo Baroncelli wrote: >> On 08/07/2014 09:36 AM, Guenter Roeck wrote: >>> On 08/06/2014 11:52 PM, Jean Delvare wrote: >>>> Hi Guenter, >>>> >>>> O

Re: [PATCH 5/5] Export the temperatures via hwmon

2014-08-08 Thread Goffredo Baroncelli
On 08/07/2014 11:19 PM, Matt Helsley wrote: On Thu, Aug 07, 2014 at 07:50:00PM +0200, Goffredo Baroncelli wrote: On 08/07/2014 09:36 AM, Guenter Roeck wrote: On 08/06/2014 11:52 PM, Jean Delvare wrote: Hi Guenter, On Wed, 06 Aug 2014 23:20:32 -0700, Guenter Roeck wrote: Patch 4/5 is Return

Re: [PATCH 5/5] Export the temperatures via hwmon

2014-08-08 Thread Goffredo Baroncelli
On 08/08/2014 06:30 PM, Guenter Roeck wrote: On 08/08/2014 07:54 AM, Goffredo Baroncelli wrote: [...] I just noticed - hwmon_device_register_with_groups() returns ERR_PTR on errors, not NULL, so checking for a NULL return value won't help much if there is an error. Thanks, due

Re: [PATCH 5/5] Export the temperatures via hwmon

2014-08-07 Thread Goffredo Baroncelli
On 08/07/2014 08:16 PM, Guenter Roeck wrote: > On Thu, Aug 07, 2014 at 07:50:00PM +0200, Goffredo Baroncelli wrote: >> When (if) these patches will be accepted I want to write another solution, >> but definitely not now. And even if it would work for me, it

[PATCH 1/5] Update drivers names to the ones invoked by i2c-powermac

2014-08-07 Thread Goffredo Baroncelli
Update drivers names to the ones invoked by i2c-powermac: - therm_ds1775 -> MAC,ds1775 - therm_adm1030 -> MAC,adm1030 The background fan control loop is started from the devices probing methods. Signed-off-by: Goffredo Baroncelli --- drivers/macintosh/therm_windtunnel.

[PATCH][v4] therm_windtunnel does not work properly on PowerMac G4

2014-08-07 Thread Goffredo Baroncelli
blem. In the bug discussion there is a patch like the my ones. See also https://lists.ozlabs.org/pipermail/linuxppc-dev/2012-July/099561.html -- gpg @keyserver.linux.it: Goffredo Baroncelli Key fingerprint BBF5 1610 0B64 DAC6 5F7D 17B2 0EDA 9B37 8B82 E0B5 -- To unsubscribe from this li

[PATCH 3/5] Add the "verbose" module option.

2014-08-07 Thread Goffredo Baroncelli
Add a "verbose" option to control the message in the kernel log verbose = 0 no message verbose = 1 log only the fan speed changes verbose = 2 log the fan speed changes and the temperature changes Signed-off-by: Goffredo Baroncelli --- drivers/macintosh/therm_windtun

[PATCH 4/5] Return the fan speed via sysfs

2014-08-07 Thread Goffredo Baroncelli
Return the fan speed via sysfs: /sys/devices/temperature/fan_level This patch is copied from the Bryan Christianson's patch (see debian bug #741663) Signed-off-by: Goffredo Baroncelli --- drivers/macintosh/therm_windtunnel.c | 10 +- 1 file changed, 9 insertions(+), 1 deletion(-) diff

[PATCH 5/5] Export the temperatures via hwmon

2014-08-07 Thread Goffredo Baroncelli
U temp2_input: 36750 temp2_label: Case temp3_input: 37750 temp3_label: Case2 The Case2 temperature is the sensor temperature inside the adm1030. Signed-off-by: Goffredo Baroncelli --- drivers/macintosh/therm_windtunnel.c | 94 ++-- 1 file changed, 90 insertions(+), 4 dele

[PATCH 2/5] Remove attach_method because un-used

2014-08-07 Thread Goffredo Baroncelli
Remove attach_method because i2c-powermac is in charge to instantiate the driver directly. Signed-off-by: Goffredo Baroncelli --- drivers/macintosh/therm_windtunnel.c | 35 --- 1 file changed, 35 deletions(-) diff --git a/drivers/macintosh/therm_windtunnel.c b

Re: [PATCH 5/5] Export the temperatures via hwmon

2014-08-07 Thread Goffredo Baroncelli
com/source/drivers/macintosh/therm_windtunnel.c#L153 and the adm1030 datasheet. I am reluctant to change this thing because this driver is for an obsolete platform (apple stopped the powermac in 2004/2005), and bad or good I have to suppose that it works. Moreover I am not in the position

Re: [PATCH 3/5] Add the "verbose" module option.

2014-08-07 Thread Goffredo Baroncelli
On 08/07/2014 06:43 PM, Jean Delvare wrote: > On Thu, 07 Aug 2014 18:29:23 +0200, Goffredo Baroncelli wrote: >> On 08/07/2014 10:52 AM, Jean Delvare wrote: >>> Le Wednesday 06 August 2014 à 21:05 +, Goffredo Baroncelli a écrit : >>>> + */ >>>> +

Re: [PATCH 3/5] Add the "verbose" module option.

2014-08-07 Thread Goffredo Baroncelli
On 08/07/2014 10:52 AM, Jean Delvare wrote: > Le Wednesday 06 August 2014 à 21:05 +0000, Goffredo Baroncelli a écrit : >> Add a "verbose" option to control the message in the kernel log >> verbose = 0 no message >> verbose = 1 log only the fan speed changes >

Re: [PATCH 3/5] Add the verbose module option.

2014-08-07 Thread Goffredo Baroncelli
On 08/07/2014 10:52 AM, Jean Delvare wrote: Le Wednesday 06 August 2014 à 21:05 +, Goffredo Baroncelli a écrit : Add a verbose option to control the message in the kernel log verbose = 0 no message verbose = 1 log only the fan speed changes verbose = 2 log the fan speed changes

Re: [PATCH 3/5] Add the verbose module option.

2014-08-07 Thread Goffredo Baroncelli
On 08/07/2014 06:43 PM, Jean Delvare wrote: On Thu, 07 Aug 2014 18:29:23 +0200, Goffredo Baroncelli wrote: On 08/07/2014 10:52 AM, Jean Delvare wrote: Le Wednesday 06 August 2014 à 21:05 +, Goffredo Baroncelli a écrit : + */ + tempchanged = x.temp != temp || x.casetemp != casetemp

Re: [PATCH 5/5] Export the temperatures via hwmon

2014-08-07 Thread Goffredo Baroncelli
not in the position to test the changes outside my hardware (1 powermac DP@1Ghz mdd) -- gpg @keyserver.linux.it: Goffredo Baroncelli (kreijackATinwind.it Key fingerprint BBF5 1610 0B64 DAC6 5F7D 17B2 0EDA 9B37 8B82 E0B5 -- To unsubscribe from this list: send the line unsubscribe linux-kernel

[PATCH 5/5] Export the temperatures via hwmon

2014-08-07 Thread Goffredo Baroncelli
: Case temp3_input: 37750 temp3_label: Case2 The Case2 temperature is the sensor temperature inside the adm1030. Signed-off-by: Goffredo Baroncelli kreij...@inwind.it --- drivers/macintosh/therm_windtunnel.c | 94 ++-- 1 file changed, 90 insertions(+), 4 deletions

[PATCH 2/5] Remove attach_method because un-used

2014-08-07 Thread Goffredo Baroncelli
Remove attach_method because i2c-powermac is in charge to instantiate the driver directly. Signed-off-by: Goffredo Baroncelli kreij...@inwind.it --- drivers/macintosh/therm_windtunnel.c | 35 --- 1 file changed, 35 deletions(-) diff --git a/drivers/macintosh

[PATCH 4/5] Return the fan speed via sysfs

2014-08-07 Thread Goffredo Baroncelli
Return the fan speed via sysfs: /sys/devices/temperature/fan_level This patch is copied from the Bryan Christianson's patch (see debian bug #741663) Signed-off-by: Goffredo Baroncelli kreij...@inwind.it --- drivers/macintosh/therm_windtunnel.c | 10 +- 1 file changed, 9 insertions(+), 1

[PATCH 3/5] Add the verbose module option.

2014-08-07 Thread Goffredo Baroncelli
Add a verbose option to control the message in the kernel log verbose = 0 no message verbose = 1 log only the fan speed changes verbose = 2 log the fan speed changes and the temperature changes Signed-off-by: Goffredo Baroncelli kreij...@inwind.it --- drivers/macintosh/therm_windtunnel.c

[PATCH][v4] therm_windtunnel does not work properly on PowerMac G4

2014-08-07 Thread Goffredo Baroncelli
discussion there is a patch like the my ones. See also https://lists.ozlabs.org/pipermail/linuxppc-dev/2012-July/099561.html -- gpg @keyserver.linux.it: Goffredo Baroncelli kreijackATinwind.it Key fingerprint BBF5 1610 0B64 DAC6 5F7D 17B2 0EDA 9B37 8B82 E0B5 -- To unsubscribe from this list: send

[PATCH 1/5] Update drivers names to the ones invoked by i2c-powermac

2014-08-07 Thread Goffredo Baroncelli
Update drivers names to the ones invoked by i2c-powermac: - therm_ds1775 - MAC,ds1775 - therm_adm1030 - MAC,adm1030 The background fan control loop is started from the devices probing methods. Signed-off-by: Goffredo Baroncelli kreij...@inwind.it --- drivers/macintosh/therm_windtunnel.c | 23

Re: [PATCH 5/5] Export the temperatures via hwmon

2014-08-07 Thread Goffredo Baroncelli
On 08/07/2014 08:16 PM, Guenter Roeck wrote: On Thu, Aug 07, 2014 at 07:50:00PM +0200, Goffredo Baroncelli wrote: When (if) these patches will be accepted I want to write another solution, but definitely not now. And even if it would work for me, it is very likely that will be accepted

Re: [PATCH 5/5] Export the temperatures via hwmon

2014-08-06 Thread Goffredo Baroncelli
On 08/07/2014 01:18 AM, Guenter Roeck wrote: > On Wed, Aug 06, 2014 at 09:05:03PM +0000, Goffredo Baroncelli wrote: >> From: Goffredo Baroncelli >> >> Export the temperature via the hwmon subsystem. >> See the list below for the sensors exported: >> >> $ c

[PATCH 2/5] Remove attach_method because un-used

2014-08-06 Thread Goffredo Baroncelli
Remove attach_method because i2c-powermac is in charge to instantiate the driver directly. Signed-off-by: Goffredo Baroncelli --- drivers/macintosh/therm_windtunnel.c | 35 --- 1 file changed, 35 deletions(-) diff --git a/drivers/macintosh/therm_windtunnel.c b

[PATCH][v3] therm_windtunnel doesn't work properly on PowerMac G4

2014-08-06 Thread Goffredo Baroncelli
instead. Signed-off-by: Benjamin Herrenschmidt [2] There is the debian bug #741663 which highlight the same problem. In the bug discussion there is a patch like the my ones. See also https://lists.ozlabs.org/pipermail/linuxppc-dev/2012-July/099561.html -- gpg @keyserver.linux.it: Goffre

[PATCH 3/5] Add the "verbose" module option.

2014-08-06 Thread Goffredo Baroncelli
Add a "verbose" option to control the message in the kernel log verbose = 0 no message verbose = 1 log only the fan speed changes verbose = 2 log the fan speed changes and the temperature changes Signed-off-by: Goffredo Baroncelli --- drivers/macintosh/therm_windtun

[PATCH 1/5] Update drivers names to the ones invoked by i2c-powermac

2014-08-06 Thread Goffredo Baroncelli
Update drivers names to the ones invoked by i2c-powermac: - therm_ds1775 -> MAC,ds1775 - therm_adm1030 -> MAC,adm1030 The background fan control loop is started from the devices probing methods. Signed-off-by: Goffredo Baroncelli --- drivers/macintosh/therm_windtunnel.

[PATCH 5/5] Export the temperatures via hwmon

2014-08-06 Thread Goffredo Baroncelli
From: Goffredo Baroncelli Export the temperature via the hwmon subsystem. See the list below for the sensors exported: $ cd /sys/devices/temperature/hwmon/hwmon0 $ echo "name: $(cat name)"; for i in temp*; do echo "$i: $(cat $i)"; done name: therm_windtunnel temp1_input: 59

[PATCH 4/5] Return the fan speed via sysfs

2014-08-06 Thread Goffredo Baroncelli
Return the fan speed via sysfs: /sys/devices/temperature/fan_level This patch is copied from the Bryan Christianson's patch (see debian bug #741663) Signed-off-by: Goffredo Baroncelli --- drivers/macintosh/therm_windtunnel.c | 10 +- 1 file changed, 9 insertions(+), 1 deletion(-) diff

[PATCH 4/5] Return the fan speed via sysfs

2014-08-06 Thread Goffredo Baroncelli
Return the fan speed via sysfs: /sys/devices/temperature/fan_level This patch is copied from the Bryan Christianson's patch (see debian bug #741663) Signed-off-by: Goffredo Baroncelli kreij...@inwind.it --- drivers/macintosh/therm_windtunnel.c | 10 +- 1 file changed, 9 insertions(+), 1

[PATCH 5/5] Export the temperatures via hwmon

2014-08-06 Thread Goffredo Baroncelli
From: Goffredo Baroncelli kreij...@iniwnd.it Export the temperature via the hwmon subsystem. See the list below for the sensors exported: $ cd /sys/devices/temperature/hwmon/hwmon0 $ echo name: $(cat name); for i in temp*; do echo $i: $(cat $i); done name: therm_windtunnel temp1_input: 59312

[PATCH 1/5] Update drivers names to the ones invoked by i2c-powermac

2014-08-06 Thread Goffredo Baroncelli
Update drivers names to the ones invoked by i2c-powermac: - therm_ds1775 - MAC,ds1775 - therm_adm1030 - MAC,adm1030 The background fan control loop is started from the devices probing methods. Signed-off-by: Goffredo Baroncelli kreij...@inwind.it --- drivers/macintosh/therm_windtunnel.c | 23

[PATCH][v3] therm_windtunnel doesn't work properly on PowerMac G4

2014-08-06 Thread Goffredo Baroncelli
Herrenschmidt b...@kernel.crashing.org [2] There is the debian bug #741663 which highlight the same problem. In the bug discussion there is a patch like the my ones. See also https://lists.ozlabs.org/pipermail/linuxppc-dev/2012-July/099561.html -- gpg @keyserver.linux.it: Goffredo Baroncelli

[PATCH 3/5] Add the verbose module option.

2014-08-06 Thread Goffredo Baroncelli
Add a verbose option to control the message in the kernel log verbose = 0 no message verbose = 1 log only the fan speed changes verbose = 2 log the fan speed changes and the temperature changes Signed-off-by: Goffredo Baroncelli kreij...@inwind.it --- drivers/macintosh/therm_windtunnel.c

[PATCH 2/5] Remove attach_method because un-used

2014-08-06 Thread Goffredo Baroncelli
Remove attach_method because i2c-powermac is in charge to instantiate the driver directly. Signed-off-by: Goffredo Baroncelli kreij...@inwind.it --- drivers/macintosh/therm_windtunnel.c | 35 --- 1 file changed, 35 deletions(-) diff --git a/drivers/macintosh

Re: [PATCH 5/5] Export the temperatures via hwmon

2014-08-06 Thread Goffredo Baroncelli
On 08/07/2014 01:18 AM, Guenter Roeck wrote: On Wed, Aug 06, 2014 at 09:05:03PM +, Goffredo Baroncelli wrote: From: Goffredo Baroncelli kreij...@iniwnd.it Export the temperature via the hwmon subsystem. See the list below for the sensors exported: $ cd /sys/devices/temperature/hwmon

Re: [PATCH 3/4] Add the "verbose" module option.

2014-08-04 Thread Goffredo Baroncelli
On 08/04/2014 10:46 AM, Jean Delvare wrote: > Le Sunday 03 August 2014 à 18:36 +0200, Goffredo Baroncelli a écrit : >> On 08/03/2014 05:52 PM, Jean Delvare wrote: >>> On Sun, 03 Aug 2014 17:12:57 +0200, Goffredo Baroncelli wrote: >>>> On 08/03/201

Re: [PATCH 3/4] Add the verbose module option.

2014-08-04 Thread Goffredo Baroncelli
On 08/04/2014 10:46 AM, Jean Delvare wrote: Le Sunday 03 August 2014 à 18:36 +0200, Goffredo Baroncelli a écrit : On 08/03/2014 05:52 PM, Jean Delvare wrote: On Sun, 03 Aug 2014 17:12:57 +0200, Goffredo Baroncelli wrote: On 08/03/2014 04:12 PM, Jean Delvare wrote: +(verbose 0

Re: [PATCH 4/4] Return the fan speed via sysfs

2014-08-03 Thread Goffredo Baroncelli
On 08/03/2014 05:59 PM, Jean Delvare wrote: > On Sun, 03 Aug 2014 17:27:17 +0200, Goffredo Baroncelli wrote: >> On 08/03/2014 04:17 PM, Jean Delvare wrote: >>> On Fri, 1 Aug 2014 14:00:50 +, Goffredo Baroncelli wrote: >>>> Return the fan speed via sysfs:

Re: [PATCH 3/4] Add the "verbose" module option.

2014-08-03 Thread Goffredo Baroncelli
On 08/03/2014 05:52 PM, Jean Delvare wrote: > On Sun, 03 Aug 2014 17:12:57 +0200, Goffredo Baroncelli wrote: >> On 08/03/2014 04:12 PM, Jean Delvare wrote: >>>> + (verbose > 0 && level >= 0)) { >>>> + print_temp(&qu

Re: [PATCH 4/4] Return the fan speed via sysfs

2014-08-03 Thread Goffredo Baroncelli
On 08/03/2014 04:17 PM, Jean Delvare wrote: > On Fri, 1 Aug 2014 14:00:50 +0000, Goffredo Baroncelli wrote: >> Return the fan speed via sysfs: >> /sys/devices/temperature/fan_level > > Good idea. Even better would be if the driver would expose a standard > hwmon inter

Re: [PATCH 3/4] Add the "verbose" module option.

2014-08-03 Thread Goffredo Baroncelli
On 08/03/2014 04:12 PM, Jean Delvare wrote: > Hi Goffredo, > > You messed up your Cc's ;-) I fight hard with git-send-email In the next trip I will check two times ! > > On Fri, 1 Aug 2014 14:00:49 +, Goffredo Baroncelli wrote: >> The "verbose&qu

Re: [PATCH 3/4] Add the verbose module option.

2014-08-03 Thread Goffredo Baroncelli
On 08/03/2014 04:12 PM, Jean Delvare wrote: Hi Goffredo, You messed up your Cc's ;-) I fight hard with git-send-email In the next trip I will check two times ! On Fri, 1 Aug 2014 14:00:49 +, Goffredo Baroncelli wrote: The verbose option controls the message in the kernel log

Re: [PATCH 4/4] Return the fan speed via sysfs

2014-08-03 Thread Goffredo Baroncelli
On 08/03/2014 04:17 PM, Jean Delvare wrote: On Fri, 1 Aug 2014 14:00:50 +, Goffredo Baroncelli wrote: Return the fan speed via sysfs: /sys/devices/temperature/fan_level Good idea. Even better would be if the driver would expose a standard hwmon interface for the temperature values. Fan

Re: [PATCH 3/4] Add the verbose module option.

2014-08-03 Thread Goffredo Baroncelli
On 08/03/2014 05:52 PM, Jean Delvare wrote: On Sun, 03 Aug 2014 17:12:57 +0200, Goffredo Baroncelli wrote: On 08/03/2014 04:12 PM, Jean Delvare wrote: + (verbose 0 level = 0)) { + print_temp(CPU-temp: , temp ); + if (casetemp) + print_temp(, Case

Re: [PATCH 4/4] Return the fan speed via sysfs

2014-08-03 Thread Goffredo Baroncelli
On 08/03/2014 05:59 PM, Jean Delvare wrote: On Sun, 03 Aug 2014 17:27:17 +0200, Goffredo Baroncelli wrote: On 08/03/2014 04:17 PM, Jean Delvare wrote: On Fri, 1 Aug 2014 14:00:50 +, Goffredo Baroncelli wrote: Return the fan speed via sysfs: /sys/devices/temperature/fan_level Good idea

[PATCH 3/4] Add the "verbose" module option.

2014-08-01 Thread Goffredo Baroncelli
The "verbose" option controls the message in the kernel log verbose = 0 no message verbose = 1 log only the fan speed changes verbose = 2 log the fan speed changes and the temperature changes Signed-off-by: Goffredo Baroncelli --- drivers/macintosh/therm_windtun

[PATCH][v2] therm_windtunnel doesn't work properly on PowerMac G4

2014-08-01 Thread Goffredo Baroncelli
Hi All, On a PowerMac G4 I noticed that between the kernel v3.2 and v3.14 I lost the fan management. I found on internet other references to this kind of problem [2] **How reproduce: - booting with the kernel 3.2, the fan is "quite" silent. The module therm_windtunnel is loaded and in the log

[PATCH 2/4] Remove attach_method because un-used

2014-08-01 Thread Goffredo Baroncelli
Remove attach_method because i2c-powermac is in charge to instantiate the driver directly. Signed-off-by: Goffredo Baroncelli --- drivers/macintosh/therm_windtunnel.c | 35 --- 1 file changed, 35 deletions(-) diff --git a/drivers/macintosh/therm_windtunnel.c b

[PATCH 4/4] Return the fan speed via sysfs

2014-08-01 Thread Goffredo Baroncelli
Return the fan speed via sysfs: /sys/devices/temperature/fan_level This patch is copied from the Bryan Christianson's patch (see debian bug #741663) Signed-off-by: Goffredo Baroncelli --- drivers/macintosh/therm_windtunnel.c | 10 +- 1 file changed, 9 insertions(+), 1 deletion(-) diff

[PATCH 1/4] Update drivers names to the ones invoked by i2c-powermac

2014-08-01 Thread Goffredo Baroncelli
Update drivers names to the ones invoked by i2c-powermac: - therm_ds1775 -> MAC,ds1775 - therm_adm1030 -> MAC,adm1030 The background fan control loop is started from the devices probing methods. Signed-off-by: Goffredo Baroncelli --- drivers/macintosh/therm_windtunnel.

[PATCH 1/4] Update drivers names to the ones invoked by i2c-powermac

2014-08-01 Thread Goffredo Baroncelli
Update drivers names to the ones invoked by i2c-powermac: - therm_ds1775 - MAC,ds1775 - therm_adm1030 - MAC,adm1030 The background fan control loop is started from the devices probing methods. Signed-off-by: Goffredo Baroncelli kreij...@inwind.it --- drivers/macintosh/therm_windtunnel.c | 23

[PATCH 4/4] Return the fan speed via sysfs

2014-08-01 Thread Goffredo Baroncelli
Return the fan speed via sysfs: /sys/devices/temperature/fan_level This patch is copied from the Bryan Christianson's patch (see debian bug #741663) Signed-off-by: Goffredo Baroncelli kreij...@inwind.it --- drivers/macintosh/therm_windtunnel.c | 10 +- 1 file changed, 9 insertions(+), 1

[PATCH][v2] therm_windtunnel doesn't work properly on PowerMac G4

2014-08-01 Thread Goffredo Baroncelli
Hi All, On a PowerMac G4 I noticed that between the kernel v3.2 and v3.14 I lost the fan management. I found on internet other references to this kind of problem [2] **How reproduce: - booting with the kernel 3.2, the fan is quite silent. The module therm_windtunnel is loaded and in the log

[PATCH 2/4] Remove attach_method because un-used

2014-08-01 Thread Goffredo Baroncelli
Remove attach_method because i2c-powermac is in charge to instantiate the driver directly. Signed-off-by: Goffredo Baroncelli kreij...@inwind.it --- drivers/macintosh/therm_windtunnel.c | 35 --- 1 file changed, 35 deletions(-) diff --git a/drivers/macintosh

[PATCH 3/4] Add the verbose module option.

2014-08-01 Thread Goffredo Baroncelli
The verbose option controls the message in the kernel log verbose = 0 no message verbose = 1 log only the fan speed changes verbose = 2 log the fan speed changes and the temperature changes Signed-off-by: Goffredo Baroncelli kreij...@inwind.it --- drivers/macintosh/therm_windtunnel.c | 37

Re: [PATCH 3/3] therm_windtunnel doesn't work properly on PowerMac G4

2014-07-31 Thread Goffredo Baroncelli
d. The subject was the only way to group the patches. The next time I will use git send-email (which I hate !), so this problem will disappear. > > On Wed, 30 Jul 2014 22:50:57 +0200, Goffredo Baroncelli wrote: >> Add the "log_temp" and "verbose" module parameters

Re: [PATCH 0/3] therm_windtunnel doesn't work properly on PowerMac G4

2014-07-31 Thread Goffredo Baroncelli
Thanks again, > -- gpg @keyserver.linux.it: Goffredo Baroncelli (kreijackATinwind.it> Key fingerprint BBF5 1610 0B64 DAC6 5F7D 17B2 0EDA 9B37 8B82 E0B5 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@v

Re: [PATCH 0/3] therm_windtunnel doesn't work properly on PowerMac G4

2014-07-31 Thread Goffredo Baroncelli
@keyserver.linux.it: Goffredo Baroncelli (kreijackATinwind.it Key fingerprint BBF5 1610 0B64 DAC6 5F7D 17B2 0EDA 9B37 8B82 E0B5 -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org

Re: [PATCH 3/3] therm_windtunnel doesn't work properly on PowerMac G4

2014-07-31 Thread Goffredo Baroncelli
was the only way to group the patches. The next time I will use git send-email (which I hate !), so this problem will disappear. On Wed, 30 Jul 2014 22:50:57 +0200, Goffredo Baroncelli wrote: Add the log_temp and verbose module parameters. I think this is a good idea, much better than build-time

[PATCH 2/3] therm_windtunnel doesn't work properly on PowerMac G4

2014-07-30 Thread Goffredo Baroncelli
Remove attach_method because un-used: now i2c-powermac is in charge to instantiate the driver. Signed-off-by: Goffredo Baroncelli --- drivers/macintosh/therm_windtunnel.c | 35 -- 1 file changed, 35 deletions(-) diff --git a/drivers/macintosh

[PATCH 1/3] therm_windtunnel doesn't work properly on PowerMac G4

2014-07-30 Thread Goffredo Baroncelli
Rename the driver name therm_ds1775 -> MAC,ds1775 therm_adm1030 -> MAC,adm1030 Start the background fan control loop from the devices probing methods. Signed-off-by: Goffredo Baroncelli --- drivers/macintosh/therm_windtunnel.c | 15 +-- 1 file chang

[PATCH 3/3] therm_windtunnel doesn't work properly on PowerMac G4

2014-07-30 Thread Goffredo Baroncelli
Add the "log_temp" and "verbose" module parameters. log_tempenable/disable the temperature logging verbose enable/disable the fan tune logging Signed-off-by: Goffredo Baroncelli --- drivers/macintosh/therm_windtunnel.c | 19 +-- 1 file ch

[PATCH 0/3] therm_windtunnel doesn't work properly on PowerMac G4

2014-07-30 Thread Goffredo Baroncelli
e my ones. See also https://lists.ozlabs.org/pipermail/linuxppc-dev/2012-July/099561.html -- gpg @keyserver.linux.it: Goffredo Baroncelli (kreijackATinwind.it> Key fingerprint BBF5 1610 0B64 DAC6 5F7D 17B2 0EDA 9B37 8B82 E0B5 -- To unsubscribe from this list: send the line "unsubscribe linu

[PATCH 0/3] therm_windtunnel doesn't work properly on PowerMac G4

2014-07-30 Thread Goffredo Baroncelli
like the my ones. See also https://lists.ozlabs.org/pipermail/linuxppc-dev/2012-July/099561.html -- gpg @keyserver.linux.it: Goffredo Baroncelli (kreijackATinwind.it Key fingerprint BBF5 1610 0B64 DAC6 5F7D 17B2 0EDA 9B37 8B82 E0B5 -- To unsubscribe from this list: send the line unsubscribe

[PATCH 3/3] therm_windtunnel doesn't work properly on PowerMac G4

2014-07-30 Thread Goffredo Baroncelli
Add the log_temp and verbose module parameters. log_tempenable/disable the temperature logging verbose enable/disable the fan tune logging Signed-off-by: Goffredo Baroncelli kreij...@inwind.it --- drivers/macintosh/therm_windtunnel.c | 19 +-- 1 file changed, 13

[PATCH 1/3] therm_windtunnel doesn't work properly on PowerMac G4

2014-07-30 Thread Goffredo Baroncelli
Rename the driver name therm_ds1775 - MAC,ds1775 therm_adm1030 - MAC,adm1030 Start the background fan control loop from the devices probing methods. Signed-off-by: Goffredo Baroncelli kreij...@inwind.it --- drivers/macintosh/therm_windtunnel.c | 15 +-- 1 file

[PATCH 2/3] therm_windtunnel doesn't work properly on PowerMac G4

2014-07-30 Thread Goffredo Baroncelli
Remove attach_method because un-used: now i2c-powermac is in charge to instantiate the driver. Signed-off-by: Goffredo Baroncelli kreij...@inwind.it --- drivers/macintosh/therm_windtunnel.c | 35 -- 1 file changed, 35 deletions(-) diff --git a/drivers

  1   2   >