Abhinav, On 2023-01-30 13:22:03, Abhinav Kumar wrote: > Hi Marijn > > On 1/30/2023 12:16 PM, Marijn Suijten wrote: > > On 2023-01-24 15:52:46, Kuogee Hsieh wrote: > > > > <snip> > > > > If only replying to a small chunk somewhere in the middle of a diff > > and/or large review, please cut out unnecessary bits to make your reply > > easier to find :) > > > >>>> + data = (dsc->flatness_min_qp & 0x1f); > >>>> + data |= (dsc->flatness_max_qp & 0x1f) << 5; > >>>> + data |= (dsc_info->det_thresh_flatness & 0xff) << 10; > >>> dpu_hw_dsc.c computes this on the fly. After removing that, and > >>> using initial_lines from the function parameters, only > >>> dsc_info->num_active_ss_per_enc remains. Do you really need that > >>> msm_display_dsc_info struct here, do you need it at all? > >> > >> I ported these code from our down stream code base. > >> > >> I make it work first, then clean it up will follow. > >> > >> I submit it for review since it looks like you guy like to have code sooner. > > > > Correct, I was looking forward to these patches albeit complete with the > > promised DSI support from Jessica, which still seems to be pending. > > > > DSI support is still being worked upon. > > I dont think we promised DSC 1.2 will come with DSI together in the same > series. It was always going to be DSC 1.2 + DP followed by another > series from Jessica for DSI. > > Lets set the expectations right. Not saying that these patches were promised as part of this series (as said, "which still seem to be pending"), just making clear that this series if of no use to me (no hurry to get the code in my hands sooner) until the DSI patches are also shared which I would have started working on myself if I didn't know QUIC was picking it up to distract from the current v1.1 broken-ness on SM8150 and SM8250. To set my (and at least Neil's) expectations straight as well: DSC 1.2 HW support should come in a separate series without DP support. Smaller series (not to mention appropriately split-up patches) lead to a decrease in scope, less dependencies and hopefully more efficient v2 - for all involved. - Marijn
