<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Wed, Jul 8, 2026 at 9:05 PM Patrick Robb <<a href="mailto:patrickrobb1997@gmail.com">patrickrobb1997@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">Actually, please see the AI code review of this patch. Specifically, this comment looks correct to me. What are your thoughts?<div><br>**File:** `dts/framework/testbed_model/linux_session.py`, line 228<br><br>The code checks `lshw_result.return_code != 0 and lshw_result.stdout == ""` but the logic is incorrect. If the command fails, you should raise an error. The `and` should likely be `or`:</div></div></blockquote><div><br></div><div>This is definitely a valid issue, I will send out a patch fixing this shortly.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div><br></div><div>Please read through the other suggestions too. There is some fluff but also some good advice.</div></div></blockquote><div><br></div><div>For sure, I will include changes from those suggestions in the subsequent patch as well.</div></div></div>