On Wed, Aug 10, 2011 at 09:45:17AM -0500, Kumar Gala wrote: > > On Aug 10, 2011, at 9:15 AM, Robin Holt wrote: > > > On Wed, Aug 10, 2011 at 03:47:43PM +0200, Wolfgang Grandegger wrote: > >> Hi Robin, > >> > >> On 08/10/2011 05:06 AM, Robin Holt wrote: > >>> In working with the socketcan developers, we have come to the conclusion > >>> the Documentation...fsl-flexcan.txt device tree documentation needs to > >>> be cleaned up. The driver does not depend upon any properties other > >> > >> Your first sentence could be misleading. Please just describe what the > >> patch does and why, something like: > >> > >> "This patch cleans up the documentation of the device-tree binding for > >> the Flexcan devices on Freescale's PowerPC and ARM cores. Extra > >> properties are not needed as the frequency of the source clock is > >> fixed..." and so on. > > > > I borrowed heavily from your message. ;) > > > >>> than the required properties so we are removing the file. Additionally, > >>> the p1010*dts* files are not following the standard for node naming in > >>> that they have a trailing -v1.0. > >> > >>> Signed-off-by: Robin Holt <[email protected]> > >>> To: Marc Kleine-Budde <[email protected]>, > >>> To: Wolfgang Grandegger <[email protected]>, > >>> To: U Bhaskar-B22300 <[email protected]> > >>> To: Scott Wood <[email protected]> > >>> Cc: [email protected], > >>> Cc: [email protected], > >>> Cc: PPC list <[email protected]> > >>> Cc: Kumar Gala <[email protected]> > >>> --- > >>> .../devicetree/bindings/net/can/fsl-flexcan.txt | 61 > >>> -------------------- > >>> arch/powerpc/boot/dts/p1010rdb.dts | 8 --- > >>> arch/powerpc/boot/dts/p1010si.dtsi | 8 +- > >>> 3 files changed, 4 insertions(+), 73 deletions(-) > >>> delete mode 100644 > >>> Documentation/devicetree/bindings/net/can/fsl-flexcan.txt > > I don't understand how we can do this? What binding spec covers the P1010 > CAN support if you remove this?
We have added it back in based upon an earlier comment from Wolfgang. I will post a new version shortly. Robin _______________________________________________ Socketcan-core mailing list [email protected] https://lists.berlios.de/mailman/listinfo/socketcan-core
