RE: [PATCH 1/2] powerpc/qman: Change fsl,qman-channel-id to cell-index

2015-08-20 Thread Roy Pledge
12, 2015 6:46 PM To: Pledge Roy-R01356 Cc: linuxppc-dev@lists.ozlabs.org; devicet...@vger.kernel.org; Bucur Madalin-Cristian-B32716 Subject: Re: [PATCH 1/2] powerpc/qman: Change fsl,qman-channel-id to cell- index On Tue, 2015-05-12 at 16:19 -0500, Pledge Roy-R01356 wrote

Re: [PATCH 1/2] powerpc/qman: Change fsl,qman-channel-id to cell-index

2015-08-20 Thread Scott Wood
On Thu, Aug 20, 2015 at 09:52:46AM -0500, Pledge Roy-R01356 wrote: I guess my point isn't getting through - channel-id and cell-index are too independent concepts that are coincidentally the same. Cell-index is only used by u-boot and is used to determine the portal number. It is absolutely

Re: [PATCH 1/2] powerpc/qman: Change fsl,qman-channel-id to cell-index

2015-08-19 Thread Scott Wood
To: Pledge Roy-R01356 Cc: linuxppc-dev@lists.ozlabs.org; devicet...@vger.kernel.org; Bucur Madalin-Cristian-B32716 Subject: Re: [PATCH 1/2] powerpc/qman: Change fsl,qman-channel-id to cell- index On Tue, 2015-05-12 at 16:19 -0500, Pledge Roy-R01356 wrote: I don't believe

RE: [PATCH 1/2] powerpc/qman: Change fsl,qman-channel-id to cell-index

2015-08-19 Thread Roy Pledge
...@vger.kernel.org; Bucur Madalin-Cristian-B32716 Subject: Re: [PATCH 1/2] powerpc/qman: Change fsl,qman-channel-id to cell- index On Tue, 2015-05-12 at 16:19 -0500, Pledge Roy-R01356 wrote: I don't believe this is correct - let me explain the rational why we had two properties in the QMan

RE: [PATCH 1/2] powerpc/qman: Change fsl,qman-channel-id to cell-index

2015-05-12 Thread Roy Pledge
I don't believe this is correct - let me explain the rational why we had two properties in the QMan portal to begin with. The two properties in question are cell-index and fsl,qman-channel-id. The cell-index property is used in u-boot as an index for the software portal ID when

Re: [PATCH 1/2] powerpc/qman: Change fsl,qman-channel-id to cell-index

2015-05-12 Thread Scott Wood
On Tue, 2015-05-12 at 16:19 -0500, Pledge Roy-R01356 wrote: I don't believe this is correct - let me explain the rational why we had two properties in the QMan portal to begin with. The two properties in question are cell-index and fsl,qman-channel-id. The cell-index

RE: [PATCH 1/2] powerpc/qman: Change fsl,qman-channel-id to cell-index

2015-05-05 Thread Roy Pledge
=freescale@lists.ozlabs.org] On Behalf Of Scott Wood Sent: Friday, April 17, 2015 6:53 PM To: linuxppc-dev@lists.ozlabs.org Cc: Wood Scott-B07421; devicet...@vger.kernel.org; Bucur Madalin-Cristian-B32716 Subject: [PATCH 1/2] powerpc/qman: Change fsl,qman-channel-id to cell-index It turns out

Re: [PATCH 1/2] powerpc/qman: Change fsl,qman-channel-id to cell-index

2015-05-05 Thread Scott Wood
On Tue, 2015-05-05 at 11:04 -0500, Pledge Roy-R01356 wrote: Sorry for the slow reply. I don't believe this is correct - let me explain the rational why we had two properties in the QMan portal to begin with. The two properties in question are cell-index and fsl,qman-channel-id. The

[PATCH 1/2] powerpc/qman: Change fsl,qman-channel-id to cell-index

2015-04-17 Thread Scott Wood
It turns out that existing U-Boots will dereference NULL pointers if the device tree does not have cell-index in the portal nodes. No patch has yet been merged adding device tree nodes for this binding (except a dtsi that has not yet been referenced), nor has any driver yet been merged making use