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;
From:   Vladimir Oltean <vladimir.oltean@nxp.com>
To:     Rob Herring <robh@kernel.org>
CC:     Shawn Guo <shawnguo@kernel.org>, Lee Jones <lee.jones@linaro.org>,
        Andrew Lunn <andrew@lunn.ch>,
        Florian Fainelli <f.fainelli@gmail.com>,
        Claudiu Manoil <claudiu.manoil@nxp.com>,
        Leo Li <leoyang.li@nxp.com>, Michael Walle <michael@walle.cc>,
        "netdev@vger.kernel.org" <netdev@vger.kernel.org>,
        "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "linux-arm-kernel@lists.infradead.org" 
        <linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH 2/3] dt-bindings: mfd: add "fsl,ls1028a-qds-qixis-i2c"
 compatible to sl28cpld
Thread-Topic: [PATCH 2/3] dt-bindings: mfd: add "fsl,ls1028a-qds-qixis-i2c"
 compatible to sl28cpld
Thread-Index: AQHYE6JPJsttfCUAAUS/+BKErg2nQKyOZ8IAgAAGRwA=
Date:   Fri, 11 Feb 2022 13:24:11 +0000
Message-ID: <20220211132410.3qldvlpznlrxsgzg@skbuf>
References: <20220127172105.4085950-1-vladimir.oltean@nxp.com>
 <20220127172105.4085950-3-vladimir.oltean@nxp.com>
 <YgZeNqAbdisyeT+s@robh.at.kernel.org>
In-Reply-To: <YgZeNqAbdisyeT+s@robh.at.kernel.org>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?UppBvJ0okN2NRA00YSYEgcNib0Y9YQ+WAtohB/FA44aigWiSCU8HiUDojSfZ?=
 =?us-ascii?Q?w+9ZlClwclasBRQdx3U4i8IfsdpBGW4VzQJh5wzXnGo6dxqbPMAlQF65aWiZ?=
 =?us-ascii?Q?Rjkx+jgUkfD/KiRTCqTyO0LKCG9APeNOKraTuHZ0LzJQBfKw7ItkMI7LxB39?=
 =?us-ascii?Q?fu/45rdcpfMcOGynGztK7T3MWzeEWE2wZSwMzX3KD1xnoOmPHvv53fd1YQM3?=
 =?us-ascii?Q?jbFuM6dd1+egHLiM9K4xxnghJxJQ7kFdNIocETkAtifduDy1pV9EcSlZ8GZl?=
 =?us-ascii?Q?XJU0G3QT3YTKvG1UtXAL1f12t5eCHZIWRwkpSxCMrVTo71VxZMcomkxBuuhu?=
 =?us-ascii?Q?jkFM6crul5rUmlRc4/ctmyUYhTLq6c0mfrxsBoyy0PgEyb+9ZhlAl0gCMFCK?=
 =?us-ascii?Q?mqfnXce2yGqSBP8rkV/gGL5HBL2GpXxkW+WBPpLNgRxtZ4BFZ0avphHNYPZ1?=
 =?us-ascii?Q?GJBvhqSoOc8Fdv9lpFsvawj6QV57qX1vuAjU+Wxacgq1CD9DjPmZRyTCROwu?=
 =?us-ascii?Q?dA0ChEG8/L0lg1pNVM1+jId56A9IJue3rEL945CIa5mrUgMX3tePvX9iaT41?=
 =?us-ascii?Q?rcicFR82DTt/zYxaiqqfr49RwFRkvdH9MiL9VFc010zsbl+prs/qP7lm6Va6?=
 =?us-ascii?Q?4Yej6pT5jscyfq1HNCxD87FPVo2lYTbbXLucD4bfVddYOtJX+a+lV2go8RPl?=
 =?us-ascii?Q?xJAdsEoCF0rQBOug5M1ukPehDfBXjxq9vaF4QaTjsWDy1Y92jNN6FfRg+dA2?=
 =?us-ascii?Q?9JjkYKeTaHzYspN4zt0EJ7YKhhBNOERY/Vz9zVJSAZ4KjugPVG9TmxfnzKvS?=
 =?us-ascii?Q?lDocxLFJvEV6HB8EmbgkxJX26xZLmjQ8eJbWMXDMhwrbBWnlLDoQM44DPdP2?=
 =?us-ascii?Q?JaX0mMqODstWZFalm7nj0jMXRh3mtj/vGl1YmOG5H1NyJDk81HsC98zY/AeT?=
 =?us-ascii?Q?CfDJ1mAZoF0IJ8P0FhkxhJqgo572+RENSO7pLgyidwBOlxEehVKb9GSuS7Sj?=
 =?us-ascii?Q?txYNS7Ji7pElfecuphd0qeJrCaZ8/AgxIT9TQjw1/Mz9Z8Q79YfFrPFTaXwv?=
 =?us-ascii?Q?ZjV11wVAY1bvDvVFDmPSuGaIRk8SuZ7ZpJ5u/ve6MxI69A3b5L/ruGAbb3tP?=
 =?us-ascii?Q?kCrd4vbLWAFivORqlk+oW3uMBnHUGjCGD/93YoJwCUiYK6WjaVkKML1YGwD4?=
 =?us-ascii?Q?xy4pZd7BzRoHMGwoYFoguEKe2Ld6raeykZ32BGT/usDJYt7rxAtcQrIHiQN8?=
 =?us-ascii?Q?BMHHIgJfnijAQLJh82EvZSfHKuqR0WUTjTb2f1ktqR4P/llA40wHrs6NVRz/?=
 =?us-ascii?Q?KMqqncETZyqNFcMG97uyBPyF7o3uxQzwfceJxUxQafa9e5RvlxmwEHFRDw7s?=
 =?us-ascii?Q?OXKidH4BphCxpK+mB5i0fGZcTIJmpyPkGwmZsESqbfnU5E9hWQI7AW9hNvbs?=
 =?us-ascii?Q?bE4HKVU7eK9CiSVDiuHFIRAJB9O87M/L8ll3aQ3+Lb+XbqM/uPXo8aegXUfa?=
 =?us-ascii?Q?nhZG/2RXy9JpGlfnNFgoIm92MiiA5iLOcxTlIJ9uu7u8TOOT92GPqV2JOVmJ?=
 =?us-ascii?Q?K9gy4DSE8hoSeYdQo1Jfh00mB4rzm/JRSEf2/6l2+pMdA15Qdllrpdk4hLyS?=
 =?us-ascii?Q?WTxroNsukLAzgdf8YRYl49M=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-ID: <0DB778AC6052874E919507B14AF9A7FC@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: fbfe0b17-99d7-4fce-4c8d-08d9ed61ca5e
X-MS-Exchange-CrossTenant-originalarrivaltime: 11 Feb 2022 13:24:11.1174
 (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: paPOCuIuMWMgNUGarx6fyrQATqHLsEbaXQBA8eQHOXHHgZwwi2UeqoMC7N4zHCY0xaMiNcoRKK6sX5tMaHJSnA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM0PR04MB7009
X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE,
        RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS,T_SCC_BODY_TEXT_LINE
        autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Fri, Feb 11, 2022 at 07:01:42AM -0600, Rob Herring wrote:
> On Thu, Jan 27, 2022 at 07:21:04PM +0200, Vladimir Oltean wrote:
> > The LS1028A-QDS QIXIS FPGA has no problem working with the
> > simple-mfd-i2c.c driver, so extend the list of compatible strings to
> > include that part.
> >=20
> > Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
> > ---
> >  Documentation/devicetree/bindings/mfd/kontron,sl28cpld.yaml | 4 +++-
> >  1 file changed, 3 insertions(+), 1 deletion(-)
> >=20
> > diff --git a/Documentation/devicetree/bindings/mfd/kontron,sl28cpld.yam=
l b/Documentation/devicetree/bindings/mfd/kontron,sl28cpld.yaml
> > index eb3b43547cb6..8c1216eb36ee 100644
> > --- a/Documentation/devicetree/bindings/mfd/kontron,sl28cpld.yaml
> > +++ b/Documentation/devicetree/bindings/mfd/kontron,sl28cpld.yaml
> > @@ -16,7 +16,9 @@ description: |
> > =20
> >  properties:
> >    compatible:
> > -    const: kontron,sl28cpld
> > +    enum:
> > +      - fsl,ls1028a-qds-qixis-i2c
> > +      - kontron,sl28cpld
>=20
> Is there some relationship between these besides happening to use the=20
> same driver? Sharing a generic driver is not a reason to have the same=20
> binding doc.
>=20
> Your DT has a mux-controller which is undocuemnted in this binding.
>=20
> Rob

I'd guess they are both programmable FPGA's/CPLD's that are used for
board control. What I don't know is whether the sources for the Kontron
bit stream are derived in any way from the QIXIS.

I can look into adding a separate binding doc for the QIXIS anyway.=