Received: by 2002:a05:6a10:f347:0:0:0:0 with SMTP id d7csp2346224pxu; Mon, 7 Dec 2020 04:25:26 -0800 (PST) X-Google-Smtp-Source: ABdhPJx4Qt6O7BB3uzSkZZeaLfBAO7Ldtk8BH6GPi4vWmoLoAYIf+SdHv5R4ZjLQrPAXfUIEYtO+ X-Received: by 2002:a17:907:720e:: with SMTP id dr14mr18744097ejc.303.1607343926040; Mon, 07 Dec 2020 04:25:26 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1607343926; cv=none; d=google.com; s=arc-20160816; b=di1CxHjg70how4F9yBPbC+A8YZsi7kX06iNc6DK1ylkhtcEHVJc0mj1htVIb/ziZ/f kav/Oqmcx0rVJm/C+Je5lKm2pmVJOx6vAuyAzlcXukImiwyO9nGwIFTGVNTSjuHqC5x+ 3daigylAxW3h7wKJ+4hPEcRA6VrSgBkN8SuBvS4OBJjgE181FLWnQBu5hz15QjNja+zr OVb01WkQGGS6xwrMMhzNOaFPayUFWS6VUSsrFBmuXZhSplC4LtRIDnNGNTwOyNJySDVm 9jaiYMNq9/W7YPSDoil0Ufj3guxPEV2IBLbqiuXWkRXikr2W/lt5ZVAkZMJxvIkneA+k vrjA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-language:content-transfer-encoding :in-reply-to:mime-version:user-agent:date:message-id:from:references :cc:to:subject:dmarc-filter:sender:dkim-signature; bh=1SikiSOLDWxZ6yMf43VC0ekizit3Ea0q7Thf5whM7Ok=; b=G3827EwZN5cQtSuMb/5bro3udoYZwGZ6jK5hNUoSc95iCKaL/EQNP9963h+chFvrla +4bmYM7c3IjI1IVRl+1AvzgZF9bKQOG5i0hvoCPELhj9F+4fGsHGZca/i2lPdkQQQT48 Lkaxo4WIX2wU/RTsuWx98NZ0yNDMI//2aLaByU0ER4fP+CLticFeo/YY/7GP8RJEePE/ zEC1qfbYN3/DKtcbt27UO65IoVmkbTv5oc+FZ3ZGUENvjOtNvspeiP7TTaHYKQ11Rk+7 P2kaWHmFBqdzWZfRxb8K8VssLGjQfFa76EvORMWV1h1YUcBMJRLAzE93q3M7tq7faZio Q/Ng== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@mg.codeaurora.org header.s=smtp header.b=g1M7+W6S; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id j9si6311536ejk.570.2020.12.07.04.25.03; Mon, 07 Dec 2020 04:25:26 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@mg.codeaurora.org header.s=smtp header.b=g1M7+W6S; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727460AbgLGMXm (ORCPT + 99 others); Mon, 7 Dec 2020 07:23:42 -0500 Received: from so254-31.mailgun.net ([198.61.254.31]:27130 "EHLO so254-31.mailgun.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727182AbgLGMXm (ORCPT ); Mon, 7 Dec 2020 07:23:42 -0500 DKIM-Signature: a=rsa-sha256; v=1; c=relaxed/relaxed; d=mg.codeaurora.org; q=dns/txt; s=smtp; t=1607343804; h=Content-Transfer-Encoding: Content-Type: In-Reply-To: MIME-Version: Date: Message-ID: From: References: Cc: To: Subject: Sender; bh=1SikiSOLDWxZ6yMf43VC0ekizit3Ea0q7Thf5whM7Ok=; b=g1M7+W6SnsrQSwoXr42O9Y1b1cqlgWZ+F4SCT4Olp4msxAjhvth4QUXou+jwh8v7OZVVPcj+ exTqM7twS8GxPmi6Rlnf+x9alKg0oMJHmpCIS/z6Vrj722XQo7RoRKoIOoZDcAuDMTQM64hV OigxrvSbFsLX0LXEJVrbEpwgtIo= X-Mailgun-Sending-Ip: 198.61.254.31 X-Mailgun-Sid: WyI0MWYwYSIsICJsaW51eC1rZXJuZWxAdmdlci5rZXJuZWwub3JnIiwgImJlOWU0YSJd Received: from smtp.codeaurora.org (ec2-35-166-182-171.us-west-2.compute.amazonaws.com [35.166.182.171]) by smtp-out-n09.prod.us-west-2.postgun.com with SMTP id 5fce1e9896285165cdc18dff (version=TLS1.2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256); Mon, 07 Dec 2020 12:22:48 GMT Sender: akashast=codeaurora.org@mg.codeaurora.org Received: by smtp.codeaurora.org (Postfix, from userid 1001) id 11A68C433ED; Mon, 7 Dec 2020 12:22:48 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-caf-mail-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-2.9 required=2.0 tests=ALL_TRUSTED,BAYES_00, NICE_REPLY_A,SPF_FAIL,URIBL_BLOCKED autolearn=no autolearn_force=no version=3.4.0 Received: from [192.168.43.89] (unknown [106.205.31.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) (Authenticated sender: akashast) by smtp.codeaurora.org (Postfix) with ESMTPSA id C2072C433CA; Mon, 7 Dec 2020 12:22:43 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 smtp.codeaurora.org C2072C433CA Authentication-Results: aws-us-west-2-caf-mail-1.web.codeaurora.org; dmarc=none (p=none dis=none) header.from=codeaurora.org Authentication-Results: aws-us-west-2-caf-mail-1.web.codeaurora.org; spf=fail smtp.mailfrom=akashast@codeaurora.org Subject: Re: [PATCH 3/3] Serial: Separate out earlycon support To: Greg KH Cc: bjorn.andersson@linaro.org, devicetree@vger.kernel.org, linux-arm-msm@vger.kernel.org, linux-serial@vger.kernel.org, mka@chromium.org, dianders@chromium.org, linux-kernel@vger.kernel.org, saravanak@google.com References: <1607330847-15522-1-git-send-email-akashast@codeaurora.org> <1607330847-15522-4-git-send-email-akashast@codeaurora.org> From: Akash Asthana Message-ID: Date: Mon, 7 Dec 2020 17:52:32 +0530 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:78.0) Gecko/20100101 Thunderbird/78.5.1 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Content-Language: en-US Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Greg, On 12/7/2020 2:41 PM, Greg KH wrote: > On Mon, Dec 07, 2020 at 02:17:27PM +0530, Akash Asthana wrote: >> Separate out earlycon support from serial driver and remove it's >> dependency on QUP wrapper driver. >> >> This enable us to manage earlycon independently and we can re-use the >> same earlycon driver for android project which currently uses >> downstream version of QUP drivers. > What do you mean by "downstream" here? > >> Signed-off-by: Akash Asthana >> --- >> drivers/tty/serial/Kconfig | 9 + >> drivers/tty/serial/Makefile | 1 + >> drivers/tty/serial/qcom_geni_earlycon.c | 649 ++++++++++++++++++++++++++++++++ >> drivers/tty/serial/qcom_geni_serial.c | 97 ----- > So you are replacing 97 lines of code with 649 lines? How is this > benefiting anyone? > > confused, We have 2 versions of QUP driver, upstream version(Present in linus tree, mostly used for chromium project) and downstream version(belong to vendor part of code in GKI design, used for all the other project). There is need to enable geni earlycon in Google provided boot image for GKI to facilitate the debug until real console(belong to vendor code) is up. Currently it won't be possible because geni earlycon cannot be enabled independently, it depends on upstream QUP wrapper driver (soc/qcom/qcom-geni-se.c) and upstream serial driver(serial/qcom_geni_serial.c). With this patch I am trying to break any dependency btw earlycon hook and QUP kernel drivers, so it can be managed independently. Regards, Akash -- The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,\na Linux Foundation Collaborative Project