Re: [PATCH 1/4] i2c: at91: add compatible with microchip,sama7g5-i2c

[Michael Walle <michael@walle.cc>]

Hi,

> Add compatible and data platform struct for sama7g5 SoC.
> 
> Signed-off-by: Eugen Hristev <eugen.hristev@microchip.com>
> ---
>  drivers/i2c/at91_i2c.c | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/drivers/i2c/at91_i2c.c b/drivers/i2c/at91_i2c.c
> index 6b4c0e4804..400a3786ca 100644
> --- a/drivers/i2c/at91_i2c.c
> +++ b/drivers/i2c/at91_i2c.c
> @@ -305,6 +305,11 @@ static const struct at91_i2c_pdata sama5d2_config = {
>  	.clk_offset = 3,
>  };
>  
> +static const struct at91_i2c_pdata sama7g5_config = {
> +	.clk_max_div = 7,
> +	.clk_offset = 3,
> +};
> +
>  static const struct udevice_id at91_i2c_ids[] = {
>  { .compatible = "atmel,at91rm9200-i2c", .data = (long)&at91rm9200_config },
>  { .compatible = "atmel,at91sam9260-i2c", .data = (long)&at91sam9260_config },
> @@ -314,6 +319,7 @@ static const struct udevice_id at91_i2c_ids[] = {
>  { .compatible = "atmel,at91sam9x5-i2c", .data = (long)&at91sam9x5_config },
>  { .compatible = "atmel,sama5d4-i2c", .data = (long)&sama5d4_config },
>  { .compatible = "atmel,sama5d2-i2c", .data = (long)&sama5d2_config },
> +{ .compatible = "microchip,sama7g5-i2c", .data = (long)&sama7g5_config },

I see that this compatible string is is also used in the linux
device tree, but there is no dt binding for it in linux. Could you
add it, so the binding is approved by Rob?

-michael