Byungho An bh74...@samsung.com :
[...]
Nit: you may consider reorganizing the variables in an inverted xmas tree
fashion at some point.
Does it look better? No problem.
Marginally if not more. Consider it a guideline to avoid unusual or ugly
layout.
[...]
+priv-ioaddr +
Byungho An bh74...@samsung.com :
[...]
+static int sxgbe_hw_init(struct sxgbe_priv_data * const priv)
+{
struct sxgbe_ops *hw = priv-hw;
+ u32 ctrl_ids;
[...]
+struct sxgbe_priv_data *sxgbe_dvr_probe(struct device *device,
nit: s/dvr/drv/ ?
(several occurences in the driver)
Francois Romieu rom...@fr.zoreil.com :
Byungho An bh74...@samsung.com :
[...]
+static int sxgbe_hw_init(struct sxgbe_priv_data * const priv) {
struct sxgbe_ops *hw = priv-hw;
+ u32 ctrl_ids;
[...]
+struct sxgbe_priv_data *sxgbe_dvr_probe(struct device *device,
nit: