From: Santosh Shilimkar <santosh.shilimkar@ti.com>
To: Russell King - ARM Linux <linux@arm.linux.org.uk>
Cc: tony@atomide.com, catalin.marinas@arm.com,
linux-omap@vger.kernel.org, linux-arm-kernel@lists.infradead.org
Subject: RE: [PATCH 1/2] ARM: l2x0: Errata fix for flush by Way operationcan cause data corruption
Date: Mon, 28 Feb 2011 18:17:07 +0530 [thread overview]
Message-ID: <69dd173e3c7b4d02ef50ceaa19b2343a@mail.gmail.com> (raw)
In-Reply-To: <20110228101351.GA30781@n2100.arm.linux.org.uk>
> -----Original Message-----
> From: Russell King - ARM Linux [mailto:linux@arm.linux.org.uk]
> Sent: Monday, February 28, 2011 3:44 PM
> To: Santosh Shilimkar
> Cc: tony@atomide.com; catalin.marinas@arm.com; linux-
> omap@vger.kernel.org; linux-arm-kernel@lists.infradead.org
> Subject: Re: [PATCH 1/2] ARM: l2x0: Errata fix for flush by Way
> operationcan cause data corruption
>
> On Sun, Feb 27, 2011 at 12:00:21PM +0000, Russell King - ARM Linux
> wrote:
> > > +#else
> > > +/* Optimised out for non-errata case */
> > > +static inline void debug_writel(unsigned long val)
> > > +{
> > > }
> >
> > #define l2x0_set_debug NULL
> >
> > > +#endif
>
> I notice you got rid of the inline function. Have you tried
> building this without the errata enabled?
I accidently dropped the inline function while
incorporating the comment from you. :(
Fixed it. Updated version # 6770/1
Regards,
Santosh
WARNING: multiple messages have this Message-ID (diff)
From: santosh.shilimkar@ti.com (Santosh Shilimkar)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 1/2] ARM: l2x0: Errata fix for flush by Way operationcan cause data corruption
Date: Mon, 28 Feb 2011 18:17:07 +0530 [thread overview]
Message-ID: <69dd173e3c7b4d02ef50ceaa19b2343a@mail.gmail.com> (raw)
In-Reply-To: <20110228101351.GA30781@n2100.arm.linux.org.uk>
> -----Original Message-----
> From: Russell King - ARM Linux [mailto:linux at arm.linux.org.uk]
> Sent: Monday, February 28, 2011 3:44 PM
> To: Santosh Shilimkar
> Cc: tony at atomide.com; catalin.marinas at arm.com; linux-
> omap at vger.kernel.org; linux-arm-kernel at lists.infradead.org
> Subject: Re: [PATCH 1/2] ARM: l2x0: Errata fix for flush by Way
> operationcan cause data corruption
>
> On Sun, Feb 27, 2011 at 12:00:21PM +0000, Russell King - ARM Linux
> wrote:
> > > +#else
> > > +/* Optimised out for non-errata case */
> > > +static inline void debug_writel(unsigned long val)
> > > +{
> > > }
> >
> > #define l2x0_set_debug NULL
> >
> > > +#endif
>
> I notice you got rid of the inline function. Have you tried
> building this without the errata enabled?
I accidently dropped the inline function while
incorporating the comment from you. :(
Fixed it. Updated version # 6770/1
Regards,
Santosh
next prev parent reply other threads:[~2011-02-28 12:47 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-02-18 12:35 [PATCH 0/2] l2x0: Add PL310 Errata 727915 Santosh Shilimkar
2011-02-18 12:35 ` Santosh Shilimkar
2011-02-18 12:35 ` [PATCH 1/2] ARM: l2x0: Errata fix for flush by Way operation can cause data corruption Santosh Shilimkar
2011-02-18 12:35 ` Santosh Shilimkar
2011-02-21 9:46 ` Catalin Marinas
2011-02-21 9:46 ` Catalin Marinas
2011-02-27 12:00 ` Russell King - ARM Linux
2011-02-27 12:00 ` Russell King - ARM Linux
2011-02-28 3:45 ` [PATCH 1/2] ARM: l2x0: Errata fix for flush by Way operationcan " Santosh Shilimkar
2011-02-28 3:45 ` Santosh Shilimkar
2011-02-28 10:13 ` [PATCH 1/2] ARM: l2x0: Errata fix for flush by Way operation can " Russell King - ARM Linux
2011-02-28 10:13 ` Russell King - ARM Linux
2011-02-28 12:47 ` Santosh Shilimkar [this message]
2011-02-28 12:47 ` [PATCH 1/2] ARM: l2x0: Errata fix for flush by Way operationcan " Santosh Shilimkar
2011-03-07 12:07 ` Will Deacon
2011-03-07 12:07 ` Will Deacon
2011-03-07 12:25 ` [PATCH 1/2] ARM: l2x0: Errata fix for flush by Wayoperationcan " Santosh Shilimkar
2011-03-07 12:25 ` Santosh Shilimkar
2011-03-07 19:10 ` Russell King - ARM Linux
2011-03-07 19:10 ` Russell King - ARM Linux
2011-03-08 6:00 ` [PATCH 1/2] ARM: l2x0: Errata fix for flush by Wayoperationcancause " Santosh Shilimkar
2011-03-08 6:00 ` Santosh Shilimkar
2011-02-18 12:35 ` [PATCH 2/2] omap4: l2x0: Populate set_debug() function and enable Errata 727915 Santosh Shilimkar
2011-02-18 12:35 ` Santosh Shilimkar
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=69dd173e3c7b4d02ef50ceaa19b2343a@mail.gmail.com \
--to=santosh.shilimkar@ti.com \
--cc=catalin.marinas@arm.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-omap@vger.kernel.org \
--cc=linux@arm.linux.org.uk \
--cc=tony@atomide.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.