[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAJ6Av5PkPa_20xUYjd1h71ajZOoDxjw2sGKUrfEjb4SuoFc4bg@mail.gmail.com>
Date: Wed, 26 Dec 2012 00:28:40 -0600
From: Omar Ramirez Luna <omar.ramirez@...itl.com>
To: Chen Gang <gang.chen@...anux.com>
Cc: Greg KH <gregkh@...uxfoundation.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
eballetbo@...ebcn.com
Subject: Re: [Suggestion] drivers/staging/tidspbridge: pr_err and pr_debug for
uninitialized buffer (name buf not initialized).
On Tue, Dec 25, 2012 at 7:56 PM, Chen Gang <gang.chen@...anux.com> wrote:
> 于 2012年12月24日 22:26, Omar Ramirez Luna 写道:
>>> b: version merging issue:
>>> > in drivers/staging/tidspbridge/core/_tiomap.h
>>> > need use "#include <mach-omap2/cm3xxx.h>" instead of "#include <mach-omap2/cm2xxx_3xxx.h>"
>>> > the macro OMAP3430_CM_AUTOIDLE_PLL has already move from cm2xxx_3xxx.h to cm3xxx.h.
>>> > (it seems arch/arm/mach-omap2/ is not a suitable place for including, but we have to)
>>> > if not change, compiling will be failed.
>
> also please help checking this issue, thanks.
I also sent a patch for this:
http://www.mail-archive.com/linux-omap@vger.kernel.org/msg82565.html
However, I just saw this one which was sent 6 days ago:
https://patchwork.kernel.org/patch/1895081/
The latter includes the new header file, mine just the definitions
needed, any approach is fine by me as they both fix the compile break.
Cheers,
Omar
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists