On Mon, Dec 30, 2013 at 08:21:43AM +0530, Tushar Behera wrote:
On 28 December 2013 00:14, Greg KH gregkh@linuxfoundation.org wrote:
But what about systems that rely on a hard-coded /dev?
Look, I'm all for making everyone use devtmpfs, but just changing major:minor numbers for drivers isn't ok, as you are changing the
Please realize what you are asking for here, I really don't think you grasp it given that you didn't ask any of the maintainers of this driver about the change in the first place.
Please get approval for this patch from others within Linaro before sending it out again. Linaro has a process in place for this type of thing, please use it, otherwise it makes people like me really grumpy and upset and causes me to yell at people at their conferences.
Asking for opinion about this ... Without this patch, I can't get serial console to come up on Exynos platforms when amba-pl011 driver enabled. But Greg is particularly not at all happy with this patch. Any comments with respect to fixing this issue would be highly appreciated.
The most direct thing would seem to be to provide a fallback in the serial core which (perhaps optionally) assigns a dynamic number only if there's a conflict as an alternative to failing registration; it won't break existing systems since they wouldn't have been able to register the extra device in the first place.