On Sun, Jan 10, 2016 at 7:20 PM, xuejiancheng <xuejiancheng@xxxxxxxxxx> 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@xxxxxxxxxx> >>> Signed-off-by: Jiancheng Xue <xuejiancheng@xxxxxxxxxx> >>> --- >>> +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 -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html