> +/* Scheduler configuration for the assigning and releasing buffers for the
> + * packet passing through PPE, which is different per SoC.
> + */
> +static const struct ppe_scheduler_bm_config ipq9574_ppe_sch_bm_config[] = {
> +     {1, 0, 0, 0, 0},
> +     {1, 1, 0, 0, 0},
> +     {1, 0, 5, 0, 0},
> +     {1, 1, 5, 0, 0},
> +     {1, 0, 6, 0, 0},
> +     {1, 1, 6, 0, 0},
> +     {1, 0, 1, 0, 0},
> +     {1, 1, 1, 0, 0},

Rather than add a comment what it is, add a comment what it means.

It also looks like the first, 3 and 4 value are fixed, so do they even
need to be in the table? And the second value flip-flops?

        Andrew

Reply via email to