Hi Wolfram,
On Tuesday 17 December 2013 23:13:09 Wolfram Sang wrote:
> On Tue, Dec 17, 2013 at 10:55:31PM +0100, Laurent Pinchart wrote:
> > On Tuesday 17 December 2013 22:44:36 Wolfram Sang wrote:
> > > From: Wolfram Sang <wsa@xxxxxxxxxxxxxxxxxxxx>
> > >
> > > I decided to put the pinmuxing into the dtsi file since there is only
> > > one pinmux posiibility which one probably wants to have when using the
> > > bus.
>
> I tried to explain here why the pin-groups are in the dtsi.
>
> > > + riic0_pins: i2c0 {
> > > + renesas,groups = "riic0_scl_p1_0", "riic0_sda_p1_1";
> > > + renesas,function = "riic0";
> > > + };
> > > +
> > > + riic1_pins: i2c1 {
> > > + renesas,groups = "riic1_scl_p1_2", "riic1_sda_p1_3";
> > > + renesas,function = "riic1";
> > > + };
> > > +
> > > + riic2_pins: i2c2 {
> > > + renesas,groups = "riic2_scl_p1_4", "riic2_sda_p1_5";
> > > + renesas,function = "riic2";
> > > + };
> > > +
> > > + riic3_pins: i2c3 {
> > > + renesas,groups = "riic3_scl_p1_6", "riic3_sda_p1_7";
> > > + renesas,function = "riic3";
> > > + };
> >
> > The SoC allows other options for the I2C pin groups. Instead of declaring
> > all possible groups here, I think we should thus only add the groups that
> > are really used to the board .dts files.
>
> Really? Couldn't find any. This is why I included it here: since it is
> the only option, people probably want to use it when they activate the
> bus.
My bad, sorry, I've misread patch 1/5 and spoke too fast :-/ I thus makes
sense to add the groups here.
> > > + clock-frequency = <100000>;
> >
> > Isn't that a board-specific property ?
>
> It is meant as a sane default. Can be overridden.
OK.
--
Regards,
Laurent Pinchart
Attachment:
signature.asc
Description: This is a digitally signed message part.
