All of lore.kernel.org
 help / color / mirror / Atom feed
From: Denys Dmytriyenko <denys@ti.com>
To: Karthik Ramanan <a0393906@ti.com>
Cc: meta-arago@arago-project.org
Subject: Re: [for master, v2 1/3] tisdk-connectivity: Add support for dra7xx-evm
Date: Mon, 15 Dec 2014 15:35:25 -0500	[thread overview]
Message-ID: <20141215203524.GF28600@edge> (raw)
In-Reply-To: <1416833465-19027-1-git-send-email-a0393906@ti.com>

Karthik,

It would really help and speed review process up, if you provide a short 
change log of patch revisions. I was postponing reviewing this set until I had 
enough time to go back and compare it with previous revisions and see what has 
changed and if all the previous comments were addressed...

-- 
Denys


On Mon, Nov 24, 2014 at 06:21:03PM +0530, Karthik Ramanan wrote:
> Signed-off-by: Karthik Ramanan <a0393906@ti.com>
> ---
>  .../packagegroup-arago-tisdk-connectivity.bb       |    9 ++++++++-
>  1 file changed, 8 insertions(+), 1 deletion(-)
> 
> diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-connectivity.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-connectivity.bb
> index f6c8660..f469fdd 100644
> --- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-connectivity.bb
> +++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-connectivity.bb
> @@ -1,6 +1,6 @@
>  DESCRIPTION = "Task to install wireless packages into the target FS"
>  LICENSE = "MIT"
> -PR = "r29"
> +PR = "r30"
>  
>  inherit packagegroup
>  
> @@ -63,6 +63,13 @@ CONNECTIVITY_RDEPENDS_ti43x = "\
>      ${FIRMWARE_AND_DRIVERS} \
>  "
>  
> +CONNECTIVITY_RDEPENDS_dra7xx-evm = "\
> +    ${WLAN_COMMON} \
> +    ${BLUETOOTH_STACK} \
> +    ${@base_contains('ARAGO_BRAND','glsdk','${FIRMWARE_AND_DRIVERS}','',d)} \
> +    bt-firmware \
> +"
> +
>  RDEPENDS_${PN} = "\
>      ${CONNECTIVITY_RDEPENDS} \
>  "
> -- 
> 1.7.9.5
> 
> _______________________________________________
> meta-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago


      parent reply	other threads:[~2014-12-15 20:35 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-11-24 12:51 [for master, v2 1/3] tisdk-connectivity: Add support for dra7xx-evm Karthik Ramanan
2014-11-24 12:51 ` [for master, v2 2/3] bt-firmware: Add support for dra7xx-evm target Karthik Ramanan
2014-11-24 12:51 ` [for master, v2 3/3] bt-firmware: Override Makefile to handle UNKNOWN platform Karthik Ramanan
2014-12-15 20:35 ` Denys Dmytriyenko [this message]

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=20141215203524.GF28600@edge \
    --to=denys@ti.com \
    --cc=a0393906@ti.com \
    --cc=meta-arago@arago-project.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.