Skip to content

Conversation

@caseorum
Copy link
Contributor

Replace some open coding with the new macro

Signed-off-by: Joseph Burt caseorum@gmail.com

Replace some open coding with the new macro

Signed-off-by: Joseph Burt <caseorum@gmail.com>
@sofci
Copy link
Collaborator

sofci commented Feb 10, 2021

Can one of the admins verify this patch?

@caseorum
Copy link
Contributor Author

@lgirdwood This is a very tiny change.

Copy link
Member

@lgirdwood lgirdwood left a comment

Choose a reason for hiding this comment

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

@caseorum thanks, fwiw I think the Linux driver may have a similar calculation.

@lgirdwood lgirdwood added this to the v1.7 milestone Feb 10, 2021
@lgirdwood
Copy link
Member

test this please

@caseorum
Copy link
Contributor Author

@lgirdwood Kernel side, pcm.c already uses uses the PFN_UP() macro and utils.c uses the snd_sgbuf_aligned_pages() function, which are functionally identical.

@lgirdwood lgirdwood merged commit df899fd into thesofproject:master Feb 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants