From: Kalle Valo <kvalo@kernel.org>
To: Alexander Wilhelm <alexander.wilhelm@westermo.com>
Cc: <ath11k@lists.infradead.org>
Subject: Re: ath11k: BigEndian platform support?
Date: Tue, 11 Jul 2023 11:49:54 +0300 [thread overview]
Message-ID: <87cz0y96j1.fsf@kernel.org> (raw)
In-Reply-To: <68290980-5bfb-c88c-be78-954f9591c135@westermo.com> (Alexander Wilhelm's message of "Thu, 29 Jun 2023 11:50:03 +0200")
Alexander Wilhelm <alexander.wilhelm@westermo.com> writes:
> I am trying to get the QCN9074 module to work on a BigEndian PowerPC
> platform. My question would be, has anyone done it yet? If not, what
> kind of effort would you estimate for porting or are there any
> firmware limitations?
This is a good question. The short answer is that it _might_ work with
QCN9074 but AFAIK nobody has tested it and I'm not really optimistic.
The long answer is that the big endian support in ath11k is implemented
in a weird way which I regret big time. The idea is that the firmware
does the translation instead of ath11k driver with this flag:
/* Host software's Copy Engine configuration. */
#ifdef __BIG_ENDIAN
#define CE_ATTR_FLAGS CE_ATTR_BYTE_SWAP_DATA
#else
#define CE_ATTR_FLAGS 0
#endif
But later I was told that not all firmware branches actually support
this feature, sigh. To my knowledge QCA6390 and WCN6855 firmwares do not
support this CE_ATTR_BYTE_SWAP_DATA but I'm hoping QCN9074 firmware
would support it. Grep for BIG_ENDIAN to see more big endian specific
changes.
In ath12k the endian support was implemented in a proper way using
__le32 type family and cpu_to_le32() & co macros, but it's also
untested. It's on my todo list to convert ath11k to do the same but no
idea when I'm able to work on it. Patches very welcome.
Do let me know if you test ath11k on big endian, I'm very curious to
know the results.
--
https://patchwork.kernel.org/project/linux-wireless/list/
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
--
ath11k mailing list
ath11k@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/ath11k
prev parent reply other threads:[~2023-07-11 8:50 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-06-29 9:50 ath11k: BigEndian platform support? Alexander Wilhelm
2023-07-11 8:49 ` Kalle Valo [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87cz0y96j1.fsf@kernel.org \
--to=kvalo@kernel.org \
--cc=alexander.wilhelm@westermo.com \
--cc=ath11k@lists.infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.