| Message ID | 1477598426-28125-2-git-send-email-tthayer@opensource.altera.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On Thu, Oct 27, 2016 at 03:00:23PM -0500, tthayer@opensource.altera.com wrote: > From: Thor Thayer <tthayer@opensource.altera.com> > > Add the Arria10 DevKit System Resource Chip register and state > monitoring module to the MFD. > > Signed-off-by: Thor Thayer <tthayer@opensource.altera.com> > --- > Note: This needs to be applied to the bindings document that > was Acked & Applied but didn't reach the for-next branch. > See https://patchwork.ozlabs.org/patch/629397/ > --- > v2 Change compatible string -mon to -monitor for clarity > --- > Documentation/devicetree/bindings/mfd/altera-a10sr.txt | 9 +++++++++ > 1 file changed, 9 insertions(+) > > diff --git a/Documentation/devicetree/bindings/mfd/altera-a10sr.txt b/Documentation/devicetree/bindings/mfd/altera-a10sr.txt > index ea151f2..c47be28 100644 > --- a/Documentation/devicetree/bindings/mfd/altera-a10sr.txt > +++ b/Documentation/devicetree/bindings/mfd/altera-a10sr.txt > @@ -18,6 +18,7 @@ The A10SR consists of these sub-devices: > Device Description > ------ ---------- > a10sr_gpio GPIO Controller This should be just "gpio" BTW. > +a10sr_monitor Register and State Monitoring s/_/-/ or maybe just "monitor". Not really a generic node name to use for this. Rob
Hi Rob, On 10/31/2016 12:36 AM, Rob Herring wrote: > On Thu, Oct 27, 2016 at 03:00:23PM -0500, tthayer@opensource.altera.com wrote: >> From: Thor Thayer <tthayer@opensource.altera.com> >> >> Add the Arria10 DevKit System Resource Chip register and state >> monitoring module to the MFD. >> >> Signed-off-by: Thor Thayer <tthayer@opensource.altera.com> >> --- >> Note: This needs to be applied to the bindings document that >> was Acked & Applied but didn't reach the for-next branch. >> See https://patchwork.ozlabs.org/patch/629397/ >> --- >> v2 Change compatible string -mon to -monitor for clarity >> --- >> Documentation/devicetree/bindings/mfd/altera-a10sr.txt | 9 +++++++++ >> 1 file changed, 9 insertions(+) >> >> diff --git a/Documentation/devicetree/bindings/mfd/altera-a10sr.txt b/Documentation/devicetree/bindings/mfd/altera-a10sr.txt >> index ea151f2..c47be28 100644 >> --- a/Documentation/devicetree/bindings/mfd/altera-a10sr.txt >> +++ b/Documentation/devicetree/bindings/mfd/altera-a10sr.txt >> @@ -18,6 +18,7 @@ The A10SR consists of these sub-devices: >> Device Description >> ------ ---------- >> a10sr_gpio GPIO Controller > > This should be just "gpio" BTW. I reason I preprend a10sr_ is to distinguish this GPIO from the other GPIOs when binding to our LEDs (see below). I think the LEDs need a unique node name (unless I'm not understanding something). A less important reason I use a10sr_gpio on the node name is that I can cat the /sys/class/gpio/gpioxxx/label and see that it is associated with the a10sr instead of one of our other general GPIOs. Is there a better way to distinguish these? > >> +a10sr_monitor Register and State Monitoring > > s/_/-/ or maybe just "monitor". Not really a generic node name to use > for this. > The reason I use _ for the node name is that the DTC fails if I reference a node name with "-" but works OK for "_". For instance, I get an error if the LEDs reference "a10sr-gpio" but "a10sr_gpio" compiles ok. Thanks for reviewing! Thor > Rob >
On Mon, Oct 31, 2016 at 10:28 AM, Thor Thayer <tthayer@opensource.altera.com> wrote: > Hi Rob, > > On 10/31/2016 12:36 AM, Rob Herring wrote: >> >> On Thu, Oct 27, 2016 at 03:00:23PM -0500, tthayer@opensource.altera.com >> wrote: >>> >>> From: Thor Thayer <tthayer@opensource.altera.com> >>> >>> Add the Arria10 DevKit System Resource Chip register and state >>> monitoring module to the MFD. >>> >>> Signed-off-by: Thor Thayer <tthayer@opensource.altera.com> >>> --- >>> Note: This needs to be applied to the bindings document that >>> was Acked & Applied but didn't reach the for-next branch. >>> See https://patchwork.ozlabs.org/patch/629397/ >>> --- >>> v2 Change compatible string -mon to -monitor for clarity >>> --- >>> Documentation/devicetree/bindings/mfd/altera-a10sr.txt | 9 +++++++++ >>> 1 file changed, 9 insertions(+) >>> >>> diff --git a/Documentation/devicetree/bindings/mfd/altera-a10sr.txt >>> b/Documentation/devicetree/bindings/mfd/altera-a10sr.txt >>> index ea151f2..c47be28 100644 >>> --- a/Documentation/devicetree/bindings/mfd/altera-a10sr.txt >>> +++ b/Documentation/devicetree/bindings/mfd/altera-a10sr.txt >>> @@ -18,6 +18,7 @@ The A10SR consists of these sub-devices: >>> Device Description >>> ------ ---------- >>> a10sr_gpio GPIO Controller >> >> >> This should be just "gpio" BTW. > > > I reason I preprend a10sr_ is to distinguish this GPIO from the other GPIOs > when binding to our LEDs (see below). I think the LEDs need a unique node > name (unless I'm not understanding something). You are confusing DTS labels and node names. Labels are global and just convenience for the DTS source only instead of using full paths (and can be anything you want). The node name is local to the level it is at. > A less important reason I use a10sr_gpio on the node name is that I can cat > the /sys/class/gpio/gpioxxx/label and see that it is associated with the > a10sr instead of one of our other general GPIOs. Is there a better way to > distinguish these? label should come from the label property I think. This is one of the problems with the GPIO sysfs interface and why it is being replaced with the char dev interface. Rob > >> >>> +a10sr_monitor Register and State Monitoring >> >> >> s/_/-/ or maybe just "monitor". Not really a generic node name to use >> for this. >> > > The reason I use _ for the node name is that the DTC fails if I reference a > node name with "-" but works OK for "_". For instance, I get an error if the > LEDs reference "a10sr-gpio" but "a10sr_gpio" compiles ok. Again, I'm talking node names. You are talking labels. And yes, DTS labels can't use '-'. Rob
diff --git a/Documentation/devicetree/bindings/mfd/altera-a10sr.txt b/Documentation/devicetree/bindings/mfd/altera-a10sr.txt index ea151f2..c47be28 100644 --- a/Documentation/devicetree/bindings/mfd/altera-a10sr.txt +++ b/Documentation/devicetree/bindings/mfd/altera-a10sr.txt @@ -18,6 +18,7 @@ The A10SR consists of these sub-devices: Device Description ------ ---------- a10sr_gpio GPIO Controller +a10sr_monitor Register and State Monitoring Arria10 GPIO Required Properties: @@ -27,6 +28,10 @@ Required Properties: the second cell is used to specify flags. See ../gpio/gpio.txt for more information. +Arria10 Register and State Monitor +Required Properties: +- compatible : Should be "altr,a10sr-monitor" + Example: resource-manager@0 { @@ -43,4 +48,8 @@ Example: gpio-controller; #gpio-cells = <2>; }; + + a10sr_monitor { + compatible = "altr,a10sr-monitor"; + }; };