Hi Daniel, On 1/28/26 2:11 PM, Daniel Stone wrote: > Hi Cristian, > > On Sun, 25 Jan 2026 at 00:23, Cristian Ciocaltea > <[email protected]> wrote: >> + /* VSI packet body */ >> + for (i = 0; i < len - 3; i += 4) >> + dw_hdmi_qp_write_pkt(hdmi, buffer + 3, i, min(len - i - 3, >> 4), >> + PKT_VSI_CONTENTS1 + i); > > Given that this for loop occurs in all the users (other than when len > < 4 where it's not required), why not move it into the > dw_hdmi_qp_write_pkt() helper itself, such that the calls for each > infoframe could be dw_hdmi_qp_write_pkt(hdmi, buffer + 3, len, > PKT_VSI_CONTENTS1 /* base reg, incremented by helper */)?
Yeah, initially planned to keep the helper simple and allow more flexibility in the callbacks. Probably now it makes sense to also write the packet header via the helper, not just the body, since this is also handled similarly in all cases. Thanks, Cristian
