Re: [PATCH v2 8/8] tty: serial: qcom_geni_serial: Add early console support

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

 



Hi Bjorn,

Thanks for pulling it into your tree and helping with the verification.

On 4/9/2018 5:46 PM, Bjorn Andersson wrote:
> On Mon 09 Apr 12:38 PDT 2018, Karthikeyan Ramasubramanian wrote:
> 
>> Add early console support in Qualcomm Technologies Inc., GENI based
>> UART controller.
>>
> 
> Hi Karthikeyan,
> 
> I pulled this into my SDM845 tree and added "earlycon" to my command
> line, the result is a working console up until the serial driver is
> probed at which point the console stops.
> 
> 
> I'm expecting that this should hand over to the normal console as it is
> probed, unless I specify keep_bootcon in which case I expect all log
> entries to be printed both through the earlycon and the normal console
> (i.e. show up twice on the uart).
I suspect the issue is something to do with clocks. I will look into it
further and fix in the next submission.

This is my current hypothesis: When the driver gets probed, it registers
the uart port which in turn invokes the console setup. The console setup
enables the "se" clock (geni_resources_on()->clk_prepare_enable()).
Enabling the "se" clock is disturbing the rate that the clock is already
on. Early console write is simultaneously going on while the console is
being set up and this clock disturbance is causing the halt.
> 
>> Signed-off-by: Karthikeyan Ramasubramanian <kramasub@xxxxxxxxxxxxxx>
>> Signed-off-by: Girish Mahadevan <girishm@xxxxxxxxxxxxxx>
> 
> This is a log of how the patch got to the list (and later to mainline),
> as such your name should be the last entry in this line and you should
> add an extra Co-Developed-by to indicate that you worked on this
> together with Girish.
Ok, I will do accordingly.
> 
> Regards,
> Bjorn
> 

Regards,
Karthik.
-- 
Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
a Linux Foundation Collaborative Project
--
To unsubscribe from this list: send the line "unsubscribe linux-serial" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux PPP]     [Linux FS]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Linmodem]     [Device Mapper]     [Linux Kernel for ARM]

  Powered by Linux