From:   Vladimir Oltean <vladimir.oltean@nxp.com>
To:     Maxim Kiselev <bigunclemax@gmail.com>
CC:     "fido_max@inbox.ru" <fido_max@inbox.ru>,
        Rob Herring <robh+dt@kernel.org>,
        Michael Ellerman <mpe@ellerman.id.au>,
        Benjamin Herrenschmidt <benh@kernel.crashing.org>,
        Paul Mackerras <paulus@samba.org>,
        Andrew Lunn <andrew@lunn.ch>,
        "David S. Miller" <davem@davemloft.net>,
        "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
        "linuxppc-dev@lists.ozlabs.org" <linuxppc-dev@lists.ozlabs.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH] powerpc: dts: t1040rdb: fix ports names for Seville
 Ethernet switch
Thread-Topic: [PATCH] powerpc: dts: t1040rdb: fix ports names for Seville
 Ethernet switch
Thread-Index: AQHX/WognYLuoHin80+YgXjEGjdBC6xK/32AgAAG3gCAAYlvgIAPYBoAgAHvc4A=
Date:   Tue, 11 Jan 2022 13:13:56 +0000
Message-ID: <20220111131355.djuyn6bbirqtsama@skbuf>
References: <20211230104329.677138-1-bigunclemax@gmail.com>
 <20211230130003.pzwzac5xttnnksz6@skbuf>
 <CALHCpMg8ZeQUcbA1EeUpXMcay0u=QZfnZZGpPb_HAXJeHoUQvQ@mail.gmail.com>
 <20211231125247.ugne3h44pmpzliin@skbuf>
 <CALHCpMgO2bqxPcaxwg29gEGF4te1HCgCa7SdNFVoxa6JDzrCrA@mail.gmail.com>
In-Reply-To: <CALHCpMgO2bqxPcaxwg29gEGF4te1HCgCa7SdNFVoxa6JDzrCrA@mail.gmail.com>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?YARafaFnTL6IXIVAo6NRZbtzywHJTIgqGmR+NVwddI/fZxDV3umr6zIdSCdz?=
 =?us-ascii?Q?+00KmDRtjby3+c78eUxN9KcDsvonWs9uOf7KMUIvsPwxHzyngIlFoHOmHShw?=
 =?us-ascii?Q?bA3op73+92kddPdZ/cnsaDeZBfEXzXSOXly00mxyvDfGpqArkVqYLhUq9U8X?=
 =?us-ascii?Q?2aq6esy0OtEr9vzXnU70q7rdo6gG9QXcCKDz3spxd1tphZJnTIZ3/Q+0dBB/?=
 =?us-ascii?Q?eadFlQWrxy9Igfhsfc6zBcY9lVnORkq+bAmAxYuyuKQQbSoskUiZg7vGGmBB?=
 =?us-ascii?Q?of5IyEemlp4SuToU2wLYBLBlmIgkQiukrYvO3F+SeWEjNWOh2eQId+fVfpb3?=
 =?us-ascii?Q?kWwzP4xCVdjPDQRE350RZtmVgUf9aJ0DLq4QFkhhozjouS6NVisS1hhRvxVi?=
 =?us-ascii?Q?js5SgShlrSCIM+S7O00f8iA6wp2mXa1+PJObg4urKL9fmK9Hwu2uVe1zUalg?=
 =?us-ascii?Q?y61gBx4tfP7UBdChvvRc2j5Rv3tzdTNGSthjIWZe5rSOVN012jx6//M13Wgk?=
 =?us-ascii?Q?YMWj5G0meeHknWbR67U43NHG5XOxrneYx7jlEPrWLsTHMgxKARk7q60OqsCY?=
 =?us-ascii?Q?soIDWCHNRpamkoSfaxXoc0QRYOoIXF6JQoLLffOM/Nm40nNwWGIp717hD+W1?=
 =?us-ascii?Q?mlRBQEPZTAiqzDsjRlRqaUWeR8qvvzrHt9TLtI7VQRxzK6kOm/hVXFXJX15D?=
 =?us-ascii?Q?FF0Fs7fP7VeA3rE48imsVYG2Gq0YScUtgG/TwtIZe3i20Tl/srj5P/J1FkWb?=
 =?us-ascii?Q?12Ha53m1ajPV/Y6WrAvzUHAA8UmBgI+zK4wzfWk5rGjjmB4CqZHpSM1YydEU?=
 =?us-ascii?Q?Ai/oLCvZ7vXJs+BxKxVSt/hHZ0lF3G0a+ONmga7ijZB1+1H5aaQboMy6qlZp?=
 =?us-ascii?Q?n1Pdv7k5hVe+qFja/KhN97O5xt8csXWAZzizVQgPnyus1d77qtsHY4AZzu0k?=
 =?us-ascii?Q?2k+jNYa1Vkc593g1zf7ZtCjxsX0PaddrsNUoPfoDNTuDssRf+7rEXKzkrF5w?=
 =?us-ascii?Q?Ti/w1vP372VCb4ZU6IDnwY35e2xh5k/c0jnMja6DzejIkia65/Ea18fZz/Hu?=
 =?us-ascii?Q?Xiz0dI9AnM2yQy8pod4Yuq0ITOTiz4ShctrP7A1P97X8m1sjb1AxKpmQ6cDy?=
 =?us-ascii?Q?cD48IAzzMwIwA3TrSfIEUyIIVdSZVIAr6m4ENNegYkCGhVKwMqWia/CUr7Oj?=
 =?us-ascii?Q?iCMiGeqFjSSyljKpHplpY8MzCwIiJnHJxfpa4OcGglaPU9QBSrcycpc5CxsH?=
 =?us-ascii?Q?XNFWJxk4saATMpl1AMyNZro8/Be7JgBX5610c5FQ7klqCaV9aeOnQ4dIJelh?=
 =?us-ascii?Q?fCLHqpHBBKf45c62s0FTpcSeYy2GJmvORvHC8QhJ9a5eTsD9pJS62lghEZ2/?=
 =?us-ascii?Q?hUdSwR8fZ/upFrp70KHe2YeICiz63D5Bq+8/gKb2nKrHDcLuWdweXpOeE4UW?=
 =?us-ascii?Q?oAUXB7ifZtQuLjMtSnZsouKwsPo/5xaqwfLoYNxKahnKjD7Lber85YBFPnuQ?=
 =?us-ascii?Q?FB2LgnoNEezwxGJiv1G/vihmtymoMoXjvuTWvelrd9UYocxno5tNxIrefkWe?=
 =?us-ascii?Q?sdCfPsDquz8v3VqNXYjPjqfsSp9Rg3Ex1E4iD1DvMVpKfpKuL8Z1zrU8yb1w?=
 =?us-ascii?Q?GyUi6DNoAllhsQ7R9tdzpUU=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-ID: <714CD808197DFB4E9FE119FEAEC09384@eurprd04.prod.outlook.com>
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: VI1PR04MB5136.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: dc0235c1-f62d-4e45-cf78-08d9d5043927
X-MS-Exchange-CrossTenant-originalarrivaltime: 11 Jan 2022 13:13:56.4506
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: NYGtmY50bnACm58INtHaVYSuYqskabTbuCzD9kTT8s85vVMMj5HBJI4c8Oy+o7wPZVbZoOrQD99o4Pq6KyATNw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR04MB3200
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Hi Maxim,

On Mon, Jan 10, 2022 at 07:40:38AM +0000, Maxim Kiselev wrote:
> Here are photos of my boards.

Your patch is OK to change t1040rdb.dts, but please preserve the existing
port mappings in a new arch/powerpc/boot/dts/fsl/t1040rdb-rev-a.dts file.

You will also need to modify the /model and /compatible nodes of the new
device tree for Rev A, something like "fsl,T1040RDB-REV-A". Take a look
at arch/arm64/boot/dts/freescale/fsl-lx2160a-bluebox3-rev-a.dts to see
an example of what I'd like to be done.

Thanks.=