rts_pstor のパッチ検討 #台風そん

linux

ええと、drivers/staging/rts_pstor/debug.h に関わるナニがかなり微妙。影響があるらしいのは以下なソースファイルらしい。

  • ms.c
  • rtsx.c
  • rtsx_card.c
  • rtsx_chip.c
  • rtsx_scsi.c
  • rtsx_transport.c
  • sd.c
  • trace.h
  • xd.c

ms.c

おおよそ引数に struct rtsx_chip 型が渡されているので無理やり何とかなる模様。

struct rtsx_chip {
	rtsx_dev_t 		*rtsx;

ここから手繰るのかどうか。
つうかツクリ的にヘッダの方を直した方が良いのかな。つうかデバッグなマクロだから別にいいじゃん的気持ちになってきました。マクロの定義が以下なカンジになってて

#ifdef CONFIG_RTS_PSTOR_DEBUG
#define RTSX_DEBUGP(x...) printk(KERN_DEBUG RTSX_STOR x)
#define RTSX_DEBUGPN(x...) printk(KERN_DEBUG x)
#define RTSX_DEBUGPX(x...) printk(x)
#define RTSX_DEBUG(x) x
#else
#define RTSX_DEBUGP(x...)
#define RTSX_DEBUGPN(x...)
#define RTSX_DEBUGPX(x...)
#define RTSX_DEBUG(x)
#endif
  • RTSX_DEBUGPX は使われてない模様なので定義を削除
  • RTSX_DEBUGPN は RTSX_DEBUGP に統合な方向で修正

つうか以下なカンジで良いのかどうか。

#define RTSX_DEBUGP(dev, x...) dev_dbg(dev, x)

んで、呼び出しが以下なカンジになるのか。

	RTSX_DEBUGP(&chip->rtsx->pci->dev, "ms_transfer_tpc: tpc = 0x%x\n", tpc);

なんとなく気持ち的に赤黒木方面に行きつつあるなぁ (を