Message ID | 20200624150847.22672-1-digetx@gmail.com (mailing list archive) |
---|---|
Headers | show |
Series | Tegra Video Decoder driver power management corrections | expand |
On 24/06/2020 17:08, Dmitry Osipenko wrote: > Hello, > > This small series addresses a Runtime PM issue that was discovered during > of Tegra VI driver reviewing by balancing RPM usage count on RPM resume > failure. Secondly it fixes reboot on some Tegra devices due to bootloader > expecting VDE power partition to be ON at the boot time, which wasn't > happening in case of a warm re-booting (i.e. by PMC resetting). Can you rebase this on top of the media_tree master branch? I think a variant of patch 1 has already been applied. I found a mail today where you mentioned that you preferred your version (it looks like I missed that) so you'll need to rework patch 1. Sorry about this, Hans > > Changelog: > > v2: - Extended the commit's message of the "Balance runtime PM use-count on > resume failure" patch. > > - Re-send for 5.9 inclusion. > > Dmitry Osipenko (4): > media: staging: tegra-vde: Balance runtime PM use-count on resume > failure > media: staging: tegra-vde: Runtime PM is always available on Tegra > media: staging: tegra-vde: Turn ON power domain on shutdown > media: staging: tegra-vde: Power-cycle hardware on probe > > drivers/staging/media/tegra-vde/vde.c | 45 +++++++++++++++++---------- > 1 file changed, 29 insertions(+), 16 deletions(-) >
24.06.2020 18:16, Hans Verkuil пишет: > On 24/06/2020 17:08, Dmitry Osipenko wrote: >> Hello, >> >> This small series addresses a Runtime PM issue that was discovered during >> of Tegra VI driver reviewing by balancing RPM usage count on RPM resume >> failure. Secondly it fixes reboot on some Tegra devices due to bootloader >> expecting VDE power partition to be ON at the boot time, which wasn't >> happening in case of a warm re-booting (i.e. by PMC resetting). > > Can you rebase this on top of the media_tree master branch? I think a variant > of patch 1 has already been applied. I found a mail today where you mentioned > that you preferred your version (it looks like I missed that) so you'll need to > rework patch 1. Hello Hans, I'll take a look at what patches has been applied, my bad for sending the v2 too late. Thank you for the heads up!
24.06.2020 18:23, Dmitry Osipenko пишет: > 24.06.2020 18:16, Hans Verkuil пишет: >> On 24/06/2020 17:08, Dmitry Osipenko wrote: >>> Hello, >>> >>> This small series addresses a Runtime PM issue that was discovered during >>> of Tegra VI driver reviewing by balancing RPM usage count on RPM resume >>> failure. Secondly it fixes reboot on some Tegra devices due to bootloader >>> expecting VDE power partition to be ON at the boot time, which wasn't >>> happening in case of a warm re-booting (i.e. by PMC resetting). >> >> Can you rebase this on top of the media_tree master branch? I think a variant >> of patch 1 has already been applied. I found a mail today where you mentioned >> that you preferred your version (it looks like I missed that) so you'll need to >> rework patch 1. > > Hello Hans, > > I'll take a look at what patches has been applied, my bad for sending > the v2 too late. Thank you for the heads up! > I tested the already-applied variant of the patch 1 and it has the same behaviour as my variant, so it's okay. Would you want me to send a v3 without the conflicting patch 1 or you could apply the patches 2-4 from this series?
On 26/06/2020 02:17, Dmitry Osipenko wrote: > 24.06.2020 18:23, Dmitry Osipenko пишет: >> 24.06.2020 18:16, Hans Verkuil пишет: >>> On 24/06/2020 17:08, Dmitry Osipenko wrote: >>>> Hello, >>>> >>>> This small series addresses a Runtime PM issue that was discovered during >>>> of Tegra VI driver reviewing by balancing RPM usage count on RPM resume >>>> failure. Secondly it fixes reboot on some Tegra devices due to bootloader >>>> expecting VDE power partition to be ON at the boot time, which wasn't >>>> happening in case of a warm re-booting (i.e. by PMC resetting). >>> >>> Can you rebase this on top of the media_tree master branch? I think a variant >>> of patch 1 has already been applied. I found a mail today where you mentioned >>> that you preferred your version (it looks like I missed that) so you'll need to >>> rework patch 1. >> >> Hello Hans, >> >> I'll take a look at what patches has been applied, my bad for sending >> the v2 too late. Thank you for the heads up! >> > > I tested the already-applied variant of the patch 1 and it has the same > behaviour as my variant, so it's okay. > > Would you want me to send a v3 without the conflicting patch 1 or you > could apply the patches 2-4 from this series? > I'll mark 1/4 as superseded and will apply patches 2-4. No need for you to do anything. Regards, Hans
26.06.2020 09:58, Hans Verkuil пишет: > On 26/06/2020 02:17, Dmitry Osipenko wrote: >> 24.06.2020 18:23, Dmitry Osipenko пишет: >>> 24.06.2020 18:16, Hans Verkuil пишет: >>>> On 24/06/2020 17:08, Dmitry Osipenko wrote: >>>>> Hello, >>>>> >>>>> This small series addresses a Runtime PM issue that was discovered during >>>>> of Tegra VI driver reviewing by balancing RPM usage count on RPM resume >>>>> failure. Secondly it fixes reboot on some Tegra devices due to bootloader >>>>> expecting VDE power partition to be ON at the boot time, which wasn't >>>>> happening in case of a warm re-booting (i.e. by PMC resetting). >>>> >>>> Can you rebase this on top of the media_tree master branch? I think a variant >>>> of patch 1 has already been applied. I found a mail today where you mentioned >>>> that you preferred your version (it looks like I missed that) so you'll need to >>>> rework patch 1. >>> >>> Hello Hans, >>> >>> I'll take a look at what patches has been applied, my bad for sending >>> the v2 too late. Thank you for the heads up! >>> >> >> I tested the already-applied variant of the patch 1 and it has the same >> behaviour as my variant, so it's okay. >> >> Would you want me to send a v3 without the conflicting patch 1 or you >> could apply the patches 2-4 from this series? >> > > I'll mark 1/4 as superseded and will apply patches 2-4. No need for you > to do anything. Thank you!