Skip to content

cpu, sam3: fix periph/dac poweron#8013

Merged
haukepetersen merged 1 commit intoRIOT-OS:masterfrom
smlng:cpu/sam3/fix_periph_dac
Nov 13, 2017
Merged

cpu, sam3: fix periph/dac poweron#8013
haukepetersen merged 1 commit intoRIOT-OS:masterfrom
smlng:cpu/sam3/fix_periph_dac

Conversation

@smlng
Copy link
Copy Markdown
Member

@smlng smlng commented Nov 13, 2017

factored out from #7995, as requested.

fixes poweron function for DAC of SAM3 cpu.

@smlng smlng added Area: drivers Area: Device drivers Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation labels Nov 13, 2017
@smlng smlng added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Nov 13, 2017
PMC->PMC_PCER1 = (1 << (ID_DACC - 32));
DACC->DACC_CHER = PMC_BIT;
DACC->DACC_CHER = (1 << line);
PMC->PMC_PCER1 = PMC_BIT;
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems incorrect to me: at least the order is broken now -> the module clock gate bit needs to be enabled before the actual module enable bit...

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

right, will fix

@smlng smlng force-pushed the cpu/sam3/fix_periph_dac branch from 3f48267 to eb8c55d Compare November 13, 2017 11:03
@smlng
Copy link
Copy Markdown
Member Author

smlng commented Nov 13, 2017

@haukepetersen fixed order

Copy link
Copy Markdown
Contributor

@haukepetersen haukepetersen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK

@haukepetersen
Copy link
Copy Markdown
Contributor

all green -> go

@haukepetersen haukepetersen merged commit ba6f2a4 into RIOT-OS:master Nov 13, 2017
@smlng smlng deleted the cpu/sam3/fix_periph_dac branch November 13, 2017 12:18
@aabadie aabadie added this to the Release 2018.01 milestone Jan 18, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: drivers Area: Device drivers CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants