Claude review: accel/ivpu: Apply minor code style cleanups to align with kernel style

[Claude Code Review Bot <claude-review@example.com>]

Patch Review

**Change 1: Remove `(bool)` cast from `drm_gem_is_imported()` (line 104→105)**

```c
-		 (bool)drm_gem_is_imported(&bo->base.base));
+		 drm_gem_is_imported(&bo->base.base));
```

Correct. `drm_gem_is_imported()` is defined in `include/drm/drm_gem.h:646` as returning `bool`, so the cast is redundant. Note that the remaining `(bool)` casts on `bo->base.pages` and `bo->base.sgt` (line 33) are intentionally kept — those convert pointers to boolean values for the `%d` format specifier, which is a different situation.

**Change 2: Use `drm_gem_is_imported()` in `ivpu_bo_bind()` (line 113→114)**

```c
-	if (bo->base.base.import_attach)
+	if (drm_gem_is_imported(&bo->base.base))
```

Correct. `drm_gem_is_imported()` does `return !!obj->import_attach;` which is semantically identical when used in a boolean/`if` context. Using the accessor is preferred over direct field access.

**Change 3: Use `kzalloc_obj()` in `ivpu_gem_create_object()` (line 122→123)**

```c
-	bo = kzalloc(sizeof(*bo), GFP_KERNEL);
+	bo = kzalloc_obj(*bo);
```

Correct. The `kzalloc_obj(P, ...)` macro expands to `__alloc_objs(kzalloc, default_gfp(__VA_ARGS__), typeof(P), 1)`, where `default_gfp()` with no extra args defaults to `GFP_KERNEL`. So `kzalloc_obj(*bo)` is equivalent to `kzalloc(sizeof(*bo), GFP_KERNEL)`.

**Copyright year update (line 95→96)**

```c
- * Copyright (C) 2020-2023 Intel Corporation
+ * Copyright (C) 2020-2026 Intel Corporation
```

Fine, though this jumps from 2023 to 2026 — presumably no changes were copyrightable in 2024-2025, so this is acceptable.

**Minor nit:** The commit message says "Replace direct import_attach test with `drm_gem_is_imported()` in `ivpu_bo_bind()`" but doesn't mention that the same function's return value also had a `(bool)` cast removed in `ivpu_dbg_bo()`. The third bullet does mention removing the cast, but doesn't clarify it's on the same helper function call. This is a very minor clarity issue in the commit message.

**Verdict:** Patch looks good. No functional or correctness concerns. All three changes align with current kernel coding conventions.

---
Generated by Claude Code Patch Reviewer