Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Thu, 27 Oct 2022 21:16:32 -0500
From:   Nishanth Menon <nm@ti.com>
To:     Matt Ranostay <mranostay@ti.com>
CC:     <vigneshr@ti.com>, <robh+dt@kernel.org>,
        <devicetree@vger.kernel.org>,
        <linux-arm-kernel@lists.infradead.org>,
        <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH v3 0/9] J721S2: Add support for additional IPs
Message-ID: <20221028021632.lzrsfna5v75hnz6n@payable>
References: <20220921031327.4135-1-mranostay@ti.com>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline
In-Reply-To: <20220921031327.4135-1-mranostay@ti.com>
User-Agent: NeoMutt/20171215
Precedence: bulk

On 20:13-20220920, Matt Ranostay wrote:
> The following series of patches add support for the following
> on J721S2 common processor board,

Thanks Matt for picking these up.. however,

> Matt Ranostay (2):
>   arm64: dts: ti: k3-j721s2-main: Add SERDES and WIZ device tree node
>   dt-bindings: PCI: Add host mode device-id for j721s2 platform


It gets hard when we mix driver bindings with dts patches. Please do not
do that - in this case, the binding would be better go through  Lorenzo
Pieralisi <lorenzo.pieralisi@arm.com> (who needs to be in the To list)
and the corresponding list linux-pci@vger.kernel.org in CC

This allows driver bindings to be independently merged of the dts
changes - also allows dts maintainers to pick patches after ensuring
bindings (and any corresponding driver changes) are merged into master
(which locks things down). please disconnect the series such that the
dts can alone be picked up without dtbs_check problems.

That said, i'd rather we rebase this after Andrew's cleanup series in
[1] and follow the same pattern for new nodes being added.


[1] https://lore.kernel.org/all/20221020160305.18711-1-afd@ti.com/

-- 
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D