Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

REALTEK_RTL8195AM IPv6 build fails #7829

Closed
juhaylinen opened this issue Aug 20, 2018 · 6 comments
Closed

REALTEK_RTL8195AM IPv6 build fails #7829

juhaylinen opened this issue Aug 20, 2018 · 6 comments

Comments

@juhaylinen
Copy link
Contributor

juhaylinen commented Aug 20, 2018

Description

Target: REALTEK_RTL8195AM
Toolchain: GCC_ARM
mbed-cli 1.5.0
Mbed OS versions 5.9.0 - 5.9.5 and master (sha f15dbf2)
Steps to reproduce:
For example, import mbed-os-example-wifi

mbed import mbed-os-example-wifi
cd mbed-os-example-wifi

Enable IPv6

"target_overrides": {
        "*": {
            "platform.stdio-convert-newlines": true,
            "lwip.ipv4-enabled": false,
            "lwip.ipv6-enabled": true
        },

Build
mbed compile -t GCC_ARM -m REALTEK_RTL8195AM

Fixed in this commit #7056 but it seems to be missing from 5.9.0 - 5.9.5

This commit #7669 brings back the "dhcps.h" and "dhcps.c" files which cause the build issue in Mbed OS master

[ ] Question
[ ] Enhancement
[X] Bug

@ciarmcom
Copy link
Member

ARM Internal Ref: MBOTRIAGE-1534

@0xc0170
Copy link
Contributor

0xc0170 commented Aug 21, 2018

cc @ARMmbed/team-realtek

@samchuarm
Copy link

Hi @prashantrar @M-ichae-l , can you please help address this build fail asap? Thanks

@M-ichae-l
Copy link
Contributor

@samchuarm
For the #7669, It brings back the "dhcps.h" and "dhcps.c". Please ask the PR creator to remove the two files (remove empty folder also). RTL8195AM does not need the two files. Realtek team would like to review the related commit before it merged. Thank you!

@0xc0170
Copy link
Contributor

0xc0170 commented Aug 23, 2018

Thanks @M-ichae-l . I'll revert them soon

@samchuarm
Copy link

It seems this is fixed now?

0xc0170 added a commit that referenced this issue Aug 28, 2018
JarkkoPaso pushed a commit to JarkkoPaso/mbed-os that referenced this issue Aug 29, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants