Skip to content

Commit 3d237a8

Browse files
committed
phy: Document what the various pss_signal_* fields represent
Between the padding and conjugation applied, it's non-obvious what the contents of these fields are. Also get rid of unused pss_signal_time_scale
1 parent ec29b0c commit 3d237a8

File tree

1 file changed

+7
-4
lines changed
  • lib/include/srsran/phy/sync

1 file changed

+7
-4
lines changed

lib/include/srsran/phy/sync/pss.h

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -76,12 +76,15 @@ typedef struct SRSRAN_API {
7676
uint32_t frame_size;
7777
uint32_t N_id_2;
7878
uint32_t fft_size;
79-
cf_t* pss_signal_freq_full[3];
8079

81-
cf_t* pss_signal_time[3];
82-
cf_t* pss_signal_time_scale[3];
80+
// pss_signal_freq: One raw ZC sequence for each N_id_2, without the DC bin
81+
// pss_signal_time: conj(IDFT{pss_signal_freq outer frequencies padded to fft_size}) right-padded to frame_size+fft_size
82+
// pss_signal_freq_full: DFT{pss_signal_time}
83+
// Note: Conjugate is applied so we can find the correlation with convolution functions
84+
cf_t pss_signal_freq[3][SRSRAN_PSS_LEN];
85+
cf_t* pss_signal_time[3];
86+
cf_t* pss_signal_freq_full[3];
8387

84-
cf_t pss_signal_freq[3][SRSRAN_PSS_LEN]; // One sequence for each N_id_2
8588
cf_t* tmp_input;
8689
cf_t* conv_output;
8790
float* conv_output_abs;

0 commit comments

Comments
 (0)