Re: [PATCH v2 1/5] Sound: SOC: TAS571x: added missing register literals

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

 



Hi Mark,

On 29.03.2016 23:33, Mark Brown wrote:
On Tue, Mar 29, 2016 at 09:39:34AM +0200, Petr Kulhavy wrote:

The list of TAS571x registers was incomplete.
Added the missing register definitions up to register 0x25
According to the driver the device has registers up to 0xff?

That is indeed true. But from address 0x29 on (0x26 to 0x28 are reserved) the register width varies between 20, 12 and 8 bytes, which I'm afraid the register map is unable to represent.

  /* device registers */
+#define TAS571X_CLK_CTRL_REG		0x00
+#define TAS571X_DEV_ID_REG		0x01
+#define TAS571X_ERR_STATUS_REG		0x02
+#define TAS571X_SYS_CTRL_1_REG		0x03
These look like volatile registers but the device has a register cache
and we're not adding a list of volatile registers (or readable registers
for that matter).
That's a good point, thanks! 0x03 is a regular RW register but 0x00 to 0x02 are indeed volatile.
Is it better to make them read-only, or volatile?

Petr
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel



[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux