> +/* 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