public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* staging: fbtft: CONFIG_ARCH_BCM2708?
@ 2015-01-20  8:46 Paul Bolle
  2015-01-20 11:11 ` Noralf Trønnes
  0 siblings, 1 reply; 2+ messages in thread
From: Paul Bolle @ 2015-01-20  8:46 UTC (permalink / raw)
  To: Thomas Petazzoni, Noralf Tronnes
  Cc: Valentin Rothberg, Greg Kroah-Hartman, devel, linux-kernel

Thomas, Noralf, 

Your commit c296d5f9957c ("staging: fbtft: core support") is included in
today's linux-next (ie, next-20150120). I noticed because a script I use
to check linux-next spotted a problem in it.

See, that commit adds two checks for CONFIG_ARCH_BCM2708. But there's no
Kconfig symbol ARCH_BCM2708. (A comment in
drivers/mmc/host/sdhci-bcm2835.c does suggest there's an out of tree
"Main bcm2708 linux port".)

Is the code to add the Kconfig symbol ARCH_BCM2708 queued somewhere?

Thanks,


Paul Bolle


^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: staging: fbtft: CONFIG_ARCH_BCM2708?
  2015-01-20  8:46 staging: fbtft: CONFIG_ARCH_BCM2708? Paul Bolle
@ 2015-01-20 11:11 ` Noralf Trønnes
  0 siblings, 0 replies; 2+ messages in thread
From: Noralf Trønnes @ 2015-01-20 11:11 UTC (permalink / raw)
  To: Paul Bolle, Thomas Petazzoni
  Cc: Valentin Rothberg, Greg Kroah-Hartman, devel, linux-kernel

Den 20.01.2015 09:46, skrev Paul Bolle:
> Thomas, Noralf,
>
> Your commit c296d5f9957c ("staging: fbtft: core support") is included in
> today's linux-next (ie, next-20150120). I noticed because a script I use
> to check linux-next spotted a problem in it.
>
> See, that commit adds two checks for CONFIG_ARCH_BCM2708. But there's no
> Kconfig symbol ARCH_BCM2708. (A comment in
> drivers/mmc/host/sdhci-bcm2835.c does suggest there's an out of tree
> "Main bcm2708 linux port".)
>
> Is the code to add the Kconfig symbol ARCH_BCM2708 queued somewhere?
Hi Paul,

ARCH_BCM2708 is used instead of ARCH_BCM2835 in the Raspberry Pi kernel 
fork.
The optimization code enabled by ARCH_BCM2708 in 
drivers/staging/fbtft/fbtft-io.c can be removed.
There will be another take on the I/O stage for those drivers later.

I can make a patch for this. Just need to find out how to setup git to 
send paches for me.

Thanks for spotting this.


Noralf.


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2015-01-20 11:20 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-01-20  8:46 staging: fbtft: CONFIG_ARCH_BCM2708? Paul Bolle
2015-01-20 11:11 ` Noralf Trønnes

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox