Re: [PATCH] Fix "ibm,processor-radix-AP-encodings"

2016-10-11 Thread Aneesh Kumar K.V
Michael Ellerman writes: > Oliver O'Halloran writes: > >> On Wed, Sep 28, 2016 at 12:43 PM, Aneesh Kumar K.V >> wrote: >>> Balbir Singh writes: >>> The top 3 bits of the lower order byte should

Re: [PATCH] Fix "ibm,processor-radix-AP-encodings"

2016-10-10 Thread Michael Ellerman
Oliver O'Halloran writes: > On Wed, Sep 28, 2016 at 12:43 PM, Aneesh Kumar K.V > wrote: >> Balbir Singh writes: >> >>> The top 3 bits of the lower order byte should contain the >>> AP encoding, we assume the top 3 bits

Re: [PATCH] Fix "ibm,processor-radix-AP-encodings"

2016-10-09 Thread Oliver O'Halloran
On Wed, Sep 28, 2016 at 12:43 PM, Aneesh Kumar K.V wrote: > Balbir Singh writes: > >> The top 3 bits of the lower order byte should contain the >> AP encoding, we assume the top 3 bits of the MSB. Balbir, could you reword this so it says

Re: [PATCH] Fix "ibm,processor-radix-AP-encodings"

2016-09-27 Thread Aneesh Kumar K.V
Balbir Singh writes: > The top 3 bits of the lower order byte should contain the > AP encoding, we assume the top 3 bits of the MSB. Are you sure, Power architecture documents always confuse about MSB vs lowe order bytes. ? > > Signed-off-by: Balbir Singh

[PATCH] Fix "ibm,processor-radix-AP-encodings"

2016-09-27 Thread Balbir Singh
The top 3 bits of the lower order byte should contain the AP encoding, we assume the top 3 bits of the MSB. Signed-off-by: Balbir Singh --- - Detected while reviewing Chris Smart's patch to add radix-AP-encoding to skiboot - Also fixed typo (sift/shift)