From mboxrd@z Thu Jan 1 00:00:00 1970 From: Scott Wood Subject: Re: [PATCH] clk: corenet: Update the clock bindings Date: Thu, 23 Jan 2014 20:35:34 -0600 Message-ID: <1390530934.24905.639.camel@snotra.buserror.net> References: <1390269732-22798-1-git-send-email-Yuantian.Tang@freescale.com> <1390437863.24905.549.camel@snotra.buserror.net> <1390511008.24905.581.camel@snotra.buserror.net> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Tang Yuantian-B29983 Cc: "galak-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org" , "linuxppc-dev-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org" , "devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , Kushwaha Prabhakar-B32579 List-Id: devicetree@vger.kernel.org On Thu, 2014-01-23 at 20:33 -0600, Tang Yuantian-B29983 wrote: > > > > Instead, how about a note like this near the top of the file: > > > > > > > > All references to "1.0" and "2.0" refer to the QorIQ chassis version > > > > to which the chip complies. > > > > > > > > Chassis Version Example Chips > > > > --------------- ------------- > > > > 1.0 p4080, p5020, p5040 > > > > 2.0 t4240, b4860, t1040 > > > > > > > Better, I will update. > > > > > > > > > > > BTW, this binding and the associated driver really should be called > > > > "qoriq-clock", not "corenet-clock". This would match the compatible > > > > string, and it doesn't really have much to do with corenet (which is > > > > part of the QorIQ chassis v1 and v2, but not *this* part). Do you > > > > know if the chassis v3 clock interface will be similar enough to > > share a driver? > > > > > > > Doesn't QorIQ include some low-end socs, like p1022, p1020? > > > > Yes, but those aren't "QorIQ Chassis 1.0" or "QorIQ Chassis 2.0". > > They're mpc85xx-family chips. > > > > In any case, if "qoriq" makes sense for the compatible, I don't see why > > it doesn't make sense for the driver. > > > So, "Corenet" is appropriate for driver. > If something should change, that must be compatible string. No. Corenet is a bus interconnect, not a chip family (despite abuse of the name in other contexts in Linux/U-Boot). And the binding with qoriq has already been accepted. -Scott -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html