public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Marek Vasut <marex@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 1/3] EHCI: Exynos: Add fdt support
Date: Mon, 3 Dec 2012 15:45:22 +0100	[thread overview]
Message-ID: <201212031545.22633.marex@denx.de> (raw)
In-Reply-To: <1354541409-9642-2-git-send-email-rajeshwari.s@samsung.com>

Dear Rajeshwari Shinde,

> Adding fdt support to ehci-exynos in order to parse
> register base addresses from the device node.
> 
> Signed-off-by: Vivek Gautam <gautam.vivek@samsung.com>
[...]

$ ./tools/checkpatch.pl \[PATCH\ 1_3\]\ EHCI_Exynos_Add\ fdt\ support.mbox 
WARNING: line over 80 characters
#136: FILE: drivers/usb/host/ehci-exynos.c:108:
+       usb = (struct exynos_usb_phy *)fdtdec_get_addr(gd->fdt_blob, node, 
"phyreg");

WARNING: line over 80 characters
#177: FILE: drivers/usb/host/ehci-exynos.c:155:
+       usb = (struct exynos_usb_phy *)fdtdec_get_addr(gd->fdt_blob, node, 
"phyreg");

total: 0 errors, 2 warnings, 85 lines checked

NOTE: Ignored message types: COMPLEX_MACRO CONSIDER_KSTRTO MINMAX 
MULTISTATEMENT_MACRO_USE_DO_WHILE

/tmp/[PATCH 1_3] EHCI_Exynos_Add fdt support.mbox has style problems, please 
review.

If any of these errors are false positives, please report
them to the maintainer, see CHECKPATCH in MAINTAINERS.

Best regards,
Marek Vasut

  reply	other threads:[~2012-12-03 14:45 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-12-03 13:30 [U-Boot] [PATCH 0/3] EXYNOS5: Add FDT support for USB Rajeshwari Shinde
2012-12-03 13:30 ` [U-Boot] [PATCH 1/3] EHCI: Exynos: Add fdt support Rajeshwari Shinde
2012-12-03 14:45   ` Marek Vasut [this message]
2012-12-03 13:30 ` [U-Boot] [PATCH 2/3] EXYNOS5: Add device node for USB Rajeshwari Shinde
2012-12-12 14:38   ` Simon Glass
2012-12-03 13:30 ` [U-Boot] [PATCH 3/3] EXYNOS5: FDT: Add compatible strings " Rajeshwari Shinde
2012-12-04  1:25   ` Simon Glass
2012-12-03 14:46 ` [U-Boot] [PATCH 0/3] EXYNOS5: Add FDT support " Marek Vasut

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=201212031545.22633.marex@denx.de \
    --to=marex@denx.de \
    --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