-
Notifications
You must be signed in to change notification settings - Fork 1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
drivers: Add FFA_CONSOLE based console driver for log #6843
Conversation
54a282e
to
0cedbe4
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @ysbnim, thanks for the patch! Please see my comments below.
6a1c24f
to
908df06
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One more comment on something I missed during my first review. With that:
Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
This console driver uses FFA_CONSOLE ABI to write the trace logs. If CFG_FFA_CONSOLE is enabled, OP-TEE will try to initialize the console driver that uses FFA interface to print trace logs. Signed-off-by: Sungbae Yoo <sungbaey@nvidia.com> Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm adding a few comments I'd like to have addressed in a new PR.
#include <drivers/serial.h> | ||
#include <kernel/thread_arch.h> | ||
|
||
#define FFA_CONSOLE_LOG_32 (0x8400008A) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is defined in core/arch/arm/include/ffa.h
already.
|
||
static void ffa_console_putc(struct serial_chip *chip __unused, int ch) | ||
{ | ||
thread_hvc(FFA_CONSOLE_LOG_32, 1, ch, 0); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The ABI supports printing more than one character at a time.
Can thread_smccc()
be used instead to support an SPMC at EL3?
This console driver uses FFA_CONSOLE ABI to write the trace logs.
If CFG_FFA_CONSOLE is enabled, OP-TEE will try to initialize the FFA console driver.