Hi,
On Mon, Sep 22, 2014 at 1:05 AM, Ian Campbell <i...@hellion.org.uk> wrote:
> On Mon, 2014-09-08 at 21:28 +0800, Chen-Yu Tsai wrote:
>> From: Oliver Schinagl <oli...@schinagl.nl>
>>
>> To setup clocks and control voltages.
>
> perhaps add "... For P2WI and PIO", since that is apparently what it is
> doing?
Sounds good. I'll expand the message to mention what the PRCM
controls, and what we actually use.
>> HdG: Rename the files from the somewhat generic pmu name to prcm.{c,h}
>> HdG: Make the prcm code only deal with the prcm, remove axp221 bits
>
> I suppose these comments aren't really meaningful in the final commit
> message, probably better to drop them.
I was trying to keep the history of the patches intact.
If Hans agress, I can drop them.
>> + * Based on sun6i sources and earlier U-Boot Allwiner A10 SPL work
>
> Typo in "Allwiner"
>
>> + reg_val = readl(&prcm->apb0_gate);
>> + reg_val |= PRCM_APB0_GATE_P2WI | PRCM_APB0_GATE_PIO;
>> + writel(reg_val, &prcm->apb0_gate);
>> +
>> + reg_val = readl(&prcm->apb0_reset);
>> + reg_val |= PRCM_APB0_RESET_P2WI | PRCM_APB0_RESET_PIO;
>> + writel(reg_val, &prcm->apb0_reset);
>
> I think these should both use the setbits helper.
Right. Will change them.
Thanks!
ChenYu
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
