https://b.hatena.ne.jp/entry/s/xenospectrum.com/linus-torvalds-rejects-google-risc-v-code-garbage/

https://lore.kernel.org/lkml/CAHk-=wjLCqUUWd8DzG+xsOn-yVL0Q=O35U9D6j6=2DUWX52ghQ@mail.gmail.com/

https://git.kernel.org/pub/scm/linux/kernel/git/riscv/linux.git/commit/?h=riscv-for-linus-6.17-mw1

+#define SBI_MPXY_MSG_PROT_VER_MAJOR(__ver) upper_16_bits(__ver)

+#define SBI_MPXY_MSG_PROT_VER_MINOR(__ver) lower_16_bits(__ver)

+#define SBI_MPXY_MSG_PROT_MKVER(__maj, __min) make_u32_from_two_u16(__maj, __min)

 

 /**

+ * make_u32_from_two_u16 - return u32 number by combining

+ * two u16 numbers.

+ * @hi: upper 16 bit number

+ * @lo: lower 16 bit number

+ */

+#define make_u32_from_two_u16(hi, lo) (((u32)(hi) << 16) | (u32)(lo))

＞ 内容見るとゴミ

＞ AI が書いたんじゃないかってレベルのゴミコード

言うほどゴミか？

関数の定義うんぬんなんて、エディタでカーソルあてりゃもわっと出てくるでしょ。

定義場所が悪いかも？とは思うけど、コードはそこまでゴミではないと思うんだけど。

追記

upper_16_bits()の定義場所は、wordpart.h？

#define upper_16_bits(n) ((u16)((n) >> 16))

make_u32_from_two_u16()をwordpart.hにいれてもいいような気がしてきた。

https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/include/linux/wordpart.h

upper_16_bits()もdefineしないで、最初から展開しておいたほうがいいんじゃないですかﾜﾗ

追記

コードの内容も、定義場所もそこまで問題ではないように見える。一番は、提出期限ギリギリ（休暇中）ってとこなのでは・・