Messages in this thread | | | Date | Thu, 13 Mar 2014 13:09:50 +0000 | From | Lee Jones <> | Subject | Re: [PATCH v2 2/3] mfd: sec-core: Add of_compatible strings for MFD cells |
| |
<snip>
> > >> Are these compatible strings documented anywhere? > > > > > > The "-clk" compatible strings are (in next patch). The "-rtc" and > > > "-regulators" are not. > > > > Hmm, this doesn't sound good. > > > > > Actually they aren't also used by child driver (each child driver parses > > > from parent of_node). > > > > Do you need to add compatible strings to mfd_cell entries of such cells > > then? > > From my point of view there isn't any special need for them (except for > clk). > > Dear Lee, > > Do you wish to have these compatibles for all of child drivers even when > they aren't used directly by them? If yes then probably I should > document them.
It's your call, either remove the unused ones or document them all.
I don't really mind either way. In the previous version I asked if there was any reason why you weren't applying compatible strings to the other cells; "subordinate drivers don't use them" is a perfectly reasonable reason.
-- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |