Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753458Ab3EOWCq (ORCPT ); Wed, 15 May 2013 18:02:46 -0400 Received: from mail-ea0-f170.google.com ([209.85.215.170]:56764 "EHLO mail-ea0-f170.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751677Ab3EOWCo convert rfc822-to-8bit (ORCPT ); Wed, 15 May 2013 18:02:44 -0400 From: Tomasz Figa To: Heiko =?ISO-8859-1?Q?St=FCbner?= Cc: Sylwester Nawrocki , Linus Walleij , Dan Williams , Vinod Koul , "linux-kernel@vger.kernel.org" , linux-samsung-soc , Kukjin Kim , "linux-arm-kernel@lists.infradead.org" , Russell King - ARM Linux Subject: Re: [RFC 2/4] dma: add dmaengine driver for Samsung s3c24xx SoCs Date: Thu, 16 May 2013 00:02:40 +0200 Message-ID: <45625193.0y3McEvmiD@flatron> User-Agent: KMail/4.10.3 (Linux/3.9.2-gentoo; KDE/4.10.3; x86_64; ; ) In-Reply-To: <201305152348.32171.heiko@sntech.de> References: <201305111330.05046.heiko@sntech.de> <5193FC08.5040002@gmail.com> <201305152348.32171.heiko@sntech.de> MIME-Version: 1.0 Content-Transfer-Encoding: 8BIT Content-Type: text/plain; charset="iso-8859-1" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2619 Lines: 70 On Wednesday 15 of May 2013 23:48:31 Heiko St?bner wrote: > Am Mittwoch, 15. Mai 2013, 23:20:08 schrieb Sylwester Nawrocki: > > On 05/15/2013 10:31 PM, Heiko St?bner wrote: > > >>> + BUG(); > > >>> > > >> > Isn't that a bit nasty. This macro should be used with care and > > >> > we > > >> > should recover if possible. dev_err()? > > > > > > runtime_config already denies any settings not in the 1,2 or 4bytes > > > range - the default-part should therefore never be reached. So if > > > any other value magically appears in the register and triggers the > > > default-part, something is seriously wrong. So my guess is, the BUG > > > might be appropriate. > > > > > > On the other hand the whole default+BUG part could also simply go > > > away, > > > for the same reasons. > > > > IMHO BUG() is not needed at all. As Linus suggested dev_err() is such > > case or WARN_ON() would be more appropriate. This has been discussed > > in the past extensively, not sure if you are aware of the other > > Linus' opinion on BUG()/BUG_ON() proliferation: > > https://lkml.org/lkml/2012/9/27/461 > Very interesting read and I'll keep this in mind in the future. What > about the other option ... i.e. simply getting rid of the whole "error > handling", as the other code paths should already make sure that only > valid values get written into the register. > > Can the value change in the register somehow on its own without kernel > intervention, or does this not happen? Hmm, it depends on hardware, I guess. Not sure how it works on this particular IP. Still, the mentioned BUG() was about a value in a driver-filled struct, wasn't it? /* Quoting the the code for reference */ > +static u32 s3c24xx_dma_getbytes_chan(struct s3c24xx_dma_chan *s3cchan) > +{ > + struct s3c24xx_dma_phy *phy = s3cchan->phy; > + struct s3c24xx_txd *txd = s3cchan->at; > + u32 tc = readl(phy->base + DSTAT) & DSTAT_CURRTC_MASK; > + > + switch (txd->dcon & DCON_DSZ_MASK) { > + case DCON_DSZ_BYTE: > + return tc; > + case DCON_DSZ_HALFWORD: > + return tc * 2; > + case DCON_DSZ_WORD: > + return tc * 4; > + default: > + break; > + } > + > + BUG(); (Btw. I don't see anything setting the DCON_DSZ bits in this field. Am I missing something?) Best regards, Tomasz -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/