14:52 <arnd> krzk, linusw__, tmlind: I was looking through leftovers from the boardfile removal, in particular drivers that use the old gpio interface and have neither DT support nor any remaining platform_data definitions. I came across the Nokia N8x0 using p54spi wireless with hardcoded GPIO lines instead and tried to address that

14:52 <arnd> the patch seemed straightforward enough, needs review and testing of course: https://www.irccloud.com/pastebin/Lg0C6U3q/

14:53 <arnd> My main struggle right now is the DT binding, as I've never actually written one from scratch, only patched existing ones. Questions:

14:53 <arnd> - is it ok to just put the driver authors as maintainers of the binding, as I don't have any interest in this binding or driver myself?

14:55 <arnd> - as the device was apparently sold by half a dozen companies at some point, should I name the binding after the one that most likely designed the hardware (intersil?) or the one that is apparently written on the chip that is actually used?

14:55 <arnd> - should I document all known IDs or just the one that I used?

14:55 <krzk> arnd: 1. yes, it should be someone interested in HW or having it, different thing is if they want to be :)

14:55 <javierm> arnd: I believe that's OK, but then your patch should probably add the DT binding doc path to the MAINTAINERS entry as well

14:55 <javierm> which seems to be P54 WIRELESS DRIVER ?

14:56 <krzk> arnd: 2. Uh, no clue here, maybe both would work.

14:57 <krzk> arnd: 3. up to you. If you know all, it is nice to document all, but it is not a requirement.

14:57 <arnd> javierm: ok, added to the maintainers file

14:57 <krzk> arnd: especially if you do not know the differences between the IDs

15:00 <arnd> krzk: I have no idea who is still using an N8x0 today, the driver was touched by a lot of people in the past that seemed to use it, but not recently

15:01 <arnd> krzk: what is the recommended way to check if the binding and the dts file are both well-formed?

15:03 <javierm> arnd: you can do `make dt_binding_check DT_SCHEMA_FILES=Documentation/devicetree/bindings/net/wireless/st,stlc45xx.yaml`

15:04 <arnd> thanks

15:05 <javierm> arnd: and `make dtbs_check` I think will check the DTS too

15:14 <frieder> javierm: arnd: `make CHECK_DTBS=y <dtb_path>` is quite useful for checking a single DTS

15:18 <javierm> frieder: ah cool, I didn't know that one. Thanks!

22:53 <linusw__> arnd: looks good to me :)