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;
Date:   Wed, 18 Aug 2021 22:34:57 +0800
From:   Chester Lin <clin@suse.com>
To:     Rob Herring <robh@kernel.org>
Cc:     devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
        linux-arm-kernel@lists.infradead.org, linux-serial@vger.kernel.org,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        Shawn Guo <shawnguo@kernel.org>,
        Krzysztof Kozlowski <krzk@kernel.org>,
        Oleksij Rempel <linux@rempel-privat.de>,
        Stefan Riedmueller <s.riedmueller@phytec.de>,
        Matthias Schiffer <matthias.schiffer@ew.tq-group.com>,
        Li Yang <leoyang.li@nxp.com>,
        Fabio Estevam <festevam@gmail.com>,
        Matteo Lisi <matteo.lisi@engicam.com>,
        Frieder Schrempf <frieder.schrempf@kontron.de>,
        Tim Harvey <tharvey@gateworks.com>,
        Jagan Teki <jagan@amarulasolutions.com>, s32@nxp.com,
        catalin-dan.udma@nxp.com, bogdan.hamciuc@nxp.com,
        bogdan.folea@nxp.com, ciprianmarian.costea@nxp.com,
        radu-nicolae.pirea@nxp.com, ghennadi.procopciuc@nxp.com,
        Matthias Brugger <matthias.bgg@gmail.com>,
        Andreas =?iso-8859-1?Q?F=E4rber?= <afaerber@suse.de>,
        "Ivan T . Ivanov" <iivanov@suse.de>, "Lee, Chun-Yi" <jlee@suse.com>
Subject: Re: [PATCH 1/8] dt-bindings: arm: fsl: add NXP S32G2 boards
Message-ID: <YR0akXYPYthDuvCh@linux-8mug>
References: <20210805065429.27485-1-clin@suse.com>
 <20210805065429.27485-2-clin@suse.com>
 <YRaxt1LCF+hWaMJU@robh.at.kernel.org>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <YRaxt1LCF+hWaMJU@robh.at.kernel.org>
MIME-Version: 1.0
X-MS-Exchange-MessageSentRepresentingType: 1
Received: from linux-8mug (118.160.215.224) by HK2PR02CA0168.apcprd02.prod.outlook.com (2603:1096:201:1f::28) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4436.19 via Frontend Transport; Wed, 18 Aug 2021 14:35:06 +0000
X-MS-PublicTrafficType: Email
X-MS-Office365-Filtering-Correlation-Id: 23ed5084-123e-413a-7c2f-08d9625564d0
X-MS-TrafficTypeDiagnostic: VI1PR04MB5630:
X-MS-Exchange-Transport-Forked: True
X-Microsoft-Antispam-PRVS: <VI1PR04MB56308D88310B16E2D24BB4D2ADFF9@VI1PR04MB5630.eurprd04.prod.outlook.com>
X-MS-Oob-TLC-OOBClassifiers: OLM:8882;
X-MS-Exchange-SenderADCheck: 1
X-MS-Exchange-AntiSpam-Relay: 0
X-Microsoft-Antispam: BCL:0;
X-Microsoft-Antispam-Message-Info: TMWGYx2J2yfRgyfj8Jwzj+6Pc/LF1jTJcSzvALkfswgzeKPyGLAgtdd9GP+3XffeWIttg/OOjHAHqQv1VC/B1f2hVymyF1aiq0/JRu7wns017w3UAq23tQTDGhqBMZV1QfBbGx3wqw7OmlZ+uj2Kdk6k6qIz9sH0um69akOtpAKx7rPw1RZg7CqzxfQY23+RpOVGvIHzQrNaNQnZzWkXifHUSkF5/S8/8eUZqyi94BX/Bkivl2ibjG2IVI6+4wgnYluysBxu0rRyn2kwJlSOnc0fgebh3+TcgIYsswpBuprkmaYVG5dYVpbFXefrKotAQpsToOsCruU2xq5bHgRaUou+tp5Pcr2oy1w8CVhFTo5OzG/F03Psksf0UkN1JzAKPw41liEZqGf6p2VJHLPDgLE7wTuGProOVm3tDRQsSr72/Ejv86zQkUObfvf/IlD7S9g65ln5nnoEZoBdrW3FemT1iafDf8POyeKYAmxqnPK62ZpW5HLKaEk3CNDUhUNOAJRj2am77O0788DoRC+JNEaN1FXvBWwV5XzdWYeiL5d9Pjo/L6bJDYzD2IjjmcKeHJtFxYF263loTsBR7lMwqJqNwCg33h6pSOjLAm+hfcQxFuWAS0lSa4POhnyLJk/4SI1HY+074QM1uCVhXV1QOg==
X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:VI1PR0402MB3439.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(366004)(83380400001)(316002)(86362001)(6916009)(54906003)(26005)(55016002)(9686003)(38100700002)(6666004)(508600001)(186003)(7416002)(66946007)(66476007)(66556008)(4326008)(5660300002)(33716001)(107886003)(6496006)(956004)(8936002)(2906002)(55236004)(8676002);DIR:OUT;SFP:1101;
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?UVtVKDifdlnSk+mL7W9Od1cxifRDV6yaV+4L94IMwC2i4WJaJAyOU0Xg+f3E?=
 =?us-ascii?Q?dQyauIkptsuGOY621/ILVbo2IVGSKw40wKP/Fpnez/Jd5lzv2S5Cy7V2NfnB?=
 =?us-ascii?Q?rreVQDqW580qvhWhOAo6RW28/cHnsB7H8oL5THJea6CzxzzxoQhQGsEGjvs1?=
 =?us-ascii?Q?OfNDxb049LIggZQgrcHJDo2+jU7bAHZ13/6bm3S6g45ssJ1E4V/1LsGupWye?=
 =?us-ascii?Q?UowtKwlah6Wv0wC3gD7SLn+3OjWScNRnVaWR7BEd8ei3/KjMFHN4mIxhhWWw?=
 =?us-ascii?Q?3dqraOAOJwPCW7doog7u9Gflm5xIjR0E7sqw+3E5GvniBZVdGEUMY78bcbTu?=
 =?us-ascii?Q?WJZiYu9NgOytV+OP4hK/eBRJOqJmM30aSVqMurYp/DS98ipMenY/raN2gRtJ?=
 =?us-ascii?Q?8JL4CaAiI/eEoG0OwCApj9TjPO7PiPBBZcMGUzueicFs9MLSqAtxiqFzbakV?=
 =?us-ascii?Q?3gQ27I90bUlfbNhWVa6QIS1Czl5mcLtXNNt+WrDdL0FaYvle2LhQmMzmgjeU?=
 =?us-ascii?Q?1MZ0Ym62i2Euino/r54MYKqjJDENKHQkd3GQMg1pdbTuunESqKhglESEOLf0?=
 =?us-ascii?Q?AiiczQ2MHfXCR3UIPke9k1p5/f1Wp1wOz0U5xTG9GaQ4GIHe46dFmKA5LKzG?=
 =?us-ascii?Q?c/XBOQ616JVTuvCVY29GYVnFFADz2TgqbINxNUe1R8HUn+OemMvpVZRnj6sP?=
 =?us-ascii?Q?/KQqt+bnZL86QiB0xYO4n0W039IZkO2gXXTccgSN/37OZRY1dN9x42Qdv+eV?=
 =?us-ascii?Q?CcyaPajeNLRtrfIZ1C2XV0EjQrmiIHqoW5nPj6b6D8f0d0RGU0bqNwBO2Xb/?=
 =?us-ascii?Q?EKU/ijmK2Iwm1YP3H8ZwenI3jV1L7k0OBtltLYOH+QvetjceA+BYX23rirsm?=
 =?us-ascii?Q?jf+fbwbHk3IEL4cCIPbMoz21aTHHqvKulm4A5qOWLwJGXIaNkS0rWeU5LhFU?=
 =?us-ascii?Q?yh/5JG3gpVtsfn0z32BSQziWg+ElGlkyWpx3y279PiM6m8napV8a8bXDy5cX?=
 =?us-ascii?Q?l6BXy+jF5a023V2DWddWQqwmMFyUMAR+GjXR0ONJWgb/3+rBZk+4Ms1zaXHr?=
 =?us-ascii?Q?bjpbxve/9hWVeKq0IJWFAMhsp3jkp22QQHUU0jLa1nQLRKMWjcOhGhcsu9E2?=
 =?us-ascii?Q?ZbsZp6EUzFk0Mk1Glxb9/DfAAx5mLAFhyqtsFYgMYx49qZfMKLWMxSiuKbUD?=
 =?us-ascii?Q?RV8gmNCjymBnDwQ3w0+1Ft7nMCbWzt7dLBj3O+w7OBc1y/Ws/xqJiMTYf6pw?=
 =?us-ascii?Q?rJGqyU01Mt7vXF3SnhcPzaOXHJPP7pIVC+Np7iLH7HPX3N4x8s+0ON8UC9yH?=
 =?us-ascii?Q?f+ws1lE6xotDFEhw4SC0OFh3?=
X-OriginatorOrg: suse.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 23ed5084-123e-413a-7c2f-08d9625564d0
X-MS-Exchange-CrossTenant-AuthSource: VI1PR0402MB3439.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 18 Aug 2021 14:35:15.6970
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: f7a17af6-1c5c-4a36-aa8b-f5be247aa4ba
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: zbpLmWbUaDgO7D0pQhXrn1Crg1xG2GzztnfFxLk/RWcFNnG8D64vRczxJS9wFKp3
X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR04MB5630
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Fri, Aug 13, 2021 at 12:53:59PM -0500, Rob Herring wrote:
> On Thu, Aug 05, 2021 at 02:54:22PM +0800, Chester Lin wrote:
> > Add bindings for S32G2's evaluation board (S32G-VNP-EVB) and reference
> > design 2 board ( S32G-VNP-RDB2).
> > 
> > Signed-off-by: Chester Lin <clin@suse.com>
> > ---
> >  Documentation/devicetree/bindings/arm/fsl.yaml | 7 +++++++
> >  1 file changed, 7 insertions(+)
> > 
> > diff --git a/Documentation/devicetree/bindings/arm/fsl.yaml b/Documentation/devicetree/bindings/arm/fsl.yaml
> > index e2097011c4b0..3914aa09e503 100644
> > --- a/Documentation/devicetree/bindings/arm/fsl.yaml
> > +++ b/Documentation/devicetree/bindings/arm/fsl.yaml
> > @@ -983,6 +983,13 @@ properties:
> >            - const: solidrun,lx2160a-cex7
> >            - const: fsl,lx2160a
> >  
> > +      - description: S32G2 based Boards
> > +        items:
> > +          - enum:
> > +              - fsl,s32g274a-evb
> > +              - fsl,s32g274a-rdb2
> > +          - const: fsl,s32g2
> 
> Given this is an entirely different family from i.MX and new?, shouldn't 
> it use 'nxp' instead of 'fsl'? Either way,

It sounds good and Radu from NXP has mentioned a similar idea for the
compatible string of linflexuart. To keep the naming consistency, should we
change all 'fsl' to 'nxp' as well? For example, we could rename the fsl.yaml
to nxp.yaml. However, changing all of them would cause some impacts, which will
need more verifications on new strings. Otherwise we would have to tolerate the
naming differences only used by s32g2.

Thanks,
Chester

> 
> Acked-by: Rob Herring <robh@kernel.org>
> 
> Rob
>