Re: [PATCH 01/10] ehci early_printk: split ehci debug driver fromearly_printk.c

[Jason Wessel]

Yinghai Lu wrote:
> Jason Wessel wrote:
>   
>> Move the dbgp early printk driver in advance of refactoring and adding
>> new code, so the changes to this code are tracked separately from the
>> move of the code.
>>
>> The drivers/usb/early directory will be the location of the current
>> and future early usb code for driving usb devices prior initializing
>> the standard interrupt driven USB drivers.
>>
>> Signed-off-by: Jason Wessel <jason.wessel@xxxxxxxxxxxxx>
>> Cc: Greg KH <gregkh@xxxxxxx>
>> Cc: Ingo Molnar <mingo@xxxxxxx>
>> Cc: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>
>> Cc: Yinghai Lu <yinghai@xxxxxxxxxx>
>> Cc: "Eric W. Biederman" <ebiederm@xxxxxxxxxxxx>
>> ---
>>  arch/x86/kernel/early_printk.c |  715 ---------------------------------------
>>  drivers/usb/Makefile           |    1 +
>>  drivers/usb/early/Makefile     |    5 +
>>  drivers/usb/early/ehci-dbgp.c  |  723 ++++++++++++++++++++++++++++++++++++++++
>>  include/linux/usb/ehci_def.h   |    6 +
>>  5 files changed, 735 insertions(+), 715 deletions(-)
>>  create mode 100644 drivers/usb/early/Makefile
>>  create mode 100644 drivers/usb/early/ehci-dbgp.c
>>     
>
> before this patch, we could use dbgp without compiling USB susbsystem in...
>
>   

And you shouldn't need it today.  You pointed out an error in the patch
which I will correct.  The drivers/Makefile needs to be changed, instead
of the drivers/usb/Makefile.

The dbgp code was changed in the patch series such that it works with or
with out the usb subsystem and with or without the earlyconsole=keep,...

All the code in the drivers/usb/early would work this way as well.  As
an example, I have an experimental UHCI serial driver that can be used
via the earlyprintk.

Thanks for the feed back.  I'll fix this for the v2 of the patch series.
Jason.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/