Mbed-os: EFM32PG12 AnalogOut driver not usable because of missing VDAC interface usage

Created on 18 Jun 2018  路  6Comments  路  Source: ARMmbed/mbed-os

Description

I recently observed that the EFM32PG12 target has no DEVICE_ANALOGOUT macro defined, so the AnalogOut driver class is not usable here.

The only EFM32 devices which have DEVICE_ANALOGOUT enabled are EFM32GG, EFM32LG and EFM32WG ...

When looking into the datasheets, I see that those have a "DAC", while the PG12 has a "VDAC".

In EMLIB there are two different interfaces for DAC and VDAC, but they look very similar.

Nevertheless, in file analogout_api.c in the EFM32 target implementation, only the DAC interface is used.

Is there a specific reason why the VDAC interface is not used on the EFM32PG12 instead of the DAC interface?

Issue request type

[x] Question
[ ] Enhancement
[ ] Bug

CLOSED silicon labs mirrored

Most helpful comment

No specific reason, other than it was probably an oversight during the porting work. I'll see what I can do to add this functionality.

They look very similar because VDAC is an improved version of DAC. It got renamed to VDAC (voltage-controlled output) since the introduction of the IDAC (current-controlled output) on some devices, to point out the different implementations.
[Mirrored to Jira]

All 6 comments

@ARMmbed/team-silabs Please review
[Mirrored to Jira]

No specific reason, other than it was probably an oversight during the porting work. I'll see what I can do to add this functionality.

They look very similar because VDAC is an improved version of DAC. It got renamed to VDAC (voltage-controlled output) since the introduction of the IDAC (current-controlled output) on some devices, to point out the different implementations.
[Mirrored to Jira]

Hi @stevew817 ... any update on this? ;)
[Mirrored to Jira]

Internal Jira reference: https://jira.arm.com/browse/IOTPART-5615

@stevew817 Did something happen on this issue?

Thank you for raising this issue. Please note we have updated our policies and
now only defects should be raised directly in GitHub. Going forward questions and
enhancements will be considered in our forums, https://forums.mbed.com/ . If this
issue is still relevant please re-raise it there.
This GitHub issue will now be closed.

Was this page helpful?
0 / 5 - 0 ratings