Re: [PATCH v2 05/11] ARM: dt: tegra30: Add device node for APB MISC

From: Prashant Gaikwad
Date: Thu Jan 03 2013 - 01:11:10 EST


On Thursday 03 January 2013 03:30 AM, Stephen Warren wrote:
On 12/27/2012 07:47 AM, Prashant Gaikwad wrote:
APB misc contains multiple registers required by different modules
such as CAR.
I don't see a DT binding document that describes what
nvidia,tegra30-apbmisc means. Also, the register range for this new node
overlaps that for the pinmux node, so they can't both "request" their
register region. You may need multiple entries in the apbmisc reg
property to avoid this.

apbmisc reg for Tegra30 can be divided into following entries:

strap registers
jtag configuration registers
pull_up/pull_down control registers
vclk control registers
tvdac registers
chip id revision registers
pad control registers

This list is not same for Tegra20 and Tegra30.

OR

another way is to add chip id revision register region to CAR node as done for pinmux node and remove apb misc node.

What is preferred?
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/