Re: [PATCH v1 0/1] video: edid: guard standard timings EDID expansion behind kconfig

[Tom Rini <trini@konsulko.com>]

[-- Attachment #1: Type: text/plain, Size: 1253 bytes --]

On Fri, Mar 14, 2025 at 06:47:26PM +0200, Svyatoslav Ryhel wrote:
> пт, 14 бер. 2025 р. о 18:21 Tom Rini <trini@konsulko.com> пише:
> >
> > On Fri, Mar 14, 2025 at 08:47:19AM +0200, Svyatoslav Ryhel wrote:
> >
> > > Since EDID only indicates supported standard timings, a large table with
> > > detailed timing information is necessary, consuming significant space. To
> > > mitigate this, the table is made configurable via kconfig, allowing it to
> > > be excluded when not needed.
> > >
> > > Svyatoslav Ryhel (1):
> > >   video: edid: guard standard timings EDID expansion behind kconfig
> > >
> > >  common/edid.c         | 6 ++++++
> > >  drivers/video/Kconfig | 6 ++++++
> > >  2 files changed, 12 insertions(+)
> >
> > To be clear, when the flag is not enabled do we have something similar
> > to the old behavior, at least in the correctly functions case?
> >
> 
> As long as I2C_EDID_STANDARD is disabled the old behavior isn't
> changed, there were performed a bit of code refactoring, in the
> previous patch by Jonas, to simplify readability but nothing more then
> that. Significant size increase is not detected as well, as long as
> I2C_EDID_STANDARD=n

Great, thanks for clarifying.

-- 
Tom

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]