<!DOCTYPE html><html><head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
</head>
<body>
<p><br>
</p>
<div class="moz-cite-prefix">On 12-06-2025 16:05, Nilawar, Badal
wrote:<br>
</div>
<blockquote type="cite" cite="mid:d4497be8-3536-47a4-99b9-58b5f1468172@intel.com">
<blockquote type="cite" style="color: #007cff;">
<blockquote type="cite" style="color: #007cff;">+static int
late_bind_fw_init(struct xe_late_bind *late_bind, u32 type)
<br>
+{
<br>
+ struct xe_device *xe = late_bind_to_xe(late_bind);
<br>
+ struct pci_dev *pdev = to_pci_dev(xe->drm.dev);
<br>
+ struct xe_late_bind_fw *lb_fw;
<br>
+ const struct firmware *fw;
<br>
+ u32 num_fans;
<br>
+ int ret;
<br>
+
<br>
+ if (!late_bind->component_added)
<br>
+ return 0;
<br>
+
<br>
+ lb_fw = &late_bind->late_bind_fw;
<br>
+
<br>
+ lb_fw->type = type;
<br>
</blockquote>
<br>
Should we validate that "type" is sane?
<br>
</blockquote>
You mean if type is not max supported type. Sure, I will add.</blockquote>
IMO we don't need to validate type, CSC will return -4 i.e. invalid
parameter error.
</body>
</html>