Re: [PATCH 13/13] ASoC: AMD: Manage ACP 2.x SRAM banks power

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Dec 04, 2015 at 06:40:40PM -0500, Alex Deucher wrote:

> +void acp_turnonoff_lower_sram_bank(void __iomem *acp_mmio, u16 bank,
> +					bool turnon)

Can we have a name like acp_set_lower_sram_bank_state() or something to
make the boolean a bit more obvious please?  You could probably also
combine this with the equivalent function for the upper bank and select
based on the value which registers to work with, that'd be much clearer
on the user side.

> +	/* If ACP_MEM_SHUT_DOWN_STS_LO is 0xFFFFFFFF, then
> +	 * shutdown sequence is complete.
> +	 */
> +	 while (acp_reg_read(acp_mmio,
> +				      mmACP_MEM_SHUT_DOWN_STS_LO)
> +				      != 0xFFFFFFFF)
> +		cpu_relax();

Strange intentation for the continuation lines (usually you'd align with
the '(' for the matching level) and this needs a timeout for safety.

Attachment: signature.asc
Description: PGP signature

_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/dri-devel

[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux