Re: [PATCH v2 1/6] media: vidtv: Rename PI definition to PI_SAMPLES

From: Andy Shevchenko

Date: Mon Mar 16 2026 - 07:22:33 EST


On Mon, Mar 16, 2026 at 11:13:24AM +0100, Hans Verkuil wrote:
>
> Just FYI: I'll take this patch. Regardless of the other patches in this series it does
> make sense.

Thank you!

> I haven't seen any updates to this series, is it abandoned?

There is no consensus which names should be chosen. I have strong opinion about
my choice, some do not agree with. Id est I like the idea to have PI as PI and
not something longer and confusing. Same for 64-bit, just some short.

My rationale is that, in most of the cases the 32-bit value will be used and
having it as is would be nice.

Another point is to have a macro that gives PI rounded to the chosen amount of
digits (to avoid / 10 / 100 / et cetera in the users). I don't know how to
implement that nicely, there is no loops or alike in C preprocessor.

--
With Best Regards,
Andy Shevchenko