On Fri, Aug 31, 2012 at 03:07:23PM +0200, Sascha Hauer wrote:
> > >   clk_register_clkdev(clk[ssi1_ipg_gate], NULL, "63fcc000.ssi");
> > >   clk_register_clkdev(clk[ssi2_ipg_gate], NULL, "50014000.ssi");
> > >   clk_register_clkdev(clk[ssi3_ipg_gate], NULL, "63fd0000.ssi");
> > > + clk_register_clkdev(clk[pwm1_ipg_gate], "ipg", "53fb4000.pwm");
> > > + clk_register_clkdev(clk[pwm1_hf_gate], "per", "53fb4000.pwm");
> > > + clk_register_clkdev(clk[pwm2_ipg_gate], "ipg", "53fb8000.pwm");
> > > + clk_register_clkdev(clk[pwm2_hf_gate], "per", "53fb8000.pwm");
> > 
> > It should be in a separate patch?
> 
> Should it? Surely I can do this, I had the feeling though that it
> belongs together.
> 
>From patch subject, I do not expect these changes in the patch.  It's
okay to have it but we need a more proper patch subject then.  

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

Reply via email to