<html xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
<meta name="Generator" content="Microsoft Word 15 (filtered medium)">
<style><!--
/* Font Definitions */
@font-face
        {font-family:PMingLiU;
        panose-1:2 2 5 0 0 0 0 0 0 0;}
@font-face
        {font-family:"Cambria Math";
        panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
        {font-family:DengXian;
        panose-1:2 1 6 0 3 1 1 1 1 1;}
@font-face
        {font-family:Calibri;
        panose-1:2 15 5 2 2 2 4 3 2 4;}
@font-face
        {font-family:"Microsoft JhengHei";
        panose-1:2 11 6 4 3 5 4 4 2 4;}
@font-face
        {font-family:"\@Microsoft JhengHei";}
@font-face
        {font-family:"\@PMingLiU";
        panose-1:2 1 6 1 0 1 1 1 1 1;}
@font-face
        {font-family:"\@DengXian";
        panose-1:2 1 6 0 3 1 1 1 1 1;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
        {margin:0cm;
        font-size:11.0pt;
        font-family:"Calibri",sans-serif;}
a:link, span.MsoHyperlink
        {mso-style-priority:99;
        color:blue;
        text-decoration:underline;}
span.EmailStyle18
        {mso-style-type:personal-reply;
        font-family:"Calibri",sans-serif;
        color:windowtext;}
.MsoChpDefault
        {mso-style-type:export-only;
        font-size:10.0pt;}
@page WordSection1
        {size:612.0pt 792.0pt;
        margin:72.0pt 72.0pt 72.0pt 72.0pt;}
div.WordSection1
        {page:WordSection1;}
--></style>
</head>
<body lang="en-TW" link="blue" vlink="purple" style="word-wrap:break-word">
<div class="WordSection1">
<p class="MsoNormal"><span lang="EN-US">Hi Tung,<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"><o:p> </o:p></span></p>
<p class="MsoNormal"><span lang="EN-US" style="mso-fareast-language:ZH-TW">I can reproduce your issue by shorting out the PWM and TACH pin.<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US" style="mso-fareast-language:ZH-TW">Are you using our evb for testing? If yes, please check there is no jumper at J80.<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US" style="mso-fareast-language:ZH-TW"><o:p> </o:p></span></p>
<p class="MsoNormal"><span lang="EN-US" style="mso-fareast-language:ZH-TW">Thanks<o:p></o:p></span></p>
<p class="MsoNormal"><o:p> </o:p></p>
<div>
<div>
<p class="MsoNormal"><span lang="EN-US" style="font-size:13.5pt;color:#9AB3E0;mso-fareast-language:ZH-TW">Best Regards,</span><span lang="EN-US" style="font-size:13.5pt;font-family:"Times New Roman",serif;color:white;mso-fareast-language:ZH-TW"><o:p></o:p></span></p>
</div>
</div>
<p class="MsoNormal"><span lang="EN-US" style="font-size:13.5pt;font-family:"Microsoft JhengHei",sans-serif;color:#83ADE0;mso-fareast-language:ZH-TW">Billy Tsai</span><o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<div style="border:none;border-top:solid #B5C4DF 1.0pt;padding:3.0pt 0cm 0cm 0cm">
<p class="MsoNormal"><b><span style="font-size:12.0pt;color:black">From: </span></b><span style="font-size:12.0pt;color:black">Tung Nguyen OS <tungnguyen@os.amperecomputing.com><br>
<b>Date: </b>Wednesday, February 16, 2022 at 3:07 PM<br>
<b>To: </b>Billy Tsai <billy_tsai@aspeedtech.com>, Troy Lee <troy_lee@aspeedtech.com>, "openbmc@lists.ozlabs.org" <openbmc@lists.ozlabs.org><br>
<b>Cc: </b>Thu Nguyen OS <thu@os.amperecomputing.com>, Thang Nguyen OS <thang@os.amperecomputing.com>, Phong Vo OS <phong@os.amperecomputing.com><br>
<b>Subject: </b>Re: [AST2600] PWM/TACH driver and dbus-sensor issue <o:p></o:p></span></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">Hi Billy,<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">I don't have the device to capture the waveform here, i will need our HW team's help. Meanwhile, debugging in the tach-aspeed-ast2600.c i have observed that:<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">+ In the case that can read the fan speed, the debug messages output like:<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"># cat /sys/class/hwmon/hwmon1/pwm1<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">120<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"># echo 110 > /sys/class/hwmon/hwmon1/pwm1<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"># cat /sys/class/hwmon/hwmon0/fan1_input<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">837053<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">[  325.499388] clk 200000000, raw_data 7 , tach_div 14336<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">[  326.070305] clk 200000000, raw_data 7 , tach_div 14336<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">[  326.641221] clk 200000000, raw_data 7 , tach_div 14336<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">[  327.212095] clk 200000000, raw_data 7 , tach_div 14336<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">[  327.783012] clk 200000000, raw_data 7 , tach_div 14336<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">+ In the issue case, i can see the code is timeout at regmap_read_poll_timeout (below)<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">150 static int aspeed_get_fan_tach_ch_rpm(struct aspeed_tach_data *priv,<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">151                                       u8 fan_tach_ch)<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">152 {<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">153         u32 raw_data, tach_div, clk_source, usec, val;<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">154         u64 rpm;<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">155         int ret;<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">156<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">157         usec = priv->tacho_channel[fan_tach_ch].sample_period;<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">158         /* Restart the Tach channel to guarantee the value is fresh */<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">159         regmap_update_bits(priv->regmap, ASPEED_TACHO_CTRL_CH(fan_tach_ch),<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">160                              TACHO_ENABLE, 0);<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">161         regmap_update_bits(priv->regmap, ASPEED_TACHO_CTRL_CH(fan_tach_ch),<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">162                              TACHO_ENABLE, TACHO_ENABLE);<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">163         ret = regmap_read_poll_timeout(<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">164                 priv->regmap, ASPEED_TACHO_STS_CH(fan_tach_ch), val,<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">165                 (val & TACHO_FULL_MEASUREMENT) && (val & TACHO_VALUE_UPDATE),<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">166                 RPM_POLLING_PERIOD_US, usec);<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">167<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">168         if (ret) {<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">169                 /* return 0 if we didn't get an answer because of timeout*/<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">170                 if (ret == -ETIMEDOUT)<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">171                         return 0; ==> TIMEOUT HERE<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">172                 else<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">173                         return ret;<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">174         }<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">175<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">176         raw_data = val & TACHO_VALUE_MASK;<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">Do you have comments on what should be checked here ?<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">________________________________________<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">From: Billy Tsai <<a href="mailto:billy_tsai@aspeedtech.com">billy_tsai@aspeedtech.com</a>><o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">Sent: Tuesday, February 15, 2022 5:33 PM<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">To: Tung Nguyen OS; Troy Lee; <a href="mailto:openbmc@lists.ozlabs.org">
openbmc@lists.ozlabs.org</a><o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">Cc: Thu Nguyen OS; Thang Nguyen OS; Phong Vo OS<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">Subject: Re: [AST2600] PWM/TACH driver and dbus-sensor issue<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">Hi Tung,<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">The register setting looks fine.<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">Can you check the connection of the FAN tach pin and BMC tach input pin?<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">If it is normal, can you check the FAN tach output waveform?<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">I test with San Ace 80 and get the result below:<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">root@ast2600-default:~# cat /sys/class/hwmon/hwmon1/fan1_input<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">3570<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">root@ast2600-default:~# echo 100 > /sys/class/hwmon/hwmon4/pwm1<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">root@ast2600-default:~# cat /sys/class/hwmon/hwmon1/fan1_input<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">3568<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">root@ast2600-default:~# echo 120 > /sys/class/hwmon/hwmon4/pwm1<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">root@ast2600-default:~# cat /sys/class/hwmon/hwmon1/fan1_input<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">4032<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">root@ast2600-default:~# echo 140 > /sys/class/hwmon/hwmon4/pwm1<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">root@ast2600-default:~# cat /sys/class/hwmon/hwmon1/fan1_input<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">4422<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">root@ast2600-default:~# echo 255 > /sys/class/hwmon/hwmon4/pwm1<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">root@ast2600-default:~# cat /sys/class/hwmon/hwmon1/fan1_input<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">6293<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">Thanks<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">Best Regards,<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">Billy Tsai<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">From: Tung Nguyen OS <<a href="mailto:tungnguyen@os.amperecomputing.com">tungnguyen@os.amperecomputing.com</a>><o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">Date: Tuesday, February 15, 2022 at 6:10 PM<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">To: Billy Tsai <<a href="mailto:billy_tsai@aspeedtech.com">billy_tsai@aspeedtech.com</a>>, Troy Lee <<a href="mailto:troy_lee@aspeedtech.com">troy_lee@aspeedtech.com</a>>, "<a href="mailto:openbmc@lists.ozlabs.org">openbmc@lists.ozlabs.org</a>"
 <<a href="mailto:openbmc@lists.ozlabs.org">openbmc@lists.ozlabs.org</a>><o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">Cc: Thu Nguyen OS <<a href="mailto:thu@os.amperecomputing.com">thu@os.amperecomputing.com</a>>, Thang Nguyen OS <<a href="mailto:thang@os.amperecomputing.com">thang@os.amperecomputing.com</a>>, Phong Vo OS <<a href="mailto:phong@os.amperecomputing.com">phong@os.amperecomputing.com</a>><o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">Subject: Re: [AST2600] PWM/TACH driver and dbus-sensor issue<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">Hi Billy,<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">Please help check:<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">root@sunmoonlake:/tmp# ./test.sh 100<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">echo 100 > /sys/class/hwmon/hwmon1/pwm1<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">cat /sys/class/hwmon/hwmon0/fan1_input<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">0<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">devmem 0x1e610000<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">0x0001101E<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">devmem 0x1e610004<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">0xFF006500<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">devmem 0x1e610008<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">0x10500000<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">devmem 0x1e61000c<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">0xA5400000<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">root@sunmoonlake:/tmp# ./test.sh 120<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">echo 120 > /sys/class/hwmon/hwmon1/pwm1<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">cat /sys/class/hwmon/hwmon0/fan1_input<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">266335<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">devmem 0x1e610000<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">0x0001101E<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">devmem 0x1e610004<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">0xFF007900<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">devmem 0x1e610008<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">0x10500000<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">devmem 0x1e61000c<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">0xF4B00016<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">root@sunmoonlake:/tmp# ./test.sh 140<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">echo 140 > /sys/class/hwmon/hwmon1/pwm1<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">cat /sys/class/hwmon/hwmon0/fan1_input<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">837053<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">devmem 0x1e610000<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">0x0001101E<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">devmem 0x1e610004<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">0xFF008D00<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">devmem 0x1e610008<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">0x10500000<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">devmem 0x1e61000c<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">0xE4B00007<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">root@sunmoonlake:/tmp# ./test.sh 255<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">echo 255 > /sys/class/hwmon/hwmon1/pwm1<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">cat /sys/class/hwmon/hwmon0/fan1_input<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">0<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">devmem 0x1e610000<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">0x0001101E<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">devmem 0x1e610004<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">0xFF000000<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">devmem 0x1e610008<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">0x10500000<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">devmem 0x1e61000c<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">0xB5C00000<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">root@sunmoonlake:/tmp#<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">Please note that i'm using the fan settings as:<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">        fan0: pwm-fan0 {<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">                compatible = "pwm-fan";<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">                pwms = <&pwm 0 40000 0>;        /* Target freq:25 kHz */<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">                cooling-min-state = <0>;<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">                cooling-max-state = <3>;<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">                #cooling-cells = <2>;<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">                cooling-levels = <0 25 128 255>;<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">        };<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">.....<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">        fan@0 {<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">                reg = <0x00>;<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">                aspeed,pulse-pr = <2>;<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">        };<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">________________________________________<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">From: Billy Tsai <<a href="mailto:billy_tsai@aspeedtech.com">billy_tsai@aspeedtech.com</a><mailto:<a href="mailto:billy_tsai@aspeedtech.com">billy_tsai@aspeedtech.com</a>>><o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">Sent: Monday, February 14, 2022 1:34 PM<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">To: Tung Nguyen OS; Troy Lee; <a href="mailto:openbmc@lists.ozlabs.org">
openbmc@lists.ozlabs.org</a><mailto:<a href="mailto:openbmc@lists.ozlabs.org">openbmc@lists.ozlabs.org</a>><o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">Cc: Thu Nguyen OS; Thang Nguyen OS; Phong Vo OS<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">Subject: Re: [AST2600] PWM/TACH driver and dbus-sensor issue<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">Hi Tung,<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">Can you check the PWM frequency and dump the register value 0x1e610000~0x1e61000c?<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">Thanks<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">Best Regards,<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">Billy Tsai<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">From: Tung Nguyen OS <<a href="mailto:tungnguyen@os.amperecomputing.com">tungnguyen@os.amperecomputing.com</a><mailto:<a href="mailto:tungnguyen@os.amperecomputing.com">tungnguyen@os.amperecomputing.com</a>>><o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">Date: Monday, February 14, 2022 at 12:24 PM<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">To: Troy Lee <<a href="mailto:troy_lee@aspeedtech.com">troy_lee@aspeedtech.com</a><mailto:<a href="mailto:troy_lee@aspeedtech.com">troy_lee@aspeedtech.com</a>>>, "<a href="mailto:openbmc@lists.ozlabs.org">openbmc@lists.ozlabs.org</a><mailto:<a href="mailto:openbmc@lists.ozlabs.org">openbmc@lists.ozlabs.org</a>>"
 <<a href="mailto:openbmc@lists.ozlabs.org">openbmc@lists.ozlabs.org</a><mailto:<a href="mailto:openbmc@lists.ozlabs.org">openbmc@lists.ozlabs.org</a>>><o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">Cc: Thu Nguyen OS <<a href="mailto:thu@os.amperecomputing.com">thu@os.amperecomputing.com</a><mailto:<a href="mailto:thu@os.amperecomputing.com">thu@os.amperecomputing.com</a>>>, Thang Nguyen OS <<a href="mailto:thang@os.amperecomputing.com">thang@os.amperecomputing.com</a><mailto:<a href="mailto:thang@os.amperecomputing.com">thang@os.amperecomputing.com</a>>>,
 Phong Vo OS <<a href="mailto:phong@os.amperecomputing.com">phong@os.amperecomputing.com</a><mailto:<a href="mailto:phong@os.amperecomputing.com">phong@os.amperecomputing.com</a>>>, Billy Tsai <<a href="mailto:billy_tsai@aspeedtech.com">billy_tsai@aspeedtech.com</a><mailto:<a href="mailto:billy_tsai@aspeedtech.com">billy_tsai@aspeedtech.com</a>>><o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">Subject: Re: [AST2600] PWM/TACH driver and dbus-sensor issue<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">Hi Troy,<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">the fan settings in device tree are similar to aspeed-ast2600a1-evb.dts, like:<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">        fan0: pwm-fan0 {<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">                compatible = "pwm-fan";<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">                pwms = <&pwm 0 40000 0>;        /* Target freq:25 kHz */<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">                cooling-min-state = <0>;<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">                cooling-max-state = <3>;<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">                #cooling-cells = <2>;<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">                cooling-levels = <0 25 128 255>;<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">        };<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">.....<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">        fan@0 {<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">                reg = <0x00>;<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">                aspeed,pulse-pr = <2>;<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">        };<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">I test with co-fan F-5010HH12B and San Ace 92 with above setting, and the result looks like in-correct as in previous email.<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">________________________________________<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">From: Troy Lee <<a href="mailto:troy_lee@aspeedtech.com">troy_lee@aspeedtech.com</a><mailto:<a href="mailto:troy_lee@aspeedtech.com">troy_lee@aspeedtech.com</a>><mailto:<a href="mailto:troy_lee@aspeedtech.com">troy_lee@aspeedtech.com</a><mailto:<a href="mailto:troy_lee@aspeedtech.com">troy_lee@aspeedtech.com</a>>>><o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">Sent: Monday, February 14, 2022 11:02 AM<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">To: Tung Nguyen OS; <a href="mailto:openbmc@lists.ozlabs.org">
openbmc@lists.ozlabs.org</a><mailto:<a href="mailto:openbmc@lists.ozlabs.org">openbmc@lists.ozlabs.org</a>><mailto:<a href="mailto:openbmc@lists.ozlabs.org">openbmc@lists.ozlabs.org</a><mailto:<a href="mailto:openbmc@lists.ozlabs.org">openbmc@lists.ozlabs.org</a>>><o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">Cc: Thu Nguyen OS; Thang Nguyen OS; Phong Vo OS; Billy Tsai<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">Subject: RE: [AST2600] PWM/TACH driver and dbus-sensor issue<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">Adding Billy.<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">Hi Tung,<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">-----Original Message-----<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">From: Tung Nguyen OS <<a href="mailto:tungnguyen@os.amperecomputing.com">tungnguyen@os.amperecomputing.com</a><mailto:<a href="mailto:tungnguyen@os.amperecomputing.com">tungnguyen@os.amperecomputing.com</a>><mailto:<a href="mailto:tungnguyen@os.amperecomputing.com">tungnguyen@os.amperecomputing.com</a><mailto:<a href="mailto:tungnguyen@os.amperecomputing.com">tungnguyen@os.amperecomputing.com</a>>>><o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">Sent: Monday, February 14, 2022 11:46 AM<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">To: Troy Lee <<a href="mailto:troy_lee@aspeedtech.com">troy_lee@aspeedtech.com</a><mailto:<a href="mailto:troy_lee@aspeedtech.com">troy_lee@aspeedtech.com</a>><mailto:<a href="mailto:troy_lee@aspeedtech.com">troy_lee@aspeedtech.com</a><mailto:<a href="mailto:troy_lee@aspeedtech.com">troy_lee@aspeedtech.com</a>>>>;
<a href="mailto:openbmc@lists.ozlabs.org">openbmc@lists.ozlabs.org</a><mailto:<a href="mailto:openbmc@lists.ozlabs.org">openbmc@lists.ozlabs.org</a>><mailto:<a href="mailto:openbmc@lists.ozlabs.org">openbmc@lists.ozlabs.org</a><mailto:<a href="mailto:openbmc@lists.ozlabs.org">openbmc@lists.ozlabs.org</a>>><o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">Cc: Thu Nguyen OS <<a href="mailto:thu@os.amperecomputing.com">thu@os.amperecomputing.com</a><mailto:<a href="mailto:thu@os.amperecomputing.com">thu@os.amperecomputing.com</a>><mailto:<a href="mailto:thu@os.amperecomputing.com">thu@os.amperecomputing.com</a><mailto:<a href="mailto:thu@os.amperecomputing.com">thu@os.amperecomputing.com</a>>>>;
 Thang Nguyen OS<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><<a href="mailto:thang@os.amperecomputing.com">thang@os.amperecomputing.com</a><mailto:<a href="mailto:thang@os.amperecomputing.com">thang@os.amperecomputing.com</a>><mailto:<a href="mailto:thang@os.amperecomputing.com">thang@os.amperecomputing.com</a><mailto:<a href="mailto:thang@os.amperecomputing.com">thang@os.amperecomputing.com</a>>>>;
 Phong Vo OS<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><<a href="mailto:phong@os.amperecomputing.com">phong@os.amperecomputing.com</a><mailto:<a href="mailto:phong@os.amperecomputing.com">phong@os.amperecomputing.com</a>><mailto:<a href="mailto:phong@os.amperecomputing.com">phong@os.amperecomputing.com</a><mailto:<a href="mailto:phong@os.amperecomputing.com">phong@os.amperecomputing.com</a>>>><o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">Subject: Re: [AST2600] PWM/TACH driver and dbus-sensor issue<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">Hi Troy,<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">Thank you for the information, i have merged the dbus-sensor patch and see<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">it creates the Fan sensors well.<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">One more issue i have observed about the fan tach report like this:<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">root@mtmitchell:~# echo 120 > /sys/class/hwmon/hwmon1/pwm1<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">root@mtmitchell:~# cat /sys/class/hwmon/hwmon0/fan1_input<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">418526<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">root@mtmitchell:~# echo 100 > /sys/class/hwmon/hwmon1/pwm1<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">root@mtmitchell:~# cat /sys/class/hwmon/hwmon0/fan1_input<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">195312<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">root@mtmitchell:~# echo 255 > /sys/class/hwmon/hwmon1/pwm1<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">root@mtmitchell:~# cat /sys/class/hwmon/hwmon0/fan1_input<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">0<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">root@mtmitchell:~# echo 50 > /sys/class/hwmon/hwmon1/pwm1<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">root@mtmitchell:~# cat /sys/class/hwmon/hwmon0/fan1_input<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">0<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">It means the fan speed is reported incorrectly somehow, but i can see the<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">fan speed changes by eyes, do you have idea on this issue ?<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">Does the aspeed,pulse-pr property of fan node set correspondingly?<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">What is the expected RPM of you fan? 418526 RPM?<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">Thanks,<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">Troy Lee<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">________________________________________<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">From: Troy Lee <<a href="mailto:troy_lee@aspeedtech.com">troy_lee@aspeedtech.com</a><mailto:<a href="mailto:troy_lee@aspeedtech.com">troy_lee@aspeedtech.com</a>><mailto:<a href="mailto:troy_lee@aspeedtech.com">troy_lee@aspeedtech.com</a><mailto:<a href="mailto:troy_lee@aspeedtech.com">troy_lee@aspeedtech.com</a>>>><o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">Sent: Friday, February 11, 2022 12:13 PM<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">To: Tung Nguyen OS; <a href="mailto:openbmc@lists.ozlabs.org">
openbmc@lists.ozlabs.org</a><mailto:<a href="mailto:openbmc@lists.ozlabs.org">openbmc@lists.ozlabs.org</a>><mailto:<a href="mailto:openbmc@lists.ozlabs.org">openbmc@lists.ozlabs.org</a><mailto:<a href="mailto:openbmc@lists.ozlabs.org">openbmc@lists.ozlabs.org</a>>><o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">Cc: Thu Nguyen OS; Thang Nguyen OS; Phong Vo OS<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">Subject: RE: [AST2600] PWM/TACH driver and dbus-sensor issue<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">Hi Tung,<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">-----Original Message-----<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">From: openbmc <openbmc-<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">bounces+troy_lee=<a href="mailto:aspeedtech.com@lists.ozlabs.org">aspeedtech.com@lists.ozlabs.org</a><mailto:<a href="mailto:aspeedtech.com@lists.ozlabs.org">aspeedtech.com@lists.ozlabs.org</a>><mailto:<a href="mailto:aspeedtech.com@lists.ozlabs.org">aspeedtech.com@lists.ozlabs.org</a><mailto:<a href="mailto:aspeedtech.com@lists.ozlabs.org">aspeedtech.com@lists.ozlabs.org</a>>>>
 On Behalf Of Tung<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">Nguyen OS<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">Sent: Wednesday, February 9, 2022 10:40 PM<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">To: <a href="mailto:openbmc@lists.ozlabs.org">openbmc@lists.ozlabs.org</a><mailto:<a href="mailto:openbmc@lists.ozlabs.org">openbmc@lists.ozlabs.org</a>><mailto:<a href="mailto:openbmc@lists.ozlabs.org">openbmc@lists.ozlabs.org</a><mailto:<a href="mailto:openbmc@lists.ozlabs.org">openbmc@lists.ozlabs.org</a>>><o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">Cc: Thu Nguyen OS <<a href="mailto:thu@os.amperecomputing.com">thu@os.amperecomputing.com</a><mailto:<a href="mailto:thu@os.amperecomputing.com">thu@os.amperecomputing.com</a>><mailto:<a href="mailto:thu@os.amperecomputing.com">thu@os.amperecomputing.com</a><mailto:<a href="mailto:thu@os.amperecomputing.com">thu@os.amperecomputing.com</a>>>>;
 Thang Nguyen OS<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><<a href="mailto:thang@os.amperecomputing.com">thang@os.amperecomputing.com</a><mailto:<a href="mailto:thang@os.amperecomputing.com">thang@os.amperecomputing.com</a>><mailto:<a href="mailto:thang@os.amperecomputing.com">thang@os.amperecomputing.com</a><mailto:<a href="mailto:thang@os.amperecomputing.com">thang@os.amperecomputing.com</a>>>>;
 Phong Vo OS<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><<a href="mailto:phong@os.amperecomputing.com">phong@os.amperecomputing.com</a><mailto:<a href="mailto:phong@os.amperecomputing.com">phong@os.amperecomputing.com</a>><mailto:<a href="mailto:phong@os.amperecomputing.com">phong@os.amperecomputing.com</a><mailto:<a href="mailto:phong@os.amperecomputing.com">phong@os.amperecomputing.com</a>>>><o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">Subject: [AST2600] PWM/TACH driver and dbus-sensor issue<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">Dear community,<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">We have ported the PWM/TACH hwmon driver for AST2600 from ASPEED's<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">repository <a href="https://github.com/AspeedTech-BMC/linux/tree/aspeed-">
https://github.com/AspeedTech-BMC/linux/tree/aspeed-</a><o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">master-<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">v5.4/drivers/hwmon to support Ampere Computing's systems. Verify with<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">AST2600 EVB, i can see the driver works. However there is a concern like:<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">1. The change of hwmon fan pwm/tach in sysfs (as below), this is<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">different from the ast2500 as the separated of the fan*_input and pwm1<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">in various hwmon directories. At this time, the driver doesn't exist<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">in the OpenBMC linux at <a href="https://github.com/openbmc/linux">
https://github.com/openbmc/linux</a>. Is OpenBMC<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">going to merge and keep this driver ?<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">Please have a look on this patch.<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><a href="https://gerrit.openbmc-project.xyz/c/openbmc/dbus-sensors/+/49253">https://gerrit.openbmc-project.xyz/c/openbmc/dbus-sensors/+/49253</a><o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">Thanks,<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">Troy Lee<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">root@mtmitchell:~# ls -la /sys/class/hwmon/hwmon hwmon0/ hwmon1/<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">hwmon2/ hwmon3/ hwmon4/ hwmon5/ hwmon6/ hwmon7/ hwmon8/<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">hwmon9/<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">root@mtmitchell:~# ls -la /sys/class/hwmon/hwmon0/<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">drwxr-xr-x    3 root     root             0 Jan  1  1970 .<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">drwxr-xr-x    3 root     root             0 Jan  1  1970 ..<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">lrwxrwxrwx    1 root     root             0 Feb  9 08:38 device -<o:p></o:p></p>
</div>
<blockquote style="border:none;border-left:solid #B5C4DF 4.5pt;padding:0cm 0cm 0cm 4.0pt;margin-left:3.75pt;margin-right:0cm" id="MAC_OUTLOOK_ATTRIBUTION_BLOCKQUOTE">
<div>
<p class="MsoNormal">../../../1e610000.pwm_tach:tach<o:p></o:p></p>
</div>
</blockquote>
<div>
<p class="MsoNormal">-r--r--r--    1 root     root          4096 Feb  9 08:38 fan10_input<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">-r--r--r--    1 root     root          4096 Feb  9 08:38 fan11_input<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">-r--r--r--    1 root     root          4096 Feb  9 08:38 fan12_input<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">-r--r--r--    1 root     root          4096 Feb  9 08:38 fan13_input<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">-r--r--r--    1 root     root          4096 Feb  9 08:38 fan14_input<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">-r--r--r--    1 root     root          4096 Feb  9 08:38 fan15_input<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">-r--r--r--    1 root     root          4096 Feb  9 08:38 fan16_input<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">-r--r--r--    1 root     root          4096 Feb  9 08:38 fan1_input<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">-r--r--r--    1 root     root          4096 Feb  9 08:38 fan2_input<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">-r--r--r--    1 root     root          4096 Feb  9 08:38 fan3_input<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">-r--r--r--    1 root     root          4096 Feb  9 08:38 fan4_input<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">-r--r--r--    1 root     root          4096 Feb  9 08:38 fan5_input<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">-r--r--r--    1 root     root          4096 Feb  9 08:38 fan6_input<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">-r--r--r--    1 root     root          4096 Feb  9 08:38 fan7_input<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">-r--r--r--    1 root     root          4096 Feb  9 08:38 fan8_input<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">-r--r--r--    1 root     root          4096 Feb  9 08:38 fan9_input<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">-r--r--r--    1 root     root          4096 Feb  9 08:38 name<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">lrwxrwxrwx    1 root     root             0 Feb  9 08:38 of_node -<o:p></o:p></p>
</div>
<blockquote style="border:none;border-left:solid #B5C4DF 4.5pt;padding:0cm 0cm 0cm 4.0pt;margin-left:3.75pt;margin-right:0cm" id="MAC_OUTLOOK_ATTRIBUTION_BLOCKQUOTE">
<div>
<p class="MsoNormal">../../../../../../../../firmware/devicetree/base/ahb/apb/pwm_tach@1e<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">610<o:p></o:p></p>
</div>
</blockquote>
<div>
<p class="MsoNormal">000/tach<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">drwxr-xr-x    2 root     root             0 Feb  9 08:38 power<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">lrwxrwxrwx    1 root     root             0 Jan  1  1970 subsystem -<o:p></o:p></p>
</div>
<blockquote style="border:none;border-left:solid #B5C4DF 4.5pt;padding:0cm 0cm 0cm 4.0pt;margin-left:3.75pt;margin-right:0cm" id="MAC_OUTLOOK_ATTRIBUTION_BLOCKQUOTE">
<div>
<p class="MsoNormal">../../../../../../../../class/hwmon<o:p></o:p></p>
</div>
</blockquote>
<div>
<p class="MsoNormal">-rw-r--r--    1 root     root          4096 Jan  1  1970 uevent<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">root@mtmitchell:~# ls -la /sys/class/hwmon/hwmon*/pwm*<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">-rw-r--r--    1 root     root          4096 Feb  9 09:13<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">/sys/class/hwmon/hwmon1/pwm1<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">-rw-r--r--    1 root     root          4096 Feb  9 09:20<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">/sys/class/hwmon/hwmon2/pwm1<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">-rw-r--r--    1 root     root          4096 Feb  9 08:38<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">/sys/class/hwmon/hwmon3/pwm1<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">-rw-r--r--    1 root     root          4096 Feb  9 08:38<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">/sys/class/hwmon/hwmon4/pwm1<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">-rw-r--r--    1 root     root          4096 Feb  9 08:38<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">/sys/class/hwmon/hwmon5/pwm1<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">-rw-r--r--    1 root     root          4096 Feb  9 08:38<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">/sys/class/hwmon/hwmon6/pwm1<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">-rw-r--r--    1 root     root          4096 Feb  9 08:38<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">/sys/class/hwmon/hwmon7/pwm1<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">-rw-r--r--    1 root     root          4096 Feb  9 08:38<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">/sys/class/hwmon/hwmon8/pwm1<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">2. With above changes, the dbus-sensor for FAN/PWM shall not work<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">because of the compatibility. So if OpenBMC want to keep above<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">PWM/TACH driver, does any sides have the plan to support the fan<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">sensor, if not, we can join and do it.<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">Thank you and best regards,<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">Tung<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
</div>
</body>
</html>