On Tue, Oct 18, 2011 at 12:44:07AM -0500, Kumar Gala wrote:
On Aug 16, 2011, at 10:32 PM, Robin Holt wrote:
quoted
David,
The following set of patches have been reviewed by the above parties and
all comments have been integrated. Although the patches stray from the
drivers/net/can directory, the diversions are related to changes for
the flexcan driver.
The patch set is based upon your net-next-2.6 tree's commit 6c37e46.
Could you please queue these up for the next appropriate push to Linus'
tree?
Thanks,
Robin Holt
Robin,
Do you remember why we went with just 'fsl,p1010-flexcan' as the device tree compatible? Do we feel the flex can on P1010 isn't the same as on MPC5xxx? or the ARM SoCs?
The decision was due to the fact there is no true "generic" fsl.flexcan
chip free of any SOC implementation and therefore not something which
could be separately defined. That decision was made by Grant Likely.
I will inline that email below.
Robin
On Mon, Aug 15, 2011 at 09:13:50AM -0600, Grant Likely wrote:
On Mon, Aug 15, 2011 at 9:03 AM, Robin Holt [off-list ref] wrote:
quoted
Grant,
Earlier, you had asked for a more specific name for the compatible
property of the Freescale flexcan device. I still have not gotten a
more specific answer. Hopefully Marc can give you more details about
the flexcan implementations.
If there is no ip core version, then just stick with the
fsl,<soc>-flexcan name and drop "fsl,flexcan". Marketing may say
flexcan is flexcan, but hardware engineers like to change things.
Trying to be too generic in compatible values will just lead to
problems in the future.
Thanks,
Robin