Re: [PATCH] serial: tegra: Don't print error on probe deferral

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Jul 03, 2023 at 12:37:59PM +0100, Jon Hunter wrote:
> If the Tegra serial driver is probe before clocks are available then the
> following error is seen on boot:
> 
>  serial-tegra 3100000.serial: Couldn't get the clock
> 
> This has been observed on Jetson AGX Orin. Fix this by calling
> dev_err_probe() instead of dev_err() to avoid printing an error on probe
> deferral.
> 
> Fixes: e9ea096dd225 ("serial: tegra: add serial driver")
> Signed-off-by: Jon Hunter <jonathanh@xxxxxxxxxx>
> ---
>  drivers/tty/serial/serial-tegra.c | 6 ++----
>  1 file changed, 2 insertions(+), 4 deletions(-)

Patch looks good to me, though I'm not sure the Fixes: tag is useful. It
points to a commit that is 10 years old and the error message is
probably not worth backporting to every single actively maintained
stable release. I'm not sure it's worth backporting at all because it's
purely cosmetic.

That's up to Greg and Jiri, so from the Tegra side:

Acked-by: Thierry Reding <treding@xxxxxxxxxx>

Attachment: signature.asc
Description: PGP signature


[Index of Archives]     [ARM Kernel]     [Linux ARM]     [Linux ARM MSM]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux