7448 pll registers

7448 pll registers

[Leisner, Martin]

I'm going to get DFS into the 7448 (it looks like I'm going to take a
different
tactic then I see in 2.6.20.16 -- it hinges on powermac, and I don't
know if it
even compiles).

In include/asm-powerpc, its missing definitions for HID1/{PC4,PC5}.

bash2 :2 mleisner@linuxcom-01 05:18:58; rcsdiff -u reg.h
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
RCS file: reg.h,v
retrieving revision 1.1
diff -u -r1.1 reg.h
--- reg.h       2007/09/05 20:45:36     1.1
+++ reg.h       2007/09/05 20:46:56
@@ -253,6 +253,8 @@
 #define HID1_PC1       (1<<15)         /* 7450 PLL_CFG[1] */
 #define HID1_PC2       (1<<14)         /* 7450 PLL_CFG[2] */
 #define HID1_PC3       (1<<13)         /* 7450 PLL_CFG[3] */
+#define HID1_PC4       (1<<12)         /* ???? PLL_CFG[4] */
+#define HID1_PC5       (1<<17)         /* 7448 specific bit --
PLL_CFG[5] */
 #define HID1_SYNCBE    (1<<11)         /* 7450 ABE for sync, eieio */
 #define HID1_ABE       (1<<10)         /* 7450 Address Broadcast Enable
*/
 #define HID1_PS                (1<<16)         /* 750FX PLL selection
*/


PC4 seems "generic", PC5 is a "MPC7448 specific bit"

I'm not sure what HID1_PS means ( 750FX PLL selection ), its the same
bit as=20
HID1_PC0.

marty

Re: 7448 pll registers

[Kumar Gala]

On Sep 5, 2007, at 4:24 PM, Leisner, Martin wrote:

> I'm going to get DFS into the 7448 (it looks like I'm going to take a
> different
> tactic then I see in 2.6.20.16 -- it hinges on powermac, and I don't
> know if it
> even compiles).
>
> In include/asm-powerpc, its missing definitions for HID1/{PC4,PC5}.
>
> bash2 :2 mleisner@linuxcom-01 05:18:58; rcsdiff -u reg.h
> ===================================================================
> RCS file: reg.h,v
> retrieving revision 1.1
> diff -u -r1.1 reg.h
> --- reg.h       2007/09/05 20:45:36     1.1
> +++ reg.h       2007/09/05 20:46:56
> @@ -253,6 +253,8 @@
>  #define HID1_PC1       (1<<15)         /* 7450 PLL_CFG[1] */
>  #define HID1_PC2       (1<<14)         /* 7450 PLL_CFG[2] */
>  #define HID1_PC3       (1<<13)         /* 7450 PLL_CFG[3] */
> +#define HID1_PC4       (1<<12)         /* ???? PLL_CFG[4] */

7455 for PLL_CFG[4]

> +#define HID1_PC5       (1<<17)         /* 7448 specific bit --
> PLL_CFG[5] */
>  #define HID1_SYNCBE    (1<<11)         /* 7450 ABE for sync, eieio */
>  #define HID1_ABE       (1<<10)         /* 7450 Address Broadcast  
> Enable
> */
>  #define HID1_PS                (1<<16)         /* 750FX PLL selection
> */
>
>
> PC4 seems "generic", PC5 is a "MPC7448 specific bit"
>
> I'm not sure what HID1_PS means ( 750FX PLL selection ), its the same
> bit as
> HID1_PC0.

Do you want this patch in the kernel?  If so we'll need a signed-off-by.

- k