システムメンテナンスのお知らせ

現在ATmega88というアトメル社製のマイコンのプログラミングを行っています。コンパイラはwin_avrを使用しています。

現在void tx_test(cc1101_client * client)というcc1101_clientポインターを引数とした関数を作成したのですが、この関数内でu16 tx_fifo_set(txfifo_data * txfifo)という txfifo_dataポインターを引数とする関数を呼び出したいと思っているのですが、これをコンパイルすると

error: incompatible type for argument 1 of 'tx_fifo_set'というエラーが出力されコンパイルできない状態になります。なぜこのようなエラーになるのでしょうか?





u16 tx_fifo_set(txfifo_data * txfifo)
{


-------------------(中略)-------------------------------------------------------------


}








void tx_test(cc1101_client * client)
{
u8 i=0;
u8 new_input_data=0;

-------------------(中略)-------------------------------------------------------------


while(1)
{
switch(status)
{


-------------------(中略)-------------------------------------------------------------

case FIFO_SETUP:
if(cnt_sent_packet > 100)
{
status = DATA_WAVEOUT_QUIT;
break;
}


//alert_data_get(&client);

cc1100_cmd_flush_tx();
cc1100_cmd_idle();

cc1100_cmd_calibrate();
cc1100_cfg_gdo0(CC1100_GDOx_SYNC_WORD);
mdelay(5);

client->status.sessionflag = 0x02;//SYN flag set
client->status.ivent_flag = 0xa5;


//tx_length = tx_fifo_set(client);
//tx_fifo_set(&client);
tx_fifo_set(client->txfifo);


udelay(10);

#if 1
cc1100_read_reg_uartout((CC1100_REG_TXBYTES | CC1100_ACCESS_STATUS));
#endif

status = DATA_WAVEOUT;
break;


-------------------(中略)-------------------------------------------------------------


}

gooドクター

A 回答 (5件)

struct _cc1101_client_ で、



> txfifo_data txfifo;

と定義されてるんだから、client->txfifoはtxfifo_data*じゃなくてtxfifo_dataでしょう。

> tx_fifo_set(&client->txfifo);

とすればいいのでは?

> これらのポインター構造を乗せさせて頂きます。

という言い回しも意味不明だし、ポインターについての理解が不足しているのではないですか?
    • good
    • 0
この回答へのお礼

回答頂きありがとうございました。コンパイルエラーを回避することができました。ポインターについての理解を早急に行いたいと思います。助かりました。

お礼日時:2011/02/25 14:45

#1です。



もう他の方が回答なさっておられるので、正解は
tx_fifo_set(&(client->txfifo));
になります。

client内のtxfifoは実体、すなわち、
txfifo_dataであってtxfifo_data*ではないですから。

したがって、client内のtxfifoのアドレスを引き渡す必要があります。

たぶん学生さんだと思いますが、
1)エラーメッセージの読み方を学ぶ
からはじめて
「ポインタの概念を理解する」
までたどり着けば問題ないと思います。

蛇足ながら、
初心者向けには日経BPの「プログラムはなぜ動くのか」がおすすめです。
(というか、学生さんには無関係ですが、このくらいも知らないで仕事(組み込み系)はまともにできないと思いますので)
    • good
    • 0
この回答へのお礼

回答頂きありがとうございました。コンパイルエラーを回避することができました。ポインターについての理解を早急に行いたいと思います。助かりました。

ご紹介頂きました書籍も参考にさせて頂きます。

お礼日時:2011/02/25 14:46

tx_fifo_set(client);


tx_fifo_set(&client);
tx_fifo_set(client->txfifo);

これが全部だめだったなら、次にやってみるのは、

tx_fifo_set(&client->txfifo);

多分、

strcut cc1101_client
{
// この中で、
txfifo_data txfifo;
// とか書いているのでしょう(憶測)
}

もしも、
txfifo_data *txfifo;
と書かれていたら、
tx_fifo_set(client->txfifo);
で通るはずだし。

と思ったら、そのとおりだったようで。
    • good
    • 0
この回答へのお礼

回答頂きありがとうございました。コンパイルエラーを回避することができました。ポインターについての理解を早急に行いたいと思います。助かりました。

お礼日時:2011/02/25 14:45

> tx_fifo_set(client->txfifo);


の「client->txfifo」が本当に「cc1101_client」型のポインタなのかですかね。
後、本題には全然関係ないんですが、構造体の名称に「cc1101_client」とかつけて分かりにくくありませんか?

この回答への補足

なお、こちらのサイトより現在問題になっているATmega88のプロジェクトとソースコードがダウンロードできるようになっております。もしよろしければ、おつきあいどうぞよろしくお願い致します。

補足日時:2011/02/23 11:58
    • good
    • 0
この回答へのお礼

回答頂きありがとうございます。ちょっとポインタの名称の付け方についても勉強不足で簡単に考え過ぎているかもしれないです。その点も考え直してみたいと思います。

お礼日時:2011/02/23 11:56

Cの標準規格に沿っていると仮定しても、cc1101_clientの(たぶん構造体?)中身が書いていないことには回答できないと思いますよ。

あまり一般的なものでもなさそうだし。

ありそうなのは、中身が実体かポインタのポインタでつけるべき記号がついていない、辺りがくさいと思いますが。

というか、メッセージがそのまま回答だと。

この回答への補足

なお、こちらのサイトより現在問題になっているATmega88のプロジェクトとソースコードがダウンロードできるようになっております。もしよろしければ、おつきあいどうぞよろしくお願い致します。

URL書き忘れてしまい補足をこちらで書かせて頂きます。失礼しました。
http://sourceforge.jp/projects/cc1101driver/svn/ …

ATmega88のフォルダが今回の問題になっているソースコードです。

補足日時:2011/02/23 11:59
    • good
    • 0
この回答へのお礼

回答頂きありがとうございます。
一応、今回みて頂いた
txfifo_data
cc1101_client
これらのポインター構造を乗せさせて頂きます。

typedef struct _ctl_data_
{

u8 adj_freq;
u8 add_sub;
u8 sleep_transmit_time;

}ctl_data;



typedef struct _rtc_time_
{

u8 chk_hour;
u8 chk_min;
u8 curt_hour;
u8 curt_min;

}rtc_time;



typedef struct _txfifo_data_
{
//u8 length;
uint16_t length;
u8 data[20];

}txfifo_data;


typedef struct _rxfifo_data_
{
//u8 length;
uint16_t length;
u8 data[20];
u8 rssi;
u8 lqi;

}rxfifo_data;



typedef struct _client_status_
{

//u8 tx_length;
u8 type;
u8 serial[6];
u8 sessionflag;
u8 ivent_flag;
u8 bat_val;
//u8 master_rssi;
//u8 master_lqi;
u8 reserve[8];

}client_status;



typedef struct _cc1101_client_
{
u8 i;
client_status status;
txfifo_data txfifo;
rxfifo_data rxfifo;
rtc_time rtc;
ctl_data ctl;

}cc1101_client;


書き方に問題があるだけなのではということなので、
u16 tx_fifo_set(txfifo_data * txfifo)
この関数の書き方についても今一度見直してみるつもりです。

助かります。

お礼日時:2011/02/23 11:30

お探しのQ&Aが見つからない時は、教えて!gooで質問しましょう!

このQ&Aを見た人はこんなQ&Aも見ています

gooドクター

このQ&Aを見た人がよく見るQ&A

人気Q&Aランキング