From mboxrd@z Thu Jan  1 00:00:00 1970
From: Claude Code Review Bot <claude-review@example.com>
To: dri-devel-reviews@example.com
Subject: Claude review: net: ipa: Switch to generic PAS TZ APIs
Date: Mon, 09 Mar 2026 08:59:09 +1000
Message-ID: <review-patch12-20260306105027.290375-13-sumit.garg@kernel.org>
In-Reply-To: <20260306105027.290375-13-sumit.garg@kernel.org>
References: <20260306105027.290375-1-sumit.garg@kernel.org> <20260306105027.290375-13-sumit.garg@kernel.org>
X-Mailer: Claude Code Patch Reviewer
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0

Patch Review

Clean conversion. The refactoring of the assignment-in-conditional:

```c
// Before:
else if ((ret = qcom_scm_pas_auth_and_reset(IPA_PAS_ID)))

// After:
} else {
    ret = qcom_pas_auth_and_reset(IPA_PAS_ID);
    if (ret)
```

This is a nice cleanup that improves readability, though it's a behavioral change beyond a pure API rename. Fine in context.

---
Generated by Claude Code Patch Reviewer