Date:   Wed, 1 Mar 2023 16:36:25 +0200
From:   Vladimir Oltean <vladimir.oltean@nxp.com>
To:     Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Cc:     Daniel Scally <djrscally@gmail.com>,
        Heikki Krogerus <heikki.krogerus@linux.intel.com>,
        Sakari Ailus <sakari.ailus@linux.intel.com>,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        "Rafael J. Wysocki" <rafael@kernel.org>,
        linux-acpi@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: Implementation of fwnode_operations :: device_get_match_data()
 for software nodes?
Message-ID: <20230301143625.7kdnzujlv4psbhla@skbuf>
References: <20230223203713.hcse3mkbq3m6sogb@skbuf>
 <Y/0uC1LgeWR0V0ts@smile.fi.intel.com>
 <20230227234411.jqmwshzkeyx6iqyo@skbuf>
 <Y/9iLBWAO37y6lZZ@smile.fi.intel.com>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <Y/9iLBWAO37y6lZZ@smile.fi.intel.com>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?68H1AvyqsUP1N6X3k2Q3EhiaXj1Axm/j3SPJSOWMgR/QspfbpW6l1UgQIGKL?=
 =?us-ascii?Q?eFkUyYjRT2D+n1RrGqUpTpABUHsDR87luQ1E+kzJ0BQtuyV7Pv5jOaGll9Du?=
 =?us-ascii?Q?lSDOH5uo2/4SaLiUSmBzrGi422DoDIo/3bdTfOyDCVx6KNvbqVFkEt+lLXV2?=
 =?us-ascii?Q?LMYwtuIGp02ZRfamnmeLg6bdgN6byFLOSGaNkp+sTVv8ZaQgz3lw6OiePQTL?=
 =?us-ascii?Q?5n2a4vPmZVDS3weAHItbQ7YQWW1h0DFmLb2fFg2rb+n14otgbeKR12+Z7ZuH?=
 =?us-ascii?Q?6KgreOJqLehsMo7wquEstOPVjoYYuiMvTT8e8Cx3aRT+jbVO6MTPXP+wiQoP?=
 =?us-ascii?Q?ShYtb3p4qTmMv4NcK4+FVuwum7YpmijZhRVLxk9K55QbvUcCOfzC7FELLzfa?=
 =?us-ascii?Q?Q+jzqEqOUfN2Dsjripyq5w+QhrXQglRSnOu3/mCOGwt9vagXNazM88u8B9jx?=
 =?us-ascii?Q?zxb2/cn/EtWjThGxnSjt4bWXr1uyprc5aQTQpWwC9fx2BVYkOV7cronLniRn?=
 =?us-ascii?Q?mnujRikvteofwyYQlplM2YLrMk38HggrMggHqHr69OIanJIDeTsNohsQPinA?=
 =?us-ascii?Q?zxxOCxHM3dMmnhHo5PjwBzmW1Kj+M/HWdVcUB4LkQu9rXd11kRTQY3H7iPVl?=
 =?us-ascii?Q?Gu+fgbMADBQ8hbMLjLJ0JtUpjO9Jj2V1u4e9HPz/eHECEGvuEYT0VX5lfRaN?=
 =?us-ascii?Q?y1b1oPZpL8jYilghwQ4Ee4IjduHqwR52R9KBbyUNSqj97t1fFGn2q16tCCFB?=
 =?us-ascii?Q?5xLhEo88u1YPTe+t/Fln2jTXp9XCq/t9TLiWowfmApFWmFIe0MpO7Yt6G+mi?=
 =?us-ascii?Q?JPvSeYgHoG+6P6srilcWTxrOrmIfyTvhxRmEtZj6uZ1g8cuNOOFXDGaOjzQb?=
 =?us-ascii?Q?K9Sq5GlRRtEsdek2nGApYOUfi/O0HnPKy2djpiwBzdzP+8SlCx4zWx3gc2J6?=
 =?us-ascii?Q?KsideLHlV0XbZ+Ss7B7KF60rGN6N/N9vFSFVsjqDSNFBIcwHzx8fG9Xfe+Jb?=
 =?us-ascii?Q?tQryMBeZIWEc2+ytNa8Cis/MKiyK7WoVX4X60aYV4o6k0UiKrxNK+g1dFHCC?=
 =?us-ascii?Q?zgRvhqRkwE+2UFS8zJRtTQiK5nSeANqHi/DqWYn+4OOIq0XdsqmYTaanB1Li?=
 =?us-ascii?Q?KObjPMDLaBBTldriaeA4nnNUHYjz18l2AezJ2itDWKrXW0BMCWMqKEbBb855?=
 =?us-ascii?Q?UlHan5wDXD98di5ON3yXVZm6xr+ytCWr3qwgB+569EyQBBVsQrGUVJ9EDAeO?=
 =?us-ascii?Q?7/yfN885yPSzMeyu4AZW25UZIk829fInVyZAhkC3xoSATzM07imYRkrAa45h?=
 =?us-ascii?Q?hrfQu5Ni/PH2s8Q4PUAt3zhOhx46okU38kVxR1OJgw9AOU0O8XieEiqV61BW?=
 =?us-ascii?Q?LWSIKMGSMxs9fu1UwpuHoFRejiIL1gtfyCGBXusbxSDyKkxXvsHuyERqMVej?=
 =?us-ascii?Q?71u0/ZXbQ4VlMaUa34jCbAQlCQoJMS7LiTNzof5na1nhXokCpoqpXQqHapCT?=
 =?us-ascii?Q?i7ENsYM9knS41u3it/LrsXbCu2JAdHhv5DOjvZCwsJYtsjP/61tsfMCFOOAm?=
 =?us-ascii?Q?DzTmNBxSPIir+VBoiwOb6so6g6Yav10e8BT+JvgbCuSVWS8U6Lp6WoitjBOP?=
 =?us-ascii?Q?pA=3D=3D?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: a2483353-6957-4246-b517-08db1a62576f
X-MS-Exchange-CrossTenant-AuthSource: AM0PR04MB6452.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 01 Mar 2023 14:36:28.0362
 (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: ZyhzhNws92TE57vp2WlamD/ew548DMQb/1Sj1axii3E8oIQFci2mcn2xfLIG+1abZmf/joF1+kkPRWLFlyHRVQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM7PR04MB7175
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Wed, Mar 01, 2023 at 04:33:16PM +0200, Andy Shevchenko wrote:
> On Tue, Feb 28, 2023 at 01:44:11AM +0200, Vladimir Oltean wrote:
> > On Tue, Feb 28, 2023 at 12:26:19AM +0200, Andy Shevchenko wrote:
> > > I believe that there are few reasons for that:
> > > 1) (besides that what Heikki mentioned);
> > > 2) the software nodes only for quirks, seems you are trying to implement
> > > something that should have to be implemented as proper DT / ACPI device node.
> > > 
> > > Can you elaborate why do you need that (as you see no other board file requires
> > > this)?
> > 
> > Trying to keep the answer short while still answering the question.
> 
> Thank you, this is helpful to understand what you want.
> 
> Random idea #N+1 based on what you told is: how about DT / ACPI overlays?
> Random idea #N+2 is: have you considered FPGA approach?
> 
> So, as far as I got it right the device _can_ be considered as hotpluggable
> blackbox with a lot of hardware onboard. This is very much reminds me FPGA
> sitting behind PCIe hotplug capable interface.
> 
> What do we have now there? Can we spread the same approach for your case?
> 
> Because to me board files here looks like a hack.
> 
> P.S.
> Yeah, I know that SPI is not hotpluggable bus per se. It may be that
> we actually need to reboot machine after plugging in/out the device.

Can you please give me some clearer references for #N+1 and #N+2?
I haven't considered either of those options and I'm not sure what that
would entail.