Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1271309

Re: [PATCHv2 0/3] Devicetree bindings for Ion

Show all headers | View raw

On Monday 16 November 2015 16:57:32 Laura Abbott wrote:
> Hi,
> 
> This is another attempt at devicetree bindings for Ion. The big complaint from
> v1 was that too much unnecessary data was being pushed into devicetree.
> v2 takes a different approach of using just compatbile strings for the heaps.
> This gets closer to the devicetree philosophy of describing just the hardware.
> Each heap ultimately represents some kind of memory that exists in the system.
> The compatible string indicates the match for how to handle the type of memory
> on this system. The details like heap-id, name etc. are now pushed out to C
> files. This makes Ion heaps look closer to something like a quirks framework.
> (I'd argue this isn't a complete mischaracterization given the type of setup
> Ion gets used for...) The one downside here is that this leads to more new
> bindings for each board that gets added.
> 
> This version also includes a sample C file which shows what the structure
> might look like. As always, your comments and reviews are appreciated.

I'm still a bit unsure about the concept of hardwiring ion in the DT
bindings. It's not just Linux-specific, it's specific to the implementation
of one or two GPU drivers, and if we fix the bindings for Ion, we might
never be able to migrate them away from this framework.

	Arnd
--
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/

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCHv2 0/3] Devicetree bindings for Ion Laura Abbott <laura@labbott.name> - 2015-11-17 02:00 +0100
  [PATCHv2 1/3] ion: Devicetree bindings for Ion Laura Abbott <laura@labbott.name> - 2015-11-17 02:00 +0100
  [PATCHv2 2/3] staging: ion: Add files for parsing the devicetree Laura Abbott <laura@labbott.name> - 2015-11-17 02:00 +0100
    Re: [PATCHv2 2/3] staging: ion: Add files for parsing the devicetree Dan Carpenter <dan.carpenter@oracle.com> - 2015-11-17 07:20 +0100
  Re: [PATCHv2 0/3] Devicetree bindings for Ion Arnd Bergmann <arnd@arndb.de> - 2015-11-17 16:20 +0100
    Re: [PATCHv2 0/3] Devicetree bindings for Ion Laura Abbott <laura@labbott.name> - 2015-11-17 20:10 +0100

csiph-web