[U-Boot] [PATCH 1/4 v2] serial: Add Zynq serial driver
Marek Vasut
marex at denx.de
Fri Sep 14 06:47:57 CEST 2012
Dear Joe Hershberger,
> Hi Marek,
>
> On Thu, Sep 13, 2012 at 9:01 AM, Marek Vasut <marex at denx.de> wrote:
> > Dear Michal Simek,
> >
> >> On 09/13/2012 02:33 PM, Marek Vasut wrote:
> >> > Dear Michal Simek,
> >> >
> >> >> On 09/13/2012 11:21 AM, Marek Vasut wrote:
> >> >>> Dear Michal Simek,
> >> >>>
> >> >>>> The driver is used on Xilinx Zynq platform.
> >> >>>>
> >> >>>> Signed-off-by: Michal Simek <monstr at monstr.eu>
> >> >>>>
> >> >>>> ---
> >> >>>> v2: Use Zynq name instead of Dragonfire and XPSS/XDFUART
> >> >>>>
> >> >>>> Rename driver name
> >> >>>> Remove driver description
> >> >>>>
> >> >>>> ---
> >> >>>>
> >> >>>> drivers/serial/Makefile | 1 +
> >> >>>> drivers/serial/serial_zynq.c | 200
> >> >>>>
> >> >>>> ++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 201
> >> >>>> insertions(+), 0 deletions(-)
> >> >>>>
> >> >>>> create mode 100644 drivers/serial/serial_zynq.c
> >> >>>
> >> >>> [...]
> >> >>>
> >> >>> It looks ok, but can you make it support CONFIG_SERIAL_MULTI right
> >> >>> away please?
> >> >>
> >> >> Yes, it will add serial_multi in the next patch.
> >> >> Can you give me your ACK or reviewed-by line? :-)
> >> >
> >> > Just squash them into one patch please.
> >>
> >> Done.
> >
> > Thanks ... my idea is to switch to serial_multi completely, then rework
> > stdio, unify serial on top of it and clean up the whole subsystem.
> > Thanks for helping!
>
> I have a patch that I'll send soon that moves the "nulldev" driver to
> the serial driver (if CONFIG_SERIAL_MULTI is defined) so that it is
> possible to avoid serial init.
Thanks, this will break my massive patchset though. Can you check
git://git.denx.de/u-boot-marex.git / stdio branch ? I did something to nulldev
there. Probably ignore the top 7 patches, they're bogus and need further work.
It's all still work in progress to some point.
> I had the problem that the serial port
> on my Zynq product lives in the fabric instead of using the hard-core
> ones.
I see ... why don't you implement nulldev_serial instead and leave nulldev stdio
as is?
> This means that the FPGA neds to be configured first, but since
> I'm using u-boot for that too, I need it to start as null and switch
> to the fabric serial after the FPGA is configured. Anyway, just more
> support for supporting CONFIG_SERIAL_MULTI.
>
> -Joe
Best regards,
Marek Vasut
More information about the U-Boot
mailing list