From: Heiko Schocher <hs@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v2 0/5] ARM: at91: improve peripheral and system clock handle functions
Date: Thu, 10 Dec 2015 07:02:26 +0100 [thread overview]
Message-ID: <56691572.2030007@denx.de> (raw)
In-Reply-To: <B256D81BAE5131468A838E5D7A243641D63B951F@penmbx01>
Hello Wenyou,
Am 10.12.2015 um 03:15 schrieb Yang, Wenyou:
> Hi Heiko,
>
>
>> -----Original Message-----
>> From: Heiko Schocher [mailto:hs at denx.de]
>> Sent: 2015?12?9? 18:01
>> To: Yang, Wenyou <Wenyou.Yang@atmel.com>
>> Cc: U-Boot Mailing List <u-boot@lists.denx.de>; andreas.devel at googlemail.com
>> Subject: Re: [PATCH v2 0/5] ARM: at91: improve peripheral and system clock
>> handle functions
>>
>> Hello Wenyou,
>>
>> Am 09.12.2015 um 08:48 schrieb Wenyou Yang:
>>> To reduce the duplicated code, rework the peripheral's and system's
>>> clock handle functions, use these functions to replace the clock
>>> handle code.
>>>
>>> Changes in v2:
>>> - fix checkpatch warning.
>>>
>>> Wenyou Yang (5):
>>> ARM: at91: asm/at91_pmc.h: fix trival register offset
>>> ARM: at91: clock: add a new file to handle clock
>>> ARM: cpu: at91: clean up peripheral clock code
>>> board: atmel: clean up peripheral clock code
>>> drivers: at91: clean up peripheral clock code
>>>
>>> arch/arm/mach-at91/Makefile | 1 +
>>> arch/arm/mach-at91/arm920t/at91rm9200_devices.c | 13 +---
>>> arch/arm/mach-at91/arm920t/timer.c | 4 +-
>>> arch/arm/mach-at91/arm926ejs/at91sam9260_devices.c | 36 +++-------
>>> arch/arm/mach-at91/arm926ejs/at91sam9261_devices.c | 28 ++------
>>> arch/arm/mach-at91/arm926ejs/at91sam9263_devices.c | 37 +++-------
>>> .../mach-at91/arm926ejs/at91sam9m10g45_devices.c | 33 +++------
>>> arch/arm/mach-at91/arm926ejs/at91sam9n12_devices.c | 40 +++--------
>>> arch/arm/mach-at91/arm926ejs/at91sam9rl_devices.c | 28 ++------
>>> arch/arm/mach-at91/arm926ejs/at91sam9x5_devices.c | 38 +++--------
>>> arch/arm/mach-at91/arm926ejs/clock.c | 7 --
>>> arch/arm/mach-at91/arm926ejs/cpu.c | 1 -
>>> arch/arm/mach-at91/arm926ejs/timer.c | 5 +-
>>> arch/arm/mach-at91/armv7/clock.c | 26 -------
>>> arch/arm/mach-at91/armv7/cpu.c | 1 -
>>> arch/arm/mach-at91/armv7/timer.c | 1 -
>>> arch/arm/mach-at91/clock.c | 72 ++++++++++++++++++++
>>> arch/arm/mach-at91/include/mach/at91_pmc.h | 8 +--
>>> arch/arm/mach-at91/include/mach/clk.h | 2 +
>>> arch/arm/mach-at91/phy.c | 1 -
>>> arch/arm/mach-at91/sdram.c | 1 -
>>> board/atmel/at91rm9200ek/at91rm9200ek.c | 1 -
>>> board/atmel/at91rm9200ek/led.c | 5 +-
>>> board/atmel/at91sam9260ek/at91sam9260ek.c | 15 ++--
>>> board/atmel/at91sam9261ek/at91sam9261ek.c | 8 +--
>>> board/atmel/at91sam9261ek/led.c | 6 +-
>>> board/atmel/at91sam9263ek/at91sam9263ek.c | 23 ++-----
>>> board/atmel/at91sam9263ek/led.c | 9 +--
>>> board/atmel/at91sam9m10g45ek/at91sam9m10g45ek.c | 20 ++----
>>> board/atmel/at91sam9m10g45ek/led.c | 7 +-
>>> board/atmel/at91sam9n12ek/at91sam9n12ek.c | 6 +-
>>> board/atmel/at91sam9rlek/at91sam9rlek.c | 18 ++---
>>> board/atmel/at91sam9rlek/led.c | 7 +-
>>> board/atmel/at91sam9x5ek/at91sam9x5ek.c | 10 +--
>>> board/atmel/sama5d2_xplained/sama5d2_xplained.c | 1 -
>>> board/atmel/sama5d3_xplained/sama5d3_xplained.c | 5 +-
>>> board/atmel/sama5d3xek/sama5d3xek.c | 5 +-
>>> board/atmel/sama5d4_xplained/sama5d4_xplained.c | 5 +-
>>> board/atmel/sama5d4ek/sama5d4ek.c | 5 +-
>>> board/bluewater/snapper9260/snapper9260.c | 15 ++--
>>> board/calao/usb_a9263/usb_a9263.c | 11 ++-
>>> board/egnite/ethernut5/ethernut5.c | 20 +++---
>>> board/esd/meesc/meesc.c | 15 ++--
>>> board/mini-box/picosam9g45/led.c | 7 +-
>>> board/mini-box/picosam9g45/picosam9g45.c | 17 ++---
>>> board/ronetix/pm9261/led.c | 7 +-
>>> board/ronetix/pm9261/pm9261.c | 22 ++----
>>> board/ronetix/pm9263/led.c | 7 +-
>>> board/ronetix/pm9263/pm9263.c | 19 ++----
>>> board/ronetix/pm9g45/pm9g45.c | 20 ++----
>>> board/siemens/corvus/board.c | 6 +-
>>> board/siemens/smartweb/smartweb.c | 3 +-
>>> board/siemens/taurus/taurus.c | 3 +-
>>> drivers/net/at91_emac.c | 9 ++-
>>> drivers/usb/host/ehci-atmel.c | 8 ---
>>> drivers/usb/host/ohci-at91.c | 20 ++----
>>> 56 files changed, 252 insertions(+), 496 deletions(-)
>>> create mode 100644 arch/arm/mach-at91/clock.c
>>
>> I applied all your patches (the v2 versions), but now tftp does not work anymore
>> on the smartweb board ...
>>
>> I try to find some time to get more details ...
>
> Sorry about it.
>
> It is my fault, incorrectly used the indicator & to achieve the address of variable of pmc->pcer and pmc->pcdr.
Ah, ok! Good catch, I see your v3 on the ML ... added them to my setup [1],
but I cannot try them yet, as my board (accurate ethernet) is broken ...
I must first reanimate it (no debugger connected to it yet ...)
> Thank you very much for your findings.
You are welcome. Currently all work is done automatically [2] ... I just
have to pass a list of patchworknumbers to my testsetup, and they get
automagically tested on the board ...
bye,
Heiko
[1] https://github.com/hsdenx/tbot/commit/9bdea9400c2ea9a521c679a83bfeb2879db841e8
[2] http://xeidos.ddns.net/buildbot/builders/smartweb_dfu/builds/28/steps/shell/logs/tbotlog
>
>>
>> bye,
>> Heiko
>> --
>> DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
>> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
>
>
> Best Regards,
> Wenyou Yang
>
--
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
next prev parent reply other threads:[~2015-12-10 6:02 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-12-09 7:48 [U-Boot] [PATCH v2 0/5] ARM: at91: improve peripheral and system clock handle functions Wenyou Yang
2015-12-09 7:48 ` [U-Boot] [PATCH v2 1/5] ARM: at91: asm/at91_pmc.h: fix trival register offset Wenyou Yang
2015-12-09 7:48 ` [U-Boot] [PATCH v2 2/5] ARM: at91: clock: add a new file to handle clock Wenyou Yang
2015-12-09 7:48 ` [U-Boot] [PATCH v2 3/5] ARM: cpu: at91: clean up peripheral clock code Wenyou Yang
2015-12-09 7:48 ` [U-Boot] [PATCH v2 4/5] board: atmel: " Wenyou Yang
2015-12-09 7:48 ` [U-Boot] [PATCH v2 5/5] drivers: at91: " Wenyou Yang
2015-12-09 10:00 ` [U-Boot] [PATCH v2 0/5] ARM: at91: improve peripheral and system clock handle functions Heiko Schocher
2015-12-10 2:15 ` Yang, Wenyou
2015-12-10 6:02 ` Heiko Schocher [this message]
2015-12-10 6:07 ` Yang, Wenyou
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=56691572.2030007@denx.de \
--to=hs@denx.de \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox