Hi Mark

On 05/04/18 07:13, Mark Brown wrote:
> On Thu, May 03, 2018 at 09:36:42PM +0000, Yixun Lan wrote:
> 
>>  static const struct of_device_id meson_spicc_of_match[] = {
>> -    { .compatible = "amlogic,meson-gx-spicc", },
>> -    { .compatible = "amlogic,meson-axg-spicc", },
>> +    {
>> +            .compatible     = "amlogic,meson-gx-spicc",
>> +            .data           = &meson_spicc_gx_data,
>> +    },
>> +    {
>> +            .compatible = "amlogic,meson-axg-spicc",
>> +            .data           = &meson_spicc_axg_data,
>> +    },
>>      { /* sentinel */ }
>>  };
>>  MODULE_DEVICE_TABLE(of, meson_spicc_of_match);
> 
> This is changing the DT bindings but there's no update to the DT binding
> documentation, please add an update there.
> 

thanks for your comments, but I actually didn't get what's your point here.

Here I only introduce the dt compatible data to differentiate the
old/new controller, the compatible name is not changed, and none of the
property is introduced.

Yixun

Reply via email to