Hi Dirk, On Wed, May 25, 2016 at 10:58 AM, Dirk Behme <dirk.behme@xxxxxxxxxxxx> wrote: > Instead of hard coding the product register in rcar_du_crtc.c, > read it based on the device tree. > > This patch series is based on > > https://git.kernel.org/cgit/linux/kernel/git/geert/renesas-drivers.git/log/?h=topic/gen3-latest > renesas-drivers-2016-05-17-v4.6 > 4717ef6d59c3204e385c > Revert "ASoC: simple-card: Add pm callbacks to platform driver" > > It's boot tested on r8a7795 Salvator-X and checked that the same > product register value is read without and with this patch series. Thanks for your series! Given the use of PRR you saw is in a patch that's definitely not ready for upstream, I think adding a full-fledged PRR driver is a bit premature. So far we've always planned to handle differences in ES versions using the compatible value, cfr. "renesas,sata-r8a7790-es1". But in general we target the latest (production) version in upstream. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds