Re: [PATCH 2/2] dt: bindings: Add binding for ESP8089 wifi chips

On Monday, August 8, 2016 11:59:25 AM CEST Chen-Yu Tsai wrote:

Hi Arnd,

On Mon, Aug 8, 2016 at 3:59 AM, Arnd Bergmann [off-list ref] wrote:

quoted

On Sunday, August 7, 2016 8:57:49 PM CEST Hans de Goede wrote:

quoted

+Espressif ESP8089 wireless SDIO devices
+
+This node provides properties for controlling the ESP8089 wireless device.
+The node is expected to be specified as a child node to the SDIO controller
+that connects the device to the system.
+
+Required properties:
+
+ - compatible : Should be "esp,esp8089".

I think it would be good to standardize a compatible string for sdio
like we have it for other discoverable buses, based on the
vendor/device ID (or whatever this uses), using something like

compatible = "sdio1234,5678", "esp,esp8089";

Is there a central repository for SDIO vendor/device IDs similar to
PCI or USB IDs?

No idea. I believe the existing repositories for PCI and USB are not
public either, there is just the information at http://pcidatabase.com/,
http://pci-ids.ucw.cz/ or http://www.linux-usb.org/usb.ids that
are maintained by volunteers and that are not authoritarive.

I found one list at https://github.com/systemd/systemd/blob/master/hwdb/sdio.ids
but it's probably very incomplete.

The vendor ID Espressif uses is 0x6666, which seems kind of bogus to me.

It probably is, but that's not important as long as nothing else uses
that ID.

	Arnd

Hi,
08.08.2016, 17:30, "Arnd Bergmann" [off-list ref]:

On Monday, August 8, 2016 11:59:25 AM CEST Chen-Yu Tsai wrote:

quoted

?Hi Arnd,

?On Mon, Aug 8, 2016 at 3:59 AM, Arnd Bergmann [off-list ref] wrote:
?> On Sunday, August 7, 2016 8:57:49 PM CEST Hans de Goede wrote:
?>> +Espressif ESP8089 wireless SDIO devices
?>> +
?>> +This node provides properties for controlling the ESP8089 wireless device.
?>> +The node is expected to be specified as a child node to the SDIO controller
?>> +that connects the device to the system.
?>> +
?>> +Required properties:
?>> +
?>> + - compatible : Should be "esp,esp8089".
?>
?> I think it would be good to standardize a compatible string for sdio
?> like we have it for other discoverable buses, based on the
?> vendor/device ID (or whatever this uses), using something like
?>
?> compatible = "sdio1234,5678", "esp,esp8089";

?Is there a central repository for SDIO vendor/device IDs similar to
?PCI or USB IDs?

No idea. I believe the existing repositories for PCI and USB are not
public either, there is just the information at http://pcidatabase.com/,
http://pci-ids.ucw.cz/ or http://www.linux-usb.org/usb.ids that
are maintained by volunteers and that are not authoritarive.

I found one list at https://github.com/systemd/systemd/blob/master/hwdb/sdio.ids
but it's probably very incomplete.

Maybe it should be maintained as a independent project...

quoted

?The vendor ID Espressif uses is 0x6666, which seems kind of bogus to me.

It probably is, but that's not important as long as nothing else uses
that ID.

Yes... I think the USB vendor ID of Allwinner, 0x1f3a, is also nothing authoritative.

????????Arnd

Thanks,
Icenowy