[greybus-dev] Re: [PATCH 2/2] staging: vc04_services: bcm2835-camera: avoid -Wformat-truncation

Laurent Pinchart,

Thanks for your feedback.

I see your point regarding the complexity of the current construct. I’ll take some time to consider the best approach and follow up once I have a clearer picture.

Masaharu Noguchi

On Sat, Aug 30, 2025 at 11:29:23PM +0200, Laurent Pinchart wrote:

The fact that we need such a complicated construct is a sign that there's something wrong in our APIs. The above code is too complicated for what it does, making it less readable, more difficult to maintain, and more bug-prone. I don't know if we need yet another sprintf variant in the kernel, or something else, but this doens't seem to be what we need.

...

}

-- Regards,

Laurent Pinchart