Re: [PATCH net-next 7/7] net: ipa: allow arbitrary number of interconnects

2021-01-18 Thread Alex Elder
On 1/18/21 1:58 PM, Jakub Kicinski wrote: > But it does sound a little too much like aligning with the vendor > driver for the sake of aligning with the vendor driver. This makes > the review for someone not familiar with the vendor driver hard, > and raises questions like "is this really needed

Re: [PATCH net-next 7/7] net: ipa: allow arbitrary number of interconnects

2021-01-18 Thread Jakub Kicinski
On Sun, 17 Jan 2021 10:03:41 -0600 Alex Elder wrote: > On 1/16/21 9:12 PM, Jakub Kicinski wrote: > > On Fri, 15 Jan 2021 06:50:50 -0600 Alex Elder wrote: > >> Currently we assume that the IPA hardware has exactly three > >> interconnects. But that won't be guaranteed for all platforms, > >> so a

Re: [PATCH net-next 7/7] net: ipa: allow arbitrary number of interconnects

2021-01-17 Thread Alex Elder
On 1/16/21 9:12 PM, Jakub Kicinski wrote: On Fri, 15 Jan 2021 06:50:50 -0600 Alex Elder wrote: Currently we assume that the IPA hardware has exactly three interconnects. But that won't be guaranteed for all platforms, so allow any number of interconnects to be specified in the configuration dat

Re: [PATCH net-next 7/7] net: ipa: allow arbitrary number of interconnects

2021-01-16 Thread Jakub Kicinski
On Fri, 15 Jan 2021 06:50:50 -0600 Alex Elder wrote: > Currently we assume that the IPA hardware has exactly three > interconnects. But that won't be guaranteed for all platforms, > so allow any number of interconnects to be specified in the > configuration data. > > For each platform, define an

[PATCH net-next 7/7] net: ipa: allow arbitrary number of interconnects

2021-01-15 Thread Alex Elder
Currently we assume that the IPA hardware has exactly three interconnects. But that won't be guaranteed for all platforms, so allow any number of interconnects to be specified in the configuration data. For each platform, define an array of interconnect data entries (still associated with the IPA