External email: Use caution opening links or attachments
21.01.2020 07:41, Sameer Pujar ÐÐÑÐÑ:
On 1/20/2020 9:28 PM, Dmitry Osipenko wrote:I'll help with the testing once all obvious problems will be fixed.
External email: Use caution opening links or attachmentsOh yes! seems like I overlooked the macro values. Thanks for pointing
Hello Sameer,
20.01.2020 17:23, Sameer Pujar ÐÐÑÐÑ:
[snip]
Tegra30 and Tegra124 have an identical CIF programming helper function.[snip]
-#define TEGRA124_AUDIOCIF_CTRL_FIFO_THRESHOLD_SHIFT 24The AUDIOCIF_CTRL bitfields are not the same on T30 and T124, why are
-#define TEGRA124_AUDIOCIF_CTRL_FIFO_THRESHOLD_MASK_US 0x3f
-#define TEGRA124_AUDIOCIF_CTRL_FIFO_THRESHOLD_MASK
(TEGRA124_AUDIOCIF_CTRL_FIFO_THRESHOLD_MASK_US <<
TEGRA124_AUDIOCIF_CTRL_FIFO_THRESHOLD_SHIFT)
-
-/* Channel count minus 1 */
-#define TEGRA30_AUDIOCIF_CTRL_AUDIO_CHANNELS_SHIFT 24
-#define TEGRA30_AUDIOCIF_CTRL_AUDIO_CHANNELS_MASK_US 7
-#define TEGRA30_AUDIOCIF_CTRL_AUDIO_CHANNELS_MASK
(TEGRA30_AUDIOCIF_CTRL_AUDIO_CHANNELS_MASK_US <<
TEGRA30_AUDIOCIF_CTRL_AUDIO_CHANNELS_SHIFT)
you claiming that programming is identical? Have you actually tried to
test these patches on T30?
this. I will retain separate CIF function for Tegra30.
I do not have a Tegra30 board with me and hence could not test anything
specific to it apart from build sanity.
If someone can help me test I would really appreciate.