Re: [PATCH] spi: sirf: drop redundant sirf,marco-spi compatible string

[Barry Song <21cnbao@xxxxxxxxx>]

2015-01-06 5:27 GMT+08:00 Mark Brown <broonie@xxxxxxxxxx>:
> On Sat, Jan 03, 2015 at 05:04:44PM +0800, Barry Song wrote:
>> From: Barry Song <Baohua.Song@xxxxxxx>
>>
>> "sirf,marco-spi" is redundant as all SPI controllers in CSR SiRFSoC are
>> compatible with prima2-spi.
>> at the same time, the whole marco project was dropped and its replacement
>> atlas7 is also compatible with prima2 in SPI.
>
> I don't see any pressing reason to do this (the cost is basically zero)
> and the compatible string seems to be used in a DTSI file, though
> perhaps that's going away?

its value is zero but it is necessary.
this should be clear as marco has been dropped in dtsi and machine level.
and there is no any reference for this string even in current codes.

i think i have been wrong from this commit:
https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/drivers/spi/spi-sirf.c?id=f3b8a8ecc5922d9dff303ae2fadc1eae608a6f7c

>
> I'd also say that the binding document update is missing but it seems
> this binding is undocumented.  :/

yes. i realized i missed to include the DT doc when i sent the spi
patch at the first time even though it always exists in my local tree.
i will add the missed one.

-barry
--
To unsubscribe from this list: send the line "unsubscribe linux-spi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html