Quoting Abhinav Kumar (2022-06-15 22:59:25)
> Hi Dmitry
>
> On 6/15/2022 10:55 PM, Dmitry Baryshkov wrote:
> > On 14/06/2022 22:32, Abhinav Kumar wrote:
> >> intf and wb resources are not dependent on the rm global
> >> state so need not be allocated during dpu_encoder_virt_atomic_mode_set().
> >>
> >> Move the allocation of intf and wb resources to
> >> dpu_encoder_setup_display()
> >> so that we can utilize the hw caps even during atomic_check() phase.
> >>
> >> Since dpu_encoder_setup_display() already has protection against
> >> setting invalid intf_idx and wb_idx, these checks can now
> >> be dropped as well.
> >>
> >> Fixes: e02a559a720f ("make changes to dpu_encoder to support virtual
> >> encoder")
> >
> > Please adjust the Fixes tags in all three commits. I didn't notice this
> > beforehand and Stephen has complained.

I think Stephen is Stephen Rothwell.

> >
> Is something wrong with the tag? Format and hash looked right to me.

        $ git config pretty.fixes
        Fixes: %h ("%s")
        $ git help fixes
        'fixes' is aliased to 'show -s --pretty=fixes'
        $ git fixes e02a559a720f
        Fixes: e02a559a720f ("drm/msm/dpu: make changes to dpu_encoder to
support virtual encoder")

it's missing the drm/msm/dpu prefix.

Reply via email to