Re: [PATCH] cpsw: Fix interrupt storm among other things

From: Pantelis Antoniou
Date: Wed Jan 30 2013 - 08:51:40 EST


Hi Vaibhav,

On Jan 30, 2013, at 3:47 PM, Bedia, Vaibhav wrote:

> Hi Antoniou,
>
> On Wed, Jan 30, 2013 at 19:07:19, Pantelis Antoniou wrote:
>> Hi Vaibhav,
>>
>> On Jan 30, 2013, at 3:29 PM, Bedia, Vaibhav wrote:
>>
>>> On Wed, Jan 30, 2013 at 18:14:30, Pantelis Antoniou wrote:
>>>> Hi Vaibhav,
>>>>
>>>> On Jan 30, 2013, at 2:38 PM, Bedia, Vaibhav wrote:
>>>>
>>>>> On Wed, Jan 30, 2013 at 16:38:50, Pantelis Antoniou wrote:
>>>>> [...]
>>>>>>
>>>>>> TBH I haven't found a simple way to print out the silicon revision number.
>>>>>> Anyone on the list know a quick and dirty method?
>>>>>>
>>>>>
>>>>> You can dump the DEVICE_ID register @ 0x44e10600.
>>>>> Bits 31:28 should be 0 for PG1.0 and 1 for PG2.0.
>>>>>
>>>>
>>>> Thanks this works perfectly:
>>>>
>>>> original-bone:
>>>> root@beaglebone:~# devmem2 0x44e10600 w
>>>> Read at address 0x44E10600 (0xb6ff4600): 0x0B94402E
>>>>
>>>> bone-black:
>>>> root@beaglebone:~# devmem2 0x44e10600 w
>>>> Read at address 0x44E10600 (0xb6fcc600): 0x1B94402E
>>>>
>>>
>>> I just re-read the mail-chain and I am confused here.
>>> So the patch in question is meant for Bone-A4 which has
>>> PG1.0?
>>>
>>
>> It is a general bug fix. The problem was discovered only on
>> the bone black which has PG2.0 silicon. The driver has been
>> tested and it works on the original bone with PG1.0 as well.
>>
>
> But Mugunthan mentioned that he doesn't see this on an EVM
> with PG2.0 silicon... is there any board dependency here?
>

I don't know, but I doubt it. How about we wait for Mugunthan to
send us what are the DEVICE_ID contents for his board.

> Regards,
> Vaibhav

Regards

-- Pantelis

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/