Re: [RFC PATCH v2 2/2] kselftest: devices: Add sample board file for google,spherion

[Nícolas F. R. A. Prado]

On Tue, Nov 28, 2023 at 12:10:46AM +0000, Bird, Tim wrote:
> > -----Original Message-----
> > From: Nícolas F. R. A. Prado <nfraprado@xxxxxxxxxxxxx>
> > Add a sample board file describing the file's format and with the list
> > of devices expected to be probed on the google,spherion machine as an
> > example.
> > 
> > Signed-off-by: Nícolas F. R. A. Prado <nfraprado@xxxxxxxxxxxxx>
> > ---
> > 
> > (no changes since v1)
> > 
> >  .../testing/selftests/devices/boards/google,spherion | 12 ++++++++++++
> 
> Overall, while trying to maintain a comprehensive set of board definitions
> seems hard, I think having a few as examples is useful.
> 
> I'm not a big fan of naming these with a comma in the name.  Is there a reason
> you are not using dash or underscore?

I'm using the name that we get from the DT compatible, so the right file can be
automatically selected by the test.

> 
> Do you anticipate a convention of  <producer> <board-or-product-name> tuples for
> the filename?

I'd just stick to the DT compatible as it's the simplest option and should work
just the same, assuming I understood correctly what you mean.

Thanks,
Nícolas