muriscv_nn_avgpool_s16 channel loop

[christian-herber-nxp]

In the average pooling function, you use a while loop on the chCnt, and also set the vector length to it size_t vl = vsetvl_e32m8(chCnt);.

End of the loop, you decrement chCnt--;.

I had a hard time understanding why this would be done. For one, the loop could be converted to a for loop, but also you are just decrementing the vector length until you are working with a vector length of 1.

Was it maybe intended to decrement the chCnt variable by the vector length at the end of the while loop?
like
chCnt -= vl; pSrc += vl;

On a side note, you might not be able to get super long vector length anyway, because the channel count is likely to be low, right? Wouldn't it make sense to apply the vector operation along a different dimension?

[PhilippvK]

We will look into this for release v0.3.0 scheduled for end of Jan 2025. Sorry for the delays!