va_dec_av1: Add max_frame_width/height_minus_1#843
va_dec_av1: Add max_frame_width/height_minus_1#843nowrep wants to merge 1 commit intointel:masterfrom
Conversation
XinfengZhang
left a comment
There was a problem hiding this comment.
not understand , this structure is frame level, each frame already has current width and height. from my understanding , max_frame_width_minus_1 max_frame_height_minus_1. should in seq level to determine the maximum resolution. why we need it for one frame ?
a54607a to
9c2d2ae
Compare
|
Yes, these values are in sequence header, but there is only
It's not maximum resolution according to seq level, it's maximum resolution used in the sequence and this can differ from frame resolution with multiple spatial layers. For example |
yes, there could be multiple resolution in one sequence, but I am wondering why HW need this, because you set parameters per picture/frame. so each frame has it's own resolution. it should be enough. |
AMD VCN needs this to correctly decode av1-1-b8-22-svc-L2T1 and av1-1-b8-22-svc-L2T2.