* [PATCH v11 0/5] flexcan/powerpc: Add support for powerpc flexcan (freescale p1010) @ 2011-08-10 16:27 Robin Holt 2011-08-10 16:27 ` [PATCH v11 2/5] flexcan: Abstract off read/write for big/little endian Robin Holt [not found] ` <1312993670-23999-1-git-send-email-holt-sJ/iWh9BUns@public.gmane.org> 0 siblings, 2 replies; 20+ messages in thread From: Robin Holt @ 2011-08-10 16:27 UTC (permalink / raw) To: Robin Holt, Kumar Gala, Wolfgang Grandegger, U Bhaskar-B22300 Cc: socketcan-core-0fE9KPoRgkgATYTw5x5z8w, netdev-u79uwXL29TY76Z2rM5mHXA, PPC list, Marc Kleine-Budde With all the patches applied, my p1010rdb works for communicating between its two can ports and also can communicate with an external PSOC. I have done no testing beyond compile testing on an arm system as I have no access to an arm based system. For the first three patches in the series, I believe they are all ready for forwarding to David S. Miller for the netdev tree. I think patch 4 is ready for submission to the PPC85xx maintainer. Patch 5 changed from the previous post by adding a second compatible string for the fsl,flexcan-p1010 and bringing back the Documentation file in a more correct form. Thanks, Robin Holt ^ permalink raw reply [flat|nested] 20+ messages in thread
* [PATCH v11 2/5] flexcan: Abstract off read/write for big/little endian. 2011-08-10 16:27 [PATCH v11 0/5] flexcan/powerpc: Add support for powerpc flexcan (freescale p1010) Robin Holt @ 2011-08-10 16:27 ` Robin Holt [not found] ` <1312993670-23999-1-git-send-email-holt-sJ/iWh9BUns@public.gmane.org> 1 sibling, 0 replies; 20+ messages in thread From: Robin Holt @ 2011-08-10 16:27 UTC (permalink / raw) To: Robin Holt, Kumar Gala, Wolfgang Grandegger, U Bhaskar-B22300 Cc: Robin Holt, Marc Kleine-Budde, Wolfgang Grandegger, U Bhaskar-B22300, socketcan-core, netdev, PPC list Make flexcan driver handle register reads in the appropriate endianess. This was a basic search and replace and then define some inlines. Signed-off-by: Robin Holt <holt@sgi.com> Acked-by: Marc Kleine-Budde <mkl@pengutronix.de> Acked-by: Wolfgang Grandegger <wg@grandegger.com> Cc: U Bhaskar-B22300 <B22300@freescale.com> Cc: socketcan-core@lists.berlios.de Cc: netdev@vger.kernel.org Cc: PPC list <linuxppc-dev@lists.ozlabs.org> --- drivers/net/can/flexcan.c | 140 ++++++++++++++++++++++++++------------------ 1 files changed, 83 insertions(+), 57 deletions(-) diff --git a/drivers/net/can/flexcan.c b/drivers/net/can/flexcan.c index 586b2cd..68cbe52 100644 --- a/drivers/net/can/flexcan.c +++ b/drivers/net/can/flexcan.c @@ -190,6 +190,31 @@ static struct can_bittiming_const flexcan_bittiming_const = { }; /* + * Abstract off the read/write for arm versus ppc. + */ +#if defined(__BIG_ENDIAN) +static inline u32 flexcan_read(void __iomem *addr) +{ + return in_be32(addr); +} + +static inline void flexcan_write(u32 val, void __iomem *addr) +{ + out_be32(addr, val); +} +#else +static inline u32 flexcan_read(void __iomem *addr) +{ + return readl(addr); +} + +static inline void flexcan_write(u32 val, void __iomem *addr) +{ + writel(val, addr); +} +#endif + +/* * Swtich transceiver on or off */ static void flexcan_transceiver_switch(const struct flexcan_priv *priv, int on) @@ -210,9 +235,9 @@ static inline void flexcan_chip_enable(struct flexcan_priv *priv) struct flexcan_regs __iomem *regs = priv->base; u32 reg; - reg = readl(®s->mcr); + reg = flexcan_read(®s->mcr); reg &= ~FLEXCAN_MCR_MDIS; - writel(reg, ®s->mcr); + flexcan_write(reg, ®s->mcr); udelay(10); } @@ -222,9 +247,9 @@ static inline void flexcan_chip_disable(struct flexcan_priv *priv) struct flexcan_regs __iomem *regs = priv->base; u32 reg; - reg = readl(®s->mcr); + reg = flexcan_read(®s->mcr); reg |= FLEXCAN_MCR_MDIS; - writel(reg, ®s->mcr); + flexcan_write(reg, ®s->mcr); } static int flexcan_get_berr_counter(const struct net_device *dev, @@ -232,7 +257,7 @@ static int flexcan_get_berr_counter(const struct net_device *dev, { const struct flexcan_priv *priv = netdev_priv(dev); struct flexcan_regs __iomem *regs = priv->base; - u32 reg = readl(®s->ecr); + u32 reg = flexcan_read(®s->ecr); bec->txerr = (reg >> 0) & 0xff; bec->rxerr = (reg >> 8) & 0xff; @@ -266,15 +291,15 @@ static int flexcan_start_xmit(struct sk_buff *skb, struct net_device *dev) if (cf->can_dlc > 0) { u32 data = be32_to_cpup((__be32 *)&cf->data[0]); - writel(data, ®s->cantxfg[FLEXCAN_TX_BUF_ID].data[0]); + flexcan_write(data, ®s->cantxfg[FLEXCAN_TX_BUF_ID].data[0]); } if (cf->can_dlc > 3) { u32 data = be32_to_cpup((__be32 *)&cf->data[4]); - writel(data, ®s->cantxfg[FLEXCAN_TX_BUF_ID].data[1]); + flexcan_write(data, ®s->cantxfg[FLEXCAN_TX_BUF_ID].data[1]); } - writel(can_id, ®s->cantxfg[FLEXCAN_TX_BUF_ID].can_id); - writel(ctrl, ®s->cantxfg[FLEXCAN_TX_BUF_ID].can_ctrl); + flexcan_write(can_id, ®s->cantxfg[FLEXCAN_TX_BUF_ID].can_id); + flexcan_write(ctrl, ®s->cantxfg[FLEXCAN_TX_BUF_ID].can_ctrl); kfree_skb(skb); @@ -462,8 +487,8 @@ static void flexcan_read_fifo(const struct net_device *dev, struct flexcan_mb __iomem *mb = ®s->cantxfg[0]; u32 reg_ctrl, reg_id; - reg_ctrl = readl(&mb->can_ctrl); - reg_id = readl(&mb->can_id); + reg_ctrl = flexcan_read(&mb->can_ctrl); + reg_id = flexcan_read(&mb->can_id); if (reg_ctrl & FLEXCAN_MB_CNT_IDE) cf->can_id = ((reg_id >> 0) & CAN_EFF_MASK) | CAN_EFF_FLAG; else @@ -473,12 +498,12 @@ static void flexcan_read_fifo(const struct net_device *dev, cf->can_id |= CAN_RTR_FLAG; cf->can_dlc = get_can_dlc((reg_ctrl >> 16) & 0xf); - *(__be32 *)(cf->data + 0) = cpu_to_be32(readl(&mb->data[0])); - *(__be32 *)(cf->data + 4) = cpu_to_be32(readl(&mb->data[1])); + *(__be32 *)(cf->data + 0) = cpu_to_be32(flexcan_read(&mb->data[0])); + *(__be32 *)(cf->data + 4) = cpu_to_be32(flexcan_read(&mb->data[1])); /* mark as read */ - writel(FLEXCAN_IFLAG_RX_FIFO_AVAILABLE, ®s->iflag1); - readl(®s->timer); + flexcan_write(FLEXCAN_IFLAG_RX_FIFO_AVAILABLE, ®s->iflag1); + flexcan_read(®s->timer); } static int flexcan_read_frame(struct net_device *dev) @@ -514,17 +539,17 @@ static int flexcan_poll(struct napi_struct *napi, int quota) * The error bits are cleared on read, * use saved value from irq handler. */ - reg_esr = readl(®s->esr) | priv->reg_esr; + reg_esr = flexcan_read(®s->esr) | priv->reg_esr; /* handle state changes */ work_done += flexcan_poll_state(dev, reg_esr); /* handle RX-FIFO */ - reg_iflag1 = readl(®s->iflag1); + reg_iflag1 = flexcan_read(®s->iflag1); while (reg_iflag1 & FLEXCAN_IFLAG_RX_FIFO_AVAILABLE && work_done < quota) { work_done += flexcan_read_frame(dev); - reg_iflag1 = readl(®s->iflag1); + reg_iflag1 = flexcan_read(®s->iflag1); } /* report bus errors */ @@ -534,8 +559,8 @@ static int flexcan_poll(struct napi_struct *napi, int quota) if (work_done < quota) { napi_complete(napi); /* enable IRQs */ - writel(FLEXCAN_IFLAG_DEFAULT, ®s->imask1); - writel(priv->reg_ctrl_default, ®s->ctrl); + flexcan_write(FLEXCAN_IFLAG_DEFAULT, ®s->imask1); + flexcan_write(priv->reg_ctrl_default, ®s->ctrl); } return work_done; @@ -549,9 +574,9 @@ static irqreturn_t flexcan_irq(int irq, void *dev_id) struct flexcan_regs __iomem *regs = priv->base; u32 reg_iflag1, reg_esr; - reg_iflag1 = readl(®s->iflag1); - reg_esr = readl(®s->esr); - writel(FLEXCAN_ESR_ERR_INT, ®s->esr); /* ACK err IRQ */ + reg_iflag1 = flexcan_read(®s->iflag1); + reg_esr = flexcan_read(®s->esr); + flexcan_write(FLEXCAN_ESR_ERR_INT, ®s->esr); /* ACK err IRQ */ /* * schedule NAPI in case of: @@ -567,16 +592,16 @@ static irqreturn_t flexcan_irq(int irq, void *dev_id) * save them for later use. */ priv->reg_esr = reg_esr & FLEXCAN_ESR_ERR_BUS; - writel(FLEXCAN_IFLAG_DEFAULT & ~FLEXCAN_IFLAG_RX_FIFO_AVAILABLE, - ®s->imask1); - writel(priv->reg_ctrl_default & ~FLEXCAN_CTRL_ERR_ALL, + flexcan_write(FLEXCAN_IFLAG_DEFAULT & + ~FLEXCAN_IFLAG_RX_FIFO_AVAILABLE, ®s->imask1); + flexcan_write(priv->reg_ctrl_default & ~FLEXCAN_CTRL_ERR_ALL, ®s->ctrl); napi_schedule(&priv->napi); } /* FIFO overflow */ if (reg_iflag1 & FLEXCAN_IFLAG_RX_FIFO_OVERFLOW) { - writel(FLEXCAN_IFLAG_RX_FIFO_OVERFLOW, ®s->iflag1); + flexcan_write(FLEXCAN_IFLAG_RX_FIFO_OVERFLOW, ®s->iflag1); dev->stats.rx_over_errors++; dev->stats.rx_errors++; } @@ -585,7 +610,7 @@ static irqreturn_t flexcan_irq(int irq, void *dev_id) if (reg_iflag1 & (1 << FLEXCAN_TX_BUF_ID)) { /* tx_bytes is incremented in flexcan_start_xmit */ stats->tx_packets++; - writel((1 << FLEXCAN_TX_BUF_ID), ®s->iflag1); + flexcan_write((1 << FLEXCAN_TX_BUF_ID), ®s->iflag1); netif_wake_queue(dev); } @@ -599,7 +624,7 @@ static void flexcan_set_bittiming(struct net_device *dev) struct flexcan_regs __iomem *regs = priv->base; u32 reg; - reg = readl(®s->ctrl); + reg = flexcan_read(®s->ctrl); reg &= ~(FLEXCAN_CTRL_PRESDIV(0xff) | FLEXCAN_CTRL_RJW(0x3) | FLEXCAN_CTRL_PSEG1(0x7) | @@ -623,11 +648,11 @@ static void flexcan_set_bittiming(struct net_device *dev) reg |= FLEXCAN_CTRL_SMP; dev_info(dev->dev.parent, "writing ctrl=0x%08x\n", reg); - writel(reg, ®s->ctrl); + flexcan_write(reg, ®s->ctrl); /* print chip status */ dev_dbg(dev->dev.parent, "%s: mcr=0x%08x ctrl=0x%08x\n", __func__, - readl(®s->mcr), readl(®s->ctrl)); + flexcan_read(®s->mcr), flexcan_read(®s->ctrl)); } /* @@ -648,10 +673,10 @@ static int flexcan_chip_start(struct net_device *dev) flexcan_chip_enable(priv); /* soft reset */ - writel(FLEXCAN_MCR_SOFTRST, ®s->mcr); + flexcan_write(FLEXCAN_MCR_SOFTRST, ®s->mcr); udelay(10); - reg_mcr = readl(®s->mcr); + reg_mcr = flexcan_read(®s->mcr); if (reg_mcr & FLEXCAN_MCR_SOFTRST) { dev_err(dev->dev.parent, "Failed to softreset can module (mcr=0x%08x)\n", @@ -673,12 +698,12 @@ static int flexcan_chip_start(struct net_device *dev) * choose format C * */ - reg_mcr = readl(®s->mcr); + reg_mcr = flexcan_read(®s->mcr); reg_mcr |= FLEXCAN_MCR_FRZ | FLEXCAN_MCR_FEN | FLEXCAN_MCR_HALT | FLEXCAN_MCR_SUPV | FLEXCAN_MCR_WRN_EN | FLEXCAN_MCR_IDAM_C; dev_dbg(dev->dev.parent, "%s: writing mcr=0x%08x", __func__, reg_mcr); - writel(reg_mcr, ®s->mcr); + flexcan_write(reg_mcr, ®s->mcr); /* * CTRL @@ -696,7 +721,7 @@ static int flexcan_chip_start(struct net_device *dev) * (FLEXCAN_CTRL_ERR_MSK), too. Otherwise we don't get any * warning or bus passive interrupts. */ - reg_ctrl = readl(®s->ctrl); + reg_ctrl = flexcan_read(®s->ctrl); reg_ctrl &= ~FLEXCAN_CTRL_TSYN; reg_ctrl |= FLEXCAN_CTRL_BOFF_REC | FLEXCAN_CTRL_LBUF | FLEXCAN_CTRL_ERR_STATE | FLEXCAN_CTRL_ERR_MSK; @@ -704,38 +729,39 @@ static int flexcan_chip_start(struct net_device *dev) /* save for later use */ priv->reg_ctrl_default = reg_ctrl; dev_dbg(dev->dev.parent, "%s: writing ctrl=0x%08x", __func__, reg_ctrl); - writel(reg_ctrl, ®s->ctrl); + flexcan_write(reg_ctrl, ®s->ctrl); for (i = 0; i < ARRAY_SIZE(regs->cantxfg); i++) { - writel(0, ®s->cantxfg[i].can_ctrl); - writel(0, ®s->cantxfg[i].can_id); - writel(0, ®s->cantxfg[i].data[0]); - writel(0, ®s->cantxfg[i].data[1]); + flexcan_write(0, ®s->cantxfg[i].can_ctrl); + flexcan_write(0, ®s->cantxfg[i].can_id); + flexcan_write(0, ®s->cantxfg[i].data[0]); + flexcan_write(0, ®s->cantxfg[i].data[1]); /* put MB into rx queue */ - writel(FLEXCAN_MB_CNT_CODE(0x4), ®s->cantxfg[i].can_ctrl); + flexcan_write(FLEXCAN_MB_CNT_CODE(0x4), + ®s->cantxfg[i].can_ctrl); } /* acceptance mask/acceptance code (accept everything) */ - writel(0x0, ®s->rxgmask); - writel(0x0, ®s->rx14mask); - writel(0x0, ®s->rx15mask); + flexcan_write(0x0, ®s->rxgmask); + flexcan_write(0x0, ®s->rx14mask); + flexcan_write(0x0, ®s->rx15mask); flexcan_transceiver_switch(priv, 1); /* synchronize with the can bus */ - reg_mcr = readl(®s->mcr); + reg_mcr = flexcan_read(®s->mcr); reg_mcr &= ~FLEXCAN_MCR_HALT; - writel(reg_mcr, ®s->mcr); + flexcan_write(reg_mcr, ®s->mcr); priv->can.state = CAN_STATE_ERROR_ACTIVE; /* enable FIFO interrupts */ - writel(FLEXCAN_IFLAG_DEFAULT, ®s->imask1); + flexcan_write(FLEXCAN_IFLAG_DEFAULT, ®s->imask1); /* print chip status */ dev_dbg(dev->dev.parent, "%s: reading mcr=0x%08x ctrl=0x%08x\n", - __func__, readl(®s->mcr), readl(®s->ctrl)); + __func__, flexcan_read(®s->mcr), flexcan_read(®s->ctrl)); return 0; @@ -757,12 +783,12 @@ static void flexcan_chip_stop(struct net_device *dev) u32 reg; /* Disable all interrupts */ - writel(0, ®s->imask1); + flexcan_write(0, ®s->imask1); /* Disable + halt module */ - reg = readl(®s->mcr); + reg = flexcan_read(®s->mcr); reg |= FLEXCAN_MCR_MDIS | FLEXCAN_MCR_HALT; - writel(reg, ®s->mcr); + flexcan_write(reg, ®s->mcr); flexcan_transceiver_switch(priv, 0); priv->can.state = CAN_STATE_STOPPED; @@ -854,24 +880,24 @@ static int __devinit register_flexcandev(struct net_device *dev) /* select "bus clock", chip must be disabled */ flexcan_chip_disable(priv); - reg = readl(®s->ctrl); + reg = flexcan_read(®s->ctrl); reg |= FLEXCAN_CTRL_CLK_SRC; - writel(reg, ®s->ctrl); + flexcan_write(reg, ®s->ctrl); flexcan_chip_enable(priv); /* set freeze, halt and activate FIFO, restrict register access */ - reg = readl(®s->mcr); + reg = flexcan_read(®s->mcr); reg |= FLEXCAN_MCR_FRZ | FLEXCAN_MCR_HALT | FLEXCAN_MCR_FEN | FLEXCAN_MCR_SUPV; - writel(reg, ®s->mcr); + flexcan_write(reg, ®s->mcr); /* * Currently we only support newer versions of this core * featuring a RX FIFO. Older cores found on some Coldfire * derivates are not yet supported. */ - reg = readl(®s->mcr); + reg = flexcan_read(®s->mcr); if (!(reg & FLEXCAN_MCR_FEN)) { dev_err(dev->dev.parent, "Could not enable RX FIFO, unsupported core\n"); -- 1.7.2.1 ^ permalink raw reply related [flat|nested] 20+ messages in thread
[parent not found: <1312993670-23999-1-git-send-email-holt-sJ/iWh9BUns@public.gmane.org>]
* [PATCH v11 1/5] flexcan: Remove #include <mach/clock.h> [not found] ` <1312993670-23999-1-git-send-email-holt-sJ/iWh9BUns@public.gmane.org> @ 2011-08-10 16:27 ` Robin Holt 2011-08-10 16:27 ` [PATCH v11 3/5] flexcan: Add of_match to platform_device definition Robin Holt ` (2 subsequent siblings) 3 siblings, 0 replies; 20+ messages in thread From: Robin Holt @ 2011-08-10 16:27 UTC (permalink / raw) To: Robin Holt, Kumar Gala, Wolfgang Grandegger, U Bhaskar-B22300 Cc: netdev-u79uwXL29TY76Z2rM5mHXA, U Bhaskar-B22300, socketcan-core-0fE9KPoRgkgATYTw5x5z8w, Marc Kleine-Budde, PPC list, Wolfgang Grandegger powerpc does not have a mach-####/clock.h. When testing, I found neither arm nor powerpc needed the mach/clock.h at all so I removed it. Signed-off-by: Robin Holt <holt-sJ/iWh9BUns@public.gmane.org> Acked-by: Marc Kleine-Budde <mkl-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org> Acked-by: Wolfgang Grandegger <wg-5Yr1BZd7O62+XT7JhA+gdA@public.gmane.org> Cc: U Bhaskar-B22300 <B22300-KZfg59tc24xl57MIdRCFDg@public.gmane.org> Cc: socketcan-core-0fE9KPoRgkgATYTw5x5z8w@public.gmane.org Cc: netdev-u79uwXL29TY76Z2rM5mHXA@public.gmane.org Cc: PPC list <linuxppc-dev-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org> --- drivers/net/can/flexcan.c | 2 -- 1 files changed, 0 insertions(+), 2 deletions(-) diff --git a/drivers/net/can/flexcan.c b/drivers/net/can/flexcan.c index 1767811..586b2cd 100644 --- a/drivers/net/can/flexcan.c +++ b/drivers/net/can/flexcan.c @@ -35,8 +35,6 @@ #include <linux/module.h> #include <linux/platform_device.h> -#include <mach/clock.h> - #define DRV_NAME "flexcan" /* 8 for RX fifo and 2 error handling */ -- 1.7.2.1 ^ permalink raw reply related [flat|nested] 20+ messages in thread
* [PATCH v11 3/5] flexcan: Add of_match to platform_device definition. [not found] ` <1312993670-23999-1-git-send-email-holt-sJ/iWh9BUns@public.gmane.org> 2011-08-10 16:27 ` [PATCH v11 1/5] flexcan: Remove #include <mach/clock.h> Robin Holt @ 2011-08-10 16:27 ` Robin Holt 2011-08-10 16:27 ` [PATCH v11 4/5] powerpc: Add flexcan device support for p1010rdb Robin Holt 2011-08-10 16:27 ` [PATCH v11 5/5] powerpc: Fix up fsl-flexcan device tree binding Robin Holt 3 siblings, 0 replies; 20+ messages in thread From: Robin Holt @ 2011-08-10 16:27 UTC (permalink / raw) To: Robin Holt, Kumar Gala, Wolfgang Grandegger, U Bhaskar-B22300 Cc: socketcan-core-0fE9KPoRgkgATYTw5x5z8w, netdev-u79uwXL29TY76Z2rM5mHXA, devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ, U Bhaskar-B22300, Grant Likely, Marc Kleine-Budde, PPC list, Wolfgang Grandegger On powerpc, the OpenFirmware devices are not matched without specifying an of_match array. Introduce that array as that is used for matching on the Freescale P1010 processor. Signed-off-by: Robin Holt <holt-sJ/iWh9BUns@public.gmane.org> Acked-by: Marc Kleine-Budde <mkl-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org> Acked-by: Wolfgang Grandegger <wg-5Yr1BZd7O62+XT7JhA+gdA@public.gmane.org> Cc: U Bhaskar-B22300 <B22300-KZfg59tc24xl57MIdRCFDg@public.gmane.org> Cc: Grant Likely <grant.likely-s3s/WqlpOiPyB63q8FvJNQ@public.gmane.org> Cc: socketcan-core-0fE9KPoRgkgATYTw5x5z8w@public.gmane.org Cc: netdev-u79uwXL29TY76Z2rM5mHXA@public.gmane.org Cc: PPC list <linuxppc-dev-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org> Cc: devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org --- drivers/net/can/flexcan.c | 13 ++++++++++++- 1 files changed, 12 insertions(+), 1 deletions(-) diff --git a/drivers/net/can/flexcan.c b/drivers/net/can/flexcan.c index 68cbe52..662f832 100644 --- a/drivers/net/can/flexcan.c +++ b/drivers/net/can/flexcan.c @@ -1027,8 +1027,19 @@ static int __devexit flexcan_remove(struct platform_device *pdev) return 0; } +static struct of_device_id flexcan_of_match[] = { + { + .compatible = "fsl,flexcan", + }, + {}, +}; + static struct platform_driver flexcan_driver = { - .driver.name = DRV_NAME, + .driver = { + .name = DRV_NAME, + .owner = THIS_MODULE, + .of_match_table = flexcan_of_match, + }, .probe = flexcan_probe, .remove = __devexit_p(flexcan_remove), }; -- 1.7.2.1 ^ permalink raw reply related [flat|nested] 20+ messages in thread
* [PATCH v11 4/5] powerpc: Add flexcan device support for p1010rdb. [not found] ` <1312993670-23999-1-git-send-email-holt-sJ/iWh9BUns@public.gmane.org> 2011-08-10 16:27 ` [PATCH v11 1/5] flexcan: Remove #include <mach/clock.h> Robin Holt 2011-08-10 16:27 ` [PATCH v11 3/5] flexcan: Add of_match to platform_device definition Robin Holt @ 2011-08-10 16:27 ` Robin Holt [not found] ` <1312993670-23999-5-git-send-email-holt-sJ/iWh9BUns@public.gmane.org> 2011-08-10 16:27 ` [PATCH v11 5/5] powerpc: Fix up fsl-flexcan device tree binding Robin Holt 3 siblings, 1 reply; 20+ messages in thread From: Robin Holt @ 2011-08-10 16:27 UTC (permalink / raw) To: Robin Holt, Kumar Gala, Wolfgang Grandegger, U Bhaskar-B22300 Cc: , netdev-u79uwXL29TY76Z2rM5mHXA, U Bhaskar-B22300, Kumar Gala, socketcan-core-0fE9KPoRgkgATYTw5x5z8w, Marc Kleine-Budde, PPC list, Wolfgang Grandegger I added a simple clock source for the p1010rdb so the flexcan driver could determine a clock frequency. The p1010 flexcan device only has an oscillator of system bus frequency divided by 2. Signed-off-by: Robin Holt <holt-sJ/iWh9BUns@public.gmane.org> Acked-by: Marc Kleine-Budde <mkl-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>, Acked-by: Wolfgang Grandegger <wg-5Yr1BZd7O62+XT7JhA+gdA@public.gmane.org>, Cc: U Bhaskar-B22300 <B22300-KZfg59tc24xl57MIdRCFDg@public.gmane.org> Cc: socketcan-core-0fE9KPoRgkgATYTw5x5z8w@public.gmane.org, Cc: netdev-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Cc: PPC list <linuxppc-dev-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org> Cc: Kumar Gala <galak-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org> --- arch/powerpc/platforms/85xx/Kconfig | 2 + arch/powerpc/platforms/85xx/Makefile | 2 + arch/powerpc/platforms/85xx/clock.c | 52 ++++++++++++++++++++++++++++++++ arch/powerpc/platforms/85xx/p1010rdb.c | 8 +++++ 4 files changed, 64 insertions(+), 0 deletions(-) create mode 100644 arch/powerpc/platforms/85xx/clock.c diff --git a/arch/powerpc/platforms/85xx/Kconfig b/arch/powerpc/platforms/85xx/Kconfig index 498534c..c4304ae 100644 --- a/arch/powerpc/platforms/85xx/Kconfig +++ b/arch/powerpc/platforms/85xx/Kconfig @@ -70,6 +70,8 @@ config MPC85xx_RDB config P1010_RDB bool "Freescale P1010RDB" select DEFAULT_UIMAGE + select HAVE_CAN_FLEXCAN if NET && CAN + select PPC_CLOCK if CAN_FLEXCAN help This option enables support for the MPC85xx RDB (P1010 RDB) board diff --git a/arch/powerpc/platforms/85xx/Makefile b/arch/powerpc/platforms/85xx/Makefile index a971b32..cc7f381 100644 --- a/arch/powerpc/platforms/85xx/Makefile +++ b/arch/powerpc/platforms/85xx/Makefile @@ -3,6 +3,8 @@ # obj-$(CONFIG_SMP) += smp.o +obj-$(CONFIG_PPC_CLOCK) += clock.o + obj-$(CONFIG_MPC8540_ADS) += mpc85xx_ads.o obj-$(CONFIG_MPC8560_ADS) += mpc85xx_ads.o obj-$(CONFIG_MPC85xx_CDS) += mpc85xx_cds.o diff --git a/arch/powerpc/platforms/85xx/clock.c b/arch/powerpc/platforms/85xx/clock.c new file mode 100644 index 0000000..0b6db1d --- /dev/null +++ b/arch/powerpc/platforms/85xx/clock.c @@ -0,0 +1,52 @@ +/* + * Copyright 2011 SGI, inc. + * + * This code is licensed for use under the GPL V2 as published by + * the Free Software Foundation. + */ + +#include <linux/device.h> +#include <linux/err.h> +#include <linux/of.h> + +#include <asm/clk_interface.h> + +#include <sysdev/fsl_soc.h> + +/* + * p1010 needs to provide a clock source for the flexcan driver. The + * oscillator for the p1010 processor is only ever the system clock / 2. + */ + +static struct clk *mpc85xx_clk_get(struct device *dev, const char *id) +{ + if (!dev) + return ERR_PTR(-ENOENT); + + if (!dev->of_node || + !of_device_is_compatible(dev->of_node, "fsl,flexcan")) + return ERR_PTR(-ENOENT); + + return NULL; +} + +static void mpc85xx_clk_put(struct clk *clk) +{ + return; +} + +static unsigned long mpc85xx_clk_get_rate(struct clk *clk) +{ + return fsl_get_sys_freq() / 2; +} + +static struct clk_interface mpc85xx_clk_functions = { + .clk_get = mpc85xx_clk_get, + .clk_get_rate = mpc85xx_clk_get_rate, + .clk_put = mpc85xx_clk_put, +}; + +void __init mpc85xx_clk_init(void) +{ + clk_functions = mpc85xx_clk_functions; +} diff --git a/arch/powerpc/platforms/85xx/p1010rdb.c b/arch/powerpc/platforms/85xx/p1010rdb.c index d7387fa..5e52122 100644 --- a/arch/powerpc/platforms/85xx/p1010rdb.c +++ b/arch/powerpc/platforms/85xx/p1010rdb.c @@ -81,6 +81,13 @@ static void __init p1010_rdb_setup_arch(void) printk(KERN_INFO "P1010 RDB board from Freescale Semiconductor\n"); } +extern void mpc85xx_clk_init(void); + +static void __init p1010_rdb_init(void) +{ + mpc85xx_clk_init(); +} + static struct of_device_id __initdata p1010rdb_ids[] = { { .type = "soc", }, { .compatible = "soc", }, @@ -111,6 +118,7 @@ define_machine(p1010_rdb) { .name = "P1010 RDB", .probe = p1010_rdb_probe, .setup_arch = p1010_rdb_setup_arch, + .init = p1010_rdb_init, .init_IRQ = p1010_rdb_pic_init, #ifdef CONFIG_PCI .pcibios_fixup_bus = fsl_pcibios_fixup_bus, -- 1.7.2.1 ^ permalink raw reply related [flat|nested] 20+ messages in thread
[parent not found: <1312993670-23999-5-git-send-email-holt-sJ/iWh9BUns@public.gmane.org>]
* Re: [PATCH v11 4/5] powerpc: Add flexcan device support for p1010rdb. [not found] ` <1312993670-23999-5-git-send-email-holt-sJ/iWh9BUns@public.gmane.org> @ 2011-08-10 17:01 ` Kumar Gala [not found] ` <8E5FA886-038D-4DF4-8A54-DD60188A21A2-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org> 0 siblings, 1 reply; 20+ messages in thread From: Kumar Gala @ 2011-08-10 17:01 UTC (permalink / raw) To: Robin Holt Cc: netdev-u79uwXL29TY76Z2rM5mHXA, U Bhaskar-B22300, socketcan-core-0fE9KPoRgkgATYTw5x5z8w, Marc Kleine-Budde, PPC list, Wolfgang Grandegger On Aug 10, 2011, at 11:27 AM, Robin Holt wrote: > I added a simple clock source for the p1010rdb so the flexcan driver > could determine a clock frequency. The p1010 flexcan device only has > an oscillator of system bus frequency divided by 2. > > Signed-off-by: Robin Holt <holt-sJ/iWh9BUns@public.gmane.org> > Acked-by: Marc Kleine-Budde <mkl-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>, > Acked-by: Wolfgang Grandegger <wg-5Yr1BZd7O62+XT7JhA+gdA@public.gmane.org>, > Cc: U Bhaskar-B22300 <B22300-KZfg59tc24xl57MIdRCFDg@public.gmane.org> > Cc: socketcan-core-0fE9KPoRgkgATYTw5x5z8w@public.gmane.org, > Cc: netdev-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, > Cc: PPC list <linuxppc-dev-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org> > Cc: Kumar Gala <galak-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org> > --- > arch/powerpc/platforms/85xx/Kconfig | 2 + > arch/powerpc/platforms/85xx/Makefile | 2 + > arch/powerpc/platforms/85xx/clock.c | 52 ++++++++++++++++++++++++++++++++ > arch/powerpc/platforms/85xx/p1010rdb.c | 8 +++++ > 4 files changed, 64 insertions(+), 0 deletions(-) > create mode 100644 arch/powerpc/platforms/85xx/clock.c I dont understand how mpc85xx_clk_functions() ends up being associated with the frequency the flexcan is running at. This either seems to global or I'm missing something. I still think the clk / freq info should be in the device tree and handled in the driver and NOT arch/powerpc platform code. - k ^ permalink raw reply [flat|nested] 20+ messages in thread
[parent not found: <8E5FA886-038D-4DF4-8A54-DD60188A21A2-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org>]
* Re: [PATCH v11 4/5] powerpc: Add flexcan device support for p1010rdb. [not found] ` <8E5FA886-038D-4DF4-8A54-DD60188A21A2-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org> @ 2011-08-10 18:16 ` Wolfgang Grandegger [not found] ` <4E42CB01.7030700-5Yr1BZd7O62+XT7JhA+gdA@public.gmane.org> 0 siblings, 1 reply; 20+ messages in thread From: Wolfgang Grandegger @ 2011-08-10 18:16 UTC (permalink / raw) To: Kumar Gala Cc: netdev-u79uwXL29TY76Z2rM5mHXA, U Bhaskar-B22300, socketcan-core-0fE9KPoRgkgATYTw5x5z8w, Marc Kleine-Budde, PPC list On 08/10/2011 07:01 PM, Kumar Gala wrote: > > On Aug 10, 2011, at 11:27 AM, Robin Holt wrote: > >> I added a simple clock source for the p1010rdb so the flexcan driver >> could determine a clock frequency. The p1010 flexcan device only has >> an oscillator of system bus frequency divided by 2. >> >> Signed-off-by: Robin Holt <holt-sJ/iWh9BUns@public.gmane.org> >> Acked-by: Marc Kleine-Budde <mkl-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>, >> Acked-by: Wolfgang Grandegger <wg-5Yr1BZd7O62+XT7JhA+gdA@public.gmane.org>, >> Cc: U Bhaskar-B22300 <B22300-KZfg59tc24xl57MIdRCFDg@public.gmane.org> >> Cc: socketcan-core-0fE9KPoRgkgATYTw5x5z8w@public.gmane.org, >> Cc: netdev-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, >> Cc: PPC list <linuxppc-dev-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org> >> Cc: Kumar Gala <galak-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org> >> --- >> arch/powerpc/platforms/85xx/Kconfig | 2 + >> arch/powerpc/platforms/85xx/Makefile | 2 + >> arch/powerpc/platforms/85xx/clock.c | 52 ++++++++++++++++++++++++++++++++ >> arch/powerpc/platforms/85xx/p1010rdb.c | 8 +++++ >> 4 files changed, 64 insertions(+), 0 deletions(-) >> create mode 100644 arch/powerpc/platforms/85xx/clock.c > > I dont understand how mpc85xx_clk_functions() ends up being associated with the frequency the flexcan is running at. The function mpc85xx_clk_get_rate() returns "fsl_get_sys_freq() / 2" for Flexcan devices. > This either seems to global or I'm missing something. This patch extends the existing Flexcan platform driver for ARM for the PowerPC using the device tree. Due to the nice integration of the device tree (of-platform) into the platform driver and devices, the difference are quite small (see patches 1..3). Apart from the endianess issue, only the clock needs to be handled in a common way. As ARM already uses the clk interface, we found it straight-forward to implement it for the P1010, or more general for the 85xx, as well, instead of using an additional helper function. > I still think the clk / freq info should be in the device tree and handled in the driver and NOT arch/powerpc platform code. If I understand you correctly, you want the boot-loader to provide the relevant information by fixing up the device tree, which then can be handled arch-independently by the driver, right? Wolfgang. ^ permalink raw reply [flat|nested] 20+ messages in thread
[parent not found: <4E42CB01.7030700-5Yr1BZd7O62+XT7JhA+gdA@public.gmane.org>]
* Re: [PATCH v11 4/5] powerpc: Add flexcan device support for p1010rdb. [not found] ` <4E42CB01.7030700-5Yr1BZd7O62+XT7JhA+gdA@public.gmane.org> @ 2011-08-11 3:56 ` Robin Holt [not found] ` <20110811035620.GB4926-sJ/iWh9BUns@public.gmane.org> 2011-08-11 4:46 ` Kumar Gala 1 sibling, 1 reply; 20+ messages in thread From: Robin Holt @ 2011-08-11 3:56 UTC (permalink / raw) To: Wolfgang Grandegger, Marc Kleine-Budde Cc: netdev-u79uwXL29TY76Z2rM5mHXA, U Bhaskar-B22300, Kumar Gala, socketcan-core-0fE9KPoRgkgATYTw5x5z8w, PPC list On Wed, Aug 10, 2011 at 08:16:33PM +0200, Wolfgang Grandegger wrote: > On 08/10/2011 07:01 PM, Kumar Gala wrote: > > > > On Aug 10, 2011, at 11:27 AM, Robin Holt wrote: > > > >> I added a simple clock source for the p1010rdb so the flexcan driver > >> could determine a clock frequency. The p1010 flexcan device only has > >> an oscillator of system bus frequency divided by 2. > >> > >> Signed-off-by: Robin Holt <holt-sJ/iWh9BUns@public.gmane.org> > >> Acked-by: Marc Kleine-Budde <mkl-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>, > >> Acked-by: Wolfgang Grandegger <wg-5Yr1BZd7O62+XT7JhA+gdA@public.gmane.org>, > >> Cc: U Bhaskar-B22300 <B22300-KZfg59tc24xl57MIdRCFDg@public.gmane.org> > >> Cc: socketcan-core-0fE9KPoRgkgATYTw5x5z8w@public.gmane.org, > >> Cc: netdev-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, > >> Cc: PPC list <linuxppc-dev-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org> > >> Cc: Kumar Gala <galak-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org> > >> --- > >> arch/powerpc/platforms/85xx/Kconfig | 2 + > >> arch/powerpc/platforms/85xx/Makefile | 2 + > >> arch/powerpc/platforms/85xx/clock.c | 52 ++++++++++++++++++++++++++++++++ > >> arch/powerpc/platforms/85xx/p1010rdb.c | 8 +++++ > >> 4 files changed, 64 insertions(+), 0 deletions(-) > >> create mode 100644 arch/powerpc/platforms/85xx/clock.c > > > > I dont understand how mpc85xx_clk_functions() ends up being associated with the frequency the flexcan is running at. > > The function mpc85xx_clk_get_rate() returns "fsl_get_sys_freq() / 2" for > Flexcan devices. > > > This either seems to global or I'm missing something. > > This patch extends the existing Flexcan platform driver for ARM for the > PowerPC using the device tree. Due to the nice integration of the device > tree (of-platform) into the platform driver and devices, the difference > are quite small (see patches 1..3). Apart from the endianess issue, only > the clock needs to be handled in a common way. As ARM already uses the > clk interface, we found it straight-forward to implement it for the > P1010, or more general for the 85xx, as well, instead of using an > additional helper function. > > > I still think the clk / freq info should be in the device tree and handled in the driver and NOT arch/powerpc platform code. > > If I understand you correctly, you want the boot-loader to provide the > relevant information by fixing up the device tree, which then can be > handled arch-independently by the driver, right? Marc and Wolfgang, This is a very early swag at this which I worked up while driving home from dinner this evening. It works with my current config, but that includes at least one bogus patch. I will have to do more testing tomorrow. For now, it is something to ponder. Thanks, Robin ------------------------------------------------------------------------ flexcan: Prefer device tree clock frequency if available. If our CAN device's device tree node has a clock-frequency property, then use that value for the can devices clock frequency. If not, fall back to asking the platform/mach code for the clock frequency associated with the flexcan device. Too-early-to-be-signed-off-by: Robin Holt <holt-sJ/iWh9BUns@public.gmane.org> diff --git a/drivers/net/can/flexcan.c b/drivers/net/can/flexcan.c index 662f832..d6a87c9 100644 --- a/drivers/net/can/flexcan.c +++ b/drivers/net/can/flexcan.c @@ -33,6 +33,7 @@ #include <linux/kernel.h> #include <linux/list.h> #include <linux/module.h> +#include <linux/of.h> #include <linux/platform_device.h> #define DRV_NAME "flexcan" @@ -929,12 +930,25 @@ static int __devinit flexcan_probe(struct platform_device *pdev) void __iomem *base; resource_size_t mem_size; int err, irq; + u32 clock_freq = 0; - clk = clk_get(&pdev->dev, NULL); - if (IS_ERR(clk)) { - dev_err(&pdev->dev, "no clock defined\n"); - err = PTR_ERR(clk); - goto failed_clock; + if (pdev->dev.of_node) { + u32 *clock_freq_p; + + clk = NULL; + clock_freq_p = (u32 *)of_get_property(pdev->dev.of_node, "clock-frequency", NULL); + if (clock_freq_p) + clock_freq = *clock_freq_p; + } + + if (clock_freq) { + clk = clk_get(&pdev->dev, NULL); + if (IS_ERR(clk)) { + dev_err(&pdev->dev, "no clock defined\n"); + err = PTR_ERR(clk); + goto failed_clock; + } + clock_freq = clk_get_rate(clk); } mem = platform_get_resource(pdev, IORESOURCE_MEM, 0); @@ -967,7 +981,7 @@ static int __devinit flexcan_probe(struct platform_device *pdev) dev->flags |= IFF_ECHO; /* we support local echo in hardware */ priv = netdev_priv(dev); - priv->can.clock.freq = clk_get_rate(clk); + priv->can.clock.freq = clock_freq; priv->can.bittiming_const = &flexcan_bittiming_const; priv->can.do_set_mode = flexcan_set_mode; priv->can.do_get_berr_counter = flexcan_get_berr_counter; @@ -1002,7 +1016,8 @@ static int __devinit flexcan_probe(struct platform_device *pdev) failed_map: release_mem_region(mem->start, mem_size); failed_get: - clk_put(clk); + if (clk) + clk_put(clk); failed_clock: return err; } @@ -1020,7 +1035,8 @@ static int __devexit flexcan_remove(struct platform_device *pdev) mem = platform_get_resource(pdev, IORESOURCE_MEM, 0); release_mem_region(mem->start, resource_size(mem)); - clk_put(priv->clk); + if (priv->clk) + clk_put(priv->clk); free_candev(dev); ^ permalink raw reply related [flat|nested] 20+ messages in thread
[parent not found: <20110811035620.GB4926-sJ/iWh9BUns@public.gmane.org>]
* Re: [PATCH v11 4/5] powerpc: Add flexcan device support for p1010rdb. [not found] ` <20110811035620.GB4926-sJ/iWh9BUns@public.gmane.org> @ 2011-08-11 7:35 ` Wolfgang Grandegger 0 siblings, 0 replies; 20+ messages in thread From: Wolfgang Grandegger @ 2011-08-11 7:35 UTC (permalink / raw) To: Robin Holt Cc: netdev-u79uwXL29TY76Z2rM5mHXA, U Bhaskar-B22300, Kumar Gala, socketcan-core-0fE9KPoRgkgATYTw5x5z8w, Marc Kleine-Budde, PPC list On 08/11/2011 05:56 AM, Robin Holt wrote: > On Wed, Aug 10, 2011 at 08:16:33PM +0200, Wolfgang Grandegger wrote: >> On 08/10/2011 07:01 PM, Kumar Gala wrote: >>> >>> On Aug 10, 2011, at 11:27 AM, Robin Holt wrote: >>> >>>> I added a simple clock source for the p1010rdb so the flexcan driver >>>> could determine a clock frequency. The p1010 flexcan device only has >>>> an oscillator of system bus frequency divided by 2. >>>> >>>> Signed-off-by: Robin Holt <holt-sJ/iWh9BUns@public.gmane.org> >>>> Acked-by: Marc Kleine-Budde <mkl-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>, >>>> Acked-by: Wolfgang Grandegger <wg-5Yr1BZd7O62+XT7JhA+gdA@public.gmane.org>, >>>> Cc: U Bhaskar-B22300 <B22300-KZfg59tc24xl57MIdRCFDg@public.gmane.org> >>>> Cc: socketcan-core-0fE9KPoRgkgATYTw5x5z8w@public.gmane.org, >>>> Cc: netdev-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, >>>> Cc: PPC list <linuxppc-dev-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org> >>>> Cc: Kumar Gala <galak-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org> >>>> --- >>>> arch/powerpc/platforms/85xx/Kconfig | 2 + >>>> arch/powerpc/platforms/85xx/Makefile | 2 + >>>> arch/powerpc/platforms/85xx/clock.c | 52 ++++++++++++++++++++++++++++++++ >>>> arch/powerpc/platforms/85xx/p1010rdb.c | 8 +++++ >>>> 4 files changed, 64 insertions(+), 0 deletions(-) >>>> create mode 100644 arch/powerpc/platforms/85xx/clock.c >>> >>> I dont understand how mpc85xx_clk_functions() ends up being associated with the frequency the flexcan is running at. >> >> The function mpc85xx_clk_get_rate() returns "fsl_get_sys_freq() / 2" for >> Flexcan devices. >> >>> This either seems to global or I'm missing something. >> >> This patch extends the existing Flexcan platform driver for ARM for the >> PowerPC using the device tree. Due to the nice integration of the device >> tree (of-platform) into the platform driver and devices, the difference >> are quite small (see patches 1..3). Apart from the endianess issue, only >> the clock needs to be handled in a common way. As ARM already uses the >> clk interface, we found it straight-forward to implement it for the >> P1010, or more general for the 85xx, as well, instead of using an >> additional helper function. >> >>> I still think the clk / freq info should be in the device tree and handled in the driver and NOT arch/powerpc platform code. >> >> If I understand you correctly, you want the boot-loader to provide the >> relevant information by fixing up the device tree, which then can be >> handled arch-independently by the driver, right? > > Marc and Wolfgang, > > This is a very early swag at this which I worked up while driving home from dinner > this evening. It works with my current config, but that includes at least one > bogus patch. I will have to do more testing tomorrow. For now, it is something > to ponder. Yes, that's what Kumar is proposing. Fine for me with the P1010. It just needs some proper U-Boot implementation. > Thanks, > Robin > ------------------------------------------------------------------------ > > flexcan: Prefer device tree clock frequency if available. > > If our CAN device's device tree node has a clock-frequency property, > then use that value for the can devices clock frequency. If not, fall > back to asking the platform/mach code for the clock frequency associated > with the flexcan device. > > Too-early-to-be-signed-off-by: Robin Holt <holt-sJ/iWh9BUns@public.gmane.org> > > diff --git a/drivers/net/can/flexcan.c b/drivers/net/can/flexcan.c > index 662f832..d6a87c9 100644 > --- a/drivers/net/can/flexcan.c > +++ b/drivers/net/can/flexcan.c > @@ -33,6 +33,7 @@ > #include <linux/kernel.h> > #include <linux/list.h> > #include <linux/module.h> > +#include <linux/of.h> > #include <linux/platform_device.h> > > #define DRV_NAME "flexcan" > @@ -929,12 +930,25 @@ static int __devinit flexcan_probe(struct platform_device *pdev) > void __iomem *base; > resource_size_t mem_size; > int err, irq; > + u32 clock_freq = 0; > > - clk = clk_get(&pdev->dev, NULL); > - if (IS_ERR(clk)) { > - dev_err(&pdev->dev, "no clock defined\n"); > - err = PTR_ERR(clk); > - goto failed_clock; > + if (pdev->dev.of_node) { > + u32 *clock_freq_p; Should be: const u32 *clock_freq_p; > + > + clk = NULL; > + clock_freq_p = (u32 *)of_get_property(pdev->dev.of_node, "clock-frequency", NULL); No cast please. > + if (clock_freq_p) > + clock_freq = *clock_freq_p; > + } Wolfgang. ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH v11 4/5] powerpc: Add flexcan device support for p1010rdb. [not found] ` <4E42CB01.7030700-5Yr1BZd7O62+XT7JhA+gdA@public.gmane.org> 2011-08-11 3:56 ` Robin Holt @ 2011-08-11 4:46 ` Kumar Gala [not found] ` <634AB7A6-1CDA-41B3-8A5D-01F29EF01521-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org> 1 sibling, 1 reply; 20+ messages in thread From: Kumar Gala @ 2011-08-11 4:46 UTC (permalink / raw) To: Wolfgang Grandegger Cc: netdev-u79uwXL29TY76Z2rM5mHXA, U Bhaskar-B22300, socketcan-core-0fE9KPoRgkgATYTw5x5z8w, Marc Kleine-Budde, PPC list On Aug 10, 2011, at 1:16 PM, Wolfgang Grandegger wrote: > On 08/10/2011 07:01 PM, Kumar Gala wrote: >> >> On Aug 10, 2011, at 11:27 AM, Robin Holt wrote: >> >>> I added a simple clock source for the p1010rdb so the flexcan driver >>> could determine a clock frequency. The p1010 flexcan device only has >>> an oscillator of system bus frequency divided by 2. >>> >>> Signed-off-by: Robin Holt <holt-sJ/iWh9BUns@public.gmane.org> >>> Acked-by: Marc Kleine-Budde <mkl-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>, >>> Acked-by: Wolfgang Grandegger <wg-5Yr1BZd7O62+XT7JhA+gdA@public.gmane.org>, >>> Cc: U Bhaskar-B22300 <B22300-KZfg59tc24xl57MIdRCFDg@public.gmane.org> >>> Cc: socketcan-core-0fE9KPoRgkgATYTw5x5z8w@public.gmane.org, >>> Cc: netdev-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, >>> Cc: PPC list <linuxppc-dev-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org> >>> Cc: Kumar Gala <galak-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org> >>> --- >>> arch/powerpc/platforms/85xx/Kconfig | 2 + >>> arch/powerpc/platforms/85xx/Makefile | 2 + >>> arch/powerpc/platforms/85xx/clock.c | 52 ++++++++++++++++++++++++++++++++ >>> arch/powerpc/platforms/85xx/p1010rdb.c | 8 +++++ >>> 4 files changed, 64 insertions(+), 0 deletions(-) >>> create mode 100644 arch/powerpc/platforms/85xx/clock.c >> >> I dont understand how mpc85xx_clk_functions() ends up being associated with the frequency the flexcan is running at. > > The function mpc85xx_clk_get_rate() returns "fsl_get_sys_freq() / 2" for > Flexcan devices. > >> This either seems to global or I'm missing something. > > This patch extends the existing Flexcan platform driver for ARM for the > PowerPC using the device tree. Due to the nice integration of the device > tree (of-platform) into the platform driver and devices, the difference > are quite small (see patches 1..3). Apart from the endianess issue, only > the clock needs to be handled in a common way. As ARM already uses the > clk interface, we found it straight-forward to implement it for the > P1010, or more general for the 85xx, as well, instead of using an > additional helper function. I see, that. What concerns me is there are numerous clocks / frequencies that exist inside a MPC85xx/P1010 SOC. The code I'm seeing does NOT seem to do anything to relate this clock JUST to the flexcan. >> I still think the clk / freq info should be in the device tree and handled in the driver and NOT arch/powerpc platform code. > > If I understand you correctly, you want the boot-loader to provide the > relevant information by fixing up the device tree, which then can be > handled arch-independently by the driver, right? Yes, that is part of what I want. - k ^ permalink raw reply [flat|nested] 20+ messages in thread
[parent not found: <634AB7A6-1CDA-41B3-8A5D-01F29EF01521-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org>]
* Re: [PATCH v11 4/5] powerpc: Add flexcan device support for p1010rdb. [not found] ` <634AB7A6-1CDA-41B3-8A5D-01F29EF01521-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org> @ 2011-08-11 7:26 ` Wolfgang Grandegger 2011-08-11 10:42 ` Robin Holt 1 sibling, 0 replies; 20+ messages in thread From: Wolfgang Grandegger @ 2011-08-11 7:26 UTC (permalink / raw) To: Kumar Gala Cc: socketcan-core-0fE9KPoRgkgATYTw5x5z8w, netdev-u79uwXL29TY76Z2rM5mHXA, Marc Kleine-Budde, U Bhaskar-B22300, PPC list On 08/11/2011 06:46 AM, Kumar Gala wrote: > > On Aug 10, 2011, at 1:16 PM, Wolfgang Grandegger wrote: > >> On 08/10/2011 07:01 PM, Kumar Gala wrote: >>> >>> On Aug 10, 2011, at 11:27 AM, Robin Holt wrote: >>> >>>> I added a simple clock source for the p1010rdb so the flexcan driver >>>> could determine a clock frequency. The p1010 flexcan device only has >>>> an oscillator of system bus frequency divided by 2. >>>> >>>> Signed-off-by: Robin Holt <holt-sJ/iWh9BUns@public.gmane.org> >>>> Acked-by: Marc Kleine-Budde <mkl-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>, >>>> Acked-by: Wolfgang Grandegger <wg-5Yr1BZd7O62+XT7JhA+gdA@public.gmane.org>, >>>> Cc: U Bhaskar-B22300 <B22300-KZfg59tc24xl57MIdRCFDg@public.gmane.org> >>>> Cc: socketcan-core-0fE9KPoRgkgATYTw5x5z8w@public.gmane.org, >>>> Cc: netdev-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, >>>> Cc: PPC list <linuxppc-dev-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org> >>>> Cc: Kumar Gala <galak-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org> >>>> --- >>>> arch/powerpc/platforms/85xx/Kconfig | 2 + >>>> arch/powerpc/platforms/85xx/Makefile | 2 + >>>> arch/powerpc/platforms/85xx/clock.c | 52 ++++++++++++++++++++++++++++++++ >>>> arch/powerpc/platforms/85xx/p1010rdb.c | 8 +++++ >>>> 4 files changed, 64 insertions(+), 0 deletions(-) >>>> create mode 100644 arch/powerpc/platforms/85xx/clock.c >>> >>> I dont understand how mpc85xx_clk_functions() ends up being associated with the frequency the flexcan is running at. >> >> The function mpc85xx_clk_get_rate() returns "fsl_get_sys_freq() / 2" for >> Flexcan devices. >> >>> This either seems to global or I'm missing something. >> >> This patch extends the existing Flexcan platform driver for ARM for the >> PowerPC using the device tree. Due to the nice integration of the device >> tree (of-platform) into the platform driver and devices, the difference >> are quite small (see patches 1..3). Apart from the endianess issue, only >> the clock needs to be handled in a common way. As ARM already uses the >> clk interface, we found it straight-forward to implement it for the >> P1010, or more general for the 85xx, as well, instead of using an >> additional helper function. > > I see, that. What concerns me is there are numerous clocks / frequencies that exist inside a MPC85xx/P1010 SOC. The code I'm seeing does NOT seem to do anything to relate this clock JUST to the flexcan. The clk interface is not commonly used on PowerPC, I know. It's just to provide compatibility with ARM. An alternative would be to use some helper function. >>> I still think the clk / freq info should be in the device tree and handled in the driver and NOT arch/powerpc platform code. >> >> If I understand you correctly, you want the boot-loader to provide the >> relevant information by fixing up the device tree, which then can be >> handled arch-independently by the driver, right? > > Yes, that is part of what I want. This works fine if we just have *one* fixed clock source and frequency. When there are choices (source and divider) it does make sense to allow the user to select the frequency via DTS file entries for Linux. Then we need arch-specific code anyway (to set the relevant registers). Furthermore we rely on the boot-loader (the usual argument) which is not a problem for new boards (with new boot-loader), of course. Wolfgang. ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH v11 4/5] powerpc: Add flexcan device support for p1010rdb. [not found] ` <634AB7A6-1CDA-41B3-8A5D-01F29EF01521-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org> 2011-08-11 7:26 ` Wolfgang Grandegger @ 2011-08-11 10:42 ` Robin Holt [not found] ` <20110811104252.GC4926-sJ/iWh9BUns@public.gmane.org> 1 sibling, 1 reply; 20+ messages in thread From: Robin Holt @ 2011-08-11 10:42 UTC (permalink / raw) To: Kumar Gala Cc: netdev-u79uwXL29TY76Z2rM5mHXA, U Bhaskar-B22300, socketcan-core-0fE9KPoRgkgATYTw5x5z8w, PPC list, Wolfgang Grandegger On Wed, Aug 10, 2011 at 11:46:27PM -0500, Kumar Gala wrote: > > On Aug 10, 2011, at 1:16 PM, Wolfgang Grandegger wrote: > > > On 08/10/2011 07:01 PM, Kumar Gala wrote: > >> > >> On Aug 10, 2011, at 11:27 AM, Robin Holt wrote: > >> > >>> I added a simple clock source for the p1010rdb so the flexcan driver > >>> could determine a clock frequency. The p1010 flexcan device only has > >>> an oscillator of system bus frequency divided by 2. > >>> > >>> Signed-off-by: Robin Holt <holt-sJ/iWh9BUns@public.gmane.org> > >>> Acked-by: Marc Kleine-Budde <mkl-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>, > >>> Acked-by: Wolfgang Grandegger <wg-5Yr1BZd7O62+XT7JhA+gdA@public.gmane.org>, > >>> Cc: U Bhaskar-B22300 <B22300-KZfg59tc24xl57MIdRCFDg@public.gmane.org> > >>> Cc: socketcan-core-0fE9KPoRgkgATYTw5x5z8w@public.gmane.org, > >>> Cc: netdev-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, > >>> Cc: PPC list <linuxppc-dev-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org> > >>> Cc: Kumar Gala <galak-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org> > >>> --- > >>> arch/powerpc/platforms/85xx/Kconfig | 2 + > >>> arch/powerpc/platforms/85xx/Makefile | 2 + > >>> arch/powerpc/platforms/85xx/clock.c | 52 ++++++++++++++++++++++++++++++++ > >>> arch/powerpc/platforms/85xx/p1010rdb.c | 8 +++++ > >>> 4 files changed, 64 insertions(+), 0 deletions(-) > >>> create mode 100644 arch/powerpc/platforms/85xx/clock.c > >> > >> I dont understand how mpc85xx_clk_functions() ends up being associated with the frequency the flexcan is running at. > > > > The function mpc85xx_clk_get_rate() returns "fsl_get_sys_freq() / 2" for > > Flexcan devices. > > > >> This either seems to global or I'm missing something. > > > > This patch extends the existing Flexcan platform driver for ARM for the > > PowerPC using the device tree. Due to the nice integration of the device > > tree (of-platform) into the platform driver and devices, the difference > > are quite small (see patches 1..3). Apart from the endianess issue, only > > the clock needs to be handled in a common way. As ARM already uses the > > clk interface, we found it straight-forward to implement it for the > > P1010, or more general for the 85xx, as well, instead of using an > > additional helper function. > > I see, that. What concerns me is there are numerous clocks / > frequencies that exist inside a MPC85xx/P1010 SOC. The code I'm seeing > does NOT seem to do anything to relate this clock JUST to the flexcan. if (!dev->of_node || !of_device_is_compatible(dev->of_node, "fsl,flexcan")) return ERR_PTR(-ENOENT); That should relate it just to flexcan, right? Plus it has the added benefit of being a baby-step in the direction of implementing a clkdev type thing for powerpc which did look fairly slick to me, but I may be confused. It sounds like Wolfgang is defering to you. Give it an honest evaluation and tell me which direction you would like me to go. I don't have a strong preference either way. The alternative I gave to Wolfgang of using a flexcan property to avoid needing any clk_get_rate seems fairly hackish at this point, but I have had more time to get used to the 'hack in a 85xx clock' method. Thanks, Robin ^ permalink raw reply [flat|nested] 20+ messages in thread
[parent not found: <20110811104252.GC4926-sJ/iWh9BUns@public.gmane.org>]
* Re: [PATCH v11 4/5] powerpc: Add flexcan device support for p1010rdb. [not found] ` <20110811104252.GC4926-sJ/iWh9BUns@public.gmane.org> @ 2011-08-11 14:17 ` Kumar Gala 0 siblings, 0 replies; 20+ messages in thread From: Kumar Gala @ 2011-08-11 14:17 UTC (permalink / raw) To: Robin Holt Cc: socketcan-core-0fE9KPoRgkgATYTw5x5z8w, netdev-u79uwXL29TY76Z2rM5mHXA, U Bhaskar-B22300, PPC list, Wolfgang Grandegger On Aug 11, 2011, at 5:42 AM, Robin Holt wrote: > On Wed, Aug 10, 2011 at 11:46:27PM -0500, Kumar Gala wrote: >> >> On Aug 10, 2011, at 1:16 PM, Wolfgang Grandegger wrote: >> >>> On 08/10/2011 07:01 PM, Kumar Gala wrote: >>>> >>>> On Aug 10, 2011, at 11:27 AM, Robin Holt wrote: >>>> >>>>> I added a simple clock source for the p1010rdb so the flexcan driver >>>>> could determine a clock frequency. The p1010 flexcan device only has >>>>> an oscillator of system bus frequency divided by 2. >>>>> >>>>> Signed-off-by: Robin Holt <holt-sJ/iWh9BUns@public.gmane.org> >>>>> Acked-by: Marc Kleine-Budde <mkl-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>, >>>>> Acked-by: Wolfgang Grandegger <wg-5Yr1BZd7O62+XT7JhA+gdA@public.gmane.org>, >>>>> Cc: U Bhaskar-B22300 <B22300-KZfg59tc24xl57MIdRCFDg@public.gmane.org> >>>>> Cc: socketcan-core-0fE9KPoRgkgATYTw5x5z8w@public.gmane.org, >>>>> Cc: netdev-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, >>>>> Cc: PPC list <linuxppc-dev-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org> >>>>> Cc: Kumar Gala <galak-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org> >>>>> --- >>>>> arch/powerpc/platforms/85xx/Kconfig | 2 + >>>>> arch/powerpc/platforms/85xx/Makefile | 2 + >>>>> arch/powerpc/platforms/85xx/clock.c | 52 ++++++++++++++++++++++++++++++++ >>>>> arch/powerpc/platforms/85xx/p1010rdb.c | 8 +++++ >>>>> 4 files changed, 64 insertions(+), 0 deletions(-) >>>>> create mode 100644 arch/powerpc/platforms/85xx/clock.c >>>> >>>> I dont understand how mpc85xx_clk_functions() ends up being associated with the frequency the flexcan is running at. >>> >>> The function mpc85xx_clk_get_rate() returns "fsl_get_sys_freq() / 2" for >>> Flexcan devices. >>> >>>> This either seems to global or I'm missing something. >>> >>> This patch extends the existing Flexcan platform driver for ARM for the >>> PowerPC using the device tree. Due to the nice integration of the device >>> tree (of-platform) into the platform driver and devices, the difference >>> are quite small (see patches 1..3). Apart from the endianess issue, only >>> the clock needs to be handled in a common way. As ARM already uses the >>> clk interface, we found it straight-forward to implement it for the >>> P1010, or more general for the 85xx, as well, instead of using an >>> additional helper function. >> >> I see, that. What concerns me is there are numerous clocks / >> frequencies that exist inside a MPC85xx/P1010 SOC. The code I'm seeing >> does NOT seem to do anything to relate this clock JUST to the flexcan. > > if (!dev->of_node || > !of_device_is_compatible(dev->of_node, "fsl,flexcan")) > return ERR_PTR(-ENOENT); > > That should relate it just to flexcan, right? Plus it has the added > benefit of being a baby-step in the direction of implementing a clkdev > type thing for powerpc which did look fairly slick to me, but I may > be confused. > > It sounds like Wolfgang is defering to you. Give it an honest evaluation > and tell me which direction you would like me to go. I don't have a > strong preference either way. The alternative I gave to Wolfgang of > using a flexcan property to avoid needing any clk_get_rate seems fairly > hackish at this point, but I have had more time to get used to the > 'hack in a 85xx clock' method. For some time we've been adding 'clock-frequency' nodes in the device tree to abstract having to know this headache in the kernel and adding a bunch of SoC specific code all the time. So pushing this to the firmware is exactly where we want it for FSL PPC SoCs. We need to make sure the device tree binding has details on a 'clock-frequency' property. - k ^ permalink raw reply [flat|nested] 20+ messages in thread
* [PATCH v11 5/5] powerpc: Fix up fsl-flexcan device tree binding. [not found] ` <1312993670-23999-1-git-send-email-holt-sJ/iWh9BUns@public.gmane.org> ` (2 preceding siblings ...) 2011-08-10 16:27 ` [PATCH v11 4/5] powerpc: Add flexcan device support for p1010rdb Robin Holt @ 2011-08-10 16:27 ` Robin Holt 2011-08-10 16:56 ` Scott Wood 3 siblings, 1 reply; 20+ messages in thread From: Robin Holt @ 2011-08-10 16:27 UTC (permalink / raw) To: Robin Holt, Kumar Gala, Wolfgang Grandegger, U Bhaskar-B22300, Scott Wood Cc: , netdev-u79uwXL29TY76Z2rM5mHXA, devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ, socketcan-core-0fE9KPoRgkgATYTw5x5z8w, Marc Kleine-Budde, PPC list 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, there is not external divider beyond what the driver already works with, and the clock source can not be selected. Signed-off-by: Robin Holt <holt-sJ/iWh9BUns@public.gmane.org> Acked-by: Marc Kleine-Budde <mkl-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>, To: Wolfgang Grandegger <wg-5Yr1BZd7O62+XT7JhA+gdA@public.gmane.org>, To: U Bhaskar-B22300 <B22300-KZfg59tc24xl57MIdRCFDg@public.gmane.org> To: Scott Wood <scottwood-KZfg59tc24xl57MIdRCFDg@public.gmane.org> To: Grant Likely <grant.likely-s3s/WqlpOiPyB63q8FvJNQ@public.gmane.org> To: Kumar Gala <galak-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org> Cc: socketcan-core-0fE9KPoRgkgATYTw5x5z8w@public.gmane.org, Cc: netdev-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Cc: PPC list <linuxppc-dev-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org> Cc: devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org --- .../devicetree/bindings/net/can/fsl-flexcan.txt | 70 ++++---------------- arch/powerpc/boot/dts/p1010rdb.dts | 10 +-- arch/powerpc/boot/dts/p1010si.dtsi | 10 +-- 3 files changed, 19 insertions(+), 71 deletions(-) diff --git a/Documentation/devicetree/bindings/net/can/fsl-flexcan.txt b/Documentation/devicetree/bindings/net/can/fsl-flexcan.txt index 1a729f0..869f4ca 100644 --- a/Documentation/devicetree/bindings/net/can/fsl-flexcan.txt +++ b/Documentation/devicetree/bindings/net/can/fsl-flexcan.txt @@ -1,61 +1,17 @@ -CAN Device Tree Bindings ------------------------- -2011 Freescale Semiconductor, Inc. +Flexcan CAN contoller on Freescale's ARM and PowerPC processors -fsl,flexcan-v1.0 nodes ------------------------ -In addition to the required compatible-, reg- and interrupt-properties, you can -also specify which clock source shall be used for the controller. +Required properties: -CPI Clock- Can Protocol Interface Clock - This CLK_SRC bit of CTRL(control register) selects the clock source to - the CAN Protocol Interface(CPI) to be either the peripheral clock - (driven by the PLL) or the crystal oscillator clock. The selected clock - is the one fed to the prescaler to generate the Serial Clock (Sclock). - The PRESDIV field of CTRL(control register) controls a prescaler that - generates the Serial Clock (Sclock), whose period defines the - time quantum used to compose the CAN waveform. +- compatible : Should be "fsl,flexcan" and optionally + "fsl,flexcan-<processor>" +- reg : Offset and length of the register set for this device +- interrupts : Interrupt tuple for this device -Can Engine Clock Source - There are two sources for CAN clock - - Platform Clock It represents the bus clock - - Oscillator Clock +Example: - Peripheral Clock (PLL) - -------------- - | - --------- ------------- - | |CPI Clock | Prescaler | Sclock - | |---------------->| (1.. 256) |------------> - --------- ------------- - | | - -------------- ---------------------CLK_SRC - Oscillator Clock - -- fsl,flexcan-clock-source : CAN Engine Clock Source.This property selects - the peripheral clock. PLL clock is fed to the - prescaler to generate the Serial Clock (Sclock). - Valid values are "oscillator" and "platform" - "oscillator": CAN engine clock source is oscillator clock. - "platform" The CAN engine clock source is the bus clock - (platform clock). - -- fsl,flexcan-clock-divider : for the reference and system clock, an additional - clock divider can be specified. -- clock-frequency: frequency required to calculate the bitrate for FlexCAN. - -Note: - - v1.0 of flexcan-v1.0 represent the IP block version for P1010 SOC. - - P1010 does not have oscillator as the Clock Source.So the default - Clock Source is platform clock. -Examples: - - can0@1c000 { - compatible = "fsl,flexcan-v1.0"; - reg = <0x1c000 0x1000>; - interrupts = <48 0x2>; - interrupt-parent = <&mpic>; - fsl,flexcan-clock-source = "platform"; - fsl,flexcan-clock-divider = <2>; - clock-frequency = <fixed by u-boot>; - }; + can@1c000 { + compatible = "fsl,p1010-flexcan", "fsl,flexcan"; + reg = <0x1c000 0x1000>; + interrupts = <48 0x2>; + interrupt-parent = <&mpic>; + }; diff --git a/arch/powerpc/boot/dts/p1010rdb.dts b/arch/powerpc/boot/dts/p1010rdb.dts index 6b33b73..d6c669c 100644 --- a/arch/powerpc/boot/dts/p1010rdb.dts +++ b/arch/powerpc/boot/dts/p1010rdb.dts @@ -23,6 +23,8 @@ ethernet2 = &enet2; pci0 = &pci0; pci1 = &pci1; + can0 = &can0; + can1 = &can1; }; memory { @@ -169,14 +171,6 @@ }; }; - can0@1c000 { - fsl,flexcan-clock-source = "platform"; - }; - - can1@1d000 { - fsl,flexcan-clock-source = "platform"; - }; - usb@22000 { phy_type = "utmi"; }; diff --git a/arch/powerpc/boot/dts/p1010si.dtsi b/arch/powerpc/boot/dts/p1010si.dtsi index 7f51104..f00076b 100644 --- a/arch/powerpc/boot/dts/p1010si.dtsi +++ b/arch/powerpc/boot/dts/p1010si.dtsi @@ -140,20 +140,18 @@ interrupt-parent = <&mpic>; }; - can0@1c000 { - compatible = "fsl,flexcan-v1.0"; + can0: can@1c000 { + compatible = "fsl,p1010-flexcan", "fsl,flexcan"; reg = <0x1c000 0x1000>; interrupts = <48 0x2>; interrupt-parent = <&mpic>; - fsl,flexcan-clock-divider = <2>; }; - can1@1d000 { - compatible = "fsl,flexcan-v1.0"; + can1: can@1d000 { + compatible = "fsl,p1010-flexcan", "fsl,flexcan"; reg = <0x1d000 0x1000>; interrupts = <61 0x2>; interrupt-parent = <&mpic>; - fsl,flexcan-clock-divider = <2>; }; L2: l2-cache-controller@20000 { -- 1.7.2.1 ^ permalink raw reply related [flat|nested] 20+ messages in thread
* Re: [PATCH v11 5/5] powerpc: Fix up fsl-flexcan device tree binding. 2011-08-10 16:27 ` [PATCH v11 5/5] powerpc: Fix up fsl-flexcan device tree binding Robin Holt @ 2011-08-10 16:56 ` Scott Wood [not found] ` <4E42B83C.2040705-KZfg59tc24xl57MIdRCFDg@public.gmane.org> 0 siblings, 1 reply; 20+ messages in thread From: Scott Wood @ 2011-08-10 16:56 UTC (permalink / raw) To: Robin Holt Cc: Kumar Gala, Wolfgang Grandegger, U Bhaskar-B22300, Grant Likely, Marc Kleine-Budde, socketcan-core, netdev, PPC list, devicetree-discuss On 08/10/2011 11:27 AM, Robin Holt wrote: > -CPI Clock- Can Protocol Interface Clock > - This CLK_SRC bit of CTRL(control register) selects the clock source to > - the CAN Protocol Interface(CPI) to be either the peripheral clock > - (driven by the PLL) or the crystal oscillator clock. The selected clock > - is the one fed to the prescaler to generate the Serial Clock (Sclock). > - The PRESDIV field of CTRL(control register) controls a prescaler that > - generates the Serial Clock (Sclock), whose period defines the > - time quantum used to compose the CAN waveform. > +- compatible : Should be "fsl,flexcan" and optionally > + "fsl,flexcan-<processor>" fsl,<processor>-flexcan, and it should not be optional, and should come before "fsl,flexcan". Also may want to list fsl,p1010-rdb as a "canonical compatible" for anything which is backwards compatible with p1010's implementation. -Scott ^ permalink raw reply [flat|nested] 20+ messages in thread
[parent not found: <4E42B83C.2040705-KZfg59tc24xl57MIdRCFDg@public.gmane.org>]
* Re: [PATCH v11 5/5] powerpc: Fix up fsl-flexcan device tree binding. [not found] ` <4E42B83C.2040705-KZfg59tc24xl57MIdRCFDg@public.gmane.org> @ 2011-08-10 17:19 ` Robin Holt 2011-08-10 17:36 ` Scott Wood 0 siblings, 1 reply; 20+ messages in thread From: Robin Holt @ 2011-08-10 17:19 UTC (permalink / raw) To: Scott Wood Cc: socketcan-core-0fE9KPoRgkgATYTw5x5z8w, netdev-u79uwXL29TY76Z2rM5mHXA, devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ, U Bhaskar-B22300, Kumar Gala, Grant Likely, Marc Kleine-Budde, PPC list, Wolfgang Grandegger On Wed, Aug 10, 2011 at 11:56:28AM -0500, Scott Wood wrote: > On 08/10/2011 11:27 AM, Robin Holt wrote: > > -CPI Clock- Can Protocol Interface Clock > > - This CLK_SRC bit of CTRL(control register) selects the clock source to > > - the CAN Protocol Interface(CPI) to be either the peripheral clock > > - (driven by the PLL) or the crystal oscillator clock. The selected clock > > - is the one fed to the prescaler to generate the Serial Clock (Sclock). > > - The PRESDIV field of CTRL(control register) controls a prescaler that > > - generates the Serial Clock (Sclock), whose period defines the > > - time quantum used to compose the CAN waveform. > > +- compatible : Should be "fsl,flexcan" and optionally > > + "fsl,flexcan-<processor>" > > fsl,<processor>-flexcan, and it should not be optional, and should come > before "fsl,flexcan". > > Also may want to list fsl,p1010-rdb as a "canonical compatible" for > anything which is backwards compatible with p1010's implementation. How do I specify 'canonical compatible'? What would be the use of it in that implementation? Robin ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH v11 5/5] powerpc: Fix up fsl-flexcan device tree binding. 2011-08-10 17:19 ` Robin Holt @ 2011-08-10 17:36 ` Scott Wood [not found] ` <4E42C196.7030708-KZfg59tc24xl57MIdRCFDg@public.gmane.org> 0 siblings, 1 reply; 20+ messages in thread From: Scott Wood @ 2011-08-10 17:36 UTC (permalink / raw) To: Robin Holt Cc: Kumar Gala, Wolfgang Grandegger, U Bhaskar-B22300, Grant Likely, Marc Kleine-Budde, socketcan-core, netdev, PPC list, devicetree-discuss On 08/10/2011 12:19 PM, Robin Holt wrote: > On Wed, Aug 10, 2011 at 11:56:28AM -0500, Scott Wood wrote: >> On 08/10/2011 11:27 AM, Robin Holt wrote: >>> -CPI Clock- Can Protocol Interface Clock >>> - This CLK_SRC bit of CTRL(control register) selects the clock source to >>> - the CAN Protocol Interface(CPI) to be either the peripheral clock >>> - (driven by the PLL) or the crystal oscillator clock. The selected clock >>> - is the one fed to the prescaler to generate the Serial Clock (Sclock). >>> - The PRESDIV field of CTRL(control register) controls a prescaler that >>> - generates the Serial Clock (Sclock), whose period defines the >>> - time quantum used to compose the CAN waveform. >>> +- compatible : Should be "fsl,flexcan" and optionally >>> + "fsl,flexcan-<processor>" >> >> fsl,<processor>-flexcan, and it should not be optional, and should come >> before "fsl,flexcan". >> >> Also may want to list fsl,p1010-rdb as a "canonical compatible" for >> anything which is backwards compatible with p1010's implementation. > > How do I specify 'canonical compatible'? Something like: compatible: Should be "fsl,<processor>-flexcan" and "fsl,flexcan". An implementation should also claim any of the following compatibles that it is fully backwards compatible with: - fsl,p1010-rdb > What would be the use of it in that implementation? It limits the number of compatibles a driver has to care about, so you don't need a huge ID table just to be able to figure out whether this is a p1010-style flexcan or ARM-style. -Scott ^ permalink raw reply [flat|nested] 20+ messages in thread
[parent not found: <4E42C196.7030708-KZfg59tc24xl57MIdRCFDg@public.gmane.org>]
* Re: [PATCH v11 5/5] powerpc: Fix up fsl-flexcan device tree binding. [not found] ` <4E42C196.7030708-KZfg59tc24xl57MIdRCFDg@public.gmane.org> @ 2011-08-10 18:30 ` Robin Holt 2011-08-10 18:40 ` Scott Wood 0 siblings, 1 reply; 20+ messages in thread From: Robin Holt @ 2011-08-10 18:30 UTC (permalink / raw) To: Scott Wood Cc: socketcan-core-0fE9KPoRgkgATYTw5x5z8w, netdev-u79uwXL29TY76Z2rM5mHXA, devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ, U Bhaskar-B22300, Kumar Gala, Grant Likely, Marc Kleine-Budde, PPC list, Wolfgang Grandegger On Wed, Aug 10, 2011 at 12:36:22PM -0500, Scott Wood wrote: > On 08/10/2011 12:19 PM, Robin Holt wrote: > > On Wed, Aug 10, 2011 at 11:56:28AM -0500, Scott Wood wrote: > >> On 08/10/2011 11:27 AM, Robin Holt wrote: > >>> -CPI Clock- Can Protocol Interface Clock > >>> - This CLK_SRC bit of CTRL(control register) selects the clock source to > >>> - the CAN Protocol Interface(CPI) to be either the peripheral clock > >>> - (driven by the PLL) or the crystal oscillator clock. The selected clock > >>> - is the one fed to the prescaler to generate the Serial Clock (Sclock). > >>> - The PRESDIV field of CTRL(control register) controls a prescaler that > >>> - generates the Serial Clock (Sclock), whose period defines the > >>> - time quantum used to compose the CAN waveform. > >>> +- compatible : Should be "fsl,flexcan" and optionally > >>> + "fsl,flexcan-<processor>" > >> > >> fsl,<processor>-flexcan, and it should not be optional, and should come > >> before "fsl,flexcan". > >> > >> Also may want to list fsl,p1010-rdb as a "canonical compatible" for > >> anything which is backwards compatible with p1010's implementation. > > > > How do I specify 'canonical compatible'? > > Something like: > > compatible: Should be "fsl,<processor>-flexcan" and "fsl,flexcan". > > An implementation should also claim any of the following compatibles > that it is fully backwards compatible with: > > - fsl,p1010-rdb I am so confused. fsl,p1010-flexcan refers, in my mind at least, to a particular chiplet on the p1010 freescale processor. fsl,p1010-rdb would mean nothing to me as that is a p1010 processor with two flexcan chiplets wired to a pair of DB-9 jacks. For the driver, what additional information is being conveyed? Let's cut to the chase. Here is what I have after incorporating your earlier comment about the compatible line. Please mark this up to exactly what you are asking for. Thanks, Robin ------------------------------------------------------------------------ Flexcan CAN contoller on Freescale's ARM and PowerPC processors Required properties: - compatible : Should be "fsl,<processor>-flexcan" and "fsl,flexcan" - reg : Offset and length of the register set for this device - interrupts : Interrupt tuple for this device Example: can@1c000 { compatible = "fsl,p1010-flexcan", "fsl,flexcan"; reg = <0x1c000 0x1000>; interrupts = <48 0x2>; interrupt-parent = <&mpic>; }; ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH v11 5/5] powerpc: Fix up fsl-flexcan device tree binding. 2011-08-10 18:30 ` Robin Holt @ 2011-08-10 18:40 ` Scott Wood [not found] ` <4E42D09E.4080405-KZfg59tc24xl57MIdRCFDg@public.gmane.org> 0 siblings, 1 reply; 20+ messages in thread From: Scott Wood @ 2011-08-10 18:40 UTC (permalink / raw) To: Robin Holt Cc: Kumar Gala, Wolfgang Grandegger, U Bhaskar-B22300, Grant Likely, Marc Kleine-Budde, socketcan-core, netdev, PPC list, devicetree-discuss On 08/10/2011 01:30 PM, Robin Holt wrote: > On Wed, Aug 10, 2011 at 12:36:22PM -0500, Scott Wood wrote: >> On 08/10/2011 12:19 PM, Robin Holt wrote: >>> On Wed, Aug 10, 2011 at 11:56:28AM -0500, Scott Wood wrote: >>>> Also may want to list fsl,p1010-rdb as a "canonical compatible" for >>>> anything which is backwards compatible with p1010's implementation. >>> >>> How do I specify 'canonical compatible'? >> >> Something like: >> >> compatible: Should be "fsl,<processor>-flexcan" and "fsl,flexcan". >> >> An implementation should also claim any of the following compatibles >> that it is fully backwards compatible with: >> >> - fsl,p1010-rdb Gah, I don't know how "rdb" replaced "flexcan" in the above. Sorry for any confusion. > I am so confused. fsl,p1010-flexcan refers, in my mind at least, to > a particular chiplet on the p1010 freescale processor. It refers to a particular version of the flexcan logic, for which the hardware doc people weren't kind enough to give us a public version number. It has been common and recommended practice in such cases, when there are multiple chips containing the same device, to pick a canonical chip (such as the first one to have the device or to be supported) and have others claim compatibility with it. > fsl,p1010-rdb > would mean nothing to me as that is a p1010 processor with two flexcan > chiplets wired to a pair of DB-9 jacks. For the driver, what additional > information is being conveyed? The programming model of the flexcan chiplet. > Let's cut to the chase. Here is what I have after incorporating your > earlier comment about the compatible line. Please mark this up to > exactly what you are asking for. > > Thanks, > Robin > ------------------------------------------------------------------------ > Flexcan CAN contoller on Freescale's ARM and PowerPC processors > > Required properties: > > - compatible : Should be "fsl,<processor>-flexcan" and "fsl,flexcan" An implementation should also claim any of the following compatibles that it is fully backwards compatible with: - fsl,p1010-flexcan > - reg : Offset and length of the register set for this device > - interrupts : Interrupt tuple for this device > > Example: > > can@1c000 { > compatible = "fsl,p1010-flexcan", "fsl,flexcan"; > reg = <0x1c000 0x1000>; > interrupts = <48 0x2>; > interrupt-parent = <&mpic>; > }; > -Scott ^ permalink raw reply [flat|nested] 20+ messages in thread
[parent not found: <4E42D09E.4080405-KZfg59tc24xl57MIdRCFDg@public.gmane.org>]
* Re: [PATCH v11 5/5] powerpc: Fix up fsl-flexcan device tree binding. [not found] ` <4E42D09E.4080405-KZfg59tc24xl57MIdRCFDg@public.gmane.org> @ 2011-08-10 18:45 ` Robin Holt 0 siblings, 0 replies; 20+ messages in thread From: Robin Holt @ 2011-08-10 18:45 UTC (permalink / raw) To: Scott Wood Cc: socketcan-core-0fE9KPoRgkgATYTw5x5z8w, netdev-u79uwXL29TY76Z2rM5mHXA, devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ, U Bhaskar-B22300, Kumar Gala, Grant Likely, Marc Kleine-Budde, PPC list, Wolfgang Grandegger On Wed, Aug 10, 2011 at 01:40:30PM -0500, Scott Wood wrote: > On 08/10/2011 01:30 PM, Robin Holt wrote: > > On Wed, Aug 10, 2011 at 12:36:22PM -0500, Scott Wood wrote: > >> On 08/10/2011 12:19 PM, Robin Holt wrote: > >>> On Wed, Aug 10, 2011 at 11:56:28AM -0500, Scott Wood wrote: > >>>> Also may want to list fsl,p1010-rdb as a "canonical compatible" for > >>>> anything which is backwards compatible with p1010's implementation. > >>> > >>> How do I specify 'canonical compatible'? > >> > >> Something like: > >> > >> compatible: Should be "fsl,<processor>-flexcan" and "fsl,flexcan". > >> > >> An implementation should also claim any of the following compatibles > >> that it is fully backwards compatible with: > >> > >> - fsl,p1010-rdb > > Gah, I don't know how "rdb" replaced "flexcan" in the above. Sorry for > any confusion. > > > I am so confused. fsl,p1010-flexcan refers, in my mind at least, to > > a particular chiplet on the p1010 freescale processor. > > It refers to a particular version of the flexcan logic, for which the > hardware doc people weren't kind enough to give us a public version number. > > It has been common and recommended practice in such cases, when there > are multiple chips containing the same device, to pick a canonical chip > (such as the first one to have the device or to be supported) and have > others claim compatibility with it. > > > fsl,p1010-rdb > > would mean nothing to me as that is a p1010 processor with two flexcan > > chiplets wired to a pair of DB-9 jacks. For the driver, what additional > > information is being conveyed? > > The programming model of the flexcan chiplet. > > > Let's cut to the chase. Here is what I have after incorporating your > > earlier comment about the compatible line. Please mark this up to > > exactly what you are asking for. > > > > Thanks, > > Robin > > ------------------------------------------------------------------------ > > Flexcan CAN contoller on Freescale's ARM and PowerPC processors > > > > Required properties: > > > > - compatible : Should be "fsl,<processor>-flexcan" and "fsl,flexcan" > > An implementation should also claim any of the following compatibles > that it is fully backwards compatible with: > > - fsl,p1010-flexcan Ah, there is my confusion. I did not realize you were saying the entire preceeding 4 lines should be included. I thought you were making a comment which I did not understand. Thank you for your patience with my ignorance, Robin ^ permalink raw reply [flat|nested] 20+ messages in thread
end of thread, other threads:[~2011-08-11 14:17 UTC | newest] Thread overview: 20+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2011-08-10 16:27 [PATCH v11 0/5] flexcan/powerpc: Add support for powerpc flexcan (freescale p1010) Robin Holt 2011-08-10 16:27 ` [PATCH v11 2/5] flexcan: Abstract off read/write for big/little endian Robin Holt [not found] ` <1312993670-23999-1-git-send-email-holt-sJ/iWh9BUns@public.gmane.org> 2011-08-10 16:27 ` [PATCH v11 1/5] flexcan: Remove #include <mach/clock.h> Robin Holt 2011-08-10 16:27 ` [PATCH v11 3/5] flexcan: Add of_match to platform_device definition Robin Holt 2011-08-10 16:27 ` [PATCH v11 4/5] powerpc: Add flexcan device support for p1010rdb Robin Holt [not found] ` <1312993670-23999-5-git-send-email-holt-sJ/iWh9BUns@public.gmane.org> 2011-08-10 17:01 ` Kumar Gala [not found] ` <8E5FA886-038D-4DF4-8A54-DD60188A21A2-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org> 2011-08-10 18:16 ` Wolfgang Grandegger [not found] ` <4E42CB01.7030700-5Yr1BZd7O62+XT7JhA+gdA@public.gmane.org> 2011-08-11 3:56 ` Robin Holt [not found] ` <20110811035620.GB4926-sJ/iWh9BUns@public.gmane.org> 2011-08-11 7:35 ` Wolfgang Grandegger 2011-08-11 4:46 ` Kumar Gala [not found] ` <634AB7A6-1CDA-41B3-8A5D-01F29EF01521-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org> 2011-08-11 7:26 ` Wolfgang Grandegger 2011-08-11 10:42 ` Robin Holt [not found] ` <20110811104252.GC4926-sJ/iWh9BUns@public.gmane.org> 2011-08-11 14:17 ` Kumar Gala 2011-08-10 16:27 ` [PATCH v11 5/5] powerpc: Fix up fsl-flexcan device tree binding Robin Holt 2011-08-10 16:56 ` Scott Wood [not found] ` <4E42B83C.2040705-KZfg59tc24xl57MIdRCFDg@public.gmane.org> 2011-08-10 17:19 ` Robin Holt 2011-08-10 17:36 ` Scott Wood [not found] ` <4E42C196.7030708-KZfg59tc24xl57MIdRCFDg@public.gmane.org> 2011-08-10 18:30 ` Robin Holt 2011-08-10 18:40 ` Scott Wood [not found] ` <4E42D09E.4080405-KZfg59tc24xl57MIdRCFDg@public.gmane.org> 2011-08-10 18:45 ` Robin Holt
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).