Re: [PATCH v3 2/2] can: m_can: add Bosch M_CAN controller support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On 07/14/2014 01:54 PM, Dong Aisheng wrote:
On Mon, Jul 14, 2014 at 01:05:55PM +0530, Varka Bhadram wrote:
On 07/14/2014 12:51 PM, Dong Aisheng wrote:
+    dev_dbg(&pdev->dev, "mram_base %p sidf 0x%x %d xidf 0x%x %d rxf0
0x%x %d rxf1 0x%x %d rxb 0x%x %d txe 0x%x %d txb 0x%x %d\n",
+        priv->mram_base,
+        priv->mcfg[MRAM_SIDF].off, priv->mcfg[MRAM_SIDF].num,
+        priv->mcfg[MRAM_XIDF].off, priv->mcfg[MRAM_XIDF].num,
+        priv->mcfg[MRAM_RXF0].off, priv->mcfg[MRAM_RXF0].num,
+        priv->mcfg[MRAM_RXF1].off, priv->mcfg[MRAM_RXF1].num,
+        priv->mcfg[MRAM_RXB].off, priv->mcfg[MRAM_RXB].num,
+        priv->mcfg[MRAM_TXE].off, priv->mcfg[MRAM_TXE].num,
+        priv->mcfg[MRAM_TXB].off, priv->mcfg[MRAM_TXB].num);
+
dev_dbg() will insert the new lines in b/w. It wont print the values as
you expected.
Check this by enabling debug ...
What do you mean by b/w?
You are expecting the data to be print in format like:
pdev->dev/name: mram_base %p sidf 0x%x %d xidf 0x%x %d rxf0 0x%x %d rxf1 0x%x %d rxb 0x%x %d txe 0x%x %d txb 0x%x %d

But when we use the dev_dbg()/pr_debug()... It will put data like:
pdev->dev/name: mram_base %p sidf 0x%x
0x%x %d rxf0 0x%x
rxf1 0x%x %d rxb
....

check this by enable DEBUG...

My test showed the format is:
root@imx6qdlsolo:~# uname -a
Linux imx6qdlsolo 3.16.0-rc2-next-20140627-00006-gd55dd62-dirty #373 SMP Fri Jul 11 18:12:31 CST 2014 armv7l GNU/Linux
root@imx6qdlsolo:~# dmesg | grep m_can
m_can 20e8000.can: mram_base c0990000 sidf 0x0 0 xidf 0x0 0 rxf0 0x0 32 rxf1 0x200 0 rxb 0x200 0 txe 0x200 0 txb 0x200 1
m_can 20e8000.can: m_can device registered (regs=c0988000, irq=146)
m_can 20f0000.can: mram_base c09a0000 sidf 0x400 0 xidf 0x400 0 rxf0 0x400 32 rxf1 0x600 0 rxb 0x600 0 txe 0x600 0 txb 0x600 1
m_can 20f0000.can: m_can device registered (regs=c0998000, irq=147)

I think you got the expected result.

I faced the above problem when i use pr_debug(). Can you check with this....?

It works too.
root@imx6qdlsolo:~# dmesg | grep mram
mram_base c0990000 sidf 0x0 0 xidf 0x0 0 rxf0 0x0 32 rxf1 0x200 0 rxb 0x200 0 txe 0x200 0 txb 0x200 1
mram_base c09a0000 sidf 0x400 0 xidf 0x400 0 rxf0 0x400 32 rxf1 0x600 0 rxb 0x600 0 txe 0x600 0 txb 0x600 1

Ohooooo... I got the problem because i used pr_debug in a 'for' loop.

Any way thanks.

This is good. Thanks for clarification.

Regards
Dong Aisheng

--
Regards,
Varka Bhadram.

Regards
Dong Aisheng


--
Regards,
Varka Bhadram.

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux