Re: [PATCH 6/9] drm/ast: Factor mmc_test code in POST code

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 17 February 2017 at 05:32, Benjamin Herrenschmidt
<benh@xxxxxxxxxxxxxxxxxxx> wrote:
> There's a lot of duplication for what's essentially N copies of
> the same loop, so factor it. The upcoming AST2500 POST code adds
> more of this.
>
> Signed-off-by: Benjamin Herrenschmidt <benh@xxxxxxxxxxxxxxxxxxx>

Here is where things go fun:

Original code does the following data flexing only for the mmc_*test2
code, while this patch adds it to both test and test2.

       data = ast_mindwm(...);
       data = (data | (data >> 16)) & 0xffff;
//       ast_moutdwm(...);
       return data;

Afaict you have addressed that in the github branch/patch so things
are back to normal.
Tests are still a bit magical but not much you can/should do about it.

Regards,
Emil
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux