关于 如何实践kernel/driver/u-boot ，怎么样按照作者的思路实现自己的代码

 给你指一个方向或者说是方法， 可以提高自己编程kernel/driver/u-boot的能力，或者说写code的能力。
 
 
举个例子，
 
比如说 kernel 里面的clock时钟的初始化部分， kernel的代码也不复杂， 但是我们要达到的目标就是 根据datasheet 能自己写出来，而不仅仅能看明白， 好比说 ，
会看小说的人，不见得都会写小说， 金庸只有一个。 我们现在就要作金庸。只有试着写 ，才能作为作家。  
 
 
只是举个简单的例子：至少我是这样来练习的，
你可以对照你自己的u-boot或者omap5912 作类似的动作。 方法都是一样的。 总之， 就是截取现有的功能， 自己试着实现它， 久而久之， 能力就提高了。
 
看的时候， 对照着原来的driver 和datasheet ，了解到流程很简单，
 
1> 读取 MPLLCON 寄存器 , 比如读到 mpllcon_value 变量中，
2> 对 mpllcon_value 分别移位操作， 取得 MDIV ,PDIV ,SDIV
3> 利用公式：
Mpll = (m * Fin) / (p * 2^s)
m = (MDIV + 8), p = (PDIV + 2), s = SDIV
4> fclk = Mpll
 
5》 读取 CLKDIVN 寄存器 ，
 
6》 然后作相应的计算： 求出 FCLK , HCLK , PCLK (WATCHDOG 用到了，我们kernel就要初始化）
 

HDIVN	[1]	0: HCLK has the clock same as the FCLK.	0
		1: HCLK has the clock same as the FCLK/2.
PDIVN	[0]	0: PCLK has the clock same as the HCLK.	0
		1: PCLK has the clock same as the HCLK/2.

 
 
 
到现在流程已经知道了，  估计 2410自带的代码也忘记的差不多了，那就可以自己写了。   只有自己写， 才能发现自己的问题， 也才能领会 作者那样写的妙处，当然作者也不见得都对
 
 
我就自己重新实现了一个，
 
我自己写的时候， 就会考虑几个问题，
 
1> 我是要int 还是用 u32 ，会不会溢出呢？还有可移植性。
2> 小的临时变量 是用 u8 , 还是 u32
3> datasheet 有没有提到有些操作需要delay ，比如 delay(400) 什么的？
4> 最后 拿到板子上去验证。 看写的有没有问题？
 
 
 
自己实现的，  后面 有 2410 作者写的。 自己对比一下 。  
 
 
static inline u32 s3c2410_get_pll_bob_version(u32 Fin, u32 mpllcon_value)
{
       //u8 is enought for mdiv and pdiv and sdiv
       u8 mdiv = 0; //作者都用
       u8 pdiv = 0;
       u8 sdiv = 0;
 
       u32 mpll = 0;
 
//相比作者的代码， 这里面没有用 宏来标记一些bit 或者bitmask 。别人看起来可能一头污水， 不过标出来
//refer P240  CLOCK DIVIDER CONTROL (CLKDIVN) REGISTER     这样别人就一下子知道了。
 
       mdiv = (mpllcon_value & ((1<<20) -1)) >> 12;
       pdiv = (mpllcon_value & ((1<<10)-1)) >>4;
       sdiv = (mpllcon_value & ((1<<2)-1)) >> 0;   //我个人觉得 >> 0 很有意义。比不加强。
 
//see manual P237
       mpll = (mdiv+8) * Fin;
       mpll /=(pdiv+2);
       mpll >>= sdiv;
 
       printk("in s3c2410_get_mpll_bob_version() , mpll = %u\n",mpll);
      
       return mpll;
 
}
 
void __init s3c2410_init_clocks(int xtal)   //Õa¸öxtal¾íêÇ»ù±¾μÄêäèëÆμÂê12MHz
{
 
       u32 mpll = 0;
       u32 clock_mpllcon = 0;
       u32 fclk = 0;
       u32 hclk =0;
       u32 pclk = 0;
 
       u32 HDIVN1 = 0;
       u32 HDIVN = 0;
       u32 PDIVN = 0;
 
      
       clock_mpllcon = ioread32(S3C2410_MPLLCON);
      
       mpll = s3c2410_get_pll_bob_version((u32)xtal,clock_mpllcon);
 
       fclk = mpll;
 
       if(HDIVN1 = ioread32(S3C2410_CLKDIVN) & (1<<2))
       {
              hclk = pclk = fclk/4;  //
       }
       else {
              hclk = (HDIVN = ioread32(S3C2410_CLKDIVN) & (1<<1) )? fclk/2:fclk;
              pclk = (PDIVN = ioread32(S3C2410_CLKDIVN) & (1<<0) )? hclk/2:hclk;
       }
 
       printk("fclk = %u, hclk=%u, pclk=%u\n",fclk,hclk,pclk);
 
      
 
。。。。
 
 
+++++++++++++++++++++++++++++++++++++++++++++++++++
 
然后再对照作者的程序的着两个函数， 对比 看看差距在哪里？？？只有找出差距， 才能提高自己！
 
 
 
 
static inline unsigned int
s3c2410_get_pll(unsigned int pllval, unsigned int baseclk)
{
       unsigned int mdiv, pdiv, sdiv;
       uint64_t fvco;
//作者用了一些宏 ，但是这么简单的操作 ，弄两次似乎不是很好。这就没什么说得了， 个人喜好不同， 不算差距。
       mdiv = pllval >> S3C2410_PLLCON_MDIVSHIFT;
       pdiv = pllval >> S3C2410_PLLCON_PDIVSHIFT;
       sdiv = pllval >> S3C2410_PLLCON_SDIVSHIFT;
 
       mdiv &= S3C2410_PLLCON_MDIVMASK;
       pdiv &= S3C2410_PLLCON_PDIVMASK;
       sdiv &= S3C2410_PLLCON_SDIVMASK;
 
       fvco = (uint64_t)baseclk * (mdiv + 8);
       do_div(fvco, (pdiv + 2) << sdiv);
 
       return (unsigned int)fvco;
}
 
 
void __init s3c2410_init_clocks(int xtal)  
{
       unsigned long tmp;
       unsigned long fclk;
       unsigned long hclk;
       unsigned long pclk;  //干吗都声明成 unsigned long ，实际上 ，unsigned int 就足够了（当然int与long长度相同）
//更好的办法是  用 u32 来表示， 因为 32 bit 肯定够了。 作者的移植性不好。
 
       /* now we've got our machine bits initialised, work out what
        * clocks we've got */
 
       fclk = s3c2410_get_pll(__raw_readl(S3C2410_MPLLCON), xtal);  //用 __raw_readl（） 更是勉强，事上 ， kernel早就不推荐用 __raw_readl （）了， 应该 用 ioread32()
 
       tmp = __raw_readl(S3C2410_CLKDIVN); //同上
 
       /* work out clock scalings */
/* 这里功能上就少了一个： if(HDIVN1 = ioread32(S3C2410_CLKDIVN) & (1<<2))  ，按照datasheet ：
 

HDIVN1	[2]	Special bus clock ratio available. (1:4:4) 0: Reserved 1: HCLK has the clock same as the FCLK/4. PCLK has the clock same as the FCLK/4. Note: If this bit is "0b1", HDIVN and PDIVN must be set "0b0".   写程序的时候要体现出来这样的判断	0
HDIVN	[1]	0: HCLK has the clock same as the FCLK. 1: HCLK has the clock same as the FCLK/2.	0
PDIVN	[0]	0: PCLK has the clock same as the HCLK. 1: PCLK has the clock same as the HCLK/2.	0

 
应该要判断 bit[2] 的， 但是作者仅仅判断了 bit0和bit1 ,就是不对的。
 
 
*/
//少了对 HDIVN1 的判断。
       hclk = fclk / ((tmp & S3C2410_CLKDIVN_HDIVN) ? 2 : 1);
       pclk = hclk / ((tmp & S3C2410_CLKDIVN_PDIVN) ? 2 : 1);