Skip to content

hisilicon-osdrv-hi3516cv200: add OV2735 case to load_hisilicon#2154

Open
moontwister wants to merge 1 commit into
OpenIPC:masterfrom
moontwister:cv200-ov2735-load-hisilicon
Open

hisilicon-osdrv-hi3516cv200: add OV2735 case to load_hisilicon#2154
moontwister wants to merge 1 commit into
OpenIPC:masterfrom
moontwister:cv200-ov2735-load-hisilicon

Conversation

@moontwister
Copy link
Copy Markdown

Summary

Adds OV2735 to the existing ov2710_mipi case in cv200/load_hisilicon. OV2735 is a MIPI sensor (1- or 2-lane, RAW10) used on several Hi3518EV200 cameras (TP-Link Kasa KC110, Lenovo Snowman variants, generic Foscam-class hardware). Its MIPI pinmux + SoC clock requirements are identical to OV2710_MIPI — only the libsns / .ini differ.

Without this case, setting `fw_setenv sensor ov2735_i2c_1080p` (or `ov2735`) on cv200 falls through to the `xxxx Invalid sensor type xxxx` branch, vendor modules don't load, and `/proc/driver/hi_mipi` never gets configured.

Test plan

  • Built `hi3518ev200_ultimate_tplink-kasa-kc110` (Add TP-Link Kasa KC110 (HI3518EV200 + OV2735) builder#99) with this patch applied. `load_hisilicon` enters the new case correctly, modules load, `/proc/driver/hi_mipi` shows the configured lane / data type.
  • cv300's `load_hisilicon` already handles OV2735 (combined with OV2718 in the same case) — this brings cv200 to parity. No behaviour change for existing sensors.

Context

This is one of three small fixes split out from the closed #2151 (KC110 board package, re-filed as OpenIPC/builder#99 per @widgetii's review). Each lands a generic OV2735 cv200 fix without any board-specific scope.

🤖 Generated with Claude Code

OV2735 is a MIPI sensor (1- or 2-lane, RAW10) sold on a number of
Hi3518EV200-based cameras (e.g. TP-Link Kasa KC110). Its MIPI pinmux
and SoC clock requirements are identical to OV2710_MIPI — only the
libsns / .ini differ. Add it to the existing ov2710_mipi case so the
shared SENSOR= path works.

cv300's load_hisilicon already handles OV2735 (combined with OV2718);
this brings cv200 to parity.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@flyrouter
Copy link
Copy Markdown
Member

Good afternoon
Thank you, we will review your proposals.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants