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;
From:   Damien Le Moal <Damien.LeMoal@wdc.com>
To:     Saravana Kannan <saravanak@google.com>,
        Russell King <linux@armlinux.org.uk>,
        Neil Armstrong <narmstrong@baylibre.com>,
        Geert Uytterhoeven <geert+renesas@glider.be>,
        Magnus Damm <magnus.damm@gmail.com>,
        Tony Lindgren <tony@atomide.com>,
        Catalin Marinas <catalin.marinas@arm.com>,
        Will Deacon <will@kernel.org>,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>
CC:     Ulf Hansson <ulf.hansson@linaro.org>,
        Rob Herring <robh+dt@kernel.org>,
        "kernel-team@android.com" <kernel-team@android.com>,
        "linux-arm-kernel@lists.infradead.org" 
        <linux-arm-kernel@lists.infradead.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "linux-oxnas@groups.io" <linux-oxnas@groups.io>,
        "linux-renesas-soc@vger.kernel.org" 
        <linux-renesas-soc@vger.kernel.org>,
        "linux-omap@vger.kernel.org" <linux-omap@vger.kernel.org>,
        "linux-riscv@lists.infradead.org" <linux-riscv@lists.infradead.org>
Subject: Re: [PATCH v4 0/2] Fix simple-bus issues with fw_devlink
Thread-Topic: [PATCH v4 0/2] Fix simple-bus issues with fw_devlink
Thread-Index: AQHXtMYHtS6fHXMugU2E5Mhb0ZTigQ==
Date:   Thu, 30 Sep 2021 02:50:15 +0000
Message-ID: <DM6PR04MB70812E56DB9CB8CDC63C6535E7AA9@DM6PR04MB7081.namprd04.prod.outlook.com>
References: <20210929000735.585237-1-saravanak@google.com>
Accept-Language: en-US
Content-Language: en-US
wdcipoutbound: EOP-TRUE
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?X06IkIwWxMM7yc2mQIHgNDQg9N4auO6KR+lBimA9/FRi7v7LV20HWLzYmZtA?=
 =?us-ascii?Q?oke97pxxbAKfLRYLWOgAM0gf3tezDH1/mU7R+a9rjVJoSytLXGXhwqV0aDFt?=
 =?us-ascii?Q?9PcHC+Vm6xOeF7aucPbRQxFJzdRBtNPzA34vPk08esOZ9U1PLvQstojthh01?=
 =?us-ascii?Q?fOFFn08yf+lI71MaTB7rwqktqiZWn2imSi++f//PKYK5tX8ntXmVlXI1Ffuz?=
 =?us-ascii?Q?oAl/9yoyVcJduAwaIrxjF4qfjE1vfwM/Xs6wonevHBcAfqFSUvKaKzdbfGVe?=
 =?us-ascii?Q?6QlWPuiyZv2DSYaq8VFVDwq09coHdkNOSaB7WLNpcRP3dewu/VCAzlPRXJul?=
 =?us-ascii?Q?UQu6U5kH2VLO/MRT+VKhb6Tsuoyq4nvPUAhYZD75mAYVtZ+lXdQBlHsbjSAs?=
 =?us-ascii?Q?aOS5EelKGXN6hXswTbVL5NXaThwNQYyKY57aT0d23SHmC5rGoin6vWnGxDQE?=
 =?us-ascii?Q?l0zC/NiYlJoR2sIQDDVfqttHO0u8FccS7B3RffbfwbiosUr3uG5bbjKkWkRq?=
 =?us-ascii?Q?bW0QHxH9bH2HZP/O2sxa+sXLCh1xFDGVGAvFMRqeSmnoz/GAyJdns2WQ2rI9?=
 =?us-ascii?Q?4fjQT85X/ZXvQq+fzpSIvc5cXLz44OR2Vvd/iD2k0fMo0yEqYefIPmYzeqb5?=
 =?us-ascii?Q?SY3Zjm4lejQOQDdnMCrkPXAaAQN1Acnjm37M3PF/3hD0Svl2Q1CD0zbu2Kt3?=
 =?us-ascii?Q?3wByf/B08heGk3oplQlErDYy9Cw1YEhkIQ/aZ7FbA1k1icdSpFjcnxLUVulc?=
 =?us-ascii?Q?DxACExBcIrj8i2HmCZ3dUypEWisZ7gLu3LrapImRABo9UrszLHX5v56aVT4i?=
 =?us-ascii?Q?AiuDUrt2O4aWTNe/UezdVfJyn7SgXt41J0Lh2m5bOmwrMpNeg/4PE8QvWdoY?=
 =?us-ascii?Q?6P8Y2Scx1UPcmi4iyi8FTcn8XcMHfx9uWYkBTVupVS9KVWZ68GNgrWHGQ7oK?=
 =?us-ascii?Q?9Kfsi9pxeLSUm8sSF8s6jgxBX1cot13yxKAY+DIhzc+G1k1X7ASlsVyCGhxC?=
 =?us-ascii?Q?xWbCvNF/HWwu8WnUz5NDKLy8+p/RbZ99xuCmBfeq37C9S0Ar0jFmNDlwxLCG?=
 =?us-ascii?Q?t1GX4V4SggCws6EyemKyEA85QA7NMJ1aTIi7HvYNRnT+jqG3yFfUuBLgvxiO?=
 =?us-ascii?Q?u7yTlHqZFq5rpbkIARiezkDj9SvVAt1YG336Wy1urZWkVXNhvmksNEyy+2aR?=
 =?us-ascii?Q?eHSEQmMD64NkHkbmLbOufPntu1KhZXGnWbVYbI7IX8i/3dh93lbfumoYhN6Z?=
 =?us-ascii?Q?UI2LpRIVfSv5nzM5oidXR1VZCAdFEuZKa10kACutokw765IM2hQnRpbRgrXH?=
 =?us-ascii?Q?IncI7Wyz37gR5C/HbF8KWaOkH2ABioClp0gMWWsgfGZCATUUeytMiTuRXW3n?=
 =?us-ascii?Q?PQ9GLakb6gnVpH1ofbIIVE0xi4FoW38YePsoTI4ksSdfyq1srQ=3D=3D?=
x-ms-exchange-transport-forked: True
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: wdc.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: DM6PR04MB7081.namprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 34049619-4b3d-4422-8742-08d983bd0809
X-MS-Exchange-CrossTenant-originalarrivaltime: 30 Sep 2021 02:50:15.4713
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: b61c8803-16f3-4c35-9b17-6f65f441df86
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: CpA65XKyKOZE+IR9oHD6IBrsjq6KUjT4cKAH7796pVnlZdabKm0hFS0U1/PEYafTTZ1Ba04o0doAOrF88N1QEw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM6PR04MB6606
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On 2021/09/29 9:07, Saravana Kannan wrote:=0A=
> Ulf reported an issue[1] with fw_devlink. This series tries to fix that=
=0A=
> issue.=0A=
> =0A=
> I replicated a similar set up on my end and I confirmed:=0A=
> - A simple-bus only device is probed.=0A=
> - Another device listing simple-bus as a 2nd compatible string isn't=0A=
>   probed.=0A=
> =0A=
> v1->v2:=0A=
> - Switched to probing the simple-bus device instead of marking it as=0A=
>   NEVER_PROBES.=0A=
> =0A=
> v2->v3:=0A=
> - Moved all the code into the simple-pm-bus driver=0A=
> - Addressed Ulf's comment about the remove() code missing a check.=0A=
> =0A=
> v3->v4:=0A=
> - Added support for driver_override to accommodate the case where new=0A=
>   transparent bus compatible strings are added to DT, but the kernel hasn=
't=0A=
>   been updated yet.=0A=
> - Added/updated comments to give more details.=0A=
> - Adding Ulf's Tested-by that he have for v3 (v4 didn't change by much).=
=0A=
=0A=
For riscv Kendryte K210 boards:=0A=
=0A=
Tested-by: Damien Le Moal <damien.lemoal@wdc.com>=0A=
=0A=
> =0A=
> Thanks,=0A=
> Saravana=0A=
> [1] - https://lore.kernel.org/lkml/CAPDyKFo9Bxremkb1dDrr4OcXSpE0keVze94Cm=
=3DzrkOVxHHxBmQ@mail.gmail.com/=0A=
> =0A=
> Cc: Ulf Hansson <ulf.hansson@linaro.org>=0A=
> Cc: Geert Uytterhoeven <geert+renesas@glider.be>=0A=
> Cc: Rob Herring <robh+dt@kernel.org>=0A=
> =0A=
> Saravana Kannan (2):=0A=
>   drivers: bus: simple-pm-bus: Add support for probing simple bus only=0A=
>     devices=0A=
>   drivers: bus: Delete CONFIG_SIMPLE_PM_BUS=0A=
> =0A=
>  arch/arm/configs/multi_v7_defconfig |  1 -=0A=
>  arch/arm/configs/oxnas_v6_defconfig |  1 -=0A=
>  arch/arm/configs/shmobile_defconfig |  1 -=0A=
>  arch/arm/mach-omap2/Kconfig         |  1 -=0A=
>  arch/arm64/configs/defconfig        |  1 -=0A=
>  drivers/bus/Kconfig                 | 12 ---------=0A=
>  drivers/bus/Makefile                |  2 +-=0A=
>  drivers/bus/simple-pm-bus.c         | 42 ++++++++++++++++++++++++++---=
=0A=
>  drivers/soc/canaan/Kconfig          |  1 -=0A=
>  9 files changed, 40 insertions(+), 22 deletions(-)=0A=
> =0A=
=0A=
=0A=
-- =0A=
Damien Le Moal=0A=
Western Digital Research=0A=