* [U-Boot] [PATCH 2/2] PPC MPC83xx: Fix MPC8323ERDB build warning
@ 2013-06-14 8:21 Wolfgang Denk
2013-06-14 9:14 ` Wolfgang Denk
` (2 more replies)
0 siblings, 3 replies; 15+ messages in thread
From: Wolfgang Denk @ 2013-06-14 8:21 UTC (permalink / raw)
To: u-boot
Fix:
mpc8323erdb.c: In function 'mac_read_from_eeprom':
mpc8323erdb.c:198:3: warning: dereferencing type-punned pointer will
break strict-aliasing rules [-Wstrict-aliasing]
Signed-off-by: Wolfgang Denk <wd@denx.de>
cc: Michael Barkowski <michael.barkowski@freescale.com>
cc: Kim Phillips <kim.phillips@freescale.com>
---
board/freescale/mpc8323erdb/mpc8323erdb.c | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/board/freescale/mpc8323erdb/mpc8323erdb.c b/board/freescale/mpc8323erdb/mpc8323erdb.c
index f29b2f4..533cb08 100644
--- a/board/freescale/mpc8323erdb/mpc8323erdb.c
+++ b/board/freescale/mpc8323erdb/mpc8323erdb.c
@@ -195,7 +195,11 @@ int mac_read_from_eeprom(void)
printf("\nEEPROM @ 0x%02x read FAILED!!!\n",
CONFIG_SYS_I2C_EEPROM_ADDR);
} else {
- if (crc32(crc, buf, 24) == *(unsigned int *)&buf[24]) {
+ unsigned int crc_buf;
+
+ memcpy(&crc_buf, &buf[24], sizeof(unsigned int));
+
+ if (crc32(crc, buf, 24) == crc_buf) {
printf("Reading MAC from EEPROM\n");
for (i = 0; i < 4; i++) {
if (memcmp(&buf[i * 6], "\0\0\0\0\0\0", 6)) {
--
1.7.11.7
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [U-Boot] [PATCH 2/2] PPC MPC83xx: Fix MPC8323ERDB build warning
2013-06-14 8:21 [U-Boot] [PATCH 2/2] PPC MPC83xx: Fix MPC8323ERDB build warning Wolfgang Denk
@ 2013-06-14 9:14 ` Wolfgang Denk
2013-06-28 21:47 ` Andy Fleming
2013-06-27 6:22 ` Wolfgang Denk
2013-07-04 12:57 ` Timur Tabi
2 siblings, 1 reply; 15+ messages in thread
From: Wolfgang Denk @ 2013-06-14 9:14 UTC (permalink / raw)
To: u-boot
Dear Kim,
In message <1371198068-3581-1-git-send-email-wd@denx.de> you wrote:
>
> mpc8323erdb.c: In function 'mac_read_from_eeprom':
> mpc8323erdb.c:198:3: warning: dereferencing type-punned pointer will
> break strict-aliasing rules [-Wstrict-aliasing]
>
> Signed-off-by: Wolfgang Denk <wd@denx.de>
> cc: Michael Barkowski <michael.barkowski@freescale.com>
The address of the board maintainer bounces:
| Delivery has failed to these recipients or groups:
|
| Michael Barkowski
| (michael.barkowski at freescale.com)<mailto:michael.barkowski@freescale.com>
| The e-mail address you entered couldn't be found. Please check the
| recipient's e-mail address and try to resend the message. If the
| problem continues, please contact your helpdesk.
|
| The following organization rejected your message:
| tx30smr01.am.freescale.net.
Do you know if there is a new maintainer for this board, or should
we enter it as orphaned?
Best regards,
Wolfgang Denk
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
The IQ of the group is the lowest IQ of a member of the group divided
by the number of people in the group.
^ permalink raw reply [flat|nested] 15+ messages in thread
* [U-Boot] [PATCH 2/2] PPC MPC83xx: Fix MPC8323ERDB build warning
2013-06-14 8:21 [U-Boot] [PATCH 2/2] PPC MPC83xx: Fix MPC8323ERDB build warning Wolfgang Denk
2013-06-14 9:14 ` Wolfgang Denk
@ 2013-06-27 6:22 ` Wolfgang Denk
2013-07-04 12:57 ` Timur Tabi
2 siblings, 0 replies; 15+ messages in thread
From: Wolfgang Denk @ 2013-06-27 6:22 UTC (permalink / raw)
To: u-boot
Dear Kim,
ping!!
In message <1371198068-3581-1-git-send-email-wd@denx.de> I wrote:
> Fix:
>
> mpc8323erdb.c: In function 'mac_read_from_eeprom':
> mpc8323erdb.c:198:3: warning: dereferencing type-punned pointer will
> break strict-aliasing rules [-Wstrict-aliasing]
>
> Signed-off-by: Wolfgang Denk <wd@denx.de>
> cc: Michael Barkowski <michael.barkowski@freescale.com>
> cc: Kim Phillips <kim.phillips@freescale.com>
> ---
> board/freescale/mpc8323erdb/mpc8323erdb.c | 6 +++++-
> 1 file changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/board/freescale/mpc8323erdb/mpc8323erdb.c b/board/freescale/mpc8323erdb/mpc8323erdb.c
> index f29b2f4..533cb08 100644
> --- a/board/freescale/mpc8323erdb/mpc8323erdb.c
> +++ b/board/freescale/mpc8323erdb/mpc8323erdb.c
> @@ -195,7 +195,11 @@ int mac_read_from_eeprom(void)
> printf("\nEEPROM @ 0x%02x read FAILED!!!\n",
> CONFIG_SYS_I2C_EEPROM_ADDR);
> } else {
> - if (crc32(crc, buf, 24) == *(unsigned int *)&buf[24]) {
> + unsigned int crc_buf;
> +
> + memcpy(&crc_buf, &buf[24], sizeof(unsigned int));
> +
> + if (crc32(crc, buf, 24) == crc_buf) {
> printf("Reading MAC from EEPROM\n");
> for (i = 0; i < 4; i++) {
> if (memcmp(&buf[i * 6], "\0\0\0\0\0\0", 6)) {
> --
> 1.7.11.7
>
Best regards,
Wolfgang Denk
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Einstein argued that there must be simplified explanations of nature,
because God is not capricious or arbitrary. No such faith comforts
the software engineer. - Fred Brooks, Jr.
^ permalink raw reply [flat|nested] 15+ messages in thread
* [U-Boot] [PATCH 2/2] PPC MPC83xx: Fix MPC8323ERDB build warning
2013-06-14 9:14 ` Wolfgang Denk
@ 2013-06-28 21:47 ` Andy Fleming
2013-06-29 4:11 ` Timur Tabi
0 siblings, 1 reply; 15+ messages in thread
From: Andy Fleming @ 2013-06-28 21:47 UTC (permalink / raw)
To: u-boot
On Fri, Jun 14, 2013 at 4:14 AM, Wolfgang Denk <wd@denx.de> wrote:
> Dear Kim,
>
> In message <1371198068-3581-1-git-send-email-wd@denx.de> you wrote:
> >
> > mpc8323erdb.c: In function 'mac_read_from_eeprom':
> > mpc8323erdb.c:198:3: warning: dereferencing type-punned pointer will
> > break strict-aliasing rules [-Wstrict-aliasing]
> >
> > Signed-off-by: Wolfgang Denk <wd@denx.de>
> > cc: Michael Barkowski <michael.barkowski@freescale.com>
>
> The address of the board maintainer bounces:
>
> | Delivery has failed to these recipients or groups:
> |
> | Michael Barkowski
> | (michael.barkowski at freescale.com)<mailto:michael.barkowski@freescale.com
> >
> | The e-mail address you entered couldn't be found. Please check the
> | recipient's e-mail address and try to resend the message. If the
> | problem continues, please contact your helpdesk.
> |
> | The following organization rejected your message:
> | tx30smr01.am.freescale.net.
>
> Do you know if there is a new maintainer for this board, or should
> we enter it as orphaned?
>
Well, he hasn't been at Freescale for years, now, so that's not surprising.
It's not entirely clear to me whether that board is one we actively
support, but I suspect we can get away with maintenance moving over to Kim.
That said, I guess I shouldn't volunteer him. :)
Andy
^ permalink raw reply [flat|nested] 15+ messages in thread
* [U-Boot] [PATCH 2/2] PPC MPC83xx: Fix MPC8323ERDB build warning
2013-06-28 21:47 ` Andy Fleming
@ 2013-06-29 4:11 ` Timur Tabi
2013-06-29 6:04 ` Sinan Akman
2013-07-04 6:15 ` Wolfgang Denk
0 siblings, 2 replies; 15+ messages in thread
From: Timur Tabi @ 2013-06-29 4:11 UTC (permalink / raw)
To: u-boot
On Fri, Jun 28, 2013 at 4:47 PM, Andy Fleming <afleming@gmail.com> wrote:
> Well, he hasn't been at Freescale for years, now, so that's not surprising.
> It's not entirely clear to me whether that board is one we actively
> support, but I suspect we can get away with maintenance moving over to Kim.
> That said, I guess I shouldn't volunteer him. :)
There's been a lot of churn at Freescale over the past couple years.
I think a lot of the email addresses are no longer valid. It's
probably time for an update.
For the record, I actually have an mpc8323erdb. I'm not sure if I
want to be the maintainer for it, though.
^ permalink raw reply [flat|nested] 15+ messages in thread
* [U-Boot] [PATCH 2/2] PPC MPC83xx: Fix MPC8323ERDB build warning
2013-06-29 4:11 ` Timur Tabi
@ 2013-06-29 6:04 ` Sinan Akman
2013-07-04 6:15 ` Wolfgang Denk
1 sibling, 0 replies; 15+ messages in thread
From: Sinan Akman @ 2013-06-29 6:04 UTC (permalink / raw)
To: u-boot
On 06/29/2013 12:11 AM, Timur Tabi wrote:
> On Fri, Jun 28, 2013 at 4:47 PM, Andy Fleming <afleming@gmail.com> wrote:
>
>> Well, he hasn't been at Freescale for years, now, so that's not surprising.
>> It's not entirely clear to me whether that board is one we actively
>> support, but I suspect we can get away with maintenance moving over to Kim.
>> That said, I guess I shouldn't volunteer him. :)
>
> There's been a lot of churn at Freescale over the past couple years.
> I think a lot of the email addresses are no longer valid. It's
> probably time for an update.
>
> For the record, I actually have an mpc8323erdb. I'm not sure if I
> want to be the maintainer for it, though.
I have that board too and can run regular tests if that is
any help
-- sinan
^ permalink raw reply [flat|nested] 15+ messages in thread
* [U-Boot] [PATCH 2/2] PPC MPC83xx: Fix MPC8323ERDB build warning
2013-06-29 4:11 ` Timur Tabi
2013-06-29 6:04 ` Sinan Akman
@ 2013-07-04 6:15 ` Wolfgang Denk
1 sibling, 0 replies; 15+ messages in thread
From: Wolfgang Denk @ 2013-07-04 6:15 UTC (permalink / raw)
To: u-boot
Dear Timur Tabi,
In message <CAOZdJXWiKuGm5wOXQ5NwkPnBUgoEUSLOZPsbT1jJLn2c+M+kOw@mail.gmail.com> you wrote:
> On Fri, Jun 28, 2013 at 4:47 PM, Andy Fleming <afleming@gmail.com> wrote:
>
> > Well, he hasn't been at Freescale for years, now, so that's not surprising.
> > It's not entirely clear to me whether that board is one we actively
> > support, but I suspect we can get away with maintenance moving over to Kim.
> > That said, I guess I shouldn't volunteer him. :)
>
> There's been a lot of churn at Freescale over the past couple years.
> I think a lot of the email addresses are no longer valid. It's
> probably time for an update.
>
> For the record, I actually have an mpc8323erdb. I'm not sure if I
> want to be the maintainer for it, though.
Well, while you're discussing a potential new maintainer for this
board, can please anybody ACK/NAK or - even better - pull that patch
so we can fix the build warnings for the release?
Thanks.
Best regards,
Wolfgang Denk
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
This is now. Later is later.
^ permalink raw reply [flat|nested] 15+ messages in thread
* [U-Boot] [PATCH 2/2] PPC MPC83xx: Fix MPC8323ERDB build warning
2013-06-14 8:21 [U-Boot] [PATCH 2/2] PPC MPC83xx: Fix MPC8323ERDB build warning Wolfgang Denk
2013-06-14 9:14 ` Wolfgang Denk
2013-06-27 6:22 ` Wolfgang Denk
@ 2013-07-04 12:57 ` Timur Tabi
2013-07-08 18:19 ` [U-Boot] [PATCH 2/2 V2] " Wolfgang Denk
` (2 more replies)
2 siblings, 3 replies; 15+ messages in thread
From: Timur Tabi @ 2013-07-04 12:57 UTC (permalink / raw)
To: u-boot
On Fri, Jun 14, 2013 at 3:21 AM, Wolfgang Denk <wd@denx.de> wrote:
> diff --git a/board/freescale/mpc8323erdb/mpc8323erdb.c b/board/freescale/mpc8323erdb/mpc8323erdb.c
> index f29b2f4..533cb08 100644
> --- a/board/freescale/mpc8323erdb/mpc8323erdb.c
> +++ b/board/freescale/mpc8323erdb/mpc8323erdb.c
> @@ -195,7 +195,11 @@ int mac_read_from_eeprom(void)
> printf("\nEEPROM @ 0x%02x read FAILED!!!\n",
> CONFIG_SYS_I2C_EEPROM_ADDR);
> } else {
> - if (crc32(crc, buf, 24) == *(unsigned int *)&buf[24]) {
> + unsigned int crc_buf;
This should be a uint32_t, because you specifically want only four bytes.
If you fix that,
Acked-by: Timur Tabi <timur@tabi.org>
^ permalink raw reply [flat|nested] 15+ messages in thread
* [U-Boot] [PATCH 2/2 V2] PPC MPC83xx: Fix MPC8323ERDB build warning
2013-07-04 12:57 ` Timur Tabi
@ 2013-07-08 18:19 ` Wolfgang Denk
2013-07-08 22:04 ` Timur Tabi
2013-07-08 18:23 ` Wolfgang Denk
2013-07-14 17:42 ` [U-Boot] [PATCH 2/2 V3] " Wolfgang Denk
2 siblings, 1 reply; 15+ messages in thread
From: Wolfgang Denk @ 2013-07-08 18:19 UTC (permalink / raw)
To: u-boot
Fix:
mpc8323erdb.c: In function 'mac_read_from_eeprom':
mpc8323erdb.c:198:3: warning: dereferencing type-punned pointer will
break strict-aliasing rules [-Wstrict-aliasing]
Signed-off-by: Wolfgang Denk <wd@denx.de>
cc: Timur Tabi <timur@tabi.org>
cc: Kim Phillips <kim.phillips@freescale.com>
---
V2: use uint32_t for crc_buf to make sure we always get exactly
32 bit; thanks to Timur Tabi for pointing out.
board/freescale/mpc8323erdb/mpc8323erdb.c | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/board/freescale/mpc8323erdb/mpc8323erdb.c b/board/freescale/mpc8323erdb/mpc8323erdb.c
index f29b2f4..533cb08 100644
--- a/board/freescale/mpc8323erdb/mpc8323erdb.c
+++ b/board/freescale/mpc8323erdb/mpc8323erdb.c
@@ -195,7 +195,11 @@ int mac_read_from_eeprom(void)
printf("\nEEPROM @ 0x%02x read FAILED!!!\n",
CONFIG_SYS_I2C_EEPROM_ADDR);
} else {
- if (crc32(crc, buf, 24) == *(unsigned int *)&buf[24]) {
+ uint32_t crc_buf;
+
+ memcpy(&crc_buf, &buf[24], sizeof(unsigned int));
+
+ if (crc32(crc, buf, 24) == crc_buf) {
printf("Reading MAC from EEPROM\n");
for (i = 0; i < 4; i++) {
if (memcmp(&buf[i * 6], "\0\0\0\0\0\0", 6)) {
--
1.7.11.7
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [U-Boot] [PATCH 2/2 V2] PPC MPC83xx: Fix MPC8323ERDB build warning
2013-07-04 12:57 ` Timur Tabi
2013-07-08 18:19 ` [U-Boot] [PATCH 2/2 V2] " Wolfgang Denk
@ 2013-07-08 18:23 ` Wolfgang Denk
2013-07-14 17:42 ` [U-Boot] [PATCH 2/2 V3] " Wolfgang Denk
2 siblings, 0 replies; 15+ messages in thread
From: Wolfgang Denk @ 2013-07-08 18:23 UTC (permalink / raw)
To: u-boot
Fix:
mpc8323erdb.c: In function 'mac_read_from_eeprom':
mpc8323erdb.c:198:3: warning: dereferencing type-punned pointer will
break strict-aliasing rules [-Wstrict-aliasing]
Signed-off-by: Wolfgang Denk <wd@denx.de>
cc: Timur Tabi <timur@tabi.org>
cc: Kim Phillips <kim.phillips@freescale.com>
---
V2: use uint32_t for crc_buf to make sure we always get exactly
32 bit; thanks to Timur Tabi for pointing out.
board/freescale/mpc8323erdb/mpc8323erdb.c | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/board/freescale/mpc8323erdb/mpc8323erdb.c b/board/freescale/mpc8323erdb/mpc8323erdb.c
index f29b2f4..533cb08 100644
--- a/board/freescale/mpc8323erdb/mpc8323erdb.c
+++ b/board/freescale/mpc8323erdb/mpc8323erdb.c
@@ -195,7 +195,11 @@ int mac_read_from_eeprom(void)
printf("\nEEPROM @ 0x%02x read FAILED!!!\n",
CONFIG_SYS_I2C_EEPROM_ADDR);
} else {
- if (crc32(crc, buf, 24) == *(unsigned int *)&buf[24]) {
+ uint32_t crc_buf;
+
+ memcpy(&crc_buf, &buf[24], sizeof(unsigned int));
+
+ if (crc32(crc, buf, 24) == crc_buf) {
printf("Reading MAC from EEPROM\n");
for (i = 0; i < 4; i++) {
if (memcmp(&buf[i * 6], "\0\0\0\0\0\0", 6)) {
--
1.7.11.7
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [U-Boot] [PATCH 2/2 V2] PPC MPC83xx: Fix MPC8323ERDB build warning
2013-07-08 18:19 ` [U-Boot] [PATCH 2/2 V2] " Wolfgang Denk
@ 2013-07-08 22:04 ` Timur Tabi
2013-07-14 17:44 ` Wolfgang Denk
0 siblings, 1 reply; 15+ messages in thread
From: Timur Tabi @ 2013-07-08 22:04 UTC (permalink / raw)
To: u-boot
On 07/08/2013 01:19 PM, Wolfgang Denk wrote:
> + uint32_t crc_buf;
> +
> + memcpy(&crc_buf, &buf[24], sizeof(unsigned int));
Well, this is wrong, because it should be sizeof(uint32_t). Or maybe
sizeof(crc_buf).
--
Timur Tabi
^ permalink raw reply [flat|nested] 15+ messages in thread
* [U-Boot] [PATCH 2/2 V3] PPC MPC83xx: Fix MPC8323ERDB build warning
2013-07-04 12:57 ` Timur Tabi
2013-07-08 18:19 ` [U-Boot] [PATCH 2/2 V2] " Wolfgang Denk
2013-07-08 18:23 ` Wolfgang Denk
@ 2013-07-14 17:42 ` Wolfgang Denk
2013-07-14 22:24 ` Timur Tabi
2013-07-16 14:36 ` [U-Boot] [U-Boot, 2/2, " Tom Rini
2 siblings, 2 replies; 15+ messages in thread
From: Wolfgang Denk @ 2013-07-14 17:42 UTC (permalink / raw)
To: u-boot
Fix:
mpc8323erdb.c: In function 'mac_read_from_eeprom':
mpc8323erdb.c:198:3: warning: dereferencing type-punned pointer will
break strict-aliasing rules [-Wstrict-aliasing]
Signed-off-by: Wolfgang Denk <wd@denx.de>
cc: Timur Tabi <timur@tabi.org>
cc: Kim Phillips <kim.phillips@freescale.com>
---
V2: use uint32_t for crc_buf to make sure we always get exactly
32 bit; thanks to Timur Tabi for pointing out.
V3: also use uint32_t for the size of the memcpy(), again thanks to
Timur Tabi for pointing out.
board/freescale/mpc8323erdb/mpc8323erdb.c | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/board/freescale/mpc8323erdb/mpc8323erdb.c b/board/freescale/mpc8323erdb/mpc8323erdb.c
index f29b2f4..533cb08 100644
--- a/board/freescale/mpc8323erdb/mpc8323erdb.c
+++ b/board/freescale/mpc8323erdb/mpc8323erdb.c
@@ -195,7 +195,11 @@ int mac_read_from_eeprom(void)
printf("\nEEPROM @ 0x%02x read FAILED!!!\n",
CONFIG_SYS_I2C_EEPROM_ADDR);
} else {
- if (crc32(crc, buf, 24) == *(unsigned int *)&buf[24]) {
+ uint32_t crc_buf;
+
+ memcpy(&crc_buf, &buf[24], sizeof(uint32_t));
+
+ if (crc32(crc, buf, 24) == crc_buf) {
printf("Reading MAC from EEPROM\n");
for (i = 0; i < 4; i++) {
if (memcmp(&buf[i * 6], "\0\0\0\0\0\0", 6)) {
--
1.7.11.7
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [U-Boot] [PATCH 2/2 V2] PPC MPC83xx: Fix MPC8323ERDB build warning
2013-07-08 22:04 ` Timur Tabi
@ 2013-07-14 17:44 ` Wolfgang Denk
0 siblings, 0 replies; 15+ messages in thread
From: Wolfgang Denk @ 2013-07-14 17:44 UTC (permalink / raw)
To: u-boot
Dear Timur Tabi,
In message <51DB3787.4000308@tabi.org> you wrote:
> On 07/08/2013 01:19 PM, Wolfgang Denk wrote:
> > + uint32_t crc_buf;
> > +
> > + memcpy(&crc_buf, &buf[24], sizeof(unsigned int));
>
> Well, this is wrong, because it should be sizeof(uint32_t). Or maybe
> sizeof(crc_buf).
Indedd. Sorry. V3 on the way.
Thanks!
Best regards,
Wolfgang Denk
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Your own mileage may vary.
^ permalink raw reply [flat|nested] 15+ messages in thread
* [U-Boot] [PATCH 2/2 V3] PPC MPC83xx: Fix MPC8323ERDB build warning
2013-07-14 17:42 ` [U-Boot] [PATCH 2/2 V3] " Wolfgang Denk
@ 2013-07-14 22:24 ` Timur Tabi
2013-07-16 14:36 ` [U-Boot] [U-Boot, 2/2, " Tom Rini
1 sibling, 0 replies; 15+ messages in thread
From: Timur Tabi @ 2013-07-14 22:24 UTC (permalink / raw)
To: u-boot
Wolfgang Denk wrote:
> mpc8323erdb.c: In function 'mac_read_from_eeprom':
> mpc8323erdb.c:198:3: warning: dereferencing type-punned pointer will
> break strict-aliasing rules [-Wstrict-aliasing]
>
> Signed-off-by: Wolfgang Denk<wd@denx.de>
> cc: Timur Tabi<timur@tabi.org>
> cc: Kim Phillips<kim.phillips@freescale.com>
Ack.
--
Timur Tabi
^ permalink raw reply [flat|nested] 15+ messages in thread
* [U-Boot] [U-Boot, 2/2, V3] PPC MPC83xx: Fix MPC8323ERDB build warning
2013-07-14 17:42 ` [U-Boot] [PATCH 2/2 V3] " Wolfgang Denk
2013-07-14 22:24 ` Timur Tabi
@ 2013-07-16 14:36 ` Tom Rini
1 sibling, 0 replies; 15+ messages in thread
From: Tom Rini @ 2013-07-16 14:36 UTC (permalink / raw)
To: u-boot
On Sun, Jul 14, 2013 at 07:42:40PM +0200, Wolfgang Denk wrote:
> Fix:
>
> mpc8323erdb.c: In function 'mac_read_from_eeprom':
> mpc8323erdb.c:198:3: warning: dereferencing type-punned pointer will
> break strict-aliasing rules [-Wstrict-aliasing]
>
> Signed-off-by: Wolfgang Denk <wd@denx.de>
> cc: Timur Tabi <timur@tabi.org>
> cc: Kim Phillips <kim.phillips@freescale.com>
Applied to u-boot/master, thanks!
--
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/20130716/ef70527b/attachment.pgp>
^ permalink raw reply [flat|nested] 15+ messages in thread
end of thread, other threads:[~2013-07-16 14:36 UTC | newest]
Thread overview: 15+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-06-14 8:21 [U-Boot] [PATCH 2/2] PPC MPC83xx: Fix MPC8323ERDB build warning Wolfgang Denk
2013-06-14 9:14 ` Wolfgang Denk
2013-06-28 21:47 ` Andy Fleming
2013-06-29 4:11 ` Timur Tabi
2013-06-29 6:04 ` Sinan Akman
2013-07-04 6:15 ` Wolfgang Denk
2013-06-27 6:22 ` Wolfgang Denk
2013-07-04 12:57 ` Timur Tabi
2013-07-08 18:19 ` [U-Boot] [PATCH 2/2 V2] " Wolfgang Denk
2013-07-08 22:04 ` Timur Tabi
2013-07-14 17:44 ` Wolfgang Denk
2013-07-08 18:23 ` Wolfgang Denk
2013-07-14 17:42 ` [U-Boot] [PATCH 2/2 V3] " Wolfgang Denk
2013-07-14 22:24 ` Timur Tabi
2013-07-16 14:36 ` [U-Boot] [U-Boot, 2/2, " Tom Rini
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.