Re: [PATCH 4/4] dt: binding: Add Qualcomm wcn36xx WiFi binding

From: Andy Gross
Date: Tue Dec 29 2015 - 14:11:24 EST


On Sun, Dec 27, 2015 at 05:34:27PM -0800, Bjorn Andersson wrote:
> Add binding representing the Qualcomm wcn3620/60/80 WiFi block.
>
> Signed-off-by: Bjorn Andersson <bjorn.andersson@xxxxxxxxxxxxxx>
> ---

<snip>

> +
> +- qcom,wcnss-mmio:
> + Usage: required
> + Value type: <prop-encoed-array>

nit: encoded

> + Definition: should specify base address and size of the WiFi related
> + registers of WCNSS
> +
> +- qcom,state:
> + Usage: required
> + Value type: <prop-encoded-array>
> + Definition: should specify the tx-enable and tx-ring-empty state
> + references
> +

<snip>

Otherwise looks good.

Reviewed-by: Andy Gross <andy.gross@xxxxxxxxxx>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/