> Agreed, and this was my initial approach, but I also wanted a minimal > fix for David to pull into "net" while we can properly resolve this for > "net-next" see below.
O.K, so in that case, this is fine. > Making the ops const was my initial approach but there are several > challenges to making it possible right now which I will address against > net-next: > > - register/unregister_switch_driver actually do modify dsa_switch_ops > while updating the list pointer, so we need to encapsulate > dsa_switch_ops into a dsa_switch_driver plus a list member O.K, this is dsa v1. I had v2 in mind. Yes, the list needs abstracting. Thanks Andrew