<feed xmlns='http://www.w3.org/2005/Atom'>
<title>rockbox/firmware/drivers/rtc, branch puzzles</title>
<subtitle>My Rockbox tree</subtitle>
<link rel='alternate' type='text/html' href='https://www.franklinwei.com/cgit/rockbox/'/>
<entry>
<title>imx233: generate register headers using headergen_v2 and update code for it</title>
<updated>2016-05-28T14:49:22+00:00</updated>
<author>
<name>Amaury Pouly</name>
<email>amaury.pouly@gmail.com</email>
</author>
<published>2016-05-24T19:29:56+00:00</published>
<link rel='alternate' type='text/html' href='https://www.franklinwei.com/cgit/rockbox/commit/?id=eac1ca22bd4a6c1849880d0f8b6764befb60bc21'/>
<id>eac1ca22bd4a6c1849880d0f8b6764befb60bc21</id>
<content type='text'>
NOTE: this commit does not introduce any change, ideally even the binary should
be almost the same. I checked the disassembly by hand and there are only a few
differences here and there, mostly the compiler decides to compile very close
expressions slightly differently. I tried to run the new code on several targets
to make sure and saw no difference.

The major syntax changes of the new headers are as follows:
- BF_{WR,SET,CLR} are now superpowerful and allows to set several fileds at once:
  BF_WR(reg, field1(value1), field2(value2), ...)
- BF_CS (use like BF_WR) does a write to reg_CLR and then reg_SET instead of RMW
- there is no more need for macros like BF_{WR_,SET,CLR}_V, since one can simply
  BF_WR with field_V(name)
- the old BF_SETV macro has no trivial equivalent and is replaced with its
  its equivalent for BF_WR(reg_SET, ...)

I also rename the register headers: "regs/regs-x.h" -&gt; "regs/x.h" to avoid the
redundant "regs".

Final note: the registers were generated using the following command:
./headergen_v2 -g imx -o ../../firmware/target/arm/imx233/regs/ desc/regs-stmp3{600,700,780}.xml

Change-Id: I7485e8b4315a0929a8edb63e7fa1edcaa54b1edc
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
NOTE: this commit does not introduce any change, ideally even the binary should
be almost the same. I checked the disassembly by hand and there are only a few
differences here and there, mostly the compiler decides to compile very close
expressions slightly differently. I tried to run the new code on several targets
to make sure and saw no difference.

The major syntax changes of the new headers are as follows:
- BF_{WR,SET,CLR} are now superpowerful and allows to set several fileds at once:
  BF_WR(reg, field1(value1), field2(value2), ...)
- BF_CS (use like BF_WR) does a write to reg_CLR and then reg_SET instead of RMW
- there is no more need for macros like BF_{WR_,SET,CLR}_V, since one can simply
  BF_WR with field_V(name)
- the old BF_SETV macro has no trivial equivalent and is replaced with its
  its equivalent for BF_WR(reg_SET, ...)

I also rename the register headers: "regs/regs-x.h" -&gt; "regs/x.h" to avoid the
redundant "regs".

Final note: the registers were generated using the following command:
./headergen_v2 -g imx -o ../../firmware/target/arm/imx233/regs/ desc/regs-stmp3{600,700,780}.xml

Change-Id: I7485e8b4315a0929a8edb63e7fa1edcaa54b1edc
</pre>
</div>
</content>
</entry>
<entry>
<title>Don't return pointer to array allocated on the stack</title>
<updated>2015-01-13T20:40:19+00:00</updated>
<author>
<name>Thomas Jarosch</name>
<email>tomj@simonv.com</email>
</author>
<published>2015-01-13T20:40:19+00:00</published>
<link rel='alternate' type='text/html' href='https://www.franklinwei.com/cgit/rockbox/commit/?id=1212edd352e924111bb3db6330755b33f13fbe71'/>
<id>1212edd352e924111bb3db6330755b33f13fbe71</id>
<content type='text'>
It will be out of scope once the function is left.

cppcheck reported:
[firmware/drivers/rtc/rtc_zenvisionm.c:31]: (error) Pointer to local array variable returned.
[firmware/drivers/rtc/rtc_zenvisionm.c:38]: (error) Pointer to local array variable returned.

Change-Id: Ibf28ba9b3d20cadcaff22398e143488c86746660
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
It will be out of scope once the function is left.

cppcheck reported:
[firmware/drivers/rtc/rtc_zenvisionm.c:31]: (error) Pointer to local array variable returned.
[firmware/drivers/rtc/rtc_zenvisionm.c:38]: (error) Pointer to local array variable returned.

Change-Id: Ibf28ba9b3d20cadcaff22398e143488c86746660
</pre>
</div>
</content>
</entry>
<entry>
<title>sonynwz: fix time handling</title>
<updated>2013-09-26T15:01:44+00:00</updated>
<author>
<name>Amaury Pouly</name>
<email>amaury.pouly@gmail.com</email>
</author>
<published>2013-09-26T15:01:44+00:00</published>
<link rel='alternate' type='text/html' href='https://www.franklinwei.com/cgit/rockbox/commit/?id=0897794f445c1cbe5c0da4fb95d203491c3871a7'/>
<id>0897794f445c1cbe5c0da4fb95d203491c3871a7</id>
<content type='text'>
Be consistent with the OF and use RTC PERSISTENT2 as offset

Change-Id: Ic8b970cbc4096d1f4efa0380a3af51c93484fe58
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Be consistent with the OF and use RTC PERSISTENT2 as offset

Change-Id: Ic8b970cbc4096d1f4efa0380a3af51c93484fe58
</pre>
</div>
</content>
</entry>
<entry>
<title>imx233: implement alarm wake up</title>
<updated>2013-07-01T22:45:00+00:00</updated>
<author>
<name>Amaury Pouly</name>
<email>amaury.pouly@gmail.com</email>
</author>
<published>2013-07-01T22:36:21+00:00</published>
<link rel='alternate' type='text/html' href='https://www.franklinwei.com/cgit/rockbox/commit/?id=d8024df1052966aa554424d88712244ae5978f2a'/>
<id>d8024df1052966aa554424d88712244ae5978f2a</id>
<content type='text'>
This adds the application part of alarm wake up. On some targets
like the Fuze+, it will also require a bootloader change to make
sure that the device doesn't power down on alarm wake up (for
example on the Fuze+ the bootloader requires the power button to
be hold sufficiently long, thus preventing alarm wake up to work)

Change-Id: I5d01957852355fddbd48110d3d75a5533f07879e
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This adds the application part of alarm wake up. On some targets
like the Fuze+, it will also require a bootloader change to make
sure that the device doesn't power down on alarm wake up (for
example on the Fuze+ the bootloader requires the power button to
be hold sufficiently long, thus preventing alarm wake up to work)

Change-Id: I5d01957852355fddbd48110d3d75a5533f07879e
</pre>
</div>
</content>
</entry>
<entry>
<title>zenxfi3: RTC is used in same was as in fuze+, using seconds-since-1970 plus an offset</title>
<updated>2012-05-26T22:16:40+00:00</updated>
<author>
<name>Bertrik Sikken</name>
<email>bertrik@sikken.nl</email>
</author>
<published>2012-05-26T22:14:42+00:00</published>
<link rel='alternate' type='text/html' href='https://www.franklinwei.com/cgit/rockbox/commit/?id=fca9e7bf8d48f2fcc4c8b21cc29aabf2ce053900'/>
<id>fca9e7bf8d48f2fcc4c8b21cc29aabf2ce053900</id>
<content type='text'>
Change-Id: Iab2e6e15c790c26d3bf2679e9f965a409d162783
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Change-Id: Iab2e6e15c790c26d3bf2679e9f965a409d162783
</pre>
</div>
</content>
</entry>
<entry>
<title>imx233: move rtc init from drivers/ to firmware/</title>
<updated>2012-05-19T10:57:42+00:00</updated>
<author>
<name>Amaury Pouly</name>
<email>amaury.pouly@gmail.com</email>
</author>
<published>2012-05-19T10:57:04+00:00</published>
<link rel='alternate' type='text/html' href='https://www.franklinwei.com/cgit/rockbox/commit/?id=692338629ba1d21a4f78d75bd4baa2f0ff74960e'/>
<id>692338629ba1d21a4f78d75bd4baa2f0ff74960e</id>
<content type='text'>
Change-Id: Id816987a4bb1191d5c9cf8a85c51d75fbb2da7c8
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Change-Id: Id816987a4bb1191d5c9cf8a85c51d75fbb2da7c8
</pre>
</div>
</content>
</entry>
<entry>
<title>as3514 rtc: sansa clip zip uses 1970/1/1 base date instead of special AS3525v2 date</title>
<updated>2012-04-17T19:33:59+00:00</updated>
<author>
<name>Bertrik Sikken</name>
<email>bertrik@sikken.nl</email>
</author>
<published>2012-04-17T19:24:49+00:00</published>
<link rel='alternate' type='text/html' href='https://www.franklinwei.com/cgit/rockbox/commit/?id=a92a0fecca5ddd2d1500b35218b4891bc13be9b6'/>
<id>a92a0fecca5ddd2d1500b35218b4891bc13be9b6</id>
<content type='text'>
Change-Id: I32329c82da93c705a5cb4af8a1a8b4e85a9c9b37
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Change-Id: I32329c82da93c705a5cb4af8a1a8b4e85a9c9b37
</pre>
</div>
</content>
</entry>
<entry>
<title>Convert IMX233 RTC driver (used in fuze+) to use mktime and gmtime</title>
<updated>2012-03-13T18:27:55+00:00</updated>
<author>
<name>Bertrik Sikken</name>
<email>bertrik@sikken.nl</email>
</author>
<published>2012-03-11T14:59:55+00:00</published>
<link rel='alternate' type='text/html' href='https://www.franklinwei.com/cgit/rockbox/commit/?id=47115ba8347f857fa5f59dc4326127e5849ea4eb'/>
<id>47115ba8347f857fa5f59dc4326127e5849ea4eb</id>
<content type='text'>
Change-Id: Ie71d495509a81db5e02078398f5d722c00136072
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Change-Id: Ie71d495509a81db5e02078398f5d722c00136072
</pre>
</div>
</content>
</entry>
<entry>
<title>Introduce gmtime.c and use it in the AS3514 RTC driver</title>
<updated>2012-03-11T21:10:54+00:00</updated>
<author>
<name>Bertrik Sikken</name>
<email>bertrik@sikken.nl</email>
</author>
<published>2012-03-08T20:21:31+00:00</published>
<link rel='alternate' type='text/html' href='https://www.franklinwei.com/cgit/rockbox/commit/?id=727e8aa6817be8816bc32184f2e2eb36b90b4a9e'/>
<id>727e8aa6817be8816bc32184f2e2eb36b90b4a9e</id>
<content type='text'>
Change-Id: I00a09ae28a68f8153fb8fa854fea741ddfb0bf09
Reviewed-on: http://gerrit.rockbox.org/175
Tested-by: Bertrik Sikken &lt;bertrik@sikken.nl&gt;
Reviewed-by: Michael Sevakis &lt;jethead71@rockbox.org&gt;
Reviewed-by: Rafaël Carré &lt;rafael.carre@gmail.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Change-Id: I00a09ae28a68f8153fb8fa854fea741ddfb0bf09
Reviewed-on: http://gerrit.rockbox.org/175
Tested-by: Bertrik Sikken &lt;bertrik@sikken.nl&gt;
Reviewed-by: Michael Sevakis &lt;jethead71@rockbox.org&gt;
Reviewed-by: Rafaël Carré &lt;rafael.carre@gmail.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>cowond2's power-target.h -&gt; rename in pmu-target.h</title>
<updated>2012-01-07T22:39:14+00:00</updated>
<author>
<name>Rafaël Carré</name>
<email>rafael.carre@gmail.com</email>
</author>
<published>2012-01-07T22:39:14+00:00</published>
<link rel='alternate' type='text/html' href='https://www.franklinwei.com/cgit/rockbox/commit/?id=f3ad239a76f010a874777dc463d90882772fbff3'/>
<id>f3ad239a76f010a874777dc463d90882772fbff3</id>
<content type='text'>
other targets (ipod nano2g / classic) use that name for pmu

git-svn-id: svn://svn.rockbox.org/rockbox/trunk@31618 a1c6a512-1295-4272-9138-f99709370657
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
other targets (ipod nano2g / classic) use that name for pmu

git-svn-id: svn://svn.rockbox.org/rockbox/trunk@31618 a1c6a512-1295-4272-9138-f99709370657
</pre>
</div>
</content>
</entry>
</feed>
