Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH] Linux kernel module for Waveshare 3.2inch RPi LCD (B) V3 touch controller
Date: Thu, 8 Jan 2015 21:24:13 +0100	[thread overview]
Message-ID: <20150108212413.4d3aee4a@free-electrons.com> (raw)
In-Reply-To: <1420747796-28272-1-git-send-email-ps.report@gmx.net>

Dear Peter Seiderer,

On Thu,  8 Jan 2015 21:09:55 +0100, Peter Seiderer wrote:
> This kernel module enables the (easy?) using of the ADS7846 SPI touch controller
> on Waveshare 3.2 LCD (B) modules.
> 
> Alternatives would be:
> - the old way: add device definition/register call to arch/arm/mach-bcm2708/bcm2708.c
> - the new way: use device tree support and add an entrie for ads7846

I'm sorry, this is not your fault, but this ads7846_device thing is an
absolute abomination. Why is RaspberryPi *THE* platform propagating all
the worst possible abominations? Why is it *THE* platform doing all the
non-standard stuff? Why is it so crappy.

People, please read this: if you want to learn embedded Linux, don't
buy Raspberry Pi. It is the worst platform ever in terms of embedded
Linux development best practices. Ever.

Do yourself a favor, and buy properly supported stuff: the Atmel
Xplained comes to mind, the BeagleBone Black as well of course.

Sorry, but this ads7846_device is just pure crap. The alternatives you
mentioned are the only good solutions: change your board file if you're
using an old style ARM kernel, or change your Device Tree if you're
using a new style ARM kernel.

I don't know what the policy of Buildroot should be about accepting
crap, but on my side, I will not add such a package to Buildroot.

Again: don't take this personally, this is absolutely not against you.
Just me being fed up with Raspberry Pi crap.

Best regards,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

  parent reply	other threads:[~2015-01-08 20:24 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-01-08 20:09 [Buildroot] [PATCH] Linux kernel module for Waveshare 3.2inch RPi LCD (B) V3 touch controller Peter Seiderer
2015-01-08 20:09 ` [Buildroot] [PATCH] ads7846-device: new package Peter Seiderer
2015-01-08 20:24 ` Thomas Petazzoni [this message]
  -- strict thread matches above, loose matches on Subject: below --
2015-01-09 20:21 [Buildroot] [PATCH] Linux kernel module for Waveshare 3.2inch RPi LCD (B) V3 touch controller Peter Seiderer
2015-01-09 21:40 ` Yann E. MORIN
2015-01-09 22:48   ` Yann E. MORIN
2015-01-10  1:13 Peter Seiderer
2015-01-10 11:03 ` Yann E. MORIN
2015-01-10 20:36 Peter Seiderer
2015-01-10 21:58 ` Yann E. MORIN

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=20150108212413.4d3aee4a@free-electrons.com \
    --to=thomas.petazzoni@free-electrons.com \
    --cc=buildroot@busybox.net \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox