public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Lukasz Majewski <l.majewski@samsung.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 0/3] add dfu support for at91 sam9260 based boards
Date: Tue, 21 Jul 2015 07:36:41 +0200	[thread overview]
Message-ID: <20150721073641.3335bf2c@amdc2363> (raw)
In-Reply-To: <55AD11A4.7040109@denx.de>

Hi Heiko,

> Hello Lukasz,
> 
> Am 20.07.2015 um 16:03 schrieb Lukasz Majewski:
> > Hi Marek,
> >
> >> add dfu supprt for at91 sam9260 based boards. The USB
> >> gadget driver is ported from linux:
> >>
> >> b2ba27a5c56ff: usb: gadget: at91_udc: move prepare clk into process
> >> context
> >>
> >> it drops a lot of checkpatch warnings/errors:
> >>
> >> checkpatch.pl found 12 error(s), 31 warning(s), 43 checks(s)
> >>
> >> but for further updates I did not fix them.
> >> The errors are all from this sort:
> >>
> >> error: drivers/usb/gadget/at91_udc.c,87: space prohibited before
> >> open square bracket '['
> >>
> >> a lot of "line over 80 characters" warnings ...
> >>
> >>
> >> Heiko Schocher (3):
> >>    ARM: at91: add cpu.h
> >>    usb: gadget: at91_udc: add support for at91_udc
> >>    at91, taurus, smartweb: add dfu support
> >
> > Marek, do you plan to review this patch series?
> >
> > (I would need at least one review before I pull the code).
> 
> Maybe a Reviewed-by or Acked-by from you (and Marek) is enough,
> as it is maybe pickedup by Andreas Biessmann (at91 maintainer)
> 

No problem. I will review the code.

> Thanks!
> 
> bye,
> Heiko
> >
> >>
> >>   arch/arm/mach-at91/include/mach/cpu.h |  149 +++
> >>   board/siemens/smartweb/smartweb.c     |   29 +
> >>   board/siemens/taurus/taurus.c         |   27 +
> >>   drivers/usb/gadget/Makefile           |    1 +
> >>   drivers/usb/gadget/at91_udc.c         | 2203
> >> +++++++++++++++++++++++++++++++++
> >> drivers/usb/gadget/at91_udc.h         |  171 +++
> >> include/configs/smartweb.h            |   34 +-
> >> include/configs/taurus.h              |   28 +-
> >> include/linux/usb/at91_udc.h          |   20 + 9 files changed,
> >> 2657 insertions(+), 5 deletions(-) create mode 100644
> >> arch/arm/mach-at91/include/mach/cpu.h create mode 100644
> >> drivers/usb/gadget/at91_udc.c create mode 100644
> >> drivers/usb/gadget/at91_udc.h create mode 100644
> >> include/linux/usb/at91_udc.h
> >>
> >
> >
> >
> 



-- 
Best regards,

Lukasz Majewski

Samsung R&D Institute Poland (SRPOL) | Linux Platform Group

  reply	other threads:[~2015-07-21  5:36 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-06-15 12:42 [U-Boot] [PATCH 0/3] add dfu support for at91 sam9260 based boards Heiko Schocher
2015-06-15 12:42 ` [U-Boot] [PATCH 1/3] ARM: at91: add cpu.h Heiko Schocher
2015-07-21  1:59   ` Marek Vasut
2015-07-22 14:40   ` Alexandre Belloni
2015-06-15 12:43 ` [U-Boot] [PATCH 2/3] usb: gadget: at91_udc: add support for at91_udc Heiko Schocher
2015-07-21  6:59   ` Lukasz Majewski
2015-08-10  5:16     ` Heiko Schocher
2015-06-15 12:43 ` [U-Boot] [PATCH 3/3] at91, taurus, smartweb: add dfu support Heiko Schocher
2015-07-21  2:02   ` Marek Vasut
2015-07-21  7:02   ` Lukasz Majewski
2015-07-20 14:03 ` [U-Boot] [PATCH 0/3] add dfu support for at91 sam9260 based boards Lukasz Majewski
2015-07-20 15:20   ` Heiko Schocher
2015-07-21  5:36     ` Lukasz Majewski [this message]
2015-07-21  2:03   ` Marek Vasut
2015-08-10  5:14     ` Heiko Schocher

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20150721073641.3335bf2c@amdc2363 \
    --to=l.majewski@samsung.com \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox