All of lore.kernel.org
 help / color / mirror / Atom feed
From: Tony Lindgren <tony@atomide.com>
To: Kishon Vijay Abraham I <kishon@ti.com>
Cc: t-kristo@ti.com, bcousson@baylibre.com,
	linux-omap@vger.kernel.org, devicetree@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org, linux@arm.linux.org.uk,
	nsekhar@ti.com
Subject: Re: [PATCH 0/4] omap: Fix broken address translation for pbias
Date: Thu, 3 Sep 2015 08:10:06 -0700	[thread overview]
Message-ID: <20150903151005.GU4215@atomide.com> (raw)
In-Reply-To: <1441279673-28105-1-git-send-email-kishon@ti.com>

* Kishon Vijay Abraham I <kishon@ti.com> [150903 04:31]:
> pbias device stopped having memory resource after
> "ARM: dts: <platform>: add minimal l4 bus layout with control module
> support" got merged. This results in platform_get_resource returning
> -EINVAL in pbias driver. This is because address translation fails
> while trying to convert the address to resource which happens
> during the device creation process during boot up.
> 
> Even though after [1], reg property in pbias dt node is un-used,
> it might be an issue if we plan to use it at a later point of time.
> 
> Patch series is created on top of [1].
> 
> [1] -> https://lkml.org/lkml/2015/9/3/51
> 
> Verified the address in omap4 panda, omap5 uevm and dra72 boards.

Your other series is obviously needed for v4.3.. But seems these
are also needed for stable? If so, which versions, v4.2+?

Can you please just do a single patch with multiple Fixes tags
as they fix regression for multiple commits? Doing tons of one
liner patches for the same thing just creates commit log noise..

Remember to not Cc stable when posting, I'll add that tag when
committing if needed.

Regards,

Tony

WARNING: multiple messages have this Message-ID (diff)
From: Tony Lindgren <tony-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org>
To: Kishon Vijay Abraham I <kishon-l0cyMroinI0@public.gmane.org>
Cc: t-kristo-l0cyMroinI0@public.gmane.org,
	bcousson-rdvid1DuHRBWk0Htik3J/w@public.gmane.org,
	linux-omap-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linux-lFZ/pmaqli7XmaaqVzeoHQ@public.gmane.org,
	nsekhar-l0cyMroinI0@public.gmane.org
Subject: Re: [PATCH 0/4] omap: Fix broken address translation for pbias
Date: Thu, 3 Sep 2015 08:10:06 -0700	[thread overview]
Message-ID: <20150903151005.GU4215@atomide.com> (raw)
In-Reply-To: <1441279673-28105-1-git-send-email-kishon-l0cyMroinI0@public.gmane.org>

* Kishon Vijay Abraham I <kishon-l0cyMroinI0@public.gmane.org> [150903 04:31]:
> pbias device stopped having memory resource after
> "ARM: dts: <platform>: add minimal l4 bus layout with control module
> support" got merged. This results in platform_get_resource returning
> -EINVAL in pbias driver. This is because address translation fails
> while trying to convert the address to resource which happens
> during the device creation process during boot up.
> 
> Even though after [1], reg property in pbias dt node is un-used,
> it might be an issue if we plan to use it at a later point of time.
> 
> Patch series is created on top of [1].
> 
> [1] -> https://lkml.org/lkml/2015/9/3/51
> 
> Verified the address in omap4 panda, omap5 uevm and dra72 boards.

Your other series is obviously needed for v4.3.. But seems these
are also needed for stable? If so, which versions, v4.2+?

Can you please just do a single patch with multiple Fixes tags
as they fix regression for multiple commits? Doing tons of one
liner patches for the same thing just creates commit log noise..

Remember to not Cc stable when posting, I'll add that tag when
committing if needed.

Regards,

Tony
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

WARNING: multiple messages have this Message-ID (diff)
From: tony@atomide.com (Tony Lindgren)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 0/4] omap: Fix broken address translation for pbias
Date: Thu, 3 Sep 2015 08:10:06 -0700	[thread overview]
Message-ID: <20150903151005.GU4215@atomide.com> (raw)
In-Reply-To: <1441279673-28105-1-git-send-email-kishon@ti.com>

* Kishon Vijay Abraham I <kishon@ti.com> [150903 04:31]:
> pbias device stopped having memory resource after
> "ARM: dts: <platform>: add minimal l4 bus layout with control module
> support" got merged. This results in platform_get_resource returning
> -EINVAL in pbias driver. This is because address translation fails
> while trying to convert the address to resource which happens
> during the device creation process during boot up.
> 
> Even though after [1], reg property in pbias dt node is un-used,
> it might be an issue if we plan to use it at a later point of time.
> 
> Patch series is created on top of [1].
> 
> [1] -> https://lkml.org/lkml/2015/9/3/51
> 
> Verified the address in omap4 panda, omap5 uevm and dra72 boards.

Your other series is obviously needed for v4.3.. But seems these
are also needed for stable? If so, which versions, v4.2+?

Can you please just do a single patch with multiple Fixes tags
as they fix regression for multiple commits? Doing tons of one
liner patches for the same thing just creates commit log noise..

Remember to not Cc stable when posting, I'll add that tag when
committing if needed.

Regards,

Tony

  parent reply	other threads:[~2015-09-03 15:10 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-09-03 11:27 [PATCH 0/4] omap: Fix broken address translation for pbias Kishon Vijay Abraham I
2015-09-03 11:27 ` Kishon Vijay Abraham I
2015-09-03 11:27 ` Kishon Vijay Abraham I
2015-09-03 11:27 ` [PATCH 1/4] ARM: dts: omap24xx: fix " Kishon Vijay Abraham I
2015-09-03 11:27   ` Kishon Vijay Abraham I
2015-09-03 11:27   ` Kishon Vijay Abraham I
2015-09-03 11:27 ` [PATCH 2/4] ARM: dts: omap4: " Kishon Vijay Abraham I
2015-09-03 11:27   ` Kishon Vijay Abraham I
2015-09-03 11:27   ` Kishon Vijay Abraham I
2015-09-03 11:27 ` [PATCH 3/4] ARM: dts: omap5: " Kishon Vijay Abraham I
2015-09-03 11:27   ` Kishon Vijay Abraham I
2015-09-03 11:27   ` Kishon Vijay Abraham I
2015-09-03 11:27 ` [PATCH 4/4] ARM: dts: dra7: " Kishon Vijay Abraham I
2015-09-03 11:27   ` Kishon Vijay Abraham I
2015-09-03 11:27   ` Kishon Vijay Abraham I
2015-09-03 15:10 ` Tony Lindgren [this message]
2015-09-03 15:10   ` [PATCH 0/4] omap: Fix broken " Tony Lindgren
2015-09-03 15:10   ` Tony Lindgren
2015-09-04 11:03   ` Kishon Vijay Abraham I
2015-09-04 11:03     ` Kishon Vijay Abraham I
2015-09-04 11:03     ` 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=20150903151005.GU4215@atomide.com \
    --to=tony@atomide.com \
    --cc=bcousson@baylibre.com \
    --cc=devicetree@vger.kernel.org \
    --cc=kishon@ti.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-omap@vger.kernel.org \
    --cc=linux@arm.linux.org.uk \
    --cc=nsekhar@ti.com \
    --cc=t-kristo@ti.com \
    /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.