RE: [PATCHv2 resend] Fix compilation error when CONFIG_CPU_FREQ_S3C24XX_DEBUGFS selected.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Denis Kuzmenko wrote:
> 
> On 12/22/2011 12:04 AM, Kukjin Kim wrote:
> > Denis Kuzmenko wrote:
> >>
> >> On 12/21/2011 09:04 AM, Kukjin Kim wrote:
> >>> Denis Kuzmenko wrote:
> >>>>
> >>>> Fix compilation error when CONFIG_CPU_FREQ_S3C24XX_DEBUGFS selected.
> >>>>
> >>>> Signed-off-by: Denis Kuzmenko <linux@xxxxxxxxxxxxxx>
> >>>> ---
> >>>>
> >>>> Patch is against 3.2-rc4.
> >>>> CONFIG_S3C2410_IOTIMING is not selected for MACH_MINI2440. Because of
> >> this
> >>>> the
> >>>> file s3c2410-iotiming.c is not ever compiled and enabling
> >>>> CONFIG_CPU_FREQ_S3C24XX_DEBUGFS option caused undefined reference to
> >>>> function
> >>>> s3c2410_iotiming_debugfs defined in that file.
> s3c2410_iotiming_debugfs
> >>>> defined
> >>>> as NULL for this case.
> >>>>
> >>> Hi Denis,
> >>>
> >>> Well, there is no error when CONFIG_CPU_FREQ_S3C24XX_DEBUGFS selected
> in
> >> my
> >>> case and s3c2410-iotiming.c is built...when is it happened? I just
> >> selected
> >>> every regarding configuration in kernel menuconfig...
> >>
> >> Please, try attached config (it's still failing in v3.2-rc6).
> >>
> > Hi Denis,
> >
> > Yes, when s3c2410 stuff is not selected, it happened.
> > Look at me soon and thanks.
> Hi, Kukjin,
> 
> Sorry, can you please explain as I can't understand. Especially second
> sentence.
> 
Oops, mis-typing. Let me look at that with your patch :)

Thanks.

Best regards,
Kgene.
--
Kukjin Kim <kgene.kim@xxxxxxxxxxx>, Senior Engineer,
SW Solution Development Team, Samsung Electronics Co., Ltd.

--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux SoC Development]     [Linux Rockchip Development]     [Linux USB Development]     [Video for Linux]     [Linux Audio Users]     [Linux SCSI]     [Yosemite News]

  Powered by Linux