[prev in list] [next in list] [prev in thread] [next in thread] 

List:       linux-can
Subject:    Re: [PATCH v3 2/2] can: flexcan: adding platform specific details for LS1021A
From:       Marc Kleine-Budde <mkl () pengutronix ! de>
Date:       2017-11-23 12:33:34
Message-ID: ca3f6c50-65b7-4c47-1564-99e4b6518cd9 () pengutronix ! de
[Download RAW message or body]

[Attachment #2 (multipart/mixed)]


On 11/23/2017 01:01 PM, Pankaj Bansal wrote:
>>> I got this version from Bill Of Material details of LS1021A SOC.
>>
>> Oh, nice. Do you have access to the vf610 list to get the IP core version?
> 
> I am sorry but I don't have access to vf610 list.

Ok.

>>> Actually in LS1021A Rev1 RX-FIFO mode is broken. Rev1 is initial
>>> release, and Rev2 Is final release. Which is why this comment.
>>
>> Ahh, can you please add this to the table above.
> 
> I have already added LS1021A in table.

Yes, but not the information that the r1 has a broken FIFO.

>> You're not using the mailbox mode anymore, right? 
> 
> No, we are using timestamp based RX fifo.
> 
>> This means the driver should work with the broken r1 silicon, too?
> 
> I would think so. But as r1 is not officially supported version, I
> have not checked it.

Ok - I don't mind if nxp doesn't support it or not...

>> Are any of these silicons out in
>> the wild? Does it make sense to use "fsl,ls1021ar1-flexcan" for this
>> compatible?
> 
> No. r1 silicon has not been delivered to any customer. We need not to support
> r1 silicon in compatible, as its not official supported version.

...but if it's not out in the wild we don't have to add it. And wait
until someone complains :)

Marc

-- 
Pengutronix e.K.                  | Marc Kleine-Budde           |
Industrial Linux Solutions        | Phone: +49-231-2826-924     |
Vertretung West/Dortmund          | Fax:   +49-5121-206917-5555 |
Amtsgericht Hildesheim, HRA 2686  | http://www.pengutronix.de   |


["signature.asc" (application/pgp-signature)]
--
To unsubscribe from this list: send the line "unsubscribe linux-can" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic