* [U-Boot] [GIT PULL] mpc83xx: mpc8308rdb enhancements
@ 2012-09-17 23:36 Kim Phillips
2012-09-18 1:18 ` Tom Rini
0 siblings, 1 reply; 6+ messages in thread
From: Kim Phillips @ 2012-09-17 23:36 UTC (permalink / raw)
To: u-boot
Hi Tom,
Ira improved mpc8308rdb h/w support by making it more comprehensive.
Please pull:
The following changes since commit a6f0c4faa4c65a7b7048b12c9d180d7e1aad1721:
Merge branch 'master' of git://git.denx.de/u-boot-avr32 (2012-09-04 09:17:27 +0200)
are available in the git repository at:
git://git.denx.de/u-boot-mpc83xx.git master
for you to fetch changes up to 014d1dfc669ab2295d7fa4ec4b9f00a4004917a0:
mpc8308rdb: add support for eSDHC MMC controller (2012-09-17 17:47:12 -0500)
----------------------------------------------------------------
Ira W. Snyder (5):
mpc8xxx_spi: fix SPI support on MPC8308RDB
mpc8308rdb: add support for Spansion SPI flash on header J8
spansion: add support for S25FL256S1
mpc8308rdb: add support for FIT images
mpc8308rdb: add support for eSDHC MMC controller
board/freescale/mpc8308rdb/mpc8308rdb.c | 58 +++++++++++++++++++++++++++++++++
drivers/mtd/spi/spansion.c | 8 +++++
drivers/spi/mpc8xxx_spi.c | 4 +++
include/configs/MPC8308RDB.h | 31 ++++++++++++++++++
4 files changed, 101 insertions(+)
Thanks,
Kim
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [GIT PULL] mpc83xx: mpc8308rdb enhancements
2012-09-17 23:36 [U-Boot] [GIT PULL] mpc83xx: mpc8308rdb enhancements Kim Phillips
@ 2012-09-18 1:18 ` Tom Rini
2012-09-18 3:09 ` Ira W. Snyder
0 siblings, 1 reply; 6+ messages in thread
From: Tom Rini @ 2012-09-18 1:18 UTC (permalink / raw)
To: u-boot
On Mon, Sep 17, 2012 at 06:36:56PM -0500, Kim Phillips wrote:
> Hi Tom,
>
> Ira improved mpc8308rdb h/w support by making it more comprehensive.
> Please pull:
>
> The following changes since commit a6f0c4faa4c65a7b7048b12c9d180d7e1aad1721:
>
> Merge branch 'master' of git://git.denx.de/u-boot-avr32 (2012-09-04 09:17:27 +0200)
>
> are available in the git repository at:
>
>
> git://git.denx.de/u-boot-mpc83xx.git master
>
> for you to fetch changes up to 014d1dfc669ab2295d7fa4ec4b9f00a4004917a0:
>
> mpc8308rdb: add support for eSDHC MMC controller (2012-09-17 17:47:12 -0500)
>
> ----------------------------------------------------------------
> Ira W. Snyder (5):
> mpc8xxx_spi: fix SPI support on MPC8308RDB
> mpc8308rdb: add support for Spansion SPI flash on header J8
> spansion: add support for S25FL256S1
> mpc8308rdb: add support for FIT images
> mpc8308rdb: add support for eSDHC MMC controller
>
> board/freescale/mpc8308rdb/mpc8308rdb.c | 58 +++++++++++++++++++++++++++++++++
> drivers/mtd/spi/spansion.c | 8 +++++
> drivers/spi/mpc8xxx_spi.c | 4 +++
> include/configs/MPC8308RDB.h | 31 ++++++++++++++++++
> 4 files changed, 101 insertions(+)
A number of boards (P2020RDB-PC_36BIT is the one I looked at the log in)
now fail:
spansion.c:74:3: error: unknown field 'page_size' specified in
initializer
Rejected, please fix, sorry.
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20120917/7e4e47ad/attachment.pgp>
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [GIT PULL] mpc83xx: mpc8308rdb enhancements
2012-09-18 1:18 ` Tom Rini
@ 2012-09-18 3:09 ` Ira W. Snyder
2012-09-18 14:57 ` Tom Rini
0 siblings, 1 reply; 6+ messages in thread
From: Ira W. Snyder @ 2012-09-18 3:09 UTC (permalink / raw)
To: u-boot
On Mon, Sep 17, 2012 at 06:18:12PM -0700, Tom Rini wrote:
> On Mon, Sep 17, 2012 at 06:36:56PM -0500, Kim Phillips wrote:
> > Hi Tom,
> >
> > Ira improved mpc8308rdb h/w support by making it more comprehensive.
> > Please pull:
> >
> > The following changes since commit a6f0c4faa4c65a7b7048b12c9d180d7e1aad1721:
> >
> > Merge branch 'master' of git://git.denx.de/u-boot-avr32 (2012-09-04 09:17:27 +0200)
> >
> > are available in the git repository at:
> >
> >
> > git://git.denx.de/u-boot-mpc83xx.git master
> >
> > for you to fetch changes up to 014d1dfc669ab2295d7fa4ec4b9f00a4004917a0:
> >
> > mpc8308rdb: add support for eSDHC MMC controller (2012-09-17 17:47:12 -0500)
> >
> > ----------------------------------------------------------------
> > Ira W. Snyder (5):
> > mpc8xxx_spi: fix SPI support on MPC8308RDB
> > mpc8308rdb: add support for Spansion SPI flash on header J8
> > spansion: add support for S25FL256S1
> > mpc8308rdb: add support for FIT images
> > mpc8308rdb: add support for eSDHC MMC controller
> >
> > board/freescale/mpc8308rdb/mpc8308rdb.c | 58 +++++++++++++++++++++++++++++++++
> > drivers/mtd/spi/spansion.c | 8 +++++
> > drivers/spi/mpc8xxx_spi.c | 4 +++
> > include/configs/MPC8308RDB.h | 31 ++++++++++++++++++
> > 4 files changed, 101 insertions(+)
>
> A number of boards (P2020RDB-PC_36BIT is the one I looked at the log in)
> now fail:
> spansion.c:74:3: error: unknown field 'page_size' specified in
> initializer
>
> Rejected, please fix, sorry.
>
> --
> Tom
Tom,
Go ahead and drop the patch "spansion: add support for S25FL256S1" from
the series. The rest should apply without any trouble.
Michal Simek beat me to it in "sf: spansion: Add support for S25FL256S"
(4a4cb4e1114). He got the idcode1 wrong (it should be 0x0219 instead of
0x2019), but I saw a followup patch on the mailing list to fix it.
Thanks,
Ira
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [GIT PULL] mpc83xx: mpc8308rdb enhancements
2012-09-18 3:09 ` Ira W. Snyder
@ 2012-09-18 14:57 ` Tom Rini
2012-09-18 21:22 ` Kim Phillips
0 siblings, 1 reply; 6+ messages in thread
From: Tom Rini @ 2012-09-18 14:57 UTC (permalink / raw)
To: u-boot
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
On 09/17/12 20:09, Ira W. Snyder wrote:
> On Mon, Sep 17, 2012 at 06:18:12PM -0700, Tom Rini wrote:
>> On Mon, Sep 17, 2012 at 06:36:56PM -0500, Kim Phillips wrote:
>>> Hi Tom,
>>>
>>> Ira improved mpc8308rdb h/w support by making it more
>>> comprehensive. Please pull:
>>>
>>> The following changes since commit
>>> a6f0c4faa4c65a7b7048b12c9d180d7e1aad1721:
>>>
>>> Merge branch 'master' of git://git.denx.de/u-boot-avr32
>>> (2012-09-04 09:17:27 +0200)
>>>
>>> are available in the git repository at:
>>>
>>>
>>> git://git.denx.de/u-boot-mpc83xx.git master
>>>
>>> for you to fetch changes up to
>>> 014d1dfc669ab2295d7fa4ec4b9f00a4004917a0:
>>>
>>> mpc8308rdb: add support for eSDHC MMC controller (2012-09-17
>>> 17:47:12 -0500)
>>>
>>> ----------------------------------------------------------------
>>>
>>>
Ira W. Snyder (5):
>>> mpc8xxx_spi: fix SPI support on MPC8308RDB mpc8308rdb: add
>>> support for Spansion SPI flash on header J8 spansion: add
>>> support for S25FL256S1 mpc8308rdb: add support for FIT images
>>> mpc8308rdb: add support for eSDHC MMC controller
>>>
>>> board/freescale/mpc8308rdb/mpc8308rdb.c | 58
>>> +++++++++++++++++++++++++++++++++ drivers/mtd/spi/spansion.c
>>> | 8 +++++ drivers/spi/mpc8xxx_spi.c | 4 +++
>>> include/configs/MPC8308RDB.h | 31
>>> ++++++++++++++++++ 4 files changed, 101 insertions(+)
>>
>> A number of boards (P2020RDB-PC_36BIT is the one I looked at the
>> log in) now fail: spansion.c:74:3: error: unknown field
>> 'page_size' specified in initializer
>>
>> Rejected, please fix, sorry.
>>
>> -- Tom
>
> Tom,
>
> Go ahead and drop the patch "spansion: add support for S25FL256S1"
> from the series. The rest should apply without any trouble.
>
> Michal Simek beat me to it in "sf: spansion: Add support for
> S25FL256S" (4a4cb4e1114). He got the idcode1 wrong (it should be
> 0x0219 instead of 0x2019), but I saw a followup patch on the
> mailing list to fix it.
This is good to know. My preference however is for a new pull request
with whatever corrections Kim deems necessary. Thanks!
- --
Tom
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.11 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://www.enigmail.net/
iQIcBAEBAgAGBQJQWIvDAAoJENk4IS6UOR1WJ28P/1Kg3p/mTJdkgDFjKjzKURer
Nv9IGEMYEF5VxA7DY7CVDutCMGeeZIf+/oiU7gsBH0Qk/NxgOmn9GYzr/IDfAuQu
69DKyQ6M0n01yk9C4/w5GjT+cAfrp9oz30Y1hKaq7TOVp79pRDX/CRfxVp/Mk8RL
QkDq/QghCzIhC5VH/JP3544VbpSIKdMkTdOoTrzG7319YYZgvYrvEh0w4vBAnqtK
SB6zaIQZYelVFKsl5EnhcB2WCnwRc6jM23nd+wRycd/VsQanrQsr8xlTo11apvBm
JmXcNUNeVuRgZFpwISuzsaAdxgsUw9TifP2Di854RzG8GoVhUlcvKCV0KWD8Q2WA
5/9O2sl7fKYmAsYCQWrjdpHBjSJZpFOwA3E0K+aohCCPX840i5imOZZWlyOr2VTF
k20dUpcIkmi1c8gaJGkHJzsRYhragY8pWTpIxBN3PFjc/MGSWB8loRseg/rJ2BDi
fb2KAVbcp9j/vF60FDc4LSuSIaIdw3tbsAlC/mk4NLRCqPdFJUYYm0pwyAoZycFK
eFb+73i+vVSUkBThQuJrGW+t9EdVgy8xO2Be7dDjqgAeD166YnQynaicfSfe6Eck
lkE5ExuW3ehUDU7ooEMApYXMKkP+m6YrlGBkoN/ShzLsCHbaBejEDtYJfWp1WYaI
3pTd7PFAQM+CnuAZjLK9
=/6OP
-----END PGP SIGNATURE-----
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [GIT PULL] mpc83xx: mpc8308rdb enhancements
2012-09-18 14:57 ` Tom Rini
@ 2012-09-18 21:22 ` Kim Phillips
2012-09-18 23:28 ` Tom Rini
0 siblings, 1 reply; 6+ messages in thread
From: Kim Phillips @ 2012-09-18 21:22 UTC (permalink / raw)
To: u-boot
On Tue, 18 Sep 2012 07:57:08 -0700
Tom Rini <trini@ti.com> wrote:
> On 09/17/12 20:09, Ira W. Snyder wrote:
> > Go ahead and drop the patch "spansion: add support for S25FL256S1"
> > from the series. The rest should apply without any trouble.
> >
> > Michal Simek beat me to it in "sf: spansion: Add support for
> > S25FL256S" (4a4cb4e1114). He got the idcode1 wrong (it should be
> > 0x0219 instead of 0x2019), but I saw a followup patch on the
> > mailing list to fix it.
>
> This is good to know. My preference however is for a new pull request
> with whatever corrections Kim deems necessary. Thanks!
rebased on current master, with Ira's spansion patch omitted.
Please pull:
The following changes since commit 037e9d33db9fc86c1e671d64077b2d1a19df7f24:
Merge branch 'master' of git://git.denx.de/u-boot-i2c (2012-09-17 11:39:03 -0700)
are available in the git repository at:
git://git.denx.de/u-boot-mpc83xx.git master
for you to fetch changes up to db1fc7d28e8947c402149ded4597970fdb3e5571:
mpc8308rdb: add support for eSDHC MMC controller (2012-09-18 16:16:45 -0500)
----------------------------------------------------------------
Ira W. Snyder (4):
mpc8xxx_spi: fix SPI support on MPC8308RDB
mpc8308rdb: add support for Spansion SPI flash on header J8
mpc8308rdb: add support for FIT images
mpc8308rdb: add support for eSDHC MMC controller
board/freescale/mpc8308rdb/mpc8308rdb.c | 58 +++++++++++++++++++++++++++++++++
drivers/spi/mpc8xxx_spi.c | 4 +++
include/configs/MPC8308RDB.h | 31 ++++++++++++++++++
3 files changed, 93 insertions(+)
Thanks,
Kim
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [GIT PULL] mpc83xx: mpc8308rdb enhancements
2012-09-18 21:22 ` Kim Phillips
@ 2012-09-18 23:28 ` Tom Rini
0 siblings, 0 replies; 6+ messages in thread
From: Tom Rini @ 2012-09-18 23:28 UTC (permalink / raw)
To: u-boot
On Tue, Sep 18, 2012 at 04:22:01PM -0500, Kim Phillips wrote:
> On Tue, 18 Sep 2012 07:57:08 -0700
> Tom Rini <trini@ti.com> wrote:
>
> > On 09/17/12 20:09, Ira W. Snyder wrote:
> > > Go ahead and drop the patch "spansion: add support for S25FL256S1"
> > > from the series. The rest should apply without any trouble.
> > >
> > > Michal Simek beat me to it in "sf: spansion: Add support for
> > > S25FL256S" (4a4cb4e1114). He got the idcode1 wrong (it should be
> > > 0x0219 instead of 0x2019), but I saw a followup patch on the
> > > mailing list to fix it.
> >
> > This is good to know. My preference however is for a new pull request
> > with whatever corrections Kim deems necessary. Thanks!
>
> rebased on current master, with Ira's spansion patch omitted.
> Please pull:
>
> The following changes since commit 037e9d33db9fc86c1e671d64077b2d1a19df7f24:
>
> Merge branch 'master' of git://git.denx.de/u-boot-i2c (2012-09-17 11:39:03 -0700)
>
> are available in the git repository at:
>
> git://git.denx.de/u-boot-mpc83xx.git master
>
> for you to fetch changes up to db1fc7d28e8947c402149ded4597970fdb3e5571:
>
> mpc8308rdb: add support for eSDHC MMC controller (2012-09-18 16:16:45 -0500)
>
> ----------------------------------------------------------------
> Ira W. Snyder (4):
> mpc8xxx_spi: fix SPI support on MPC8308RDB
> mpc8308rdb: add support for Spansion SPI flash on header J8
> mpc8308rdb: add support for FIT images
> mpc8308rdb: add support for eSDHC MMC controller
>
> board/freescale/mpc8308rdb/mpc8308rdb.c | 58 +++++++++++++++++++++++++++++++++
> drivers/spi/mpc8xxx_spi.c | 4 +++
> include/configs/MPC8308RDB.h | 31 ++++++++++++++++++
> 3 files changed, 93 insertions(+)
Applied to u-boot/master, thanks. In the future, please start a new
thread, it helps me track. Thanks again!
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20120918/dabad604/attachment.pgp>
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2012-09-18 23:28 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-09-17 23:36 [U-Boot] [GIT PULL] mpc83xx: mpc8308rdb enhancements Kim Phillips
2012-09-18 1:18 ` Tom Rini
2012-09-18 3:09 ` Ira W. Snyder
2012-09-18 14:57 ` Tom Rini
2012-09-18 21:22 ` Kim Phillips
2012-09-18 23:28 ` Tom Rini
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox