From mboxrd@z Thu Jan 1 00:00:00 1970 From: Roger Quadros Subject: Re: [PATCH 09/13] mfd: omap-usb-host: Add device tree support and binding information Date: Wed, 6 Feb 2013 10:56:39 +0200 Message-ID: <51121AC7.6070107@ti.com> References: <1359993540-20780-1-git-send-email-rogerq@ti.com> <1359993540-20780-10-git-send-email-rogerq@ti.com> <20130205142049.GC26842@e106331-lin.cambridge.arm.com> <51111A41.4050607@ti.com> <20130205161141.GD26842@e106331-lin.cambridge.arm.com> Mime-Version: 1.0 Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20130205161141.GD26842-NuALmloUBlrZROr8t4l/smS4ubULX0JqMm0uRHvK7Nw@public.gmane.org> Sender: linux-usb-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Mark Rutland Cc: "tony-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org" , "linux-lFZ/pmaqli7XmaaqVzeoHQ@public.gmane.org" , "devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org" , "linux-usb-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , "balbi-l0cyMroinI0@public.gmane.org" , "stern-nwvwT67g6+6dFdvTe/nMLpVzexx5G7lz@public.gmane.org" , Samuel Ortiz , "linux-omap-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , "linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org" List-Id: devicetree@vger.kernel.org On 02/05/2013 06:11 PM, Mark Rutland wrote: > [...] > >>>> + >>>> +- single_ulpi_bypass: Must be present if the controller contains a single >>>> + ULPI bypass control bit. e.g. OMAP3 silicon <= ES2.1 >>> >>> Again it would be nicer to have '-' rather than '_' here. It might be worth >>> prefixing this "ti,". >> >> Is prefixing with "ti" really required? how does it better? > > I thought single-ulpi-bypass sounded rather generic, but it probably is > specific enough as-is. I don't know enough about USB hardware to have strong > feelings either way. > Yes, it is specific to the TI silicon. I'll leave it as it is then. cheers, -roger -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html