Claude review: dma-buf: heaps: use designated initializer for exp_info

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

Patch Review

**Subject**: `dma-buf: heaps: use designated initializer for exp_info`  
**Files Modified**: `drivers/dma-buf/heaps/cma_heap.c`, `drivers/dma-buf/heaps/system_heap.c`  
**Verdict**: ✅ **Good cleanup** (pure refactoring)

#### Technical Review

Pure refactoring patch to use C99 designated initializers instead of field-by-field assignment.

**Changes in cma_heap.c** (lines 585-598):
```c
-   struct dma_heap_export_info exp_info;
+   struct dma_heap_export_info exp_info = {
+       .name = name,
+       .ops = &cma_heap_ops,
+   };
    ...
-   exp_info.name = name;
-   exp_info.ops = &cma_heap_ops;
    exp_info.priv = cma_heap;
```

**Analysis**:
✅ Modern C style  
✅ Zero-initializes all unspecified fields automatically  
✅ Makes follow-up patches cleaner (PATCH 4 adds new field)  
✅ No functional change  
✅ Reduces lines of code  

**Question**: Why is `exp_info.priv = cma_heap;` kept as separate assignment instead of being in the initializer? 

Answer: Because `cma_heap` is allocated just before (line 591-593), so it's not available at initialization time. This is fine.

**Correctness Check**:
- Before: `exp_info` on stack, uninitialized, then `.name` and `.ops` assigned
- After: `exp_info` on stack, zero-initialized, with `.name` and `.ops` set
- Both: `.priv` assigned later

This is semantically identical. ✅

---

---
Generated by Claude Code Patch Reviewer