| Message ID | 1425561211-31003-3-git-send-email-d-gerlach@ti.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On Thu, 5 Mar 2015, Dave Gerlach wrote: > RTC hwmod is needed for proper operation of PM features like > rtcwake and rtc-only mode so reuse the am33xx rtc hwmod. > > Signed-off-by: Dave Gerlach <d-gerlach@ti.com> > --- > arch/arm/mach-omap2/omap_hwmod_43xx_data.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/arch/arm/mach-omap2/omap_hwmod_43xx_data.c b/arch/arm/mach-omap2/omap_hwmod_43xx_data.c > index 8eb8592..9070535 100644 > --- a/arch/arm/mach-omap2/omap_hwmod_43xx_data.c > +++ b/arch/arm/mach-omap2/omap_hwmod_43xx_data.c > @@ -889,6 +889,7 @@ static struct omap_hwmod_ocp_if *am43xx_hwmod_ocp_ifs[] __initdata = { > &am43xx_l4_ls__dss, > &am43xx_l4_ls__dss_dispc, > &am43xx_l4_ls__dss_rfbi, > + &am33xx_l4_wkup__rtc, > NULL, > }; Thanks, queued for v4.1. - Paul
Paul, On 03/05/2015 10:26 PM, Paul Walmsley wrote: > On Thu, 5 Mar 2015, Dave Gerlach wrote: > >> RTC hwmod is needed for proper operation of PM features like >> rtcwake and rtc-only mode so reuse the am33xx rtc hwmod. >> >> Signed-off-by: Dave Gerlach <d-gerlach@ti.com> >> --- >> arch/arm/mach-omap2/omap_hwmod_43xx_data.c | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/arch/arm/mach-omap2/omap_hwmod_43xx_data.c b/arch/arm/mach-omap2/omap_hwmod_43xx_data.c >> index 8eb8592..9070535 100644 >> --- a/arch/arm/mach-omap2/omap_hwmod_43xx_data.c >> +++ b/arch/arm/mach-omap2/omap_hwmod_43xx_data.c >> @@ -889,6 +889,7 @@ static struct omap_hwmod_ocp_if *am43xx_hwmod_ocp_ifs[] __initdata = { >> &am43xx_l4_ls__dss, >> &am43xx_l4_ls__dss_dispc, >> &am43xx_l4_ls__dss_rfbi, >> + &am33xx_l4_wkup__rtc, >> NULL, >> }; > > Thanks, queued for v4.1. Thanks, but please note as I just commented in Patch 1 of this series, without the ti,no-init flag in place that is introduced there this patch will cause the am43x-epos-evm to fail to boot. Regards, Dave > > > - Paul >
Hi Dave, On Fri, 6 Mar 2015, Dave Gerlach wrote: > Paul, > On 03/05/2015 10:26 PM, Paul Walmsley wrote: > > On Thu, 5 Mar 2015, Dave Gerlach wrote: > > > >> RTC hwmod is needed for proper operation of PM features like > >> rtcwake and rtc-only mode so reuse the am33xx rtc hwmod. > >> > >> Signed-off-by: Dave Gerlach <d-gerlach@ti.com> > >> --- > >> arch/arm/mach-omap2/omap_hwmod_43xx_data.c | 1 + > >> 1 file changed, 1 insertion(+) > >> > >> diff --git a/arch/arm/mach-omap2/omap_hwmod_43xx_data.c b/arch/arm/mach-omap2/omap_hwmod_43xx_data.c > >> index 8eb8592..9070535 100644 > >> --- a/arch/arm/mach-omap2/omap_hwmod_43xx_data.c > >> +++ b/arch/arm/mach-omap2/omap_hwmod_43xx_data.c > >> @@ -889,6 +889,7 @@ static struct omap_hwmod_ocp_if *am43xx_hwmod_ocp_ifs[] __initdata = { > >> &am43xx_l4_ls__dss, > >> &am43xx_l4_ls__dss_dispc, > >> &am43xx_l4_ls__dss_rfbi, > >> + &am33xx_l4_wkup__rtc, > >> NULL, > >> }; > > > > Thanks, queued for v4.1. > > Thanks, but please note as I just commented in Patch 1 of this series, without > the ti,no-init flag in place that is introduced there this patch will cause the > am43x-epos-evm to fail to boot. If that's so, shouldn't it appear in the series after patch 3, then? If only patches 1 and 2 are applied, then won't the boot be broken on am43x-epos-evm ? - Paul
Paul, On 03/06/2015 11:44 AM, Paul Walmsley wrote: > Hi Dave, > > On Fri, 6 Mar 2015, Dave Gerlach wrote: > >> Paul, >> On 03/05/2015 10:26 PM, Paul Walmsley wrote: >>> On Thu, 5 Mar 2015, Dave Gerlach wrote: >>> >>>> RTC hwmod is needed for proper operation of PM features like >>>> rtcwake and rtc-only mode so reuse the am33xx rtc hwmod. >>>> >>>> Signed-off-by: Dave Gerlach <d-gerlach@ti.com> >>>> --- >>>> arch/arm/mach-omap2/omap_hwmod_43xx_data.c | 1 + >>>> 1 file changed, 1 insertion(+) >>>> >>>> diff --git a/arch/arm/mach-omap2/omap_hwmod_43xx_data.c b/arch/arm/mach-omap2/omap_hwmod_43xx_data.c >>>> index 8eb8592..9070535 100644 >>>> --- a/arch/arm/mach-omap2/omap_hwmod_43xx_data.c >>>> +++ b/arch/arm/mach-omap2/omap_hwmod_43xx_data.c >>>> @@ -889,6 +889,7 @@ static struct omap_hwmod_ocp_if *am43xx_hwmod_ocp_ifs[] __initdata = { >>>> &am43xx_l4_ls__dss, >>>> &am43xx_l4_ls__dss_dispc, >>>> &am43xx_l4_ls__dss_rfbi, >>>> + &am33xx_l4_wkup__rtc, >>>> NULL, >>>> }; >>> >>> Thanks, queued for v4.1. >> >> Thanks, but please note as I just commented in Patch 1 of this series, without >> the ti,no-init flag in place that is introduced there this patch will cause the >> am43x-epos-evm to fail to boot. > > If that's so, shouldn't it appear in the series after patch 3, then? > If only patches 1 and 2 are applied, then won't the boot be broken on > am43x-epos-evm ? Hmm yes you are correct that would be the case, seems I should have swapped the order. I've gotten into the habit of putting dt patches last to enable what gets introduced previously, guess it's not always the best thing to do. Thanks for pointing this out. Regards, Dave > > - Paul >
On Fri, 6 Mar 2015, Dave Gerlach wrote: > On 03/06/2015 11:44 AM, Paul Walmsley wrote: > > On Fri, 6 Mar 2015, Dave Gerlach wrote: > >> On 03/05/2015 10:26 PM, Paul Walmsley wrote: > >>> On Thu, 5 Mar 2015, Dave Gerlach wrote: > >>> > >>>> RTC hwmod is needed for proper operation of PM features like > >>>> rtcwake and rtc-only mode so reuse the am33xx rtc hwmod. > >>>> > >>>> Signed-off-by: Dave Gerlach <d-gerlach@ti.com> > >>> > >>> Thanks, queued for v4.1. > >> > >> Thanks, but please note as I just commented in Patch 1 of this series, without > >> the ti,no-init flag in place that is introduced there this patch will cause the > >> am43x-epos-evm to fail to boot. > > > > If that's so, shouldn't it appear in the series after patch 3, then? > > If only patches 1 and 2 are applied, then won't the boot be broken on > > am43x-epos-evm ? > > Hmm yes you are correct that would be the case, seems I should have swapped the > order. I've gotten into the habit of putting dt patches last to enable what gets > introduced previously, guess it's not always the best thing to do. Thanks for > pointing this out. OK dropped until this thing is sorted out. - Paul
diff --git a/arch/arm/mach-omap2/omap_hwmod_43xx_data.c b/arch/arm/mach-omap2/omap_hwmod_43xx_data.c index 8eb8592..9070535 100644 --- a/arch/arm/mach-omap2/omap_hwmod_43xx_data.c +++ b/arch/arm/mach-omap2/omap_hwmod_43xx_data.c @@ -889,6 +889,7 @@ static struct omap_hwmod_ocp_if *am43xx_hwmod_ocp_ifs[] __initdata = { &am43xx_l4_ls__dss, &am43xx_l4_ls__dss_dispc, &am43xx_l4_ls__dss_rfbi, + &am33xx_l4_wkup__rtc, NULL, };
RTC hwmod is needed for proper operation of PM features like rtcwake and rtc-only mode so reuse the am33xx rtc hwmod. Signed-off-by: Dave Gerlach <d-gerlach@ti.com> --- arch/arm/mach-omap2/omap_hwmod_43xx_data.c | 1 + 1 file changed, 1 insertion(+)