Closed Bug 907046 Opened 11 years ago Closed 7 years ago

Remove libnfc-nxp from B2G base manifest

Categories

(Firefox OS Graveyard :: NFC, defect)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: allstars.chh, Unassigned)

Details

Currently there are some devices which don't use libnfc-nxp as their NFC libraries. 
So we should remove libnfc-nxp from base manifest, and only enable it for some particular devices.
AFAIK, Nexus-S uses libnfc-nxp.

Michael, do you know what devices other than Nexus-S use libnfc-nxp?

thanks
No idea. However, if we're not planning to support libnfc-nxp, we can remove it regardless since it'll make no difference.
Oh, wait
From Nexus-4, there's also a library called libnfc_ndef.so, which seems from libnfc_nxp.
using readelf to dump the symbols

Symbol table '.dynsym' contains 11 entries:
   Num:    Value  Size Type    Bind   Vis      Ndx Name
     0: 00000000     0 NOTYPE  LOCAL  DEFAULT  UND 
     1: 00000000     0 FUNC    GLOBAL DEFAULT  UND __cxa_finalize
     2: 00000000     0 FUNC    GLOBAL DEFAULT  UND __cxa_atexit
     3: 00000000     0 FUNC    GLOBAL DEFAULT  UND __aeabi_unwind_cpp_pr0
     4: 000004e3   472 FUNC    GLOBAL DEFAULT    8 phFriNfc_NdefRecord_GetRecords
     5: 000006bb    68 FUNC    GLOBAL DEFAULT    8 phFriNfc_NdefRecord_GetLength
     6: 00000701   220 FUNC    GLOBAL DEFAULT    8 phFriNfc_NdefRecord_Parse
     7: 000007dd   264 FUNC    GLOBAL DEFAULT    8 phFriNfc_NdefRecord_Generate
     8: 00002004     0 NOTYPE  GLOBAL DEFAULT  ABS _edata
     9: 00002004     0 NOTYPE  GLOBAL DEFAULT  ABS __bss_start
    10: 00002004     0 NOTYPE  GLOBAL DEFAULT  ABS _end


Yes, it's from libnfc-nxp, I think.
No longer blocks: b2g-nfc
Sony Rhine devices do use NXP
Flags: needinfo?(afarden)
Yes, and numerous other devices use nxp. If we wish to port Firefox OS to those devices (as per the new goal of porting ALL the things!) we will need to provide support here for nxp based nfc devices.
Flags: needinfo?(afarden)
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.