Re: [PATCH v5 7/7] arch/x86: Comment out efi_set_rtc_mmss()
On Fri, 13 Jun, at 07:00:23PM, Daniel Kiper wrote: > efi_set_rtc_mmss() is never used to set RTC due to bugs found > on many EFI platforms. It is set directly by mach_set_rtc_mmss(). > > Signed-off-by: Daniel Kiper > --- > arch/x86/platform/efi/efi.c |6 ++ > 1 file changed, 6 insertions(+) > > diff --git a/arch/x86/platform/efi/efi.c b/arch/x86/platform/efi/efi.c > index deb4f05..bd3e080 100644 > --- a/arch/x86/platform/efi/efi.c > +++ b/arch/x86/platform/efi/efi.c > @@ -244,6 +244,11 @@ static efi_status_t __init > phys_efi_set_virtual_address_map( > return status; > } > > +#if 0 > +/* > + * efi_set_rtc_mmss() is never used to set RTC due to bugs found > + * on many EFI platforms. It is set directly by mach_set_rtc_mmss(). > + */ > int efi_set_rtc_mmss(const struct timespec *now) > { > unsigned long nowtime = now->tv_sec; > @@ -279,6 +284,7 @@ int efi_set_rtc_mmss(const struct timespec *now) > } > return 0; > } > +#endif > > void efi_get_time(struct timespec *now) > { As others have said in this thread - just delete this sucker. I did make an attempt to get rid of all the time functions recently, but chickened out at the last minute because we were nearing the merge window, http://article.gmane.org/gmane.linux.kernel.janitors/30082 So yes, please go ahead and delete this function. -- Matt Fleming, Intel Open Source Technology Center -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH v5 7/7] arch/x86: Comment out efi_set_rtc_mmss()
On Fri, 13 Jun, at 07:00:23PM, Daniel Kiper wrote: efi_set_rtc_mmss() is never used to set RTC due to bugs found on many EFI platforms. It is set directly by mach_set_rtc_mmss(). Signed-off-by: Daniel Kiper daniel.ki...@oracle.com --- arch/x86/platform/efi/efi.c |6 ++ 1 file changed, 6 insertions(+) diff --git a/arch/x86/platform/efi/efi.c b/arch/x86/platform/efi/efi.c index deb4f05..bd3e080 100644 --- a/arch/x86/platform/efi/efi.c +++ b/arch/x86/platform/efi/efi.c @@ -244,6 +244,11 @@ static efi_status_t __init phys_efi_set_virtual_address_map( return status; } +#if 0 +/* + * efi_set_rtc_mmss() is never used to set RTC due to bugs found + * on many EFI platforms. It is set directly by mach_set_rtc_mmss(). + */ int efi_set_rtc_mmss(const struct timespec *now) { unsigned long nowtime = now-tv_sec; @@ -279,6 +284,7 @@ int efi_set_rtc_mmss(const struct timespec *now) } return 0; } +#endif void efi_get_time(struct timespec *now) { As others have said in this thread - just delete this sucker. I did make an attempt to get rid of all the time functions recently, but chickened out at the last minute because we were nearing the merge window, http://article.gmane.org/gmane.linux.kernel.janitors/30082 So yes, please go ahead and delete this function. -- Matt Fleming, Intel Open Source Technology Center -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH v5 7/7] arch/x86: Comment out efi_set_rtc_mmss()
On Fri, 13 Jun 2014, Daniel Kiper wrote: > efi_set_rtc_mmss() is never used to set RTC due to bugs found > on many EFI platforms. It is set directly by mach_set_rtc_mmss(). > > Signed-off-by: Daniel Kiper > --- > arch/x86/platform/efi/efi.c |6 ++ > 1 file changed, 6 insertions(+) > > diff --git a/arch/x86/platform/efi/efi.c b/arch/x86/platform/efi/efi.c > index deb4f05..bd3e080 100644 > --- a/arch/x86/platform/efi/efi.c > +++ b/arch/x86/platform/efi/efi.c > @@ -244,6 +244,11 @@ static efi_status_t __init > phys_efi_set_virtual_address_map( > return status; > } > > +#if 0 > +/* > + * efi_set_rtc_mmss() is never used to set RTC due to bugs found > + * on many EFI platforms. It is set directly by mach_set_rtc_mmss(). > + */ > int efi_set_rtc_mmss(const struct timespec *now) > { > unsigned long nowtime = now->tv_sec; > @@ -279,6 +284,7 @@ int efi_set_rtc_mmss(const struct timespec *now) > } > return 0; > } > +#endif "Commenting out" code like that is not a good idea: it leaves dead-rotting functions in the middle of your source file. If the function is not used, just go ahead and remove it. We can use git to get it back. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [PATCH v5 7/7] arch/x86: Comment out efi_set_rtc_mmss()
On Fri, 13 Jun 2014, Daniel Kiper wrote: efi_set_rtc_mmss() is never used to set RTC due to bugs found on many EFI platforms. It is set directly by mach_set_rtc_mmss(). Signed-off-by: Daniel Kiper daniel.ki...@oracle.com --- arch/x86/platform/efi/efi.c |6 ++ 1 file changed, 6 insertions(+) diff --git a/arch/x86/platform/efi/efi.c b/arch/x86/platform/efi/efi.c index deb4f05..bd3e080 100644 --- a/arch/x86/platform/efi/efi.c +++ b/arch/x86/platform/efi/efi.c @@ -244,6 +244,11 @@ static efi_status_t __init phys_efi_set_virtual_address_map( return status; } +#if 0 +/* + * efi_set_rtc_mmss() is never used to set RTC due to bugs found + * on many EFI platforms. It is set directly by mach_set_rtc_mmss(). + */ int efi_set_rtc_mmss(const struct timespec *now) { unsigned long nowtime = now-tv_sec; @@ -279,6 +284,7 @@ int efi_set_rtc_mmss(const struct timespec *now) } return 0; } +#endif Commenting out code like that is not a good idea: it leaves dead-rotting functions in the middle of your source file. If the function is not used, just go ahead and remove it. We can use git to get it back. -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/