[PATCH 1/5] W1: Add device tree support to MXC onewire master.

[Mark Rutland]

Hi,

This looks nice and simple, I just have a couple of comment.

On Wed, Jan 16, 2013 at 09:48:23AM +0000, Martin Fuzzey wrote:
> Signed-off-by: Martin Fuzzey <mfuzzey at parkeon.com>
> ---
>  drivers/w1/masters/mxc_w1.c |    9 ++++++++-
>  1 files changed, 8 insertions(+), 1 deletions(-)
> 
> diff --git a/drivers/w1/masters/mxc_w1.c b/drivers/w1/masters/mxc_w1.c
> index 708a25f..949e566 100644
> --- a/drivers/w1/masters/mxc_w1.c
> +++ b/drivers/w1/masters/mxc_w1.c
> @@ -186,9 +186,16 @@ static int mxc_w1_remove(struct platform_device *pdev)
>  	return 0;
>  }
>  
> +static struct of_device_id mxc_w1_dt_ids[] = {
> +	{ .compatible = "fsl,imx21-owire" },
> +	{ /* sentinel */ }
> +};
> +MODULE_DEVICE_TABLE(of, mxc_w1_dt_ids);
> +
>  static struct platform_driver mxc_w1_driver = {
>  	.driver = {
> -		   .name = "mxc_w1",
> +		.name = "mxc_w1",
> +		.of_match_table = mxc_w1_dt_ids,
>  	},
>  	.probe = mxc_w1_probe,
>  	.remove = mxc_w1_remove,
> 

I see there's already a binding for gpio-based w1 -- "w1-gpio". Given this is
already using the "w1" naming convention in a binding, I think it'd be worth
using the compatible string "fsl,imx21-w1", for consistency (both with the
other binding and the driver name).

It's also be worth documenting (in
Documentation/devicetree/bindings/w1/fsl-imx21-w1, presumably).

When adding bindings, it's also good to Cc devicetree-discuss.

Thanks,
Mark.