Hi Konrad, Thanks for confirming your preferences.
On 4/16/2024 5:14 AM, Konrad Dybcio wrote:
On 4/2/24 08:21, Mukesh Kumar Savaliya wrote:
Thanks Konrad for detailed review. For dt-bindings sending a separate
patch soon, rest comments tried to address and updated patch V2.
[...]
+ if (of_property_read_bool(pdev->dev.of_node, "qcom,shared-se")) {
+ gi2c->is_shared = true;
+ dev_info(&pdev->dev, "Multi-EE usecase with shared SE\n");
How would this line be useful in my kernel log?
It informs that particular SE is shared between SEs from two
subsystems, hence respective debug can happen accordingly in case of
the issue.
This amounts to "not very useful". As an end user, I couldn't care less
about the nitty-gritty of firmware-hardware interactions, so long as the
thing works. You must not spam the kernel log with debug messages, as it
slows things down and makes actually useful messages harder to spot. If
you want to keep it, use dev_dbg.
Sure, will go with the dev_dbg. I am uploading a separate patches for
i2c and GSI DMA sub system.
Konrad
