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

From: "Aneesh Kumar K.V" <[email-masked]>
Date: Tue, 11 Oct 2016 13:19:53 +0530

Michael Ellerman <[email-masked]> writes:

> Oliver O'Halloran <[email-masked]> writes:
>
>> On Wed, Sep 28, 2016 at 12:43 PM, Aneesh Kumar K.V
>> <[email-masked]> wrote:
>>> Balbir Singh <[email-masked]> 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 "Currently we wrongly assume
>> <blah>" or similar. The current commit message made me think you were
>> changing it to look at the top 3 bits of the MSB rather than changing
>> it look at the LSB.
>
> Yeah please reword it to describe what the current code does, why it's
> incorrect, and how we are fixing it.
>

I have an updated patch for this, which redo the AP-encoding parsing
based on spec update. Wil send it to the list once we finalize the
details

-aneesh