<html xmlns:v="urn:schemas-microsoft-com:vml" 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;}
@font-face
        {font-family:"Fira Code";
        panose-1:2 11 5 9 5 0 0 2 0 4;}
/* 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><!--[if gte mso 9]><xml>
<o:shapedefaults v:ext="edit" spidmax="1026" />
</xml><![endif]--><!--[if gte mso 9]><xml>
<o:shapelayout v:ext="edit">
<o:idmap v:ext="edit" data="1" />
</o:shapelayout></xml><![endif]-->
</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">The register setting looks fine.<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US">Can you check the connection of the FAN tach pin and BMC tach input pin?<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US">If </span><span lang="EN-US" style="mso-fareast-language:ZH-TW">it is normal, can you check the FAN tach output waveform?<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">I test with San Ace 80 and get the result below:<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" style="line-height:15.75pt;background:#272822"><span style="font-size:10.5pt;font-family:"Fira Code";color:#F8F8F2">root@ast2600-default:</span><span style="font-size:10.5pt;font-family:"Fira Code";color:#F92672">~</span><span style="font-size:10.5pt;font-family:"Fira Code";color:#88846F">#
 cat /sys/class/hwmon/hwmon1/fan1_input </span><span style="font-size:10.5pt;font-family:"Fira Code";color:#F8F8F2"><o:p></o:p></span></p>
<p class="MsoNormal" style="line-height:15.75pt;background:#272822"><span style="font-size:10.5pt;font-family:"Fira Code";color:#F8F8F2">3570                                                          
<o:p></o:p></span></p>
<p class="MsoNormal" style="line-height:15.75pt;background:#272822"><span style="font-size:10.5pt;font-family:"Fira Code";color:#F8F8F2">root@ast2600-default:</span><span style="font-size:10.5pt;font-family:"Fira Code";color:#F92672">~</span><span style="font-size:10.5pt;font-family:"Fira Code";color:#88846F">#
 echo 100 > /sys/class/hwmon/hwmon4/pwm1</span><span style="font-size:10.5pt;font-family:"Fira Code";color:#F8F8F2"><o:p></o:p></span></p>
<p class="MsoNormal" style="line-height:15.75pt;background:#272822"><span style="font-size:10.5pt;font-family:"Fira Code";color:#F8F8F2">root@ast2600-default:</span><span style="font-size:10.5pt;font-family:"Fira Code";color:#F92672">~</span><span style="font-size:10.5pt;font-family:"Fira Code";color:#88846F">#
 cat /sys/class/hwmon/hwmon1/fan1_input </span><span style="font-size:10.5pt;font-family:"Fira Code";color:#F8F8F2"><o:p></o:p></span></p>
<p class="MsoNormal" style="line-height:15.75pt;background:#272822"><span style="font-size:10.5pt;font-family:"Fira Code";color:#F8F8F2">3568                                                          
<o:p></o:p></span></p>
<p class="MsoNormal" style="line-height:15.75pt;background:#272822"><span style="font-size:10.5pt;font-family:"Fira Code";color:#F8F8F2">root@ast2600-default:</span><span style="font-size:10.5pt;font-family:"Fira Code";color:#F92672">~</span><span style="font-size:10.5pt;font-family:"Fira Code";color:#88846F">#
 echo 120 > /sys/class/hwmon/hwmon4/pwm1</span><span style="font-size:10.5pt;font-family:"Fira Code";color:#F8F8F2"><o:p></o:p></span></p>
<p class="MsoNormal" style="line-height:15.75pt;background:#272822"><span style="font-size:10.5pt;font-family:"Fira Code";color:#F8F8F2">root@ast2600-default:</span><span style="font-size:10.5pt;font-family:"Fira Code";color:#F92672">~</span><span style="font-size:10.5pt;font-family:"Fira Code";color:#88846F">#
 cat /sys/class/hwmon/hwmon1/fan1_input </span><span style="font-size:10.5pt;font-family:"Fira Code";color:#F8F8F2"><o:p></o:p></span></p>
<p class="MsoNormal" style="line-height:15.75pt;background:#272822"><span style="font-size:10.5pt;font-family:"Fira Code";color:#F8F8F2">4032                                                           <o:p></o:p></span></p>
<p class="MsoNormal" style="line-height:15.75pt;background:#272822"><span style="font-size:10.5pt;font-family:"Fira Code";color:#F8F8F2">root@ast2600-default:</span><span style="font-size:10.5pt;font-family:"Fira Code";color:#F92672">~</span><span style="font-size:10.5pt;font-family:"Fira Code";color:#88846F">#
 echo 140 > /sys/class/hwmon/hwmon4/pwm1</span><span style="font-size:10.5pt;font-family:"Fira Code";color:#F8F8F2"><o:p></o:p></span></p>
<p class="MsoNormal" style="line-height:15.75pt;background:#272822"><span style="font-size:10.5pt;font-family:"Fira Code";color:#F8F8F2">root@ast2600-default:</span><span style="font-size:10.5pt;font-family:"Fira Code";color:#F92672">~</span><span style="font-size:10.5pt;font-family:"Fira Code";color:#88846F">#
 cat /sys/class/hwmon/hwmon1/fan1_input </span><span style="font-size:10.5pt;font-family:"Fira Code";color:#F8F8F2"><o:p></o:p></span></p>
<p class="MsoNormal" style="line-height:15.75pt;background:#272822"><span style="font-size:10.5pt;font-family:"Fira Code";color:#F8F8F2">4422                                                          
<o:p></o:p></span></p>
<p class="MsoNormal" style="line-height:15.75pt;background:#272822"><span style="font-size:10.5pt;font-family:"Fira Code";color:#F8F8F2">root@ast2600-default:</span><span style="font-size:10.5pt;font-family:"Fira Code";color:#F92672">~</span><span style="font-size:10.5pt;font-family:"Fira Code";color:#88846F">#
 echo 255 > /sys/class/hwmon/hwmon4/pwm1</span><span style="font-size:10.5pt;font-family:"Fira Code";color:#F8F8F2"><o:p></o:p></span></p>
<p class="MsoNormal" style="line-height:15.75pt;background:#272822"><span style="font-size:10.5pt;font-family:"Fira Code";color:#F8F8F2">root@ast2600-default:</span><span style="font-size:10.5pt;font-family:"Fira Code";color:#F92672">~</span><span style="font-size:10.5pt;font-family:"Fira Code";color:#88846F">#
 cat /sys/class/hwmon/hwmon1/fan1_input </span><span style="font-size:10.5pt;font-family:"Fira Code";color:#F8F8F2"><o:p></o:p></span></p>
<p class="MsoNormal" style="line-height:15.75pt;background:#272822"><span style="font-size:10.5pt;font-family:"Fira Code";color:#F8F8F2">6293<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"><o:p> </o:p></span></p>
<p class="MsoNormal"><span lang="EN-US" style="mso-fareast-language:ZH-TW">Thanks
</span><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"><o:p> </o:p></span></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>Tuesday, February 15, 2022 at 6:10 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"><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>><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><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>><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>>, "<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>>,
 Billy Tsai <<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>>><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>><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>>><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>>>;
<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<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>>>; 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>>><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>>><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>><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>
<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">-----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>>> 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>><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<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>>>; 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>>><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>
</blockquote>
<div>
<p class="MsoNormal">master-<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">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>
</blockquote>
<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>
<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">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>
</blockquote>
<div>
<p class="MsoNormal">hwmon9/<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">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>
<div>
<p class="MsoNormal">> ../../../1e610000.pwm_tach:tach<o:p></o:p></p>
</div>
<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>
<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>
<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>
<div>
<p class="MsoNormal">> ../../../../../../../../class/hwmon<o:p></o:p></p>
</div>
<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>
</blockquote>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
</div>
</body>
</html>