All of lore.kernel.org
 help / color / mirror / Atom feed
From: kishon@ti.com (Kishon Vijay Abraham I)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v3 0/4] ARM: sunxi: Add driver for sunxi usb phy + usb dts bindings
Date: Mon, 24 Feb 2014 15:04:57 +0530	[thread overview]
Message-ID: <530B1241.1000202@ti.com> (raw)
In-Reply-To: <1393157352-21104-1-git-send-email-hdegoede@redhat.com>

Hi,

On Sunday 23 February 2014 05:39 PM, Hans de Goede wrote:
> Hi Kishon, Maxime,
> 
> Here is v3 of my sunxi-usb-phy driver it addresses all review remarks made
> in response to v2, and as such this should be the final version, changes:
> 
> -Fix check for wrong variable in error handling path pointed out by wens
> -Switch to using reg-names to differentiate between the different register
>  ranges needed
> 
> Kishon, can you please queue up the 1st patch of this series for 3.15 ?

huh.. maybe I failed to notice that before but the $subject for that patch
should be 'PHY: sunxi...' and not 'ARM: sunxi..'. Can you fix that up?

Thanks
Kishon

> 
> Maxime, can you please add patches 2-4 to your dt-for-3.15 branch? Note these
> are only the dtsi bits.
> 
> I'm holding back all the board .dts patches until it is clear how we want to
> deal with the ahci + usb regulator bits. Specifically I'm waiting for an answer
> to this question:
> 
> "I hope this helps explain my reasoning, as said I'm fine with
> either way, if you want to change over to a single file +
> explicit enabling, let me know and I'll respin the ahci dts
> patches.  Note I'm going on vacation for a week starting Monday,
> so you likely won't get a new version until next weekend."
> 
> In my last reply to "[PATCH v6 17/18] ARM: sun4i: dt: Add ahci / sata support"
> 
> I've a feeling you (Maxime) want me to rework things in to a single dtsi
> for all common regulators + explicit enabling, which works for me, but before
> doing the necessary refactoring I would like to hear that this is what you
> want from you.
> 
> Regards,
> 
> Hans
> 
> 
> p.s.
> 
> I'm going on vacation for 5 days leaving tomorrow, so I won't send the
> (refactored) board .dts file patches until next weekend.
> 
> 

WARNING: multiple messages have this Message-ID (diff)
From: Kishon Vijay Abraham I <kishon-l0cyMroinI0@public.gmane.org>
To: Hans de Goede <hdegoede-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>,
	Maxime Ripard
	<maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>
Cc: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
	devicetree <devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
	linux-sunxi-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org
Subject: Re: [PATCH v3 0/4] ARM: sunxi: Add driver for sunxi usb phy + usb dts bindings
Date: Mon, 24 Feb 2014 15:04:57 +0530	[thread overview]
Message-ID: <530B1241.1000202@ti.com> (raw)
In-Reply-To: <1393157352-21104-1-git-send-email-hdegoede-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>

Hi,

On Sunday 23 February 2014 05:39 PM, Hans de Goede wrote:
> Hi Kishon, Maxime,
> 
> Here is v3 of my sunxi-usb-phy driver it addresses all review remarks made
> in response to v2, and as such this should be the final version, changes:
> 
> -Fix check for wrong variable in error handling path pointed out by wens
> -Switch to using reg-names to differentiate between the different register
>  ranges needed
> 
> Kishon, can you please queue up the 1st patch of this series for 3.15 ?

huh.. maybe I failed to notice that before but the $subject for that patch
should be 'PHY: sunxi...' and not 'ARM: sunxi..'. Can you fix that up?

Thanks
Kishon

> 
> Maxime, can you please add patches 2-4 to your dt-for-3.15 branch? Note these
> are only the dtsi bits.
> 
> I'm holding back all the board .dts patches until it is clear how we want to
> deal with the ahci + usb regulator bits. Specifically I'm waiting for an answer
> to this question:
> 
> "I hope this helps explain my reasoning, as said I'm fine with
> either way, if you want to change over to a single file +
> explicit enabling, let me know and I'll respin the ahci dts
> patches.  Note I'm going on vacation for a week starting Monday,
> so you likely won't get a new version until next weekend."
> 
> In my last reply to "[PATCH v6 17/18] ARM: sun4i: dt: Add ahci / sata support"
> 
> I've a feeling you (Maxime) want me to rework things in to a single dtsi
> for all common regulators + explicit enabling, which works for me, but before
> doing the necessary refactoring I would like to hear that this is what you
> want from you.
> 
> Regards,
> 
> Hans
> 
> 
> p.s.
> 
> I'm going on vacation for 5 days leaving tomorrow, so I won't send the
> (refactored) board .dts file patches until next weekend.
> 
> 

  parent reply	other threads:[~2014-02-24  9:34 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-02-23 12:09 [PATCH v3 0/4] ARM: sunxi: Add driver for sunxi usb phy + usb dts bindings Hans de Goede
2014-02-23 12:09 ` Hans de Goede
2014-02-23 12:09 ` [PATCH v3 1/4] ARM: sunxi: Add driver for sunxi usb phy Hans de Goede
2014-02-23 12:09   ` Hans de Goede
2014-02-24  9:27   ` Maxime Ripard
2014-02-24  9:27     ` Maxime Ripard
2014-02-24  9:39   ` Kishon Vijay Abraham I
2014-02-24  9:39     ` Kishon Vijay Abraham I
2014-03-01 13:14     ` [linux-sunxi] " Hans de Goede
2014-03-01 13:14       ` Hans de Goede
2014-02-26 15:12   ` [linux-sunxi] " Chen-Yu Tsai
2014-02-26 15:12     ` Chen-Yu Tsai
2014-03-01 13:17     ` [linux-sunxi] " Hans de Goede
2014-03-01 13:17       ` Hans de Goede
2014-02-23 12:09 ` [PATCH v3 2/4] ARM: sun4i: dt: Add USB host bindings Hans de Goede
2014-02-23 12:09   ` Hans de Goede
2014-02-23 12:09 ` [PATCH v3 3/4] ARM: sun5i: " Hans de Goede
2014-02-23 12:09   ` Hans de Goede
2014-02-23 12:09 ` [PATCH v3 4/4] ARM: sun7i: " Hans de Goede
2014-02-23 12:09   ` Hans de Goede
2014-02-24  9:34 ` Kishon Vijay Abraham I [this message]
2014-02-24  9:34   ` [PATCH v3 0/4] ARM: sunxi: Add driver for sunxi usb phy + usb dts bindings Kishon Vijay Abraham I

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=530B1241.1000202@ti.com \
    --to=kishon@ti.com \
    --cc=linux-arm-kernel@lists.infradead.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.