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 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 2A208C433EF for ; Fri, 24 Sep 2021 20:20:42 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 0C0E960238 for ; Fri, 24 Sep 2021 20:20:42 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1346790AbhIXUWO (ORCPT ); Fri, 24 Sep 2021 16:22:14 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:34950 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1346333AbhIXUWN (ORCPT ); Fri, 24 Sep 2021 16:22:13 -0400 Received: from mail-yb1-xb30.google.com (mail-yb1-xb30.google.com [IPv6:2607:f8b0:4864:20::b30]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id B6EA1C061571 for ; Fri, 24 Sep 2021 13:20:39 -0700 (PDT) Received: by mail-yb1-xb30.google.com with SMTP id y13so8275856ybi.6 for ; Fri, 24 Sep 2021 13:20:39 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20210112; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=XyhQ38a/4uH67bAl5YGynvJ7Eg72vGZjm0Neu7EJ7Uc=; b=sugKOapzTbYiBxmeNfdWAfSXSs+U3/EV89R7DA5U1a32VM+oxYjy40oSJmo2l8hJIz WAjB+3JeAn7Ku+NF4E9dhQw4TyNqJzJ12tqzkreHVHfo9jxbpWeb4ARq4LIllJVeF2KA Vmer6jlQHrx/x2qy3jI3Z8FY1UEEh2/d2NSGQEizmxs9BImL1aBacyyfDnwEdLy1kB6I z26/ARJMI+VOt+MZdFXu1ZkaqZ+hIDz18MI6wbKPT5POdu0Bh6rR9X6jAXShrLVvuYBK x3mDJFHfGpJEQzOD0vh4/2ZPYL1wUcuuvkvhEcStBAvz8EzR/pU6HV001X0k9qoNbvgz QZuQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=XyhQ38a/4uH67bAl5YGynvJ7Eg72vGZjm0Neu7EJ7Uc=; b=q/7AARcXDmufTgV2v9XasQ1uuOfnkxI4+s1yCP7/fLn3eR2OMGZMSjkhC1nst6C9lG cgbQHsUmoJEJ1LeUGlg3DwoX9J06I0J/tSFhFVA0sW62XqSwfvopRnK87z6St2nPu4wH TJJe2OXCZLC5EK63Yi6PnD3w+c3gzzqc/A3skZw0CQkNLomHK+7HBGbCnDGLPoX9IDmh Jtq4c7B78zpxhCzL2EYYqnrtTRw8MKmDDI3AhW6ZopQ4YBycmEJiUxLtx5QTaRXrqKH/ wgb0vSV4N0IL1Ri3kyhDNl9y/fMHog+P7nuqK22rRFLZyJj71oUAgs8IeCxKVbiDWRj1 UlpQ== X-Gm-Message-State: AOAM532tBIsQy13xh1YSVh3Ys0fwsomeEyqP5iles/0np8+EwiV2gTk5 W8K/RlJFxmuEfKNnAy8rkUR6gflSgQnUroEDUOp/YA== X-Google-Smtp-Source: ABdhPJw3UmguJUAbaav5AKykn8QK+KcLgLzhmVnIKdKzvHlgFfAoVRS7LT68ls/fxAaH3sgCg9ehHFiw2VCPF+JJqhE= X-Received: by 2002:a25:2b07:: with SMTP id r7mr13368843ybr.296.1632514838758; Fri, 24 Sep 2021 13:20:38 -0700 (PDT) MIME-Version: 1.0 References: <20210904000543.2019010-1-saravanak@google.com> <20210904000543.2019010-2-saravanak@google.com> In-Reply-To: From: Saravana Kannan Date: Fri, 24 Sep 2021 13:20:02 -0700 Message-ID: Subject: Re: [PATCH v3 1/2] drivers: bus: simple-pm-bus: Add support for probing simple bus only devices To: Ulf Hansson Cc: Rob Herring , Russell King , Neil Armstrong , Geert Uytterhoeven , Magnus Damm , Tony Lindgren , Catalin Marinas , Will Deacon , Damien Le Moal , Android Kernel Team , Linux ARM , Linux Kernel Mailing List , linux-oxnas@groups.io, Linux-Renesas , linux-omap , linux-riscv Content-Type: text/plain; charset="UTF-8" Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Sep 24, 2021 at 4:50 AM Ulf Hansson wrote: > > On Thu, 9 Sept 2021 at 13:01, Ulf Hansson wrote: > > > > On Sat, 4 Sept 2021 at 02:05, Saravana Kannan wrote: > > > > > > fw_devlink could end up creating device links for bus only devices. > > > However, bus only devices don't get probed and can block probe() or > > > sync_state() [1] call backs of other devices. To avoid this, probe these > > > devices using the simple-pm-bus driver. > > > > > > However, there are instances of devices that are not simple buses (they > > > get probed by their specific drivers) that also list the "simple-bus" > > > (or other bus only compatible strings) in their compatible property to > > > automatically populate their child devices. We still want these devices > > > to get probed by their specific drivers. So, we make sure this driver > > > only probes devices that are only buses. > > > > > > [1] - https://lore.kernel.org/lkml/CAPDyKFo9Bxremkb1dDrr4OcXSpE0keVze94Cm=zrkOVxHHxBmQ@mail.gmail.com/ > > > Signed-off-by: Saravana Kannan > > > Tested-by: Saravana Kannan > > > > Tested-by: Ulf Hansson > > Saravana, Rob, > > I have been following your latest discussion in this thread - and it > looks like you guys are moving towards a consensus. > > Although, if there is anything I can do to help to complete this, just > tell me and I will jump in immediately. Looks like we settled on the allow list approach during the BoF in LPC. I'll send out a v4 with some tweaks. -Saravana