みんなの「教えて(疑問・質問)」にみんなで「答える」Q&Aコミュニティ

こんにちはゲストさん。会員登録(無料)して質問・回答してみよう!

解決済みの質問

ポインター引数の関数でコンパイルエラーが出る。

現在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;


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


}

投稿日時 - 2011-02-23 10:21:39

QNo.6544054

すぐに回答ほしいです

質問者が選んだベストアンサー

struct _cc1101_client_ で、

> txfifo_data txfifo;

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

> tx_fifo_set(&client->txfifo);

とすればいいのでは?

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

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

投稿日時 - 2011-02-23 12:04:58

お礼

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

投稿日時 - 2011-02-25 14:45:45

このQ&Aは役に立ちましたか?

0人が「このQ&Aが役に立った」と投票しています

回答(5)

ANo.5

#1です。

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

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

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

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

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

投稿日時 - 2011-02-23 12:32:37

お礼

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

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

投稿日時 - 2011-02-25 14:46:23

ANo.4

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);
で通るはずだし。

と思ったら、そのとおりだったようで。

投稿日時 - 2011-02-23 12:08:42

お礼

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

投稿日時 - 2011-02-25 14:45:55

ANo.2

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

投稿日時 - 2011-02-23 11:26:36

補足

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

投稿日時 - 2011-02-23 11:58:07

お礼

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

投稿日時 - 2011-02-23 11:56:09

ANo.1

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

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

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

投稿日時 - 2011-02-23 10:49:30

補足

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

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

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

投稿日時 - 2011-02-23 11:59:43

お礼

回答頂きありがとうございます。
一応、今回みて頂いた
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:07

あなたにオススメの質問