On 10/24/07, Timur Tabi <[EMAIL PROTECTED]> wrote:
> Jon Smirl wrote:
> > Is this consensus on how the tree should look?
> >
> > There is no attempt to describe the codec connections inside the
> > device tree.
>
> I don't think I agree with that. The device tree should indicate which codec
> is
> connected to which I2S/AC97 device.
What I meant was that there is no attempt to describe how the codec is
connected to the external world. Those connections are described in
the fabric driver.
I'm getting conflicting opinions on how the devices should be linked
into the tree. We should pick one and add it to the documentation.
The DTC experts need to tell us which way to make the pointers between
i2s and i2c for the codec. Here's a another way it could be done that
looks more like the ac97 model.
[EMAIL PROTECTED] { // PSC1
compatible = "fsl,mpc5200b-psc-i2s","fsl,mpc5200-psc-i2s";
cell-index = <0>;
reg = <2000 100>;
interrupts = <2 1 0>;
interrupt-parent = <&mpc5200_pic>;
codec0: [EMAIL PROTECTED] {
compatible = "ti,tas5508";
reg = <0>;
i2c-handle = <&[EMAIL PROTECTED]>;
};
codec1: [EMAIL PROTECTED] {
compatible = "ti,tas5508";
reg = <1>;
i2c-handle = <&[EMAIL PROTECTED]>;
};
};
[EMAIL PROTECTED] { // PSC2
compatible = "fsl,mpc5200b-psc-i2s","fsl,mpc5200-psc-i2s";
cell-index = <1>;
reg = <2200 100>;
interrupts = <2 2 0>;
interrupt-parent = <&mpc5200_pic>;
codec2: [EMAIL PROTECTED] {
compatible = "wolson,wm8750";
reg = <2>;
i2c-handle = <&[EMAIL PROTECTED]>;
};
};
[EMAIL PROTECTED] {
compatible = "fsl,mpc5200b-i2c", "fsl,mpc5200-i2c", "fsl-i2c";
#address-cells = <1>;
#size-cells = <0>;
cell-index = <0>;
reg = <3d00 40>;
interrupts = <2 f 0>;
interrupt-parent = <&mpc5200_pic>;
fsl5200-clocking;
}
----- or would this be better? ---------------------
[EMAIL PROTECTED] { // PSC1
compatible = "fsl,mpc5200b-psc-i2s","fsl,mpc5200-psc-i2s";
cell-index = <0>;
reg = <2000 100>;
interrupts = <2 1 0>;
interrupt-parent = <&mpc5200_pic>;
[EMAIL PROTECTED] {
compatible = "ti,tas5508";
reg = <0>;
};
[EMAIL PROTECTED] {
compatible = "ti,tas5508";
reg = <1>;
};
};
[EMAIL PROTECTED] { // PSC2
compatible = "fsl,mpc5200b-psc-i2s","fsl,mpc5200-psc-i2s";
cell-index = <1>;
reg = <2200 100>;
interrupts = <2 2 0>;
interrupt-parent = <&mpc5200_pic>;
[EMAIL PROTECTED] {
compatible = "wolson,wm8750";
reg = <2>;
};
};
[EMAIL PROTECTED] {
compatible = "fsl,mpc5200b-i2c", "fsl,mpc5200-i2c", "fsl-i2c";
#address-cells = <1>;
#size-cells = <0>;
cell-index = <0>;
reg = <3d00 40>;
interrupts = <2 f 0>;
interrupt-parent = <&mpc5200_pic>;
fsl5200-clocking;
i2c-handle = <&[EMAIL PROTECTED] &[EMAIL PROTECTED] &[EMAIL PROTECTED]>
}
>
> I see that you do that for the AC97 node, but not the I2S node. Why?
>
> > I'm still not clear on how to trigger the load of the fabric driver.
> > Right now I have a single kernel that works on Efika and my target
> > hardware. This gets sorted out by define_machine(xxxx). I'll write
> > some code tonight to figure out how to load drivers and match on
> > codec0, codec1, etc. But how do I probe for the fabric driver I need
> > to figure out whether to load the Efika one or my target one.
>
> I've been struggling with that one, too. To keep it simple, I have the fabric
> driver just search for all the I2S nodes in my tree, and create ASoC objects
> for
> each one it finds. There's some hackery there, but I don't think we need to
> solve all the problems at once. The only thing that *has* to be right the
> first
> time is the device tree.
>
> > [EMAIL PROTECTED] { // PSC2
> > compatible = "fsl,mpc5200b-psc-i2s","fsl,mpc5200-psc-i2s";
> > cell-index = <1>;
> > reg = <2200 100>;
> > interrupts = <2 2 0>;
> > interrupt-parent = <&mpc5200_pic>;
> > };
> >
> > [EMAIL PROTECTED] {
> > compatible = "fsl,mpc5200b-i2c", "fsl,mpc5200-i2c", "fsl-i2c";
> > #address-cells = <1>;
> > #size-cells = <0>;
> > cell-index = <0>;
> > reg = <3d00 40>;
> > interrupts = <2 f 0>;
> > interrupt-parent = <&mpc5200_pic>;
> > fsl5200-clocking;
> >
> > codec0: [EMAIL PROTECTED] {
> > compatible = "ti,tas5508";
> > reg = <0>;
> > i2s-handle = <&[EMAIL PROTECTED]>;
> > };
>
> I'd do this the other way around -- that is:
>
> [EMAIL PROTECTED] { // PSC2
> compatible = "fsl,mpc5200b-psc-i2s","fsl,mpc5200-psc-i2s";
> ...
> i2c-handle = <&codec0>; /* Or something like that */
> };
>
> The reason is because I think the I2S driver will be instantiated *first* as
> an
> I2S driver and then it will create the I2C instantiation.
>
--
Jon Smirl
[EMAIL PROTECTED]
_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev
