From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-5.8 required=3.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, MENTIONS_GIT_HOSTING,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 82CCBC433E0 for ; Fri, 5 Jun 2020 14:52:03 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 5D4D5206A2 for ; Fri, 5 Jun 2020 14:52:03 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="UT0pXXm6" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727101AbgFEOv7 (ORCPT ); Fri, 5 Jun 2020 10:51:59 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:54328 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727826AbgFEOv6 (ORCPT ); Fri, 5 Jun 2020 10:51:58 -0400 Received: from mail-yb1-xb44.google.com (mail-yb1-xb44.google.com [IPv6:2607:f8b0:4864:20::b44]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 33A0BC08C5C5 for ; Fri, 5 Jun 2020 07:51:58 -0700 (PDT) Received: by mail-yb1-xb44.google.com with SMTP id t9so4923583ybk.9 for ; Fri, 05 Jun 2020 07:51:58 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=gIdG6UEiHEb5RjVC13mngSWyhrH+zQoFoNg2xv+5i6E=; b=UT0pXXm62kWCAETgUQiiH7SuR9KrWgIUH7xO+XOxKabMOeXEO12SG7QaazeRz6Br8D z3BPm87kFauLqS3D0xvAtd2zzJ9zTG9JFO0Sxf5/1N2KxsMC7egH+e/mdFP6KUW419ft xj5dZbauTAdbE8zrymV1plXyNlQtKy/GivJm41qLbOcWvy5l5Gq4wMBhZb/50wmI7W/w RVG7hty/w46U8WFREfq12rK509On5eGVLpzLXcSEYNB2mNx9UResQ08CIIUeZUNf1sbl yVH1X5odJ9G/oZbt7oKWpomC6v4eE12zM3tnxpn0yakECSD1rGRYQb3+67tkFwkBIsUl Wc9g== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=gIdG6UEiHEb5RjVC13mngSWyhrH+zQoFoNg2xv+5i6E=; b=HiV3Q04EQ88HSQXKurxzG+PO9I32E/3OW/zIdzjxW5K8YDIxwQqHlHtrjmN9faZzAy CTIo0UIHuTGbVIP9fsWe7k/IDuNBmWiCtQI0QjP6kOJ/T1QDTTtHUZFX6dZfSiWQBkDL At2viPQ3RqXYBzBK0ufMeJ49vbupUwOsmgbO+EkYj8810Jf+5l/mz7Ovv7WvvxyBUcuq QQOrGOHv9ZnrWmlW0vwwAecgCVKpzdw/nGE1Icjb7o0k9PSlm1Y66SvCsq9Z6Cea/cJp Gvgp9E4Td5vhdRCCoRYz5BDUvan9uuVjfDQaPlsO8ZUxGKtk88ekCLVPjo8RidaQC2BO aoqQ== X-Gm-Message-State: AOAM531sjJG8W1jMxe0gHOX3Pma2GqtwLYutEw23Y4g77b7KeRGWY5d+ hwY42z+6tZ2x/LV8+SSFNyqQgZfh6MRQbpxUuv7EUA== X-Google-Smtp-Source: ABdhPJyuTGP1hAakt+L74reWI5CvDw60cyJJlrXPwu7+Qgb3VpYZNx4PYFeBCNPAXb/UaoUqkRzhlCB+z7OD6q6xaqo= X-Received: by 2002:a25:8b0c:: with SMTP id i12mr18084529ybl.371.1591368717101; Fri, 05 Jun 2020 07:51:57 -0700 (PDT) MIME-Version: 1.0 References: <20200524023815.21789-1-jonathan@marek.ca> <20200524023815.21789-2-jonathan@marek.ca> <20200529025246.GV279327@builder.lan> <20200529031520.GA1799770@builder.lan> <91eb7ee0e549b10724c724aebfd91996@codeaurora.org> <8cf134f0-381f-7765-2496-e5abd77f3087@marek.ca> <81a9d07c0c8d76abf0ef734963788884@codeaurora.org> In-Reply-To: <81a9d07c0c8d76abf0ef734963788884@codeaurora.org> From: Nicolas Dechesne Date: Fri, 5 Jun 2020 16:51:46 +0200 Message-ID: Subject: Re: [PATCH 1/6] arm64: dts: qcom: sm8150: add apps_smmu node To: Sai Prakash Ranjan Cc: Jonathan Marek , Bjorn Andersson , linux-arm-msm , Andy Gross , Rob Herring , "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , open list , devicetree-owner@vger.kernel.org Content-Type: text/plain; charset="UTF-8" Sender: devicetree-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org On Fri, Jun 5, 2020 at 4:39 PM Sai Prakash Ranjan wrote: > > Hi Nico, > > On 2020-06-05 20:01, Nicolas Dechesne wrote: > > On Fri, Jun 5, 2020 at 4:14 PM Sai Prakash Ranjan > > wrote: > >> > >> On 2020-06-05 19:40, Jonathan Marek wrote: > >> > On 6/5/20 10:03 AM, Sai Prakash Ranjan wrote: > >> >> On 2020-05-29 08:45, Bjorn Andersson wrote: > >> >>> On Thu 28 May 20:02 PDT 2020, Jonathan Marek wrote: > >> >>> > >> >>>> > >> >>>> > >> >>>> On 5/28/20 10:52 PM, Bjorn Andersson wrote: > >> >>>> > On Sat 23 May 19:38 PDT 2020, Jonathan Marek wrote: > >> >>>> > > >> >>>> > > Add the apps_smmu node for sm8150. Note that adding the iommus field for > >> >>>> > > UFS is required because initializing the iommu removes the bypass mapping > >> >>>> > > that created by the bootloader. > >> >>>> > > > >> >>>> > > >> >>>> > Unrelated to the patch itself; how do you disable the splash screen on > >> >>>> > 8150? "fastboot oem select-display-panel none" doesn't seem to work for > >> >>>> > me on the MTP - and hence this would prevent my device from booting. > >> >>>> > > >> >>>> > Thanks, > >> >>>> > Bjorn > >> >>>> > > >> >>>> > >> >>>> I don't have a MTP, but on HDK855, "fastboot oem > >> >>>> select-display-panel none" > >> >>>> combined with setting the physical switch to HDMI mode (which > >> >>>> switches off > >> >>>> the 1440x2560 panel) gets it to not setup the display at all (just > >> >>>> the > >> >>>> fastboot command isn't enough). > >> >>>> > >> >>> > >> >>> Okay, I don't think we have anything equivalent on the MTP, but good > >> >>> to > >> >>> know. > >> >>> > >> >> > >> >> Actually I tried out this in SM8150 MTP and it works fine for me, > >> >> > >> >> "fastboot set_active a; fastboot set_active b; fastboot set_active a; > >> >> fastboot oem select-display-panel none; fastboot reboot bootloader; > >> >> fastboot boot boot-sm8150.img" > >> >> > >> >> Also I need to switch slots everytime like above, otherwise I always > >> >> see some error > >> >> while loading the boot image. > >> >> > >> > > >> > What is the error? If it is "FAILED (remote: Failed to > >> > load/authenticate boot image: Load Error)" then flashing/erasing > >> > boot_a will make it go away ("fastboot erase boot_a") for the next 6 > >> > or so "failed" boots. > >> > > >> > >> Yes this exact error. > > > > The bootloader maintains a 'boot status' in one of the partition > > attributes. After a certain amount of 'failed' boot , it will switch > > to the other boot partition. It's the same thing on RB3/DB845c. In our > > release for DB845c, we are patching the bootloader so that this > > behavior is bypassed. On typical 'product' there is a user space > > application that will come and set the partition attribute to indicate > > the boot was successful. > > > > For the record, this is the patch we use on 845c: > > https://git.linaro.org/landing-teams/working/qualcomm/abl.git/commit/?h=release/LE.UM.2.3.7-09200-sda845.0&id=e3dc60213234ed626161a568ba587ddac63c5158 > > > > rebuilding EDK2/ABL requires access to signing tools.. so it might not > > be possible for everyone. but in case you can, it should be > > straightforward to reuse this patch. > > > > Thank you for these details and the patch, it's very useful. > I do have access to ABL code and the signing tools and can build one. Good. Then the next problem you will likely face is that building QCOM ABL is far from being straightforward. Why would it be? ;) That's the script we use to build it ourselves: https://git.linaro.org/ci/job/configs.git/tree/lt-qcom-bootloader/dragonboard845c/builders.sh#n61 It has a reference to sectools which we have (internally) access to, but you have it too, and you should be able to leverage most of the script. > > Thanks, > Sai > > -- > QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a > member > of Code Aurora Forum, hosted by The Linux Foundation