Applied. Thanks!
Alex
On Fri, May 31, 2024 at 11:37 AM Randy Dunlap <rdun...@infradead.org> wrote:
>
>
>
> On 5/31/24 2:32 AM, Colin Ian King wrote:
> > There are a few spelling mistakes in dml2_printf messages. Fix them.
> >
> > Signed-off-by: Colin Ian King <colin.i.k...@gmail.com>
>
>
> Reviewed-by: Randy Dunlap <rdun...@infradead.org>
>
> Thanks.
>
> > ---
> > .../dc/dml2/dml21/src/dml2_core/dml2_core_dcn4_calcs.c | 6 +++---
> > .../display/dc/dml2/dml21/src/dml2_core/dml2_core_shared.c | 6 +++---
> > 2 files changed, 6 insertions(+), 6 deletions(-)
> >
> > diff --git
> > a/drivers/gpu/drm/amd/display/dc/dml2/dml21/src/dml2_core/dml2_core_dcn4_calcs.c
> >
> > b/drivers/gpu/drm/amd/display/dc/dml2/dml21/src/dml2_core/dml2_core_dcn4_calcs.c
> > index 8062144a5a6d..e7e6751f4477 100644
> > ---
> > a/drivers/gpu/drm/amd/display/dc/dml2/dml21/src/dml2_core/dml2_core_dcn4_calcs.c
> > +++
> > b/drivers/gpu/drm/amd/display/dc/dml2/dml21/src/dml2_core/dml2_core_dcn4_calcs.c
> > @@ -5731,7 +5731,7 @@ static bool CalculatePrefetchSchedule(struct
> > dml2_core_internal_scratch *scratch
> > dml2_printf("DML: Tvm: %fus - time to fetch vm\n",
> > s->TimeForFetchingVM);
> > dml2_printf("DML: Tr0: %fus - time to fetch first row of data
> > pagetables\n", s->TimeForFetchingRowInVBlank);
> > dml2_printf("DML: Tsw: %fus = time to fetch enough pixel data
> > and cursor data to feed the scalers init position and detile\n",
> > (double)s->LinesToRequestPrefetchPixelData * s->LineTime);
> > - dml2_printf("DML: To: %fus - time for propogation from scaler
> > to optc\n", (*p->DSTYAfterScaler + ((double)(*p->DSTXAfterScaler) /
> > (double)p->myPipe->HTotal)) * s->LineTime);
> > + dml2_printf("DML: To: %fus - time for propagation from scaler
> > to optc\n", (*p->DSTYAfterScaler + ((double)(*p->DSTXAfterScaler) /
> > (double)p->myPipe->HTotal)) * s->LineTime);
> > dml2_printf("DML: Tvstartup - TSetup - Tcalc - TWait - Tpre -
> > To > 0\n");
> > dml2_printf("DML: Tslack(pre): %fus - time left over in
> > schedule\n", p->VStartup * s->LineTime - s->TimeForFetchingVM - 2 *
> > s->TimeForFetchingRowInVBlank - (*p->DSTYAfterScaler +
> > ((double)(*p->DSTXAfterScaler) / (double)p->myPipe->HTotal)) * s->LineTime
> > - p->TWait - p->TCalc - *p->TSetup);
> > dml2_printf("DML: row_bytes = dpte_row_bytes (per_pipe) =
> > PixelPTEBytesPerRow = : %u\n", p->PixelPTEBytesPerRow);
> > @@ -8268,7 +8268,7 @@ static bool dml_core_mode_support(struct
> > dml2_core_calcs_mode_support_ex *in_out
> > dml2_printf("DML::%s: mode_lib->ms.DCFCLK = %f\n", __func__,
> > mode_lib->ms.DCFCLK);
> > dml2_printf("DML::%s: mode_lib->ms.FabricClock = %f\n", __func__,
> > mode_lib->ms.FabricClock);
> > dml2_printf("DML::%s: mode_lib->ms.uclk_freq_mhz = %f\n", __func__,
> > mode_lib->ms.uclk_freq_mhz);
> > - dml2_printf("DML::%s: urgent latency tolarance = %f\n", __func__,
> > ((mode_lib->ip.rob_buffer_size_kbytes -
> > mode_lib->ip.pixel_chunk_size_kbytes) * 1024 / (mode_lib->ms.DCFCLK *
> > mode_lib->soc.return_bus_width_bytes)));
> > + dml2_printf("DML::%s: urgent latency tolerance = %f\n", __func__,
> > ((mode_lib->ip.rob_buffer_size_kbytes -
> > mode_lib->ip.pixel_chunk_size_kbytes) * 1024 / (mode_lib->ms.DCFCLK *
> > mode_lib->soc.return_bus_width_bytes)));
> > #endif
> >
> > mode_lib->ms.support.OutstandingRequestsSupport = true;
> > @@ -11089,7 +11089,7 @@ static bool dml_core_mode_programming(struct
> > dml2_core_calcs_mode_programming_ex
> > if
> > (display_cfg->plane_descriptors[k].immediate_flip &&
> > mode_lib->mp.ImmediateFlipSupportedForPipe[k] == false) {
> > mode_lib->mp.ImmediateFlipSupported =
> > false;
> > #ifdef __DML_VBA_DEBUG__
> > - dml2_printf("DML::%s: Pipe %0d not
> > supporing iflip!\n", __func__, k);
> > + dml2_printf("DML::%s: Pipe %0d not
> > supporting iflip!\n", __func__, k);
> > #endif
> > }
> > }
> > diff --git
> > a/drivers/gpu/drm/amd/display/dc/dml2/dml21/src/dml2_core/dml2_core_shared.c
> >
> > b/drivers/gpu/drm/amd/display/dc/dml2/dml21/src/dml2_core/dml2_core_shared.c
> > index f2e2250d28d3..6eb3fec87ec1 100644
> > ---
> > a/drivers/gpu/drm/amd/display/dc/dml2/dml21/src/dml2_core/dml2_core_shared.c
> > +++
> > b/drivers/gpu/drm/amd/display/dc/dml2/dml21/src/dml2_core/dml2_core_shared.c
> > @@ -1988,7 +1988,7 @@ bool dml2_core_shared_mode_support(struct
> > dml2_core_calcs_mode_support_ex *in_ou
> > dml2_printf("DML::%s: mode_lib->ms.FabricClock = %f\n", __func__,
> > mode_lib->ms.FabricClock);
> > dml2_printf("DML::%s: mode_lib->ms.uclk_freq_mhz = %f\n", __func__,
> > mode_lib->ms.uclk_freq_mhz);
> > dml2_printf("DML::%s: max_urgent_latency_us = %f\n", __func__,
> > mode_lib->ms.support.max_urgent_latency_us);
> > - dml2_printf("DML::%s: urgent latency tolarance = %f\n", __func__,
> > ((mode_lib->ip.rob_buffer_size_kbytes -
> > mode_lib->ip.pixel_chunk_size_kbytes) * 1024 / (mode_lib->ms.DCFCLK *
> > mode_lib->soc.return_bus_width_bytes)));
> > + dml2_printf("DML::%s: urgent latency tolerance = %f\n", __func__,
> > ((mode_lib->ip.rob_buffer_size_kbytes -
> > mode_lib->ip.pixel_chunk_size_kbytes) * 1024 / (mode_lib->ms.DCFCLK *
> > mode_lib->soc.return_bus_width_bytes)));
> > #endif
> >
> > mode_lib->ms.support.OutstandingRequestsSupport = true;
> > @@ -8131,7 +8131,7 @@ static bool CalculatePrefetchSchedule(struct
> > dml2_core_internal_scratch *scratch
> > dml2_printf("DML: Tvm: %fus - time to fetch vm\n",
> > s->TimeForFetchingVM);
> > dml2_printf("DML: Tr0: %fus - time to fetch first row of data
> > pagetables\n", s->TimeForFetchingRowInVBlank);
> > dml2_printf("DML: Tsw: %fus = time to fetch enough pixel data
> > and cursor data to feed the scalers init position and detile\n",
> > (double)s->LinesToRequestPrefetchPixelData * s->LineTime);
> > - dml2_printf("DML: To: %fus - time for propogation from scaler
> > to optc\n", (*p->DSTYAfterScaler + ((double)(*p->DSTXAfterScaler) /
> > (double)p->myPipe->HTotal)) * s->LineTime);
> > + dml2_printf("DML: To: %fus - time for propagation from scaler
> > to optc\n", (*p->DSTYAfterScaler + ((double)(*p->DSTXAfterScaler) /
> > (double)p->myPipe->HTotal)) * s->LineTime);
> > dml2_printf("DML: Tvstartup - TSetup - Tcalc - TWait - Tpre -
> > To > 0\n");
> > dml2_printf("DML: Tslack(pre): %fus - time left over in
> > schedule\n", p->VStartup * s->LineTime - s->TimeForFetchingVM - 2 *
> > s->TimeForFetchingRowInVBlank - (*p->DSTYAfterScaler +
> > ((double)(*p->DSTXAfterScaler) / (double)p->myPipe->HTotal)) * s->LineTime
> > - p->TWait - p->TCalc - *p->TSetup);
> > dml2_printf("DML: row_bytes = dpte_row_bytes (per_pipe) =
> > PixelPTEBytesPerRow = : %u\n", p->PixelPTEBytesPerRow);
> > @@ -10959,7 +10959,7 @@ bool dml2_core_shared_mode_programming(struct
> > dml2_core_calcs_mode_programming_e
> > if
> > (display_cfg->plane_descriptors[k].immediate_flip &&
> > mode_lib->mp.ImmediateFlipSupportedForPipe[k] == false) {
> > mode_lib->mp.ImmediateFlipSupported =
> > false;
> > #ifdef __DML_VBA_DEBUG__
> > - dml2_printf("DML::%s: Pipe %0d not
> > supporing iflip!\n", __func__, k);
> > + dml2_printf("DML::%s: Pipe %0d not
> > supporting iflip!\n", __func__, k);
> > #endif
> > }
> > }
>
> --
> #Randy
> https://people.kernel.org/tglx/notes-about-netiquette
> https://subspace.kernel.org/etiquette.html
