sm_io/chips/*: change si57x set_freq () to receive a pointer

In this way, we can standardize how the functions are called and handled.

sm_io/chips/*: change si57x set_freq () to receive a pointer
In this way, we can standardize how the functions are called and handled.
3c467e7e · Lucas Russo · f95982ad · 3c467e7e · 3c467e7e
Commit 3c467e7e authored Apr 26, 2016 by Lucas Russo
Hide whitespace changes
Inline Side-by-side

Showing with 5 additions and 2 deletions

sm_ch_si57x.h include/sm_ch_si57x.h +1 -1

sm_ch_si57x.c src/sm_io/chips/sm_ch_si57x.c +4 -1

No files found.
--- a/include/sm_ch_si57x.h
+++ b/include/sm_ch_si57x.h
@@ -40,7 +40,7 @@ smch_err_e smch_si57x_get_divs (smch_si57x_t *self, uint64_t *rfreq,
 smch_err_e smch_si57x_get_defaults (smch_si57x_t *self, double fout);

 /* Setup new frequency */
-smch_err_e smch_si57x_set_freq (smch_si57x_t *self, double frequency);
+smch_err_e smch_si57x_set_freq (smch_si57x_t *self, double *freq);
 /* Get current frequency */
 smch_err_e smch_si57x_get_freq (smch_si57x_t *self, double *freq);


--- a/src/sm_io/chips/sm_ch_si57x.c
+++ b/src/sm_io/chips/sm_ch_si57x.c
@@ -164,10 +164,13 @@ smch_err_e smch_si57x_get_defaults (smch_si57x_t *self, double fout)
    return _smch_si57x_get_defaults (self, fout);
 }

-smch_err_e smch_si57x_set_freq (smch_si57x_t *self, double frequency)
+smch_err_e smch_si57x_set_freq (smch_si57x_t *self, double *freq)
 {
    assert (self);
+    assert (freq);
+
    smch_err_e err = SMCH_SUCCESS;
+    double frequency = *freq;

    ASSERT_TEST(frequency > 0, "Invalid frequency (0 Hz)", err_exit);