Re: [PATCH] drm/edid: transition to passing struct cea_db * to cae_db_payload_len

2024-10-21 Thread Jani Nikula
On Mon, 21 Oct 2024, Vamsi Krishna Brahmajosyula wrote: > On Mon, Oct 21, 2024 at 7:12 PM Jani Nikula > wrote: >> >> On Fri, 11 Oct 2024, Vamsi Krishna Brahmajosyula >> wrote: >> > Address the FIXME in cea_db_payload_len >> > Transition to passing struct cea_db * everywhere >> >> You've

Re: [PATCH] drm/edid: transition to passing struct cea_db * to cae_db_payload_len

2024-10-21 Thread Vamsi Krishna Brahmajosyula
On Mon, Oct 21, 2024 at 7:12 PM Jani Nikula wrote: > > On Fri, 11 Oct 2024, Vamsi Krishna Brahmajosyula > wrote: > > Address the FIXME in cea_db_payload_len > > Transition to passing struct cea_db * everywhere > > You've misunderstood the comment. The point is to pass struct cea_db * > aro

Re: [PATCH] drm/edid: transition to passing struct cea_db * to cae_db_payload_len

2024-10-21 Thread Jani Nikula
On Fri, 11 Oct 2024, Vamsi Krishna Brahmajosyula wrote: > Address the FIXME in cea_db_payload_len > Transition to passing struct cea_db * everywhere You've misunderstood the comment. The point is to pass struct cea_db * around, not to stick to u8 * and drop calls to cea_db_payload_len().

Re: [PATCH] drm/edid: transition to passing struct cea_db * to cae_db_payload_len

2024-10-21 Thread Vamsi Krishna Brahmajosyula
On Fri, Oct 11, 2024 at 8:59 PM Vamsi Krishna Brahmajosyula wrote: > > Address the FIXME in cea_db_payload_len > Transition to passing struct cea_db * everywhere > > Precompute the payload length in drm_parse_cea_ext and pass to > individual parsers to avoid casting struct cea_db pointer t

[PATCH] drm/edid: transition to passing struct cea_db * to cae_db_payload_len

2024-10-11 Thread Vamsi Krishna Brahmajosyula
Address the FIXME in cea_db_payload_len Transition to passing struct cea_db * everywhere Precompute the payload length in drm_parse_cea_ext and pass to individual parsers to avoid casting struct cea_db pointer to u8 pointer where length is needed. Since the type of payload length is incon