Re: [PATCH v2 2/2] media: staging: max96712: Add support for MAX96724

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

 



Hejssan!

On Tue, Aug 27, 2024 at 07:57:35PM +0200, Niklas Söderlund wrote:
> Hej Sakari,
> 
> Tack för feedback.

Var så god!

> > > @@ -181,7 +186,8 @@ static void max96712_pattern_enable(struct max96712_priv *priv, bool enable)
> > >  	}
> > >  
> > >  	/* PCLK 75MHz. */
> > > -	max96712_write(priv, 0x0009, 0x01);
> > > +	if (!priv->max96724)
> > > +		max96712_write(priv, 0x0009, 0x01);
> > 
> > It'd be nice to have a macro for this, espeically now that the driver
> > supports more than one chip.
> 
> What do you mean by macro? To test for priv->max96724, a define for the 
> register name or something else?

Ah, I meant 0x0009 and preferrably 0x01 as well.

-- 
Hälsningar,

Sakari Ailus




[Index of Archives]     [Linux Samsung SOC]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]

  Powered by Linux