
<html>

<head>

<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">

</head>

<body>

<div dir="ltr">

<div></div>

<div>

<div>

<div dir="ltr">Hi Zehra,</div>

<div dir="ltr">I think the issue of isMultiHost() has been resolved with <a rel="noreferrer noopener" href="https://github.com/openbmc/phosphor-buttons/commit/4250785723d92b6578176a2d961d47ab66e08124">https://github.com/openbmc/phosphor-buttons/commit/4250785723d92b6578176a2d961d47ab66e08124</a>.</div>

<div dir="ltr">

<div><br>

</div>

</div>

<div id="ms-outlook-mobile-signature" dir="ltr">BR,</div>

<div id="ms-outlook-mobile-signature" dir="ltr">Thang Q. Nguyen</div>

</div>

<div> </div>

<hr style="display:inline-block;width:98%" tabindex="-1">

<div id="divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif"><b>From:</b> openbmc <openbmc-bounces+thang=os.amperecomputing.com@lists.ozlabs.org> on behalf of Patrick Williams <patrick@stwcx.xyz><br>

<b>Sent:</b> Monday, September 26, 2022 11:27 PM<br>

<b>To:</b> Naveen Moses <naveen.moses@hotmail.com><br>

<b>Cc:</b> zehra.ozdemir@inventron.com.tr <zehra.ozdemir@inventron.com.tr>; openbmc@lists.ozlabs.org <openbmc@lists.ozlabs.org>; M. Erhan Yigitbasi <erhan.yigitbasi@inventron.com.tr>; naveen.mosess@hcl.com <naveen.mosess@hcl.com><br>

<b>Subject:</b> Re: phosphor-buttons multihost control

<div> </div>

</font></div>

Hi Zehra, <br>

<br>

On Mon, Sep 26, 2022 at 07:38:39PM +0530, Naveen Moses wrote: <br>

> The host Selector object is not strictly required. The host selector object <br>

> check was only added to identify if the system is single host or multi host <br>

> system based on host selector availability. <br>

> Thanks. <br>

> <br>

> On Mon, Sep 26, 2022 at 12:58 PM <zehra.ozdemir@inventron.com.tr> wrote: <br>

> <br>

> > Hello, <br>

> > <br>

> > <br>

> > <br>

> > There is an isMultiHost() method to control to return true in case the <br>

> > host selector object is available in button-handler.cpp of the <br>

> > phosphor-buttons. And if it is not available, it does not return false and <br>

> > the project gets an error. So we covered this return line with try catch <br>

> > and it works. <br>

<br>

Based on what Naveen wrote, it would be helpful if you could submit a <br>

change to improve the implementation to not have this issue. I'm <br>

surprised no one else has ran into the issues you are seeing. <br>

<br>

> > <br>

> > But my question is, does this host selector object has to be strictly <br>

> > defined for the project to run properly? <br>

> > <br>

> > <br>

> > <br>

> > Thanks. <br>

> > <br>

<br>

-- <br>

Patrick Williams <br>

</div>

</div>

</body>

</html>