Page 4 of 4 FirstFirst ... 234
Results 31 to 37 of 37

Thread: New applesmc module for all models - Call for testing

  1. #31
    Join Date
    Dec 2005
    Location
    Greece
    Beans
    436

    Re: New applesmc module for all models - Call for testing

    Quote Originally Posted by kosumi68 View Post
    fanX_manual, fanX_min and fanX_output should all be read-write.
    Yep, those files were ok. All of it is
    Code:
    /sys/devices/platform/applesmc.768$ ll fan*
    -r--r--r-- 1 root root 4096 2010-11-23 12:34 fan1_input
    -r--r--r-- 1 root root 4096 2010-11-23 12:34 fan1_label
    -rw-r--r-- 1 root root 4096 2010-11-23 12:52 fan1_manual
    -r--r--r-- 1 root root 4096 2010-11-23 12:52 fan1_max
    -rw-r--r-- 1 root root 4096 2010-11-23 12:52 fan1_min
    -rw-r--r-- 1 root root 4096 2010-11-23 12:52 fan1_output
    -r--r--r-- 1 root root 4096 2010-11-23 12:52 fan1_safe
    -r--r--r-- 1 root root 4096 2010-11-23 12:34 fan2_input
    -r--r--r-- 1 root root 4096 2010-11-23 12:34 fan2_label
    -rw-r--r-- 1 root root 4096 2010-11-23 12:52 fan2_manual
    -r--r--r-- 1 root root 4096 2010-11-23 12:52 fan2_max
    -rw-r--r-- 1 root root 4096 2010-11-23 12:52 fan2_min
    -rw-r--r-- 1 root root 4096 2010-11-23 12:52 fan2_output
    -r--r--r-- 1 root root 4096 2010-11-23 12:52 fan2_safe
    The reason I only posted the *input files is because the echoing commands (previous post) where targetting, well, the *input file(s). I did not change those files.

    TB* sensors still show 129 degrees (without the battery of course). When the battery is plugged-in it looks fine, i.e.
    Code:
    TB0T:        +20.0°C                                    
    TB1T:        +20.0°C                                    
    TB2T:        +19.5°C                                    
    TB3T:        +19.8°C
    Removing and loading again the applesmc module does not change the permissions of the *input files (as expected I guess?).
    Last edited by Nikos.Alexandris; November 23rd, 2010 at 12:08 PM.

  2. #32
    Join Date
    May 2008
    Beans
    745

    Re: New applesmc module for all models - Call for testing

    I see, so with/without battery affects the readings, which should be ok. What I can see, the remaining question is whether the read/write failures are random or occur every time. Your estimate here would be most valuable. I am also trying to understand the amount of read/writes to fanX_manual. Since writing this particular value changes the internal fan handling in the SMC, it is probably a good idea to exclude it from the control loop in macfanctld.

    Thanks for testing!

  3. #33
    Join Date
    Dec 2005
    Location
    Greece
    Beans
    436

    Re: New applesmc module for all models - Call for testing

    Quote Originally Posted by kosumi68 View Post
    I see, so with/without battery affects the readings, which should be ok.
    Yes. But, why set to 129 degrees when readings are not available (unplugged battery)? What is the motivation behind this? Can it not be set to N/A or the like? Or something that macfanctld will understand and ignore automatically?

    Currently, the user should change the exclude option in /etc/macfanctld.conf each time the battery is being un-/plugged.

    What I can see, the remaining question is whether the read/write failures are random or occur every time. Your estimate here would be most valuable.
    Hmm... I was about to say it is "default" but now I am unsure. Loading and unloading applesmc should produce the same list of read arg fail messages right? Well, it does not. The only messages added after rmmod and modprobe are

    Code:
    [ 8836.704892] applesmc: key=295 fan=2 temp=20 acc=1 lux=2 kbd=1
    [ 8836.735361] input: applesmc as /devices/platform/applesmc.768/input/input11
    [ 8837.423779] applesmc: light sensor data length set to 10
    I am also trying to understand the amount of read/writes to fanX_manual. Since writing this particular value changes the internal fan handling in the SMC, it is probably a good idea to exclude it from the control loop in macfanctld.
    Not sure if I am able to help with this - also, currently I am a bit confused with other stuff and it will take me to dive a bit more in the SMC and fan control logic

    Cheers!

  4. #34
    Join Date
    May 2008
    Beans
    745

    Re: New applesmc module for all models - Call for testing

    Quote Originally Posted by Nikos.Alexandris View Post
    Yes. But, why set to 129 degrees when readings are not available (unplugged battery)? What is the motivation behind this? Can it not be set to N/A or the like? Or something that macfanctld will understand and ignore automatically?
    It seems the information is not available, unfortunately - the readings are simply what you get from the smc. Just to make sure: you plug/unplug the battery with the machine off, right?


    Not sure if I am able to help with this - also, currently I am a bit confused with other stuff and it will take me to dive a bit more in the SMC and fan control logic

    Cheers!
    No problem, I think we are making steady progress - thanks!

  5. #35
    Join Date
    Dec 2005
    Location
    Greece
    Beans
    436

    Re: New applesmc module for all models - Call for testing

    Quote Originally Posted by kosumi68 View Post
    It seems the information is not available, unfortunately - the readings are simply what you get from the smc. Just to make sure: you plug/unplug the battery with the machine off, right?
    Not really... But even booting with the battery not present gives the same 129 degrees readings.

    Thanks for all the work

  6. #36
    Join Date
    Aug 2010
    Beans
    1

    Re: New applesmc module for all models - Call for testing

    Not sure if this thread is dead (and if the solution is presented somewhere else) but I'm having the same problem on my mbp 6,2 12.04.

  7. #37
    Join Date
    Feb 2008
    Location
    Texas
    Beans
    29,807
    Distro
    Ubuntu 20.04 Focal Fossa

    Re: New applesmc module for all models - Call for testing

    Old thread closed.

Page 4 of 4 FirstFirst ... 234

Bookmarks

Posting Permissions

  • You may not post new threads
  • You may not post replies
  • You may not post attachments
  • You may not edit your posts
  •