Re: [PATCH] spmi-pmic-arb: support configurable number of peripherals

2015-10-15 Thread Stephen Boyd
On 10/15, Ivan T. Ivanov wrote: > > > On Oct 15, 2015, at 1:43 AM, Stephen Boyd wrote: > > > > I have to take this back. I missed the part where some pmics are on > > slave id 2 or slave id 4, so this check isn't going to work. I've > > adjusted it to use sid % 2 instead and I'll resend these tw

Re: [PATCH] spmi-pmic-arb: support configurable number of peripherals

2015-10-15 Thread Ivan T. Ivanov
> On Oct 15, 2015, at 1:43 AM, Stephen Boyd wrote: > > On 09/15/2015 11:27 AM, Stephen Boyd wrote: >> On 09/15, Ivan T. Ivanov wrote: >>> On Mon, 2015-09-14 at 18:28 -0700, Stephen Boyd wrote: On 09/14/2015 02:54 PM, Stephen Boyd wrote: > The current driver implementation supports only

Re: [PATCH] spmi-pmic-arb: support configurable number of peripherals

2015-10-14 Thread Stephen Boyd
On 09/15/2015 11:27 AM, Stephen Boyd wrote: > On 09/15, Ivan T. Ivanov wrote: >> On Mon, 2015-09-14 at 18:28 -0700, Stephen Boyd wrote: >>> On 09/14/2015 02:54 PM, Stephen Boyd wrote: The current driver implementation supports only 128 peripherals. Add support for more than 128 peripheral

Re: [PATCH] spmi-pmic-arb: support configurable number of peripherals

2015-09-15 Thread Stephen Boyd
On 09/15, Ivan T. Ivanov wrote: > > On Mon, 2015-09-14 at 18:28 -0700, Stephen Boyd wrote: > > On 09/14/2015 02:54 PM, Stephen Boyd wrote: > > > The current driver implementation supports only 128 peripherals. > > > Add support for more than 128 peripherals by taking a lazy > > > caching approach

Re: [PATCH] spmi-pmic-arb: support configurable number of peripherals

2015-09-15 Thread Ivan T. Ivanov
On Mon, 2015-09-14 at 18:28 -0700, Stephen Boyd wrote: > On 09/14/2015 02:54 PM, Stephen Boyd wrote: > > The current driver implementation supports only 128 peripherals. > > Add support for more than 128 peripherals by taking a lazy > > caching approach to the mapping tables. Instead of reading th

Re: [PATCH] spmi-pmic-arb: support configurable number of peripherals

2015-09-14 Thread Stephen Boyd
On 09/14/2015 02:54 PM, Stephen Boyd wrote: > The current driver implementation supports only 128 peripherals. > Add support for more than 128 peripherals by taking a lazy > caching approach to the mapping tables. Instead of reading the > tables at boot given some fixed size, read them on an as nee

Re: [PATCH] spmi-pmic-arb: support configurable number of peripherals

2015-09-09 Thread Stephen Boyd
On 09/09, Gilad Avidov wrote: > On Fri, 4 Sep 2015 17:50:11 -0700 > Stephen Boyd wrote: > > > On 09/04, Gilad Avidov wrote: > > > On Thu, 3 Sep 2015 17:16:30 -0700 > > > Stephen Boyd wrote: > > > > > > > On 09/03, Gilad Avidov wrote: > > I agree that the algorithm is not complicated. However,

Re: [PATCH] spmi-pmic-arb: support configurable number of peripherals

2015-09-09 Thread Gilad Avidov
On Fri, 4 Sep 2015 17:50:11 -0700 Stephen Boyd wrote: > On 09/04, Gilad Avidov wrote: > > On Thu, 3 Sep 2015 17:16:30 -0700 > > Stephen Boyd wrote: > > > > > On 09/03, Gilad Avidov wrote: > > > > +supported by HW. Default (minimum > > > > supported) is 128. + > > > > +Ex

Re: [PATCH] spmi-pmic-arb: support configurable number of peripherals

2015-09-04 Thread Stephen Boyd
On 09/04, Gilad Avidov wrote: > On Thu, 3 Sep 2015 17:16:30 -0700 > Stephen Boyd wrote: > > > On 09/03, Gilad Avidov wrote: > > > + supported by HW. Default (minimum > > > supported) is 128. + > > > +Example V1 PMIC-Arbiter: > > > > > > spmi { > > > compatible = "qc

Re: [PATCH] spmi-pmic-arb: support configurable number of peripherals

2015-09-04 Thread Gilad Avidov
On Thu, 3 Sep 2015 17:16:30 -0700 Stephen Boyd wrote: > On 09/03, Gilad Avidov wrote: > > +supported by HW. Default (minimum > > supported) is 128. + > > +Example V1 PMIC-Arbiter: > > > > spmi { > > compatible = "qcom,spmi-pmic-arb"; > > @@ -62,4 +66,32 @@ Ex

Re: [PATCH] spmi-pmic-arb: support configurable number of peripherals

2015-09-03 Thread Stephen Boyd
On 09/03, Gilad Avidov wrote: > + supported by HW. Default (minimum supported) is 128. > + > +Example V1 PMIC-Arbiter: > > spmi { > compatible = "qcom,spmi-pmic-arb"; > @@ -62,4 +66,32 @@ Example: > > interrupt-controller; > #