Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1305797

Re: [PATCH v2] mtd: spi-nor: add hisilicon spi-nor flash controller driver

Show all headers | View raw

On Sun, Jan 10, 2016 at 7:20 PM, xuejiancheng <xuejiancheng@huawei.com> wrote:
> Hi Rob,
>
> On 2016/1/9 23:42, Rob Herring wrote:
>> On Fri, Jan 08, 2016 at 09:26:45AM +0800, Jiancheng Xue wrote:
>>> add hisilicon spi-nor flash controller driver
>>>
>>> Signed-off-by: Binquan Peng <pengbinquan@huawei.com>
>>> Signed-off-by: Jiancheng Xue <xuejiancheng@huawei.com>
>>> ---

>>> +Required properties:
>>> +- compatible : Should be "hisilicon,hisi-sfc".
>>
>> I missed this last time, but you also need a more specific compatible
>> string with the SOC name in it.
>>
>
> This IP block is used in many SOCs. Do I need to add a specific compatible
> string for every SOC? Thank you.

Yes. You can keep the generic one for driver matching, but you want to
have the specific one in case there ever is a quirk/errata in a
specific SOC.

Rob

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH v2] mtd: spi-nor: add hisilicon spi-nor flash controller driver Jiancheng Xue <xuejiancheng@huawei.com> - 2016-01-08 02:50 +0100
  Re: [PATCH v2] mtd: spi-nor: add hisilicon spi-nor flash controller  driver Rob Herring <robh@kernel.org> - 2016-01-09 16:50 +0100
    Re: [PATCH v2] mtd: spi-nor: add hisilicon spi-nor flash controller  driver xuejiancheng <xuejiancheng@huawei.com> - 2016-01-11 02:30 +0100
      Re: [PATCH v2] mtd: spi-nor: add hisilicon spi-nor flash controller driver Rob Herring <robh@kernel.org> - 2016-01-11 04:20 +0100
        Re: [PATCH v2] mtd: spi-nor: add hisilicon spi-nor flash controller  driver xuejiancheng <xuejiancheng@huawei.com> - 2016-01-11 05:00 +0100

csiph-web