Claude review: MAINTAINERS: split hisilicon maintenance and add Yongbang Shi for hibmc-drm matainers

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

Patch Review

**Typo in subject:** "matainers" should be "maintainers".

**Critical: Whitespace formatting is wrong.** The MAINTAINERS file uses **tabs** after the field tag (e.g., `M:\t`), as confirmed by the existing file. The patch replaces all tab-separated lines with **spaces**:

```
-M:	Xinliang Liu <xinliang.liu@linaro.org>
+M:      Yongbang Shi <shiyongbang@huawei.com>
```

The `+` lines use spaces (likely 6 spaces) instead of a tab character. This will cause `scripts/get_maintainer.pl` and checkpatch.pl to complain, and is inconsistent with every other entry in MAINTAINERS. All lines must use a tab after the colon, not spaces.

**Missing F: path trailing slash.** The file path entries lack trailing slashes for directories:

```
+F:      drivers/gpu/drm/hisilicon/hibmc
+F:      drivers/gpu/drm/hisilicon/kirin
```

The MAINTAINERS convention for directories is to include a trailing `/` (e.g., `F: drivers/gpu/drm/hisilicon/hibmc/`). Without it, `get_maintainer.pl` may not match files within the directory correctly. The existing entry uses `F: drivers/gpu/drm/hisilicon/` with a trailing slash.

**Documentation/devicetree/bindings coverage.** The DT bindings path `Documentation/devicetree/bindings/display/hisilicon/` is only assigned to the KIRIN section. If hibmc also has (or will have) DT bindings under that path, they would have no maintainer coverage. Currently this seems reasonable since hibmc is a BMC (server management) driver that likely doesn't use DT, but it's worth confirming.

**Tian Tao remains M: for HIBMC but was removed from original entry in V1→V2 changelog context.** The changelog says "remove Tian Tao and Xinwei Kong from DRM DRIVERS FOR HISILICON KIRIN," and indeed they are only listed under HIBMC. This seems intentional and fine — just noting for clarity.

**Summary of required fixes:**
1. Fix subject typo: "matainers" → "maintainers"
2. Use tabs (not spaces) after field tags on all new lines
3. Add trailing `/` to directory paths in `F:` entries (`hibmc/`, `kirin/`)

---
Generated by Claude Code Patch Reviewer