Skip to content

fix: guard IMPISPAWBHist declaration for platforms that support it#17

Merged
themactep merged 2 commits intostablefrom
stable-local
Feb 23, 2026
Merged

fix: guard IMPISPAWBHist declaration for platforms that support it#17
themactep merged 2 commits intostablefrom
stable-local

Conversation

@themactep
Copy link
Copy Markdown
Owner

Problem

IMPISPAWBHist is not defined in T40/T41 SDK headers (only present in T20/T21/T23/T30/T31/C100). The prototype declaration in imp_hal.hpp was unconditional, causing a compile error when building for T41.

Fix

Guard the IMP_ISP_Tuning_GetAwbHist prototype with the same platform condition already used at the call site in imp_hal.cpp:

#if defined(PLATFORM_T20) || defined(PLATFORM_T21) || defined(PLATFORM_T23) || defined(PLATFORM_T30) || \
    defined(PLATFORM_T31) || defined(PLATFORM_C100)
int IMP_ISP_Tuning_GetAwbHist(IMPISPAWBHist *awb_hist);
#endif

Tested

Build verified for vanhua_z66si_t41nq_gc4023_eth (T41 SoC).

themactep and others added 2 commits February 23, 2026 03:13
IMPISPAWBHist is not defined in T40/T41 SDK headers. Guard the
IMP_ISP_Tuning_GetAwbHist prototype with the same platform condition
used at the call site in imp_hal.cpp.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@themactep themactep merged commit 6566b90 into stable Feb 23, 2026
0 of 21 checks passed
@themactep themactep deleted the stable-local branch February 23, 2026 08:17
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.

1 participant